RE: [PATCH 1/2] xsk: do not use mmap_sem

2019-02-11 Thread Weiny, Ira
> > >> --- > > >> net/xdp/xdp_umem.c | 6 ++ > > >> 1 file changed, 2 insertions(+), 4 deletions(-) > > >> > > >> diff --git a/net/xdp/xdp_umem.c b/net/xdp/xdp_umem.c index > > >> 5ab236c5c9a5..25e1e76654a8 100644 > > >> --- a/net/xdp/xdp_umem.c > > >> +++ b/net/xdp/xdp_umem.c > > >> @@

Re: [PATCH 1/2] xsk: do not use mmap_sem

2019-02-11 Thread Dan Williams
[ add Ira ] On Mon, Feb 11, 2019 at 7:33 AM Daniel Borkmann wrote: > > [ +Dan ] > > On 02/07/2019 08:43 AM, Björn Töpel wrote: > > Den tors 7 feb. 2019 kl 06:38 skrev Davidlohr Bueso : > >> > >> Holding mmap_sem exclusively for a gup() is an overkill. > >> Lets replace the call for gup_fast()

Re: [PATCH 1/2] xsk: do not use mmap_sem

2019-02-11 Thread Daniel Borkmann
[ +Dan ] On 02/07/2019 08:43 AM, Björn Töpel wrote: > Den tors 7 feb. 2019 kl 06:38 skrev Davidlohr Bueso : >> >> Holding mmap_sem exclusively for a gup() is an overkill. >> Lets replace the call for gup_fast() and let the mm take >> it if necessary. >> >> Cc: David S. Miller >> Cc: Bjorn Topel

Re: [PATCH 1/2] xsk: do not use mmap_sem

2019-02-06 Thread Björn Töpel
Den tors 7 feb. 2019 kl 06:38 skrev Davidlohr Bueso : > > Holding mmap_sem exclusively for a gup() is an overkill. > Lets replace the call for gup_fast() and let the mm take > it if necessary. > > Cc: David S. Miller > Cc: Bjorn Topel > Cc: Magnus Karlsson > CC: net...@vger.kernel.org >

[PATCH 1/2] xsk: do not use mmap_sem

2019-02-06 Thread Davidlohr Bueso
Holding mmap_sem exclusively for a gup() is an overkill. Lets replace the call for gup_fast() and let the mm take it if necessary. Cc: David S. Miller Cc: Bjorn Topel Cc: Magnus Karlsson CC: net...@vger.kernel.org Signed-off-by: Davidlohr Bueso --- net/xdp/xdp_umem.c | 6 ++ 1 file