From 686e46ce714803f47d3183c954ceaf51976157cc Mon Sep 17 00:00:00 2001 From: Corinna Vinschen Date: Mon, 19 Aug 2024 11:51:14 +0200 Subject: [PATCH] Cygwin: pipe: do short writes only once in nonblocking case too If a nonblocking write requires short writes, just try it once as in the blocking case. After all, we are nonblocking, so don't loop unnecessarily. Fixes: 170e6badb621 ("Cygwin: pipe: improve writing when pipe buffer is almost full") Signed-off-by: Corinna Vinschen --- winsup/cygwin/fhandler/pipe.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/winsup/cygwin/fhandler/pipe.cc b/winsup/cygwin/fhandler/pipe.cc index 34b295c4d..852076ccc 100644 --- a/winsup/cygwin/fhandler/pipe.cc +++ b/winsup/cygwin/fhandler/pipe.cc @@ -574,8 +574,7 @@ fhandler_pipe_fifo::raw_write (const void *ptr, size_t len) len1 = avail & ~(PIPE_BUF - 1); else len1 = 1 << (31 - __builtin_clzl (avail)); - if (!is_nonblocking ()) - short_write_once = true; + short_write_once = true; } if (isclosed ()) /* A signal handler might have closed the fd. */ { -- 2.43.5