[PULL ppc] powernv queue

[PULL ppc] powernv queue

[Cédric Le Goater]

The following changes since commit bccabb3a5d60182645c7749e89f21a9ff307a9eb:

  Update version for v6.1.0-rc2 release (2021-08-04 16:56:14 +0100)

are available in the Git repository at:

  https://github.com/legoater/qemu/ tags/pull-powernv-20210810

for you to fetch changes up to 91a6b62df830d51f2b6b2ea00b3c92231d0ba9dc:

  ppc/pnv: update skiboot to commit 820d43c0a775. (2021-08-10 14:18:51 +0200)

----------------------------------------------------------------
ppc/pnv: update skiboot image

----------------------------------------------------------------
Cédric Le Goater (1):
      ppc/pnv: update skiboot to commit 820d43c0a775.

 pc-bios/skiboot.lid | Bin 1667280 -> 2528128 bytes
 roms/skiboot        |   2 +-
 2 files changed, 1 insertion(+), 1 deletion(-)

Re: [PULL ppc] powernv queue

[Peter Maydell]

On Tue, 10 Aug 2021 at 13:46, Cédric Le Goater <clg@kaod.org> wrote:
>
> The following changes since commit bccabb3a5d60182645c7749e89f21a9ff307a9eb:
>
>   Update version for v6.1.0-rc2 release (2021-08-04 16:56:14 +0100)
>
> are available in the Git repository at:
>
>   https://github.com/legoater/qemu/ tags/pull-powernv-20210810
>
> for you to fetch changes up to 91a6b62df830d51f2b6b2ea00b3c92231d0ba9dc:
>
>   ppc/pnv: update skiboot to commit 820d43c0a775. (2021-08-10 14:18:51 +0200)
>
> ----------------------------------------------------------------
> ppc/pnv: update skiboot image
>
> ----------------------------------------------------------------
> Cédric Le Goater (1):
>       ppc/pnv: update skiboot to commit 820d43c0a775.

That skiboot changelog lists a massive set of changes, which
is correspondingly a larger risk that something in there turns
out to be a must-fix-for-release regression.

For future release cycles, can you try to get guest bios
blob updates in earlier rather than on the day we're trying
to get rc3 out, please?

thanks
-- PMM

Re: [PULL ppc] powernv queue

[Philippe Mathieu-Daudé]

On 8/10/21 5:17 PM, Peter Maydell wrote:
> On Tue, 10 Aug 2021 at 13:46, Cédric Le Goater <clg@kaod.org> wrote:
>>
>> The following changes since commit bccabb3a5d60182645c7749e89f21a9ff307a9eb:
>>
>>   Update version for v6.1.0-rc2 release (2021-08-04 16:56:14 +0100)
>>
>> are available in the Git repository at:
>>
>>   https://github.com/legoater/qemu/ tags/pull-powernv-20210810
>>
>> for you to fetch changes up to 91a6b62df830d51f2b6b2ea00b3c92231d0ba9dc:
>>
>>   ppc/pnv: update skiboot to commit 820d43c0a775. (2021-08-10 14:18:51 +0200)
>>
>> ----------------------------------------------------------------
>> ppc/pnv: update skiboot image
>>
>> ----------------------------------------------------------------
>> Cédric Le Goater (1):
>>       ppc/pnv: update skiboot to commit 820d43c0a775.
> 
> That skiboot changelog lists a massive set of changes, which
> is correspondingly a larger risk that something in there turns
> out to be a must-fix-for-release regression.

There might be a misunderstanding here, per
https://www.mail-archive.com/qemu-devel@nongnu.org/msg828857.html
I understand this pull request is targetting David PPC tree,
not the mainstream one. David first queued this patch for 6.2,
and later confirmed by Cédric:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg829160.html

> For future release cycles, can you try to get guest bios
> blob updates in earlier rather than on the day we're trying
> to get rc3 out, please?
> 
> thanks
> -- PMM
> 

Re: [PULL ppc] powernv queue

[Peter Maydell]

On Tue, 10 Aug 2021 at 16:32, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> On 8/10/21 5:17 PM, Peter Maydell wrote:
> > On Tue, 10 Aug 2021 at 13:46, Cédric Le Goater <clg@kaod.org> wrote:
> >>
> >> The following changes since commit bccabb3a5d60182645c7749e89f21a9ff307a9eb:
> >>
> >>   Update version for v6.1.0-rc2 release (2021-08-04 16:56:14 +0100)
> >>
> >> are available in the Git repository at:
> >>
> >>   https://github.com/legoater/qemu/ tags/pull-powernv-20210810
> >>
> >> for you to fetch changes up to 91a6b62df830d51f2b6b2ea00b3c92231d0ba9dc:
> >>
> >>   ppc/pnv: update skiboot to commit 820d43c0a775. (2021-08-10 14:18:51 +0200)
> >>
> >> ----------------------------------------------------------------
> >> ppc/pnv: update skiboot image
> >>
> >> ----------------------------------------------------------------
> >> Cédric Le Goater (1):
> >>       ppc/pnv: update skiboot to commit 820d43c0a775.
> >
> > That skiboot changelog lists a massive set of changes, which
> > is correspondingly a larger risk that something in there turns
> > out to be a must-fix-for-release regression.
>
> There might be a misunderstanding here, per
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg828857.html
> I understand this pull request is targetting David PPC tree,
> not the mainstream one. David first queued this patch for 6.2,
> and later confirmed by Cédric:
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg829160.html

Mmm. Also as Dan pointed out on irc firmware updates ought to
be honouring the usual softfreeze rules. So I'm going to err
on the safe side and won't be applying this.

thanks
-- PMM

Re: [PULL ppc] powernv queue

[Cédric Le Goater]

On 8/10/21 5:33 PM, Peter Maydell wrote:
> On Tue, 10 Aug 2021 at 16:32, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>>
>> On 8/10/21 5:17 PM, Peter Maydell wrote:
>>> On Tue, 10 Aug 2021 at 13:46, Cédric Le Goater <clg@kaod.org> wrote:
>>>>
>>>> The following changes since commit bccabb3a5d60182645c7749e89f21a9ff307a9eb:
>>>>
>>>>   Update version for v6.1.0-rc2 release (2021-08-04 16:56:14 +0100)
>>>>
>>>> are available in the Git repository at:
>>>>
>>>>   https://github.com/legoater/qemu/ tags/pull-powernv-20210810
>>>>
>>>> for you to fetch changes up to 91a6b62df830d51f2b6b2ea00b3c92231d0ba9dc:
>>>>
>>>>   ppc/pnv: update skiboot to commit 820d43c0a775. (2021-08-10 14:18:51 +0200)
>>>>
>>>> ----------------------------------------------------------------
>>>> ppc/pnv: update skiboot image
>>>>
>>>> ----------------------------------------------------------------
>>>> Cédric Le Goater (1):
>>>>       ppc/pnv: update skiboot to commit 820d43c0a775.
>>>
>>> That skiboot changelog lists a massive set of changes, which
>>> is correspondingly a larger risk that something in there turns
>>> out to be a must-fix-for-release regression.
>>
>> There might be a misunderstanding here, per
>> https://www.mail-archive.com/qemu-devel@nongnu.org/msg828857.html
>> I understand this pull request is targetting David PPC tree,
>> not the mainstream one. David first queued this patch for 6.2,
>> and later confirmed by Cédric:
>> https://www.mail-archive.com/qemu-devel@nongnu.org/msg829160.html
> 
> Mmm. Also as Dan pointed out on irc firmware updates ought to
> be honouring the usual softfreeze rules. So I'm going to err
> on the safe side and won't be applying this.

Yes. I should have added a 'ppc-for-6.2' prefix. 

Sorry about that. 

C. 

Re: [PULL ppc] powernv queue

[Peter Maydell]

On Tue, 10 Aug 2021 at 20:36, Cédric Le Goater <clg@kaod.org> wrote:
> Yes. I should have added a 'ppc-for-6.2' prefix.

If this wasn't intended for master, can you make sure the
cover letter includes the "PULL SUBSYSTEM whatever" subject
tag and has "not for master" in the body, as noted in
https://wiki.qemu.org/Contribute/SubmitAPullRequest
please? That way my email filters will ignore it.

thanks
-- PMM

Re: [PULL ppc] powernv queue

[Cédric Le Goater]

On 8/11/21 10:15 AM, Peter Maydell wrote:
> On Tue, 10 Aug 2021 at 20:36, Cédric Le Goater <clg@kaod.org> wrote:
>> Yes. I should have added a 'ppc-for-6.2' prefix.
> 
> If this wasn't intended for master, can you make sure the
> cover letter includes the "PULL SUBSYSTEM whatever" subject
> tag and has "not for master" in the body, as noted in
> https://wiki.qemu.org/Contribute/SubmitAPullRequest
> please? That way my email filters will ignore it.

ok. That's what I needed to know. Thanks and sorry again for the
false alarm.

C.