[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU
On 12.10.21 18:32, Ian Jackson wrote:
Hi Ian
Oleksandr writes ("Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions
for DomU"):
On 11.10.21 20:48, Oleksandr Tyshchenko wrote:
Hi Ian, Stefano
+ LOG(DEBUG, "Extended region %u: %#"PRIx64"->%#"PRIx64"\n",
If this appears to be the last version, may I please ask to remove
unneeded "\n" in the message above while committing? Thank you.
I think Stefano will be committing this but: personally I don't like
the "edit on commit" workflow. Committers are already a bottleneck
and it is easy to make uncontrolled mistakes. I find the most
convenient workflow to be to acquire a git branch from somewhere and
commit that, so if you provide a git branch with the acks folded in
and this kind of minor fix included, that would be ideal. Ie,
git-request-merge format or something like it.
Yes, I will provide a git branch later on today.
Thanks,
Ian.
--
Regards,
Oleksandr Tyshchenko
|