diff options
author | Kevin Krakauer <krakauer@google.com> | 2021-04-05 12:34:12 -0700 |
---|---|---|
committer | gVisor bot <gvisor-bot@google.com> | 2021-04-05 12:37:31 -0700 |
commit | e7b2023647df994736ccaaa99a66a8712db2c631 (patch) | |
tree | cbf5450b94532a66f2deda14b837390de77f66f9 | |
parent | e21a71bff18ba9da30a0ef977c747376d51ce8cb (diff) |
deflake semaphore test
There's no reason to actually increment the semaphore, it just introduces the
chance of a race.
PiperOrigin-RevId: 366851795
-rw-r--r-- | test/syscalls/linux/semaphore.cc | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/test/syscalls/linux/semaphore.cc b/test/syscalls/linux/semaphore.cc index 28f51a3bf..8c5732147 100644 --- a/test/syscalls/linux/semaphore.cc +++ b/test/syscalls/linux/semaphore.cc @@ -234,14 +234,6 @@ TEST(SemaphoreTest, SemTimedOpBlock) { AutoSem sem(semget(IPC_PRIVATE, 1, 0600 | IPC_CREAT)); ASSERT_THAT(sem.get(), SyscallSucceeds()); - ScopedThread th([&sem] { - absl::SleepFor(absl::Milliseconds(100)); - - struct sembuf buf = {}; - buf.sem_op = 1; - ASSERT_THAT(RetryEINTR(semop)(sem.get(), &buf, 1), SyscallSucceeds()); - }); - struct sembuf buf = {}; buf.sem_op = -1; struct timespec timeout = {}; |