mbox series

[v2,0/3] Series to implement suspend for ppc/spapr

Message ID 20190722053215.20808-1-npiggin@gmail.com (mailing list archive)
Headers show
Series Series to implement suspend for ppc/spapr | expand

Message

Nicholas Piggin July 22, 2019, 5:32 a.m. UTC
Hi, this series is rebased on top of the qmp event fix, and takes
Paolo's suggestion to implement ->wakeup for i386 before adding
ppc, which makes it much nicer.

If the first two patches can be agreed on initially and merged, I
can take the third patch through the ppc list after that.

Thanks,
Nick

Nicholas Piggin (3):
  machine: Add wakeup method to MachineClass
  i386: use machine class ->wakeup method
  spapr: Implement ibm,suspend-me

 hw/i386/pc.c           |  8 ++++++++
 hw/ppc/spapr.c         |  7 +++++++
 hw/ppc/spapr_rtas.c    | 32 ++++++++++++++++++++++++++++++++
 include/hw/boards.h    |  1 +
 include/hw/ppc/spapr.h |  3 ++-
 vl.c                   | 16 +++++++++++++++-
 6 files changed, 65 insertions(+), 2 deletions(-)

Comments

David Gibson July 22, 2019, 8:39 a.m. UTC | #1
On Mon, Jul 22, 2019 at 03:32:12PM +1000, Nicholas Piggin wrote:
> Hi, this series is rebased on top of the qmp event fix, and takes
> Paolo's suggestion to implement ->wakeup for i386 before adding
> ppc, which makes it much nicer.
> 
> If the first two patches can be agreed on initially and merged, I
> can take the third patch through the ppc list after that.

LGTM.  Now, what tree(s) do we merge this through?

> 
> Thanks,
> Nick
> 
> Nicholas Piggin (3):
>   machine: Add wakeup method to MachineClass
>   i386: use machine class ->wakeup method
>   spapr: Implement ibm,suspend-me
> 
>  hw/i386/pc.c           |  8 ++++++++
>  hw/ppc/spapr.c         |  7 +++++++
>  hw/ppc/spapr_rtas.c    | 32 ++++++++++++++++++++++++++++++++
>  include/hw/boards.h    |  1 +
>  include/hw/ppc/spapr.h |  3 ++-
>  vl.c                   | 16 +++++++++++++++-
>  6 files changed, 65 insertions(+), 2 deletions(-)
>
Paolo Bonzini July 22, 2019, 11:20 a.m. UTC | #2
On 22/07/19 10:39, David Gibson wrote:
> On Mon, Jul 22, 2019 at 03:32:12PM +1000, Nicholas Piggin wrote:
>> Hi, this series is rebased on top of the qmp event fix, and takes
>> Paolo's suggestion to implement ->wakeup for i386 before adding
>> ppc, which makes it much nicer.
>>
>> If the first two patches can be agreed on initially and merged, I
>> can take the third patch through the ppc list after that.
> 
> LGTM.  Now, what tree(s) do we merge this through?

I have just spotted a volunteer!

Acked-by: Paolo Bonzini <pbonzini@redhat.com>

Paolo

>>
>> Thanks,
>> Nick
>>
>> Nicholas Piggin (3):
>>   machine: Add wakeup method to MachineClass
>>   i386: use machine class ->wakeup method
>>   spapr: Implement ibm,suspend-me
>>
>>  hw/i386/pc.c           |  8 ++++++++
>>  hw/ppc/spapr.c         |  7 +++++++
>>  hw/ppc/spapr_rtas.c    | 32 ++++++++++++++++++++++++++++++++
>>  include/hw/boards.h    |  1 +
>>  include/hw/ppc/spapr.h |  3 ++-
>>  vl.c                   | 16 +++++++++++++++-
>>  6 files changed, 65 insertions(+), 2 deletions(-)
>>
>
David Gibson July 22, 2019, 11:42 a.m. UTC | #3
On Mon, Jul 22, 2019 at 01:20:08PM +0200, Paolo Bonzini wrote:
> On 22/07/19 10:39, David Gibson wrote:
> > On Mon, Jul 22, 2019 at 03:32:12PM +1000, Nicholas Piggin wrote:
> >> Hi, this series is rebased on top of the qmp event fix, and takes
> >> Paolo's suggestion to implement ->wakeup for i386 before adding
> >> ppc, which makes it much nicer.
> >>
> >> If the first two patches can be agreed on initially and merged, I
> >> can take the third patch through the ppc list after that.
> > 
> > LGTM.  Now, what tree(s) do we merge this through?
> 
> I have just spotted a volunteer!

Well, merging an i386 patch through the ppc tree is kinda weird, but
yeah, I can do that.
> 
> Acked-by: Paolo Bonzini <pbonzini@redhat.com>
> 
> Paolo
> 
> >>
> >> Thanks,
> >> Nick
> >>
> >> Nicholas Piggin (3):
> >>   machine: Add wakeup method to MachineClass
> >>   i386: use machine class ->wakeup method
> >>   spapr: Implement ibm,suspend-me
> >>
> >>  hw/i386/pc.c           |  8 ++++++++
> >>  hw/ppc/spapr.c         |  7 +++++++
> >>  hw/ppc/spapr_rtas.c    | 32 ++++++++++++++++++++++++++++++++
> >>  include/hw/boards.h    |  1 +
> >>  include/hw/ppc/spapr.h |  3 ++-
> >>  vl.c                   | 16 +++++++++++++++-
> >>  6 files changed, 65 insertions(+), 2 deletions(-)
> >>
> > 
>
David Gibson July 23, 2019, 3:40 a.m. UTC | #4
On Mon, Jul 22, 2019 at 01:20:08PM +0200, Paolo Bonzini wrote:
> On 22/07/19 10:39, David Gibson wrote:
> > On Mon, Jul 22, 2019 at 03:32:12PM +1000, Nicholas Piggin wrote:
> >> Hi, this series is rebased on top of the qmp event fix, and takes
> >> Paolo's suggestion to implement ->wakeup for i386 before adding
> >> ppc, which makes it much nicer.
> >>
> >> If the first two patches can be agreed on initially and merged, I
> >> can take the third patch through the ppc list after that.
> > 
> > LGTM.  Now, what tree(s) do we merge this through?
> 
> I have just spotted a volunteer!
> 
> Acked-by: Paolo Bonzini <pbonzini@redhat.com>

I've put this in the ppc-for-4.2 tree.

> 
> Paolo
> 
> >>
> >> Thanks,
> >> Nick
> >>
> >> Nicholas Piggin (3):
> >>   machine: Add wakeup method to MachineClass
> >>   i386: use machine class ->wakeup method
> >>   spapr: Implement ibm,suspend-me
> >>
> >>  hw/i386/pc.c           |  8 ++++++++
> >>  hw/ppc/spapr.c         |  7 +++++++
> >>  hw/ppc/spapr_rtas.c    | 32 ++++++++++++++++++++++++++++++++
> >>  include/hw/boards.h    |  1 +
> >>  include/hw/ppc/spapr.h |  3 ++-
> >>  vl.c                   | 16 +++++++++++++++-
> >>  6 files changed, 65 insertions(+), 2 deletions(-)
> >>
> > 
>