mirrors/JetBrainsRuntime
1
0
Fork 0
mirror of https://github.com/JetBrains/JetBrainsRuntime.git synced 2025-12-10 19:39:39 +01:00
Code Issues Packages Projects Releases Wiki Activity

8266421: Deadlock in Sound System

Browse Source 
Reviewed-by: phh, aph
Backport-of: f6f82c3168
This commit is contained in:
Sergey Bylokhov
2022-01-21 23:20:55 +00:00
committed by Vitaly Provodin
parent ee5f97e3f4
commit aa800b51bd
4 changed files with 117 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
src/java.desktop/share/classes/com/sun/media/sound/AbstractDataLine.java
View File

@@ -1,5 +1,5 @@
/*
* Copyright (c) 1999, 2016, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 1999, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -315,13 +315,7 @@ abstract class AbstractDataLine extends AbstractLine implements DataLine {
//boolean sendEvents = false;
//long position = getLongFramePosition();
synchronized (this) {
if (this.active != active) {
this.active = active;
//sendEvents = true;
}
}
this.active = active;
// $$kk: 11.19.99: take ACTIVE / INACTIVE / EOM events out;
// putting them in is technically an API change.
@@ -347,12 +341,9 @@ abstract class AbstractDataLine extends AbstractLine implements DataLine {
boolean sendEvents = false;
long position = getLongFramePosition();
synchronized (this) {
if (this.started != started) {
this.started = started;
sendEvents = true;
}
if (this.started != started) {
this.started = started;
sendEvents = true;
}
if (sendEvents) {

10
src/java.desktop/share/classes/com/sun/media/sound/AbstractLine.java
View File

@@ -1,5 +1,5 @@
/*
* Copyright (c) 1999, 2016, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 1999, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -162,11 +162,9 @@ abstract class AbstractLine implements Line {
boolean sendEvents = false;
long position = getLongFramePosition();
synchronized (this) {
if (this.open != open) {
this.open = open;
sendEvents = true;
}
if (this.open != open) {
this.open = open;
sendEvents = true;
}
if (sendEvents) {

4
src/java.desktop/share/classes/com/sun/media/sound/DirectAudioDevice.java
View File

@@ -1,5 +1,5 @@
/*
* Copyright (c) 2002, 2020, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2002, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -1252,7 +1252,7 @@ final class DirectAudioDevice extends AbstractMixer {
}
@Override
public synchronized void setMicrosecondPosition(long microseconds) {
public void setMicrosecondPosition(long microseconds) {
if (Printer.trace) Printer.trace("> DirectClip: setMicrosecondPosition: " + microseconds);
long frames = Toolkit.micros2frames(getFormat(), microseconds);

106
test/jdk/javax/sound/sampled/Clip/SetPositionHang.java Normal file
View File

@@ -0,0 +1,106 @@
/*
* Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*/
import javax.sound.sampled.AudioFormat;
import javax.sound.sampled.AudioSystem;
import javax.sound.sampled.Clip;
import javax.sound.sampled.LineUnavailableException;
/**
* @test
* @bug 8266421
* @summary Tests that Clip.setFramePosition/setMicrosecondPosition do not hang.
*/
public final class SetPositionHang implements Runnable {
private static volatile boolean testFramePosition;
private final Clip clip;
private final String thread;
private SetPositionHang(String thread, Clip clip) {
this.thread = thread;
this.clip = clip;
}
public static void main(String[] args) throws Exception {
testFramePosition = false;
test();
testFramePosition = true;
test();
}
private static void test() throws InterruptedException {
try (Clip clip = AudioSystem.getClip()) {
// prepare audio data
int frameCount = 441000; // lets say 10 seconds
AudioFormat format = new AudioFormat(44100.0f, 16, 2, true, false);
byte[] bytes = new byte[frameCount * format.getFrameSize()];
clip.open(format, bytes, 0, frameCount);
Thread t1 = new Thread(new SetPositionHang("1", clip));
Thread t2 = new Thread(new SetPositionHang("2", clip));
Thread t3 = new Thread(new SetPositionHang("3", clip));
Thread t4 = new Thread(new SetPositionHang("4", clip));
Thread t5 = new Thread(new SetPositionHang("5", clip));
t1.start();
t2.start();
t3.start();
t4.start();
t5.start();
t1.join();
t2.join();
t3.join();
t4.join();
t5.join();
} catch (LineUnavailableException | IllegalArgumentException ignored) {
// the test is not applicable
}
}
public void run() {
System.out.println("Thread " + thread + " Start");
for (int i = 0; i < 100; i++) {
// System.out.println("Thread " + thread + " Play " +
// System.currentTimeMillis() % 100000);
playSound();
try {
Thread.sleep(i);
} catch (InterruptedException e) {
throw new RuntimeException(e);
}
}
System.out.println("Thread " + thread + " Finish");
}
private void playSound() {
if (clip.isRunning()) {
clip.stop();
}
if (testFramePosition) {
clip.setFramePosition(0);
} else {
clip.setMicrosecondPosition(0);
}
clip.start();
}
}