[Qemu-devel] [PATCHv2 0/7] spapr: DRC cleanups (part III)

David Gibson posted 7 patches 8 years, 5 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170607032018.19588-1-david@gibson.dropbear.id.au
Test FreeBSD passed
Test checkpatch passed
Test docker passed
Test s390x passed
hw/ppc/spapr.c             |  30 ++---
hw/ppc/spapr_drc.c         | 269 ++++++++++++++++++++-------------------------
hw/ppc/spapr_pci.c         |  72 +++++-------
hw/ppc/trace-events        |   5 +-
include/hw/ppc/spapr_drc.h |  30 ++---
5 files changed, 171 insertions(+), 235 deletions(-)
[Qemu-devel] [PATCHv2 0/7] spapr: DRC cleanups (part III)
Posted by David Gibson 8 years, 5 months ago
A third batch of cleanups to the DRC code.  This continues to clear
away relatively simple cruft, to get a clearer look at the fundamental
state handling.

Changes since v1:
  * Some comment updates suggested by Mike Roth
  * Changed approach to the get_name cleanup, using generated on the
    fly names, instead of externally assigned names
  * Added in some cleanups to hotplug code in spapr_pci.c

David Gibson (7):
  spapr: Clean up DR entity sense handling
  spapr: Don't misuse DR-indicator in spapr_recover_pending_dimm_state()
  spapr: Clean up RTAS set-indicator
  spapr: Clean up handling of DR-indicator
  spapr: Change DRC attach & detach methods to functions
  spapr: Fold spapr_phb_{add,remove}_pci_device() into their only
    callers
  spapr: Rework DRC name handling

 hw/ppc/spapr.c             |  30 ++---
 hw/ppc/spapr_drc.c         | 269 ++++++++++++++++++++-------------------------
 hw/ppc/spapr_pci.c         |  72 +++++-------
 hw/ppc/trace-events        |   5 +-
 include/hw/ppc/spapr_drc.h |  30 ++---
 5 files changed, 171 insertions(+), 235 deletions(-)

-- 
2.9.4


Re: [Qemu-devel] [PATCHv2 0/7] spapr: DRC cleanups (part III)
Posted by Michael Roth 8 years, 5 months ago
Quoting David Gibson (2017-06-06 22:20:11)
> A third batch of cleanups to the DRC code.  This continues to clear
> away relatively simple cruft, to get a clearer look at the fundamental
> state handling.
> 
> Changes since v1:
>   * Some comment updates suggested by Mike Roth
>   * Changed approach to the get_name cleanup, using generated on the
>     fly names, instead of externally assigned names
>   * Added in some cleanups to hotplug code in spapr_pci.c

Series:

Reviewed-by: Michael Roth <mdroth@linux.vnet.ibm.com>
Acked-by: Michael Roth <mdroth@linux.vnet.ibm.com>

> 
> David Gibson (7):
>   spapr: Clean up DR entity sense handling
>   spapr: Don't misuse DR-indicator in spapr_recover_pending_dimm_state()
>   spapr: Clean up RTAS set-indicator
>   spapr: Clean up handling of DR-indicator
>   spapr: Change DRC attach & detach methods to functions
>   spapr: Fold spapr_phb_{add,remove}_pci_device() into their only
>     callers
>   spapr: Rework DRC name handling
> 
>  hw/ppc/spapr.c             |  30 ++---
>  hw/ppc/spapr_drc.c         | 269 ++++++++++++++++++++-------------------------
>  hw/ppc/spapr_pci.c         |  72 +++++-------
>  hw/ppc/trace-events        |   5 +-
>  include/hw/ppc/spapr_drc.h |  30 ++---
>  5 files changed, 171 insertions(+), 235 deletions(-)
> 
> -- 
> 2.9.4
> 


Re: [Qemu-devel] [PATCHv2 0/7] spapr: DRC cleanups (part III)
Posted by David Gibson 8 years, 5 months ago
On Wed, Jun 07, 2017 at 06:22:16PM -0500, Michael Roth wrote:
> Quoting David Gibson (2017-06-06 22:20:11)
> > A third batch of cleanups to the DRC code.  This continues to clear
> > away relatively simple cruft, to get a clearer look at the fundamental
> > state handling.
> > 
> > Changes since v1:
> >   * Some comment updates suggested by Mike Roth
> >   * Changed approach to the get_name cleanup, using generated on the
> >     fly names, instead of externally assigned names
> >   * Added in some cleanups to hotplug code in spapr_pci.c
> 
> Series:
> 
> Reviewed-by: Michael Roth <mdroth@linux.vnet.ibm.com>
> Acked-by: Michael Roth <mdroth@linux.vnet.ibm.com>

Thanks.  Patches merged into ppc-for-2.10.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson