Hi Ambroz,
- FILE_SYNCHRONOUS_IO_NONALERT ); + read_timeout ? 0 : FILE_SYNCHRONOUS_IO_NONALERT ); This looks incorrect. You probably want to compare to -1, not zero. Zero means reads must not block at all, and -1 means read must wait forever (not sure what zero timeout would work as expected with your patch, but it didn't anyway and I think it maybe is broken in another place) Also, please attach patches with disposition inline, not attachment (git can do that) - this make review somehow easier.
Regards, Andrey Ambroz Bizjak wrote:
Hi,
Changelog: server: fix async read on mailslot ------------------------------------------------------------------------