Re: [RFC PATCH v2 02/10] udp: implement GRO for plain UDP sockets.

2018-10-22 Thread Willem de Bruijn
> > > > > +static struct sk_buff *udp_gro_receive_segment(struct list_head *head, > > > + struct sk_buff *skb) > > > +{ > > > + struct udphdr *uh = udp_hdr(skb); > > > + struct sk_buff *pp = NULL; > > > + struct udphdr *uh2; > > > + struct sk_buff *p

Re: [RFC PATCH v2 02/10] udp: implement GRO for plain UDP sockets.

2018-10-22 Thread Willem de Bruijn
On Mon, Oct 22, 2018 at 6:13 AM Paolo Abeni wrote: > > On Sun, 2018-10-21 at 16:06 -0400, Willem de Bruijn wrote: > > On Fri, Oct 19, 2018 at 10:30 AM Paolo Abeni wrote: > > > > > > This is the RX counterpart of commit bec1f6f69736 ("udp: generate gso > > > with UDP_SEGMENT"). When UDP_GRO is ena

Re: [RFC PATCH v2 02/10] udp: implement GRO for plain UDP sockets.

2018-10-22 Thread Paolo Abeni
On Mon, 2018-10-22 at 13:24 +0200, Steffen Klassert wrote: > On Fri, Oct 19, 2018 at 04:25:12PM +0200, Paolo Abeni wrote: > > > > +#define UDO_GRO_CNT_MAX 64 > > Maybe better UDP_GRO_CNT_MAX? Oops, typo. Yes, sure, will address in the next iteration. > Btw. do we really need this explicit limi

Re: [RFC PATCH v2 02/10] udp: implement GRO for plain UDP sockets.

2018-10-22 Thread Steffen Klassert
On Fri, Oct 19, 2018 at 04:25:12PM +0200, Paolo Abeni wrote: > > +#define UDO_GRO_CNT_MAX 64 Maybe better UDP_GRO_CNT_MAX? Btw. do we really need this explicit limit? We should not get more than 64 packets during one napi poll cycle. > +static struct sk_buff *udp_gro_receive_segment(struct lis

Re: [RFC PATCH v2 02/10] udp: implement GRO for plain UDP sockets.

2018-10-22 Thread Paolo Abeni
On Sun, 2018-10-21 at 16:06 -0400, Willem de Bruijn wrote: > On Fri, Oct 19, 2018 at 10:30 AM Paolo Abeni wrote: > > > > This is the RX counterpart of commit bec1f6f69736 ("udp: generate gso > > with UDP_SEGMENT"). When UDP_GRO is enabled, such socket is also > > eligible for GRO in the rx path:

Re: [RFC PATCH v2 02/10] udp: implement GRO for plain UDP sockets.

2018-10-21 Thread Willem de Bruijn
On Fri, Oct 19, 2018 at 10:30 AM Paolo Abeni wrote: > > This is the RX counterpart of commit bec1f6f69736 ("udp: generate gso > with UDP_SEGMENT"). When UDP_GRO is enabled, such socket is also > eligible for GRO in the rx path: UDP segments directed to such socket > are assembled into a larger GSO

[RFC PATCH v2 02/10] udp: implement GRO for plain UDP sockets.

2018-10-19 Thread Paolo Abeni
This is the RX counterpart of commit bec1f6f69736 ("udp: generate gso with UDP_SEGMENT"). When UDP_GRO is enabled, such socket is also eligible for GRO in the rx path: UDP segments directed to such socket are assembled into a larger GSO_UDP_L4 packet. The core UDP GRO support is enabled with setso