On Sat, Sep 09, 2017 at 05:05:46PM +0200, Greg Kurz wrote: > Before resuming the huge work on PHB hotplug, here are some patches > that maybe worth to apply. > > Patches 1 to 3 are basic improvements.
I've applied these to ppc-for-2.11. > Patch 4 and 5 may be a bit controversial. Everywhere in the spapr > code where we build an FDT portion, libfdt failures cause QEMU to > exit, even on hotplug paths. Only spapr_pci doesn't do that and > propagates the error instead. My understanding is that a failure > when building the FDT is likely to happen because of a bug in QEMU. Still looking at these. > > Hence the choice to convert spapr_pci to do like the others. We may > even consider changing _FDT() to abort() instead of exit(). > > Alternatively, if libfdt failures shouldn't be necessarily fatal, > especially on post-realize paths, then we should probably introduce > an _FDT_ERR() helper to propagate errors. And use it in may places > where we currently terminate QEMU: memory hotplug, CPU hotplug, CAS, > machine reset... > -- 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
signature.asc
Description: PGP signature