Re: [PATCH v5 10/17] net: sgi: ioc3-eth: rework skb rx handling

2019-08-21 Thread Jakub Kicinski
On Wed, 21 Aug 2019 16:28:47 +0200, Thomas Bogendoerfer wrote: > > This looks like a DMA engine alignment requirement, more than an > > optimization. > > that true, there are two constraints for the rx buffers, start must be aligned > to 128 bytes and a buffer must not cross a 16kbyte boundary.

Re: [PATCH v5 10/17] net: sgi: ioc3-eth: rework skb rx handling

2019-08-21 Thread Thomas Bogendoerfer
On Mon, 19 Aug 2019 16:55:22 -0700 Jakub Kicinski wrote: > On Mon, 19 Aug 2019 18:31:33 +0200, Thomas Bogendoerfer wrote: > > Buffers alloacted by alloc_skb() are already cache aligned so there > > is no need for an extra align done by ioc3_alloc_skb. And instead > > of skb_put/skb_trim simply us

Re: [PATCH v5 10/17] net: sgi: ioc3-eth: rework skb rx handling

2019-08-19 Thread Jakub Kicinski
On Mon, 19 Aug 2019 18:31:33 +0200, Thomas Bogendoerfer wrote: > Buffers alloacted by alloc_skb() are already cache aligned so there > is no need for an extra align done by ioc3_alloc_skb. And instead > of skb_put/skb_trim simply use one skb_put after frame size is known > during receive. > > Sign

[PATCH v5 10/17] net: sgi: ioc3-eth: rework skb rx handling

2019-08-19 Thread Thomas Bogendoerfer
Buffers alloacted by alloc_skb() are already cache aligned so there is no need for an extra align done by ioc3_alloc_skb. And instead of skb_put/skb_trim simply use one skb_put after frame size is known during receive. Signed-off-by: Thomas Bogendoerfer --- drivers/net/ethernet/sgi/ioc3-eth.c |