<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Mar 9, 2018 at 11:59 AM, Dave Airlie <span dir="ltr"><<a href="mailto:airlied@gmail.com" target="_blank">airlied@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Working under embargoes does suck and quite a lot of people had seen<br>
this code but I think the process falls down a bit when Jason wrote<br>
(even tongue-in-cheek)<br>
<span class="">"You're 4.5 hours too late, I'm afraid.  I'd be happy to take some<br>
patches though. :-)"<br>
<br>
</span>I know in this case Dylan writing patches is probably the most<br>
expedient thing, but the response is off putting for post-push review.<br>
<br>
Review after something has landed is just as valid as review before.<br>
If someone spends the time<br>
reviewing something and it's already in the tree, you should still try<br>
and respond to the review in a timely fashion.<br>
<br>
If you want to push stuff quickly that has gotten lots of internal<br>
review, but you also need to realise you are have obligations to take<br>
post-review feedback more seriously :-)<span class="HOEnZb"><font color="#888888"><br>
</font></span></blockquote></div><br></div><div class="gmail_extra">Fair enough.<br></div></div>