1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197
|
/*
* Copyright (C) 2018 The Guava Authors
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.google.common.util.concurrent;
import static com.google.common.truth.Truth.assertThat;
import java.lang.reflect.Method;
import java.nio.channels.spi.AbstractInterruptibleChannel;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.locks.AbstractOwnableSynchronizer;
import java.util.concurrent.locks.LockSupport;
import junit.framework.TestCase;
public final class InterruptibleTaskTest extends TestCase {
// Regression test for a deadlock where a task could be stuck busy waiting for the task to
// transition to DONE
public void testInterruptThrows() throws Exception {
final CountDownLatch isInterruptibleRegistered = new CountDownLatch(1);
InterruptibleTask<Void> task =
new InterruptibleTask<Void>() {
@Override
Void runInterruptibly() throws Exception {
BrokenChannel bc = new BrokenChannel();
bc.doBegin();
isInterruptibleRegistered.countDown();
new CountDownLatch(1).await(); // the interrupt will wake us up
return null;
}
@Override
boolean isDone() {
return false;
}
@Override
String toPendingString() {
return "";
}
@Override
void afterRanInterruptiblySuccess(Void result) {}
@Override
void afterRanInterruptiblyFailure(Throwable error) {}
};
Thread runner = new Thread(task);
runner.start();
isInterruptibleRegistered.await();
try {
task.interruptTask();
fail();
} catch (RuntimeException expected) {
assertThat(expected)
.hasMessageThat()
.isEqualTo("I bet you didn't think Thread.interrupt could throw");
}
// We need to wait for the runner to exit. It used to be that the runner would get stuck in the
// busy loop when interrupt threw.
runner.join(TimeUnit.SECONDS.toMillis(10));
}
static final class BrokenChannel extends AbstractInterruptibleChannel {
@Override
protected void implCloseChannel() {
throw new RuntimeException("I bet you didn't think Thread.interrupt could throw");
}
void doBegin() {
super.begin();
}
}
/**
* Because Thread.interrupt() can invoke arbitrary code, it can be slow (e.g. perform IO). To
* protect ourselves from that we want to make sure that tasks don't spin too much waiting for the
* interrupting thread to complete the protocol.
*/
/*
* This test hangs (or maybe is just *very* slow) under Android.
*
* TODO(b/218700094): Ideally, get this to pass under Android. Failing that, convince ourselves
* that the test isn't exposing a real problem with InterruptibleTask, one that could matter in
* prod.
*/
@AndroidIncompatible
public void testInterruptIsSlow() throws Exception {
final CountDownLatch isInterruptibleRegistered = new CountDownLatch(1);
final SlowChannel slowChannel = new SlowChannel();
final InterruptibleTask<Void> task =
new InterruptibleTask<Void>() {
@Override
Void runInterruptibly() throws Exception {
slowChannel.doBegin();
isInterruptibleRegistered.countDown();
try {
new CountDownLatch(1).await(); // the interrupt will wake us up
} catch (InterruptedException ie) {
// continue
}
LockSupport.unpark(Thread.currentThread()); // simulate a spurious wakeup.
return null;
}
@Override
boolean isDone() {
return false;
}
@Override
String toPendingString() {
return "";
}
@Override
void afterRanInterruptiblySuccess(Void result) {}
@Override
void afterRanInterruptiblyFailure(Throwable error) {}
};
Thread runner = new Thread(task, "runner");
runner.start();
isInterruptibleRegistered.await();
// trigger the interrupt on another thread since it will block
Thread interrupter =
new Thread("Interrupter") {
@Override
public void run() {
task.interruptTask();
}
};
interrupter.start();
// this will happen once the interrupt has been set which means that
// 1. the runner has been woken up
// 2. the interrupter is stuck in the call the Thread.interrupt()
// after some period of time the runner thread should become blocked on the task because it is
// waiting for the slow interrupting thread to complete Thread.interrupt
awaitBlockedOnInstanceOf(runner, InterruptibleTask.Blocker.class);
Object blocker = LockSupport.getBlocker(runner);
assertThat(blocker).isInstanceOf(AbstractOwnableSynchronizer.class);
Method getExclusiveOwnerThread =
AbstractOwnableSynchronizer.class.getDeclaredMethod("getExclusiveOwnerThread");
getExclusiveOwnerThread.setAccessible(true);
Thread owner = (Thread) getExclusiveOwnerThread.invoke(blocker);
assertThat(owner).isSameInstanceAs(interrupter);
slowChannel.exitClose.countDown(); // release the interrupter
// We need to wait for the runner to exit. To make sure that the interrupting thread wakes it
// back up.
runner.join(TimeUnit.SECONDS.toMillis(10));
}
// waits for the given thread to be blocked on the given object
private static void awaitBlockedOnInstanceOf(Thread t, Class<?> blocker)
throws InterruptedException {
while (!isThreadBlockedOnInstanceOf(t, blocker)) {
if (t.getState() == Thread.State.TERMINATED) {
throw new RuntimeException("Thread " + t + " exited unexpectedly");
}
Thread.sleep(1);
}
}
private static boolean isThreadBlockedOnInstanceOf(Thread t, Class<?> blocker) {
return t.getState() == Thread.State.WAITING && blocker.isInstance(LockSupport.getBlocker(t));
}
static final class SlowChannel extends AbstractInterruptibleChannel {
final CountDownLatch exitClose = new CountDownLatch(1);
@Override
protected void implCloseChannel() {
Uninterruptibles.awaitUninterruptibly(exitClose);
}
void doBegin() {
super.begin();
}
}
}
|