Re: [PATCH 08/23] powerpc/eeh: Refactor eeh_reset_pe_once()

2013-06-02 Thread Gavin Shan
On Sat, Jun 01, 2013 at 02:18:50PM +1000, Benjamin Herrenschmidt wrote: >On Thu, 2013-05-30 at 16:23 +0800, Gavin Shan wrote: >> The patch changes the criteria used to judge if the PE has been >> resetted successfully. We needn't the PE status is exactly equal >> to the combo: (EEH_STATE_MMIO_ACTIV

Re: [PATCH 08/23] powerpc/eeh: Refactor eeh_reset_pe_once()

2013-05-31 Thread Benjamin Herrenschmidt
On Thu, 2013-05-30 at 16:23 +0800, Gavin Shan wrote: > The patch changes the criteria used to judge if the PE has been > resetted successfully. We needn't the PE status is exactly equal > to the combo: (EEH_STATE_MMIO_ACTIVE | EEH_STATE_DMA_ACTIVE). The comment above doesn't mean anything :-) I a

[PATCH 08/23] powerpc/eeh: Refactor eeh_reset_pe_once()

2013-05-30 Thread Gavin Shan
The patch changes the criteria used to judge if the PE has been resetted successfully. We needn't the PE status is exactly equal to the combo: (EEH_STATE_MMIO_ACTIVE | EEH_STATE_DMA_ACTIVE). Signed-off-by: Gavin Shan --- arch/powerpc/platforms/pseries/eeh.c |4 ++-- 1 files changed, 2 insert