Re: [Qemu-devel] [PATCH v2 for-1.2 00/27] Suppress unused default drives

[Anthony Liguori]

Peter Maydell <address@hidden> writes:

> On 15 August 2012 20:58, Anthony Liguori <address@hidden> wrote:
>> Peter Maydell <address@hidden> writes:
>>> On 15 August 2012 20:25, Alexander Graf <address@hidden> wrote:
>>>> Nack from my POV. Too late for 1.2. Better get this in early for 1.3.
>>
>> No, it's not too late for 1.2.
>>
>> The release process is pretty clear.  Major features needed to be posted
>> before August 1st.  The late to get non-bug fixes in is today.
>
> Yes. I don't think that means "it's OK to send out a patchset that
> isn't just doing cosmetic fixes to a generally OK previous version
> on the day of feature freeze and expect that people will have time
> to review it".
>
> Basically, if this wasn't freeze day I'd expect a patchseries like this
> to sit on the list for at least three days or so for review.
>
>> This is not a major feature but more importantly, has gone through a few
>> revisions and has gotten positive review comments.
>
> Anything touching 50 files is "major feature" IMHO, and the first
> version of this patchset went out just 6 days ago.
>
> Short rc phases only work if people are reasonably sensible about
> not putting in enormous numbers of patches right at the freeze
> deadline, IMHO. This patchset doesn't meet the "value obtained
> for amount of disruption / quality of review" bar for me, is all.

http://ozlabs.org/~rusty/index.cgi/tech/2007-05-04.html

It's not that I disagree with you.  I think this is good feedback for a
series like this.

I just don't want people sending out single sentence "Nack" emails for
patch series just because we're at the end of the release cycle.  It
sets the wrong tone IMHO.

Regards,

Anthony Liguori

>
> -- PMM