You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by ja...@apache.org on 2018/06/27 21:00:38 UTC
svn commit: r1834544 - /httpd/httpd/branches/2.4.x/STATUS
Author: jailletc36
Date: Wed Jun 27 21:00:38 2018
New Revision: 1834544
URL: http://svn.apache.org/viewvc?rev=1834544&view=rev
Log:
propose
Modified:
httpd/httpd/branches/2.4.x/STATUS
Modified: httpd/httpd/branches/2.4.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/STATUS?rev=1834544&r1=1834543&r2=1834544&view=diff
==============================================================================
--- httpd/httpd/branches/2.4.x/STATUS (original)
+++ httpd/httpd/branches/2.4.x/STATUS Wed Jun 27 21:00:38 2018
@@ -211,6 +211,14 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
2.4.x patch: svn merge -c 1832092 ^/httpd/httpd/trunk .
+1: elukey
+ *) event: based on the commit log ("use a subpool of pchild since skiplist
+ allocations will happen across threads and are only protected from
+ other skiplist operations.", per sf's review comments), it sounds like this
+ should be backported.
+ trunk patch: http://svn.apache.org/r1588806
+ 2.4.x patch: http://home.apache.org/~jailletc36/backport_1588806.patch
+ +1:
+
PATCHES/ISSUES THAT ARE BEING WORKED
[ New entries should be added at the START of the list ]
Re: svn commit: r1834544 - /httpd/httpd/branches/2.4.x/STATUS
Posted by Yann Ylavic <yl...@gmail.com>.
On Thu, Jun 28, 2018 at 3:24 PM, Eric Covener <co...@gmail.com> wrote:
> On Thu, Jun 28, 2018 at 9:18 AM Yann Ylavic <yl...@gmail.com> wrote:
>>
>> On Wed, Jun 27, 2018 at 11:00 PM, <ja...@apache.org> wrote:
>> >
>> > + *) event: based on the commit log ("use a subpool of pchild since skiplist
>> > + allocations will happen across threads and are only protected from
>> > + other skiplist operations.", per sf's review comments), it sounds like this
>> > + should be backported.
>> > + trunk patch: http://svn.apache.org/r1588806
>> > + 2.4.x patch: http://home.apache.org/~jailletc36/backport_1588806.patch
>> > + +1:
>>
>> Hmm, timer_skiplist seems to be used only when g_timer_skiplist_mtx is
>> locked, am I missing something?
>
> IIUC the risk is colliding with someone else using pchild w/ similar
> locking around just their own use of it.
Oh I see, there shouldn't be much activity on pchild during runtime
but indeed apr_skiplist could race with legal things like
apr_pool_create(..., pchild).
I wonder if apr_skiplist shouldn't handle a subpool by itself though,
by the time this patch is simple enough so I'm +1.
Re: svn commit: r1834544 - /httpd/httpd/branches/2.4.x/STATUS
Posted by Eric Covener <co...@gmail.com>.
On Thu, Jun 28, 2018 at 9:18 AM Yann Ylavic <yl...@gmail.com> wrote:
>
> On Wed, Jun 27, 2018 at 11:00 PM, <ja...@apache.org> wrote:
> >
> > + *) event: based on the commit log ("use a subpool of pchild since skiplist
> > + allocations will happen across threads and are only protected from
> > + other skiplist operations.", per sf's review comments), it sounds like this
> > + should be backported.
> > + trunk patch: http://svn.apache.org/r1588806
> > + 2.4.x patch: http://home.apache.org/~jailletc36/backport_1588806.patch
> > + +1:
>
> Hmm, timer_skiplist seems to be used only when g_timer_skiplist_mtx is
> locked, am I missing something?
IIUC the risk is colliding with someone else using pchild w/ similar
locking around just their own use of it.
Re: svn commit: r1834544 - /httpd/httpd/branches/2.4.x/STATUS
Posted by Yann Ylavic <yl...@gmail.com>.
On Wed, Jun 27, 2018 at 11:00 PM, <ja...@apache.org> wrote:
>
> + *) event: based on the commit log ("use a subpool of pchild since skiplist
> + allocations will happen across threads and are only protected from
> + other skiplist operations.", per sf's review comments), it sounds like this
> + should be backported.
> + trunk patch: http://svn.apache.org/r1588806
> + 2.4.x patch: http://home.apache.org/~jailletc36/backport_1588806.patch
> + +1:
Hmm, timer_skiplist seems to be used only when g_timer_skiplist_mtx is
locked, am I missing something?