[Skiboot] [PATCH v8 2/2] Add SBE driver support

Benjamin Herrenschmidt benh at kernel.crashing.org
Sat Apr 21 11:27:01 AEST 2018


On Fri, 2018-04-20 at 12:20 +0530, Vasant Hegde wrote:
> > 
> > Generally I still prefer if we handle one bit at a time amd loop & re-
> > read & retry, as discussed.
> 
> I thought about that.. But I don't see much benefits. In both cases we have to 
> take care of
> ordering (like process ACK before response). And we will end up clearing each 
> bit separately. So we will do additional xscom_write to clear bits.
> 
> If you really want that method I can change the code. Please let me know.

The worry is that we may complete the message before we've processed
all the bits, thus dropping the lock, allowing new messages in and
possibly other opetations to happen. It might be ok, but ...

Ben.



More information about the Skiboot mailing list