On Tue, Jun 15, 2010 at 11:41:53AM -0400, 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? There are no > "Reviewed-by:" included in the actual commits, and the requirement > for a positive review also seem to vary a lot, up to the point that > some commiters commit code that has never hit a public mailing list > before. > This is indeed something very useful that should be encouraged. Depending on the patch and the persons that have reviewed/acked it, I commit some patches only after a very quick look. -- Aurelien Jarno GPG: 1024D/F1BCDB73 aurelien@xxxxxxxxxxx http://www.aurel32.net -- 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