Re: [PATCH 1/2] pci: cleanup failover sanity check

2021-02-09 Thread Laurent Vivier
CC: Michael On 06/02/2021 13:39, Laurent Vivier wrote: > Commit a1190ab628 has added a "allow_unplug_during_migration = true" at > the end of the main "if" block, so it is not needed to set it anymore > in the previous checking. > > Remove it, to have only sub-ifs that check for needed conditions

Re: [PATCH 1/2] pci: cleanup failover sanity check

2021-02-08 Thread Jens Freimann
On Sat, Feb 06, 2021 at 01:39:54PM +0100, Laurent Vivier wrote: Commit a1190ab628 has added a "allow_unplug_during_migration = true" at the end of the main "if" block, so it is not needed to set it anymore in the previous checking. Remove it, to have only sub-ifs that check for needed conditions

Re: [PATCH 1/2] pci: cleanup failover sanity check

2021-02-06 Thread Laurent Vivier
On 06/02/2021 13:39, Laurent Vivier wrote: > Commit a1190ab628 has added a "allow_unplug_during_migration = true" at > the end of the main "if" block, so it is not needed to set it anymore > in the previous checking. > > Remove it, to have only sub-ifs that check for needed conditions and exit > i

[PATCH 1/2] pci: cleanup failover sanity check

2021-02-06 Thread Laurent Vivier
Commit a1190ab628 has added a "allow_unplug_during_migration = true" at the end of the main "if" block, so it is not needed to set it anymore in the previous checking. Remove it, to have only sub-ifs that check for needed conditions and exit if one fails. Fixes: 4f5b6a05a4e7 ("pci: add option for