<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/12/11 Liu Yuan <span dir="ltr"><<a href="mailto:namei.unix@gmail.com" target="_blank">namei.unix@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, Dec 11, 2013 at 10:59:27AM +0800, Robin Dong wrote:<br>
> Why not adding the check in gateway_forward_request() ? I think we should<br>
> check the copy number before send forward the request out.<br>
<br>
</div>I think it is better to check the request as early as possible, no? This is<br>
pointless to run the request down to gateway_forward_request(), which is queued<br>
on the work queue and might need some time to be called and then return the<br>
error to the client.<br>
<br>
What is the benefit to check inside the gateway_forward_request instead of the<br>
approch of this patch?<br></blockquote><div><br></div><div>Sorry£¬I misunderstand the gateway_forward_request() and queue_gateway_request()£¬your code is correct.</div><div><br></div><div>Reviewed-by: Robin Dong <<a href="mailto:sanbai@taobao.com">sanbai@taobao.com</a>></div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888">Yuan<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>--<br>Best Regard<br>Robin Dong
</div></div>