On Fri, 2008-03-14 at 21:20 +0900, Ishizaki Kou wrote: > This patch splits cell io-workaround code into spider-pci dependent > code and a generic part, and also adds interfaces to the generic > io-workaround mechanism. > > Signed-off-by: Kou Ishizaki <[EMAIL PROTECTED]> > ---
Hi ! I noticed that you add a second level of indirection. We already have one going to the workarounds in the first place, so that looks a bit too much to my taste. I may have missed something in your patch but if the workarounds are specific to a given bridge, they may as well set the top level indirections once straight to the right workarounds. Also, my understanding is that we are pretty much using the same bridge on the cell blades, so is there any way that can be consolidated rather than having function pointers ? I'll try to have a closer look next week, but I'm a bit worried by having all IO go through 2 level of function pointers, the PPE isn't very good at it and this will slow things down more than they already are. Cheers, Ben. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
