I have been too optimistic. While this solved one of my issues, it didn't take pipe into account.
A pipe must hold a ref to a backend until the session is actually closed, because it will increment the relevant counters. After playing around with the backend's refcount, trying to inc/decrement it in all code paths, I didn't come to a satisfying result. I still have 2 test cases (1 pipe-related) where a backend hot swap (outside of the cli thread) fails. This patch is no good, it doesn't solve anything. Best Regards, Dridi On Mon, Jun 29, 2015 at 7:23 PM, Dridi Boukelmoune <[email protected]> wrote: > Making progress on dynamic backends testing, I found a glitch in the > max_connections handling, among other issues. > > Patch discussed with fgs and phk, not yet reviewed. > > Best Regards, > Dridi _______________________________________________ varnish-dev mailing list [email protected] https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev
