because, as in the case of pretendgz, they may be chaging the next
level up.

e00023 passes now

https://github.com/varnishcache/varnish-cache/issues/3028

fiddling the filters manually really does not make much sense, in
particular not for our test-cases which are supposed to closely match
varnish-cache serial esi behavior

I _suspect_ this caused issues with make failing in dependency tracking
for me, like:

make[2]: Entering directory
'/home/slink/Devel/varnish-git/libvdp-pesi/src'
  CC       vdp_pesi.lo
mv: cannot stat '.deps/vdp_pesi.Tpo': No such file or directory
Makefile:700: recipe for target 'vdp_pesi.lo' failed
make[2]: *** [vdp_pesi.lo] Error 1

but better safe than sorry

it's too early in set_delivered() because set_closed() calls
set_delivered() on a nexus when all children are already devivered, but
that can happen from within the nexus node's fsm.

we will add extra finis when it's safe in a later commit

in the varnish-cache variant of the code this works because not VDP_Push
is used but the internal push to the parent

When subreqs still used the topreq's VDPs, we could not do this, but now
that we properly layer the VDPs again, we can (and should)

waiting for the boc happened at the wrong place

***  v1   debug|Assert error in vped_gzgz_init(), foreign/cache_esi_deliver.c line 163:
***  v1   debug|  Condition(foo->start > 0 && foo->start < foo->olen * 8) not true.

... because of T_FINAL.

Because the parent's thread (except for the root) will terminate
eventually, this is also all good, except for the case that we spend all
available threads on T_FINAL.

Leave the hard problems for later, *sigh*

for esi_level = 0, we still need to close the root in the vdp, but for
levels down, we want to make sure that the vdps are right before
anything is happening below our node

the producer should have the T_FINAL lock before making the node
available by releasing the tree lock

those VSL(SLT_Debug, 0, ...) trigger false positives while testing

or at least I fail to see how it should with just one req->vdc

...why I could have meant by topa-rent

This restores src/foreign/cache_esi_deliver.c to basically the code from
varnish-cache and avoids any unlayered pushes / crc.

Never mind that this currently never happens anyway.