On Mon, Jul 30, 2018 at 12:20 PM, Sinan Kaya <ok...@kernel.org> wrote: > +netdev, > > On 7/30/2018 9:45 AM, Alexander Duyck wrote: >> >> I haven't had a chance to work on this much myself. My understanding >> is that igb has had the barriers updated, but I don't think any of the >> other drivers have been worked over yet. > > > Unfortunately, I have recently changed jobs and I no longer have the > hardware to test my changes. I thought that you wanted to handle this > yourself. > > I haven't seen any follow ups. I wanted to double check.
As I said so far igb has been the only one updated, and that was done by a third party: 73017f4e051c8 igb: Use dma_wmb() instead of wmb() before doorbell writes > I worked with several architecture leads on 4.17. All architectures > support the updated barrier semantics now. It is time to clean up the > network drivers. Thanks for that update. As I said for now igb has the barriers updated. The idea being that igb is the test vehicle for this so if we go a kernel version or so without triggering any issues then we can follow up with the other drivers. The other thing we have to keep in mind is that unlike many other NICs we have to also deal with emulations of our devices (e1000 and e1000e) that may rely on certain barriers being used to enforce things like SMP synchronization between CPUs, so we have to be careful as we roll this out. Thanks. - Alex
