Re: [Qemu-devel] Re: KVM call minutes for June 15

[Yoshiaki Tamura]

2010/6/16 Markus Armbruster <address@hidden>:
> Anthony Liguori <address@hidden> writes:
>
>> On 06/15/2010 10:41 AM, Christoph Hellwig wrote:
>>> On Tue, Jun 15, 2010 at 08:18:12AM -0700, Chris Wright wrote:
>>>
>>>> KVM/qemu patches
>>>> - patch rate is high, documentation is low, review is low
>>>> - patches need to include better descriptions and documentation
>>>>    - will slow down patch writers
>>>>    - will make it easier for patch reviewers
>>>>
>>> What is the qemu patch review policy anyway?
>>
>> We don't really have a coherent policy.  Suggestions for improvement
>> are always appreciated.
>>
>>>    There are no
>>> "Reviewed-by:" included in the actual commits,
>>
>> Reviewed-by/Ack-by's are pretty helpful for me.  In terms of including
>> them in commit messages, if there's a strong feeling that that would
>> be helpful then it's something I can look at doing but it also
>> requires a fair bit of manual work during commit.
>
> Can't hurt reviewer motivation.  Could it be automated?  Find replies,
> extract tags.  If you want your acks to be picked up, you better make
> sure your References header works, and your tags are formatted
> correctly.

How about letting the submitter to include acked-by or reviewed-by
manually and repost?
It wouldn't make the maintainers busy.  Although the traffic would
increase, it would show the gratitude from submitter to the reviewer.

Thanks,

Yoshi

>
> [...]
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to address@hidden
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>