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.
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. >