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

Re: [PATCH 1/2] posix_fadvise() *returns* error codes but does not set errno

2017-11-03 Thread Erik Bray
On Thu, Nov 2, 2017 at 6:10 PM, Corinna Vinschen wrote: > On Nov 2 16:45, Erik M. Bray wrote: >> Also updates the fhandler_*::fadvise implementations to adhere to the same >> semantics. > > Both patches pushed. Thanks!

Re: [PATCH 2/2] posix_fallocate() *returns* error codes but does not set errno

2017-11-02 Thread Erik Bray
On Thu, Nov 2, 2017 at 4:06 PM, Corinna Vinschen wrote: > Hi Erik, > > On Nov 2 15:15, Erik M. Bray wrote: >> diff --git a/winsup/cygwin/fhandler_disk_file.cc >> b/winsup/cygwin/fhandler_disk_file.cc >> index f46e355..9d5ec30 100644 >> --- a/winsup/cygwin/fhandler_disk_file.cc >> +++

Re: [PATCH] posix_fadvise() *returns* error codes but does not set errno

2017-11-02 Thread Erik Bray
On Thu, Nov 2, 2017 at 3:58 PM, Corinna Vinschen wrote: > Hi Eric, > > On Nov 2 14:26, Erik M. Bray wrote: >> Also updates the fhandler_*::fadvise implementations to adhere to the same >> semantics. > > Good catch. I have just some style nits. > >> --- >>

Re: [PATCH] Ensure that a blocking send() on a socket returns (with success) if a signal is handled mid-transition and SA_RESTART is not set.

2017-06-07 Thread Erik Bray
On Tue, Jun 6, 2017 at 4:23 PM, Corinna Vinschen wrote: > Hi Erik, > > [vacation-induced late reply] No problem--you had been silent here for a while so I guessed you were on vacation. Welcome back! > On May 11 16:05, Erik M. Bray wrote: >> --- >>

Re: [PATCH 0/3] Updated patches for /proc//environ

2017-01-10 Thread Erik Bray
On Tue, Jan 10, 2017 at 4:41 PM, Corinna Vinschen <corinna-cyg...@cygwin.com> wrote: > On Jan 10 16:02, Erik Bray wrote: >> From: "Erik M. Bray" <erik.b...@lri.fr> >> >> Updated versions of the patch set originally submitted at >> https://cygwin.co

[PATCH 2/3] Add a _pinfo.environ() method analogous to _pinfo.cmdline(), and others.

2017-01-10 Thread Erik Bray
From: Erik M. Bray Returns the process's environment concatenated into a single block of null-terminated strings, along with the length of the environment block. Adds an associated PICOM_ENVIRON commune_process handler. --- winsup/cygwin/pinfo.cc | 83

[PATCH 3/3] Add a /proc//environ proc file handler, analogous to /proc//cmdline.

2017-01-10 Thread Erik Bray
From: Erik M. Bray --- winsup/cygwin/fhandler_process.cc | 22 ++ 1 file changed, 22 insertions(+) diff --git a/winsup/cygwin/fhandler_process.cc b/winsup/cygwin/fhandler_process.cc index 5f530a2..bbb44fa 100644 --- a/winsup/cygwin/fhandler_process.cc +++

[PATCH 1/3] Move the core environment parsing of environ_init into a new win32env_to_cygenv function.

2017-01-10 Thread Erik Bray
From: Erik M. Bray win32env_to_cygwenv handles converting wchar to char and some other minor taks. Optionally it handles converting any paths in variables to posix paths. This will be useful for implementing /proc//environ --- winsup/cygwin/environ.cc | 84

Re: [PATCH 2/3] Add a _pinfo.environ() method analogous to _pinfo.cmdline(), and others.

2017-01-10 Thread Erik Bray
On Mon, Jan 9, 2017 at 3:58 PM, Corinna Vinschen wrote: > On Jan 5 18:39, Erik M. Bray wrote: >> diff --git a/winsup/cygwin/pinfo.cc b/winsup/cygwin/pinfo.cc >> index 1ce6809..a3e376c 100644 >> --- a/winsup/cygwin/pinfo.cc >> +++ b/winsup/cygwin/pinfo.cc >> @@ -653,8

Re: [PATCH 0/3] Add support for /proc//environ

2017-01-10 Thread Erik Bray
On Mon, Jan 9, 2017 at 3:43 PM, Corinna Vinschen wrote: > Hi Erik, > > On Jan 5 18:39, erik.m.b...@gmail.com wrote: >> From: "Erik M. Bray" >> >> Per this discussion started in this thread: >> https://cygwin.com/ml/cygwin/2016-11/msg00205.html >> >>

Re: [PATCH] Return the correct value for getsockopt(SO_REUSEADDR) after setting setsockopt(SO_REUSEADDR, 1).

2017-01-09 Thread Erik Bray
On Mon, Jan 9, 2017 at 5:58 PM, Corinna Vinschen <corinna-cyg...@cygwin.com> wrote: > On Jan 9 17:36, Erik Bray wrote: >> --- >> winsup/cygwin/net.cc | 8 >> 1 file changed, 8 insertions(+) >> >> diff --git a/winsup/cygwin/net.cc b/winsup/cygwin/

[PATCH] Return the correct value for getsockopt(SO_REUSEADDR) after setting setsockopt(SO_REUSEADDR, 1).

2017-01-09 Thread Erik Bray
From: Erik M. Bray --- winsup/cygwin/net.cc | 8 1 file changed, 8 insertions(+) diff --git a/winsup/cygwin/net.cc b/winsup/cygwin/net.cc index e4805d3..b02f9e3 100644 --- a/winsup/cygwin/net.cc +++ b/winsup/cygwin/net.cc @@ -925,6 +925,14 @@ cygwin_getsockopt (int

Re: Return the correct value for sysconf(_SC_PAGESIZE)

2016-11-17 Thread Erik Bray
On Thu, Nov 17, 2016 at 10:54 AM, Corinna Vinschen <corinna-cyg...@cygwin.com> wrote: > On Nov 16 15:51, Erik Bray wrote: >> On Wed, Nov 16, 2016 at 3:01 PM, Eric Blake <ebl...@redhat.com> wrote: >> > On 11/16/2016 07:56 AM, Erik Bray wrote: >> > >&g

Re: Return the correct value for sysconf(_SC_PAGESIZE)

2016-11-16 Thread Erik Bray
On Wed, Nov 16, 2016 at 3:01 PM, Eric Blake <ebl...@redhat.com> wrote: > On 11/16/2016 07:56 AM, Erik Bray wrote: > >>> There is no good reason to use the non-POSIXy page size. It doesn't >>> help you in the least for any pagesize-related functionality. Mmap >

Re: Return the correct value for sysconf(_SC_PAGESIZE)

2016-11-16 Thread Erik Bray
On Tue, Nov 15, 2016 at 5:19 PM, Corinna Vinschen <corinna-cyg...@cygwin.com> wrote: > On Nov 15 16:47, Erik Bray wrote: >> On Tue, Nov 15, 2016 at 3:58 PM, Corinna Vinschen >> <corinna-cyg...@cygwin.com> wrote: >> > On Nov 15 14:51, Erik Bray wrote: >> &

Re: Return the correct value for sysconf(_SC_PAGESIZE)

2016-11-15 Thread Erik Bray
On Tue, Nov 15, 2016 at 3:58 PM, Corinna Vinschen <corinna-cyg...@cygwin.com> wrote: > On Nov 15 14:51, Erik Bray wrote: >> Greetings, >> >> Currently sysconf(_SC_PAGESIZE) returns the value of >> wincap.allocation_granularity()--a change I *think* had to have be

Return the correct value for sysconf(_SC_PAGESIZE)

2016-11-15 Thread Erik Bray
Greetings, Currently sysconf(_SC_PAGESIZE) returns the value of wincap.allocation_granularity()--a change I *think* had to have been made by mistake in https://cygwin.com/git/gitweb.cgi?p=newlib-cygwin.git;a=commit;f=winsup/cygwin/sysconf.cc;h=177dc6c7f6d0608ef6540fd997d9b444e324cae2 There's no