[PATCH 0/6] hw/ppc: Snapshot support for several ppc devices

Caleb Schlossin posted 6 patches 1 month, 3 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20251211220926.2865972-1-calebs@linux.ibm.com
Maintainers: Nicholas Piggin <npiggin@gmail.com>, Aditya Gupta <adityag@linux.ibm.com>, Glenn Miles <milesg@linux.ibm.com>, Alistair Francis <alistair@alistair23.me>
There is a newer version of this series
hw/ppc/pnv_adu.c             | 12 +++++++++++
hw/ppc/pnv_chiptod.c         | 38 +++++++++++++++++++++++++++++++++++
hw/ppc/pnv_core.c            | 22 ++++++++++++++++++++
hw/ppc/pnv_i2c.c             | 11 ++++++++++
hw/ppc/pnv_lpc.c             | 39 ++++++++++++++++++++++++++++++++++++
hw/ssi/pnv_spi.c             | 27 +++++++++++++++++++++++++
include/hw/ppc/pnv_chiptod.h |  2 ++
7 files changed, 151 insertions(+)
[PATCH 0/6] hw/ppc: Snapshot support for several ppc devices
Posted by Caleb Schlossin 1 month, 3 weeks ago
Add snapshot support for several ppc devices for the powernv machines.

* Adding LPC, ADU, SPI, I2C, core, and chipTOD

No specific ordering of the patches as this is ongoing development to
support the PowerVM team. Additional patches for other (non-ppc) devices
will be coming in separate patch submissions.

Thanks,
Caleb

Michael Kowal (1):
  hw/ppc: Add VMSTATE information for LPC model

Caleb Schlossin (2):
  hw/ppc: Add pnv_spi vmstate support
  hw/ppc: Add pnv_i2c vmstate support

Angelo Jaramillo (3):
  hw/ppc: pnv_adu.c added vmstate support
  hw/ppc: pnv_core.c add vmstate support
  hw/ppc: pnv_chiptod.c add vmstate support
  

 hw/ppc/pnv_adu.c             | 12 +++++++++++
 hw/ppc/pnv_chiptod.c         | 38 +++++++++++++++++++++++++++++++++++
 hw/ppc/pnv_core.c            | 22 ++++++++++++++++++++
 hw/ppc/pnv_i2c.c             | 11 ++++++++++
 hw/ppc/pnv_lpc.c             | 39 ++++++++++++++++++++++++++++++++++++
 hw/ssi/pnv_spi.c             | 27 +++++++++++++++++++++++++
 include/hw/ppc/pnv_chiptod.h |  2 ++
 7 files changed, 151 insertions(+)

-- 
2.47.3
Re: [PATCH 0/6] hw/ppc: Snapshot support for several ppc devices
Posted by Aditya Gupta 1 month, 3 weeks ago
Hello Caleb,

On 25/12/11 04:09PM, Caleb Schlossin wrote:
> Add snapshot support for several ppc devices for the powernv machines.
> 
> * Adding LPC, ADU, SPI, I2C, core, and chipTOD
> 
> No specific ordering of the patches as this is ongoing development to
> support the PowerVM team. Additional patches for other (non-ppc) devices
> will be coming in separate patch submissions.

Thanks for the patches, can you tell more background on this ? Because
afaik PowerNV doesn't seem to have vmstate mostly, are these enough to
store/restore full PowerNV context ?

The patches look good overall, though I am yet to review in detail.
Some minor nits, LPC only sets vmstate for Power10's LPC, and many
devices vmstate don't have pre/post load handlers, assuming just the
vmstate info is enough in those cases ?

Thanks,
- Aditya G

> 
> Thanks,
> Caleb
> 
> Michael Kowal (1):
>   hw/ppc: Add VMSTATE information for LPC model
> 
> Caleb Schlossin (2):
>   hw/ppc: Add pnv_spi vmstate support
>   hw/ppc: Add pnv_i2c vmstate support
> 
> Angelo Jaramillo (3):
>   hw/ppc: pnv_adu.c added vmstate support
>   hw/ppc: pnv_core.c add vmstate support
>   hw/ppc: pnv_chiptod.c add vmstate support
>   
> 
>  hw/ppc/pnv_adu.c             | 12 +++++++++++
>  hw/ppc/pnv_chiptod.c         | 38 +++++++++++++++++++++++++++++++++++
>  hw/ppc/pnv_core.c            | 22 ++++++++++++++++++++
>  hw/ppc/pnv_i2c.c             | 11 ++++++++++
>  hw/ppc/pnv_lpc.c             | 39 ++++++++++++++++++++++++++++++++++++
>  hw/ssi/pnv_spi.c             | 27 +++++++++++++++++++++++++
>  include/hw/ppc/pnv_chiptod.h |  2 ++
>  7 files changed, 151 insertions(+)
> 
> -- 
> 2.47.3
>
Re: [PATCH 0/6] hw/ppc: Snapshot support for several ppc devices
Posted by Caleb Schlossin 1 month, 3 weeks ago
Aditya,

My team has gotten snapshot support working with PowerNV and I'm now working on trying to upstream the smaller (less complicated) devices. Other, more complicated, support will be send upstream separately (PHB, XIVE, etc.).

Regarding LPC, all we had been focused on testing was Power10. If you'd like me to add support for Power8 and Power9, I can work on that in the next patch set.

- Caleb

On 12/15/25 12:08 AM, Aditya Gupta wrote:
> Hello Caleb,
> 
> On 25/12/11 04:09PM, Caleb Schlossin wrote:
>> Add snapshot support for several ppc devices for the powernv machines.
>>
>> * Adding LPC, ADU, SPI, I2C, core, and chipTOD
>>
>> No specific ordering of the patches as this is ongoing development to
>> support the PowerVM team. Additional patches for other (non-ppc) devices
>> will be coming in separate patch submissions.
> 
> Thanks for the patches, can you tell more background on this ? Because
> afaik PowerNV doesn't seem to have vmstate mostly, are these enough to
> store/restore full PowerNV context ?
> 
> The patches look good overall, though I am yet to review in detail.
> Some minor nits, LPC only sets vmstate for Power10's LPC, and many
> devices vmstate don't have pre/post load handlers, assuming just the
> vmstate info is enough in those cases ?
> 
> Thanks,
> - Aditya G
> 
>>
>> Thanks,
>> Caleb
>>
>> Michael Kowal (1):
>>   hw/ppc: Add VMSTATE information for LPC model
>>
>> Caleb Schlossin (2):
>>   hw/ppc: Add pnv_spi vmstate support
>>   hw/ppc: Add pnv_i2c vmstate support
>>
>> Angelo Jaramillo (3):
>>   hw/ppc: pnv_adu.c added vmstate support
>>   hw/ppc: pnv_core.c add vmstate support
>>   hw/ppc: pnv_chiptod.c add vmstate support
>>   
>>
>>  hw/ppc/pnv_adu.c             | 12 +++++++++++
>>  hw/ppc/pnv_chiptod.c         | 38 +++++++++++++++++++++++++++++++++++
>>  hw/ppc/pnv_core.c            | 22 ++++++++++++++++++++
>>  hw/ppc/pnv_i2c.c             | 11 ++++++++++
>>  hw/ppc/pnv_lpc.c             | 39 ++++++++++++++++++++++++++++++++++++
>>  hw/ssi/pnv_spi.c             | 27 +++++++++++++++++++++++++
>>  include/hw/ppc/pnv_chiptod.h |  2 ++
>>  7 files changed, 151 insertions(+)
>>
>> -- 
>> 2.47.3
>>