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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2010/6/16 Markus Armbruster <armbru@xxxxxxxxxx>:
> Anthony Liguori <anthony@xxxxxxxxxxxxx> 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 majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux