Re: [PATCH v4 0/7] Pegasos2 fixes and audio output support

[Daniel P . Berrangé]

On Tue, Feb 28, 2023 at 04:05:30PM +0100, BALATON Zoltan wrote:
> On Mon, 27 Feb 2023, Philippe Mathieu-Daudé wrote:
> > On 27/2/23 18:47, BALATON Zoltan wrote:
> > > On Mon, 27 Feb 2023, Bernhard Beschow wrote:
> > > > Unfortunately my patches had changes merged in. This now makes it hard 
> > > > to
> > > > show what really changed (spoiler: nothing that affects behavior).
> > > > 
> > > > As you probably noticed in the "resend" version of this iteration I 
> > > > split
> > > > off a patch introducing the priq properties. It belongs to the sub 
> > > > series
> > > > of the Pegasos2 IRQ fixes which appear unnecessary to me, so I don't 
> > > > want
> > > > to show up in `git blame` as the author of any of these changes. I
> > > > attributed it to you because this was really your change which
> > > > I'm not even
> > > > sure is legal.
> > > > 
> > > > Let's avoid such complications by keeping our series separate.
> > > 
> > > Let's cool down a bit. Philippe took some of the sm501 patches in
> > > his giant pull request (and a lot of your patches too) now so I'll
> > > wait until that lands and hope to get some review for the remaining
> > > patches too. Once that pull req is merged I'll rebase the remaining
> > > patches and resubmit the series also adding changes for reasonable
> > > review comments I get by then.
> > 
> > I'm sorry it took me so long, I was expecting these patches to be picked
> > up by other maintainers but everybody is very busy. I know you'll need
> 
> You have no reason to apologise really, you did a great job merging all the
> patches. I was thinking that because as you say every maintainer is very
> busy now and we also had CI outage for a few weeks should we consider
> extending the date until the freeze by one or two weeks? That would allow
> people to relax a bit and be able to consolidate and merge all still pending
> patches. Postponing the 8.0 release one or two weeks is probably better than
> missing a lot of changes until the next release in September. We'd still aim
> for the original freeze date but if we fail to meet that it would be more
> convenient to know there could be a possibility for extending it. But make
> it clear that this is only for this one time because of CI outage and
> additional maintainer load caused by that so not something that should be
> done regularly but under current circumstances I would consider it.

There's no need to change the release schedule IMHO. Subsystem maintainers
should continue to send pull requests as normal. Peter is still processing
PRs, albeit at a lower rate with adhoc CI. From the soft freeze POV what
matters is just that the PRs are posted on the mailing list before the
deadline. If they're posted in time, they're still valid for inclusion in
the release. Our CI allowance is reset at the end of today anyway.

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|