Re: [Qemu-devel] [PATCH 6/6] spapr: Clean up DRC set_isolation_state() path

2017-06-19 Thread David Gibson
On Mon, Jun 19, 2017 at 05:52:27PM -0500, Michael Roth wrote: > Quoting David Gibson (2017-06-08 00:09:30) > > There are substantial differences in the various paths through > > set_isolation_state(), both for setting to ISOLATED versus UNISOLATED > > state and for logical versus physical DRCs. >

Re: [Qemu-devel] [PATCH 6/6] spapr: Clean up DRC set_isolation_state() path

2017-06-19 Thread Michael Roth
Quoting David Gibson (2017-06-08 00:09:30) > There are substantial differences in the various paths through > set_isolation_state(), both for setting to ISOLATED versus UNISOLATED > state and for logical versus physical DRCs. > > So, split the set_isolation_state() method into isolate() and

Re: [Qemu-devel] [PATCH 6/6] spapr: Clean up DRC set_isolation_state() path

2017-06-19 Thread Greg Kurz
On Thu, 8 Jun 2017 15:09:30 +1000 David Gibson wrote: > There are substantial differences in the various paths through > set_isolation_state(), both for setting to ISOLATED versus UNISOLATED > state and for logical versus physical DRCs. > > So, split the

[Qemu-devel] [PATCH 6/6] spapr: Clean up DRC set_isolation_state() path

2017-06-07 Thread David Gibson
There are substantial differences in the various paths through set_isolation_state(), both for setting to ISOLATED versus UNISOLATED state and for logical versus physical DRCs. So, split the set_isolation_state() method into isolate() and unisolate() methods, and give it different implementations