<div class="gmail_quote">On Mon, Nov 12, 2012 at 10:40 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 <<a href="mailto:1352287938-7247-2-git-send-email-martin@varnish-software.com">1352287938-7247-2-git-send-email-martin@varnish-software.com</a>>, Mart<br>
<div class="im">in Blix Grydeland writes:<br>
>This is to give persistent stevedores a chance to clean up their ban<br>
>lists without having to worry about the lists changing under them<br>
>(normally the ban callbacks are called under the ban mutex, but not so<br>
>during exit).<br>
<br>
</div>I don't have a problem with the idea, but I don't like having BAN_Insert()<br>
return a failure we don't know how to handle.  I think BAN_Insert() should<br>
take ownership of the ban even when it fails, but returning a status code<br>
is probably a good idea, provided it gets used for something somewhere.<br></blockquote><div><br></div><div>Changed to make BAN_Insert() always take ownership of the ban, freeing it on failure. BAN_Insert() will still return the status code.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I think it should be used in the CLI::ban case, so that a cluster-controller<br>
can keep track of which bans have been entered on which varnish instances.<br></blockquote><div><br></div><div>CLI will get the error message.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
>Also remove the duplicate drop tail bans code. That is now only done<br>
>in the main thread loop.<br>
<br>
</div>This breaks r01030.vtc so it should probably be done in its own commit.<br></blockquote><div><br></div><div>This has been moved to a separate commit. Though I do not see how the change affects r01030.vtc, nor have I been able to make that test case fail.</div>
<div><br></div><div>Martin</div></div><div><br></div>-- <br><div><table border="0" cellpadding="0" cellspacing="0" style="font-size:12px;line-height:1.5em;font-family:'Helvetica Neue',Arial,sans-serif;color:rgb(102,102,102);width:550px;border-top-width:1px;border-top-style:solid;border-top-color:rgb(238,238,238);border-bottom-width:1px;border-bottom-style:solid;border-bottom-color:rgb(238,238,238);margin-top:20px;padding-top:5px;padding-bottom:5px">
<tbody><tr><td width="100"><a href="http://varnish-software.com" target="_blank"><img src="http://www.varnish-software.com/static/media/logo-email.png"></a><span></span><span></span></td><td><strong style="font-size:14px;color:rgb(34,34,34)">Martin Blix Grydeland</strong><br>
Senior Developer | Varnish Software AS<br>Cell: +47 21 98 92 60<br><span style="font-weight:bold">We Make Websites Fly!</span></td></tr></tbody></table></div><br>