> Am 25.08.2021 um 17:06 schrieb Yann Ylavic <ylavic....@gmail.com>:
>
> Thanks, looks good, it probably needs a backport to 2.4.x since the
> first travis failure was there (Build #1819).
> Let's see how it works after several runs before maybe.
+1
Let's see what Joe and Travis can do to wreck it.
> On Wed, Aug 25, 2021 at 5:02 PM ste...@eissing.org <ste...@eissing.org> wrote:
>>
>> It should be fixed now with r1892599, I believe.
>>
>>> Am 25.08.2021 um 13:53 schrieb Yann Ylavic <ylavic....@gmail.com>:
>>>
>>> On Wed, Aug 25, 2021 at 1:14 AM Travis CI <bui...@travis-ci.com> wrote:
>>>>
>>>> apache / httpd
>>>>
>>>> trunk
>>>>
>>>> Build #1831 was broken
>>>> 21 mins and 33 secs
>>>> Yann Ylavic
>>>> 243c5fa CHANGESET →
>>>>
>>>> mpm_{event,worker,prefork}: late stop of children processes on restart.
>>>
>>> (unrelated to this change, Build #1819 failed the same earlier).
>>>
>>> Here: https://app.travis-ci.com/github/apache/httpd/jobs/533578536#L4103
>>>
>>> It seems that when exiting a stream can be destroyed while in
>>> h2_mplx_s_task_done::s_task_done()::mst_check_data_for()
>>> when the mplx lock is released (thus stream->id faults).
>>>
>>> Should the caller(s) of mst_check_data_for() pass stream->id (under
>>> the lock) instead?
>>> This would avoid the fault but that's still a potentially destroyed
>>> stream being fifo'ed, though we are exiting so it might not matter..
>>>
>>> A better fix maybe, Stefan?
>>>
>>> Cheers;
>>> Yann.
>>