[PATCH] Cygwin: pipe: Give up to use query_hdl for non-cygwin apps.

Takashi Yano takashi.yano@nifty.ne.jp
Sun Mar 3 10:21:09 GMT 2024


On Sun, 3 Mar 2024 10:34:44 +0100 (CET)
Johannes Schindelin wrote:
> Hi Takashi,
> 
> thank you for the fix! I can confirm that it addresses the problem
> demonstrated by the reproducer in that MSYS2-runtime ticket.

Thanks for testing!

> After noticing that we enumerate all the processes (which is an expensive
> operation) just to skip all of the non-Cygwin ones anyway, I wonder if it
> wouldn't be smarter to go through the internal list of cygpids and take it
> from there, skipping the `SystemProcessInformation` calls altogether.

Yeah, that makes sens. I'll submit v2 patch.

-- 
Takashi Yano <takashi.yano@nifty.ne.jp>


More information about the Cygwin-patches mailing list