Re: [Qemu-devel] [PATCH v6 00/26] Invert Endian bit in SPARCv9 MMU TTE

[Philippe Mathieu-Daudé]

On 8/7/19 2:41 PM, address@hidden wrote:
> On 8/7/19 8:37 PM, Philippe Mathieu-Daudé wrote:
> 
>> I'm confused I think I already reviewed various patches of your previous
> ​> series but don't see my Reviewed-by tags.​>
> Apologies Philippe! I am the confused one here =/
> 
> Will append.
> 
> Thank you very much for the reviews and qemu-devel newbie tips so far. I
> have felt very welcome.

Well for a newbie you did an impressive series!

The 'Reviewed-by' or 'Tested-by' tags help the maintainers to process
patches.

Since reviewing a series is time-consuming, if you iterate over a series
without changing some patchs, you should collect and amend the tags the
reviewers gave you, this way it helps them keep track of patches
reviewed and patches waiting for review.

In my case I find it very confuse when I look at a patch I already gave
my R-b tag and the tag is not here, I re-review the patch looking for
differences.

Often a reviewer asks for easy changes, and uses "with this changes:
R-b". If you addresses his comments you can then add his tag in the next
version.

If you split a reviewed patch in various, it is also OK to keep the tags
in all the splitted patches.

Regards,

Phil.