Re: [PATCH bpf-next 1/2] xsk: update rings for load-acquire/store-release semantics

2021-03-03 Thread Björn Töpel
On Tue, 2 Mar 2021 at 11:23, Toke Høiland-Jørgensen wrote: > > Björn Töpel writes: > > > On 2021-03-01 17:08, Toke Høiland-Jørgensen wrote: > >> Björn Töpel writes: > >> > >>> From: Björn Töpel > >>> > >>> Currently, the AF_XDP rings uses smp_{r,w,}mb() fences on the > >>> kernel-side. By updat

Re: [PATCH bpf-next 1/2] xsk: update rings for load-acquire/store-release semantics

2021-03-02 Thread Toke Høiland-Jørgensen
Björn Töpel writes: > On 2021-03-01 17:08, Toke Høiland-Jørgensen wrote: >> Björn Töpel writes: >> >>> From: Björn Töpel >>> >>> Currently, the AF_XDP rings uses smp_{r,w,}mb() fences on the >>> kernel-side. By updating the rings for load-acquire/store-release >>> semantics, the full barrier o

Re: [PATCH bpf-next 1/2] xsk: update rings for load-acquire/store-release semantics

2021-03-02 Thread Björn Töpel
On 2021-03-01 17:08, Toke Høiland-Jørgensen wrote: Björn Töpel writes: From: Björn Töpel Currently, the AF_XDP rings uses smp_{r,w,}mb() fences on the kernel-side. By updating the rings for load-acquire/store-release semantics, the full barrier on the consumer side can be replaced with i

Re: [PATCH bpf-next 1/2] xsk: update rings for load-acquire/store-release semantics

2021-03-01 Thread Toke Høiland-Jørgensen
Björn Töpel writes: > From: Björn Töpel > > Currently, the AF_XDP rings uses smp_{r,w,}mb() fences on the > kernel-side. By updating the rings for load-acquire/store-release > semantics, the full barrier on the consumer side can be replaced with > improved performance as a nice side-effect. > >

[PATCH bpf-next 1/2] xsk: update rings for load-acquire/store-release semantics

2021-03-01 Thread Björn Töpel
From: Björn Töpel Currently, the AF_XDP rings uses smp_{r,w,}mb() fences on the kernel-side. By updating the rings for load-acquire/store-release semantics, the full barrier on the consumer side can be replaced with improved performance as a nice side-effect. Note that this change does *not* req