<div dir="ltr">Attached.<br><br>OK?<br><br><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 30, 2014 at 9:01 AM, Poul-Henning Kamp <span dir="ltr"><<a href="mailto:phk@phk.freebsd.dk" target="_blank">phk@phk.freebsd.dk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">--------<br>
In message <CAJV_h0bR-h=i0e-GKwZrd5fE9q-1AsF=_sJ=gFJQ=<a href="mailto:6CbiF0kow@mail.gmail.com">6CbiF0kow@mail.gmail.com</a>><br>
, Federico Schwindt writes:<br>
<br>
Two things, on trivial-ish and the other serious-ish.<br>
<br>
The trivial thing is that I think we should put the actual<br>
rollback in VRT_Rollback() (give it an VCL_HTTP argument) so that<br>
std.rollback just wraps that.<br>
<br>
The other thing is that I'm not even close to being convinced that<br>
resetting the workspace is safe anymore, and I have no idea how I<br>
can convince myself that it is, in a world full of VMODs.<br>
<br>
We may have to give up on that, and only rollback the struct http.<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20<br>
phk@FreeBSD.ORG         | TCP/IP since RFC 956<br>
FreeBSD committer       | BSD since 4.3-tahoe<br>
Never attribute to malice what can adequately be explained by incompetence.<br>
</div></div></blockquote></div><br></div>