<div dir="ltr">Thanks for the reply, any updates for the review?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 29, 2020 at 11:45 PM Jiri Denemark <<a href="mailto:jdenemar@redhat.com">jdenemar@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi.<br>
<br>
On Wed, Apr 29, 2020 at 16:03:19 +0800, Zhenyu Zheng wrote:<br>
> Hi Jiri，<br>
> <br>
> Thanks alot for the help, I've updated the series to v4 and also attached<br>
> pipeline results for each patch as suggested.<br>
<br>
I explicitly said you don't have to send a new version just for that<br>
small issue...<br>
<br>
And I don't think pipeline results are needed for each patch, one result<br>
for the whole series should be enough to assure the code compiles on all<br>
tested architectures. Reviewers will usually check themselves whether<br>
the series can be compiled after each patch. I just wanted you to take<br>
the branch with all the patches and push it to gitlab, which will start<br>
a single CI pipeline for all the changes (just like in your v3), but the<br>
branch on gitlab will contain each patch of your series as is (while in<br>
v3 you squashed all patches into a single commit and pushed that).<br>
<br>
That said, all this is mainly a guidance for future submission. It's<br>
*not* a request for yet another respin of this series.<br>
<br>
In the meantime I found two AArch64 hosts with CPUs recognized by your<br>
changes so I'm playing with them.<br>
<br>
Jirka<br>
</blockquote></div>