Ok, thanks. I did not have the other one in place. 

> Am 06.02.2017 um 22:24 schrieb Stefan Priebe - Profihost AG 
> <[email protected]>:
> 
>> Am 06.02.2017 um 22:22 schrieb Stefan Priebe - Profihost AG:
>> Hi Stefan,
>> 
>> this one does not apply on top of yann's
>> mpm_event_listener_wakeup_bug57399_V7.patch patch.
> 
> i've now used his patch to mpm and yours for mod_http2.
> 
> Stefan
> 
>> 
>> Stefan
>>> Am 06.02.2017 um 15:34 schrieb Stefan Eissing:
>>> Yes, that was mixing the order. The attached v4 compiles and runs the h2 
>>> tests for me without errors.
>>> 
>>> 
>>> 
>>> 
>>> 
>>>> Am 06.02.2017 um 14:43 schrieb Yann Ylavic <[email protected]>:
>>>> 
>>>> On Mon, Feb 6, 2017 at 2:31 PM, Stefan Eissing
>>>> <[email protected]> wrote:
>>>>> Currently running some tests. Have crashes on the original patch in my 
>>>>> test suite. Fixed one, hunting for the next...
>>>> 
>>>> I think it comes from my change that creates slave connections from
>>>> master->pool (instead of mplx's), because now slave's pool is already
>>>> destroyed when 
>>>> h2_mplx_release_and_join()->task_destroy()->h2_slave_destroy()
>>>> is called (hence the crash).
>>>> 
>>>> I restored your original code in this new (attached) patch.
>>>> 
>>>> @s.priebe, would you test this one please?
>>>> <ptrans_and_slaves_allocator-v3.patch>
>>> 
>>> Stefan Eissing
>>> 
>>> <green/>bytes GmbH
>>> Hafenstrasse 16
>>> 48155 Münster
>>> www.greenbytes.de
>>> 

Reply via email to