Re: [PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-09-04 Thread Magnus Karlsson
On Wed, Sep 4, 2019 at 2:19 PM Yauheni Kaliuta wrote: > > Hi, Magnus! > > > On Wed, 4 Sep 2019 12:25:13 +0200, Magnus Karlsson wrote: > > On Wed, Sep 4, 2019 at 8:56 AM Yauheni Kaliuta > > wrote: > >> > >> Hi, Magnus! > >> > >> > On Wed, 4 Sep 2019 08:39:24 +0200, Magnus Karlsson

Re: [PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-09-04 Thread Yauheni Kaliuta
Hi, Magnus! > On Wed, 4 Sep 2019 12:25:13 +0200, Magnus Karlsson wrote: > On Wed, Sep 4, 2019 at 8:56 AM Yauheni Kaliuta > wrote: >> >> Hi, Magnus! >> >> > On Wed, 4 Sep 2019 08:39:24 +0200, Magnus Karlsson wrote: >> >> > On Wed, Sep 4, 2019 at 7:32 AM Yauheni Kaliuta >> >

Re: [PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-09-04 Thread Magnus Karlsson
On Wed, Sep 4, 2019 at 8:56 AM Yauheni Kaliuta wrote: > > Hi, Magnus! > > > On Wed, 4 Sep 2019 08:39:24 +0200, Magnus Karlsson wrote: > > > On Wed, Sep 4, 2019 at 7:32 AM Yauheni Kaliuta > > wrote: > >> > >> Hi, Magnus! > >> > >> > On Tue, 9 Apr 2019 08:44:13 +0200, Magnus Karlsso

Re: [PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-09-03 Thread Yauheni Kaliuta
Hi, Magnus! > On Wed, 4 Sep 2019 08:39:24 +0200, Magnus Karlsson wrote: > On Wed, Sep 4, 2019 at 7:32 AM Yauheni Kaliuta > wrote: >> >> Hi, Magnus! >> >> > On Tue, 9 Apr 2019 08:44:13 +0200, Magnus Karlsson wrote: >> >> > The use of smp_rmb() and smp_wmb() creates a Linux h

Re: [PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-09-03 Thread Magnus Karlsson
On Wed, Sep 4, 2019 at 7:32 AM Yauheni Kaliuta wrote: > > Hi, Magnus! > > > On Tue, 9 Apr 2019 08:44:13 +0200, Magnus Karlsson wrote: > > > The use of smp_rmb() and smp_wmb() creates a Linux header dependency > > on barrier.h that is uneccessary in most parts. This patch implements > > th

Re: [PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-09-03 Thread Yauheni Kaliuta
Hi, Magnus! > On Tue, 9 Apr 2019 08:44:13 +0200, Magnus Karlsson wrote: > The use of smp_rmb() and smp_wmb() creates a Linux header dependency > on barrier.h that is uneccessary in most parts. This patch implements > the two small defines that are needed from barrier.h. As a bonus, the

Re: [PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-04-09 Thread Daniel Borkmann
On 04/10/2019 12:28 AM, Georg Müller wrote: > Am 09.04.19 um 13:29 schrieb Magnus Karlsson: >> On Tue, Apr 9, 2019 at 11:11 AM Daniel Borkmann wrote: >>> On 04/09/2019 08:44 AM, Magnus Karlsson wrote: The use of smp_rmb() and smp_wmb() creates a Linux header dependency on barrier.h that

Re: [PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-04-09 Thread Georg Müller
Am 09.04.19 um 13:29 schrieb Magnus Karlsson: > On Tue, Apr 9, 2019 at 11:11 AM Daniel Borkmann wrote: >> >> On 04/09/2019 08:44 AM, Magnus Karlsson wrote: >>> The use of smp_rmb() and smp_wmb() creates a Linux header dependency >>> on barrier.h that is uneccessary in most parts. This patch implem

Re: [PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-04-09 Thread Magnus Karlsson
On Tue, Apr 9, 2019 at 11:11 AM Daniel Borkmann wrote: > > On 04/09/2019 08:44 AM, Magnus Karlsson wrote: > > The use of smp_rmb() and smp_wmb() creates a Linux header dependency > > on barrier.h that is uneccessary in most parts. This patch implements > > the two small defines that are needed fro

Re: [PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-04-09 Thread Daniel Borkmann
On 04/09/2019 08:44 AM, Magnus Karlsson wrote: > The use of smp_rmb() and smp_wmb() creates a Linux header dependency > on barrier.h that is uneccessary in most parts. This patch implements > the two small defines that are needed from barrier.h. As a bonus, the > new implementations are faster than

[PATCH bpf 2/2] libbpf: remove dependency on barrier.h in xsk.h

2019-04-08 Thread Magnus Karlsson
The use of smp_rmb() and smp_wmb() creates a Linux header dependency on barrier.h that is uneccessary in most parts. This patch implements the two small defines that are needed from barrier.h. As a bonus, the new implementations are faster than the default ones as they default to sfence and lfence