Re: [PATCH] Fix two bugs in the limit of large numbers of sockets:

2017-11-07 Thread Erik Bray
On Nov 7, 2017 16:36, "Corinna Vinschen" wrote: Erik, On Nov 7 16:11, Corinna Vinschen wrote: > On Nov 7 14:44, Erik M. Bray wrote: > > * Fix the maximum number of sockets allowed in the session to 2048, > > instead of making it relative to sizeof(wsa_event). > > > > The original choice of

[PATCH] pread() returns non-zero if read beyond EOF, because NtReadFile returns EOF status but doesn't set information to 0. Need reset io status block before NtReadFile is called, so that pread() w

2017-11-07 Thread Xiaofeng Liu via cygwin-patches
--- winsup/cygwin/fhandler_disk_file.cc |    1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/winsup/cygwin/fhandler_disk_file.cc b/winsup/cygwin/fhandler_disk_file.ccindex bc8fead..525cb32 100644--- a/winsup/cygwin/fhandler_disk_file.cc+++ b/winsup/cygwin/fhandler_disk_file.cc

Re: [PATCH] Fix two bugs in the limit of large numbers of sockets:

2017-11-07 Thread Corinna Vinschen
Erik, On Nov 7 16:11, Corinna Vinschen wrote: > On Nov 7 14:44, Erik M. Bray wrote: > > * Fix the maximum number of sockets allowed in the session to 2048, > > instead of making it relative to sizeof(wsa_event). > > > > The original choice of 2048 was in order to fit the wsa_events array >

Re: [PATCH] Fix two bugs in the limit of large numbers of sockets:

2017-11-07 Thread Jon Turney
On 07/11/2017 13:44, Erik M. Bray wrote: * Fix the maximum number of sockets allowed in the session to 2048, instead of making it relative to sizeof(wsa_event). The original choice of 2048 was in order to fit the wsa_events array in the .cygwin_dll_common shared section, but there is st

Re: [PATCH] Fix two bugs in the limit of large numbers of sockets:

2017-11-07 Thread Corinna Vinschen
On Nov 7 14:44, Erik M. Bray wrote: > * Fix the maximum number of sockets allowed in the session to 2048, > instead of making it relative to sizeof(wsa_event). > > The original choice of 2048 was in order to fit the wsa_events array > in the .cygwin_dll_common shared section, but there is s

[PATCH] Fix two bugs in the limit of large numbers of sockets:

2017-11-07 Thread Erik M. Bray
* Fix the maximum number of sockets allowed in the session to 2048, instead of making it relative to sizeof(wsa_event). The original choice of 2048 was in order to fit the wsa_events array in the .cygwin_dll_common shared section, but there is still enough room to grow there to have 2048 s

Re: PATCH: pread() return non-zero if read beyond end of file

2017-11-07 Thread Corinna Vinschen
On Nov 6 21:54, Xiaofeng Liu via cygwin-patches wrote: > try again, after saving the diff in linux first. > git diff                                                                      >                        diff --git a/winsup/cygwin/fhandler_disk_file.cc > b/winsup/cygwin/fhandler_disk_fil