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

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 ?

                        -------------------------
                        |                       |
                   [ 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 :-)

Cheers,
Ben.