RE: Pollset: Thread Safe & More

2004-09-23 Thread Ed Holyat
apr and was thread safe. Don't the pools use mutexing already, is this platform specific? -Original Message- From: Paul Querna [mailto:[EMAIL PROTECTED] Sent: Thursday, September 23, 2004 11:09 AM To: Joe Orton Cc: dev@apr.apache.org Subject: Re: Pollset: Thread Safe & More On

Re: Pollset: Thread Safe & More

2004-09-23 Thread Joe Orton
On Thu, Sep 23, 2004 at 09:08:59AM -0600, Paul Querna wrote: > On Thu, 2004-09-23 at 11:38 +0100, Joe Orton wrote: > > So I'm worried this patch is really solving the problem in the wrong > > place. The thing you are exporting here in the APR API, "is the > > apr_pollset_* interface thread-safe in

Re: Pollset: Thread Safe & More

2004-09-23 Thread Paul Querna
On Thu, 2004-09-23 at 11:38 +0100, Joe Orton wrote: > So I'm worried this patch is really solving the problem in the wrong > place. The thing you are exporting here in the APR API, "is the > apr_pollset_* interface thread-safe in _add and _remove", seems really > horrible. Either an API is thread

Re: Pollset: Thread Safe & More

2004-09-23 Thread Paul Querna
On Thu, 2004-09-23 at 11:38 +0100, Joe Orton wrote: > On Wed, Sep 22, 2004 at 08:57:24PM -0600, Paul Querna wrote: > > Attached is a Patch for apr_pollset_*. > > Changes Include: > > - Replace HAS_* with USE_* to remove some complex #ifdef stuff. > > - Partially thread safe (*) > > - Removes the l

Re: Pollset: Thread Safe & More

2004-09-23 Thread Joe Orton
On Wed, Sep 22, 2004 at 08:57:24PM -0600, Paul Querna wrote: > Attached is a Patch for apr_pollset_*. > Changes Include: > - Replace HAS_* with USE_* to remove some complex #ifdef stuff. > - Partially thread safe (*) > - Removes the limitation on the number of sockets that you can add to a > polls

Pollset: Thread Safe & More

2004-09-23 Thread Paul Querna
Attached is a Patch for apr_pollset_*. Changes Include: - Replace HAS_* with USE_* to remove some complex #ifdef stuff. - Partially thread safe (*) - Removes the limitation on the number of sockets that you can add to a pollset (**) * The patch allows multiple threads to concurrently call _add()