libs/newlib-cygwin
libs
/
newlib-cygwin
mirror of git://sourceware.org/git/newlib-cygwin.git
4
0
Fork 0
Code Issues Projects Releases Wiki Activity

Cygwin: pipe: Fix notification timing of select_sem. 

- Make select_sem notify even when read/write partially.
This commit is contained in:
Takashi Yano 2021-09-09 12:08:39 +09:00 committed by Corinna Vinschen
parent 597f87294d
commit 08b8534d0a
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
winsup/cygwin/fhandler_pipe.cc
View File

@ -311,6 +311,9 @@ fhandler_pipe::raw_read (void *ptr, size_t& len)
{
status = STATUS_THREAD_SIGNALED;
nbytes += io.Information;
if (select_sem && io.Information > 0)
ReleaseSemaphore (select_sem,
get_obj_handle_count (select_sem), NULL);
break;
}
status = io.Status;
@ -365,6 +368,8 @@ fhandler_pipe::raw_read (void *ptr, size_t& len)
if (nbytes_now == 0)
break;
else if (select_sem)
ReleaseSemaphore (select_sem, get_obj_handle_count (select_sem), NULL);
}
ReleaseMutex (read_mtx);
if (evt)
@ -376,8 +381,6 @@ fhandler_pipe::raw_read (void *ptr, size_t& len)
}
else if (status == STATUS_THREAD_CANCELED)
pthread::static_cancel_self ();
if (select_sem && nbytes)
ReleaseSemaphore (select_sem, get_obj_handle_count (select_sem), NULL);
len = nbytes;
}
@ -472,6 +475,9 @@ fhandler_pipe_fifo::raw_write (const void *ptr, size_t len)
{
status = STATUS_THREAD_SIGNALED;
nbytes += io.Information;
if (select_sem && io.Information > 0)
ReleaseSemaphore (select_sem,
get_obj_handle_count (select_sem), NULL);
break;
}
status = io.Status;
@ -502,6 +508,8 @@ fhandler_pipe_fifo::raw_write (const void *ptr, size_t len)
if (nbytes_now == 0)
break;
else if (select_sem)
ReleaseSemaphore (select_sem, get_obj_handle_count (select_sem), NULL);
}
if (evt)
CloseHandle (evt);
@ -509,8 +517,6 @@ fhandler_pipe_fifo::raw_write (const void *ptr, size_t len)
set_errno (EINTR);
else if (status == STATUS_THREAD_CANCELED)
pthread::static_cancel_self ();
if (select_sem && nbytes)
ReleaseSemaphore (select_sem, get_obj_handle_count (select_sem), NULL);
return nbytes ?: -1;
}