Am 06.02.2017 um 16:06 schrieb Stefan Eissing: > It does, at the end. Traversed the directories in different order it seems. *arg* sorry
> >> Am 06.02.2017 um 16:05 schrieb Stefan Priebe - Profihost AG >> <[email protected]>: >> >> Am 06.02.2017 um 16:03 schrieb Stefan Eissing: >>> I took Yann's v3 patch and changed a tiny bit in the h2. I just added a var >>> declaration in event.c without which it did not compile for me. >> >> But your patch does not contain the changes to the event mpm. That's why >> i ask. >> >> Stefan >> >>> -Stefan >>> >>>> Am 06.02.2017 um 15:41 schrieb Stefan Priebe - Profihost AG >>>> <[email protected]>: >>>> >>>> Hi, >>>> >>>> so i should test the mpm event part from Yann + your http2 part? >>>> >>>> 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 >>>>> >>> >>> Stefan Eissing >>> >>> <green/>bytes GmbH >>> Hafenstrasse 16 >>> 48155 Münster >>> www.greenbytes.de >>> > > Stefan Eissing > > <green/>bytes GmbH > Hafenstrasse 16 > 48155 Münster > www.greenbytes.de >
