On Wed, Jun 26, 2013 at 09:57:26AM +1000, Benjamin Herrenschmidt wrote:

On Wed, 2013-06-26 at 07:49 +0800, Gavin Shan wrote:

quoted

It's something like the followings. For ER on PE#0, we will have
PE with type of EEH_PE_BUS marked as isolated, instead of the
one with EEH_PE_PHB.


        [ EEH_PE_PHB] <---> [ EEH_PE_PHB] <---> [ EEH_PE_PHB]
                                  |
                            [ EEH_PE_BUS ] PE#0
                                  |

So we actually have two PEs here ? One real (PE#0) and one imaginary
(PHB PE) with no PE# associated ?

Yes, The (PHB PE) is actually a container to all PEs under the
PHB ;-)

quoted

                        -------------------------
                        |                       |
                   [ EEH_PE_BUS ] PE#1     [ EEH_PE_BUS] PE#2

quoted

I would either not bother and collect the FF's, or make this specific
to fence and only fence.

I'd like to keep it specific to fenced PHB and it's already be
that :-)

Thanks,
Gavin