On 06/14/2018 02:04 PM, Saeed Mahameed wrote: > I was looking at the code without my fix :) > > with my fix: > release = frags->page_offset + frag_info->frag_stride > PAGE_SIZE; > > for XDP: frag_info->frag_stride is PAGE_SIZE, so release will always be > true regardless of PAGE_SIZE. > > So i guess i didn't quite understand your PowerPC concern.. can you > elaborate ? > So your maths with PAGE_SIZE=65536 and MTU 9000 frag_stride is about 9344 So if the last chunk of the page has 9100 bytes, we wont be able to use it, while really we should be able to use it.
- [net RFC] net/mlx4_en: Use frag stride in crossing page bou... Saeed Mahameed
- Re: [net RFC] net/mlx4_en: Use frag stride in crossing... Eric Dumazet
- Re: [net RFC] net/mlx4_en: Use frag stride in cros... Saeed Mahameed
- Re: [net RFC] net/mlx4_en: Use frag stride in ... Eric Dumazet
- Re: [net RFC] net/mlx4_en: Use frag stride... Saeed Mahameed
- Re: [net RFC] net/mlx4_en: Use frag s... Saeed Mahameed
- Re: [net RFC] net/mlx4_en: Use fr... Saeed Mahameed
- Re: [net RFC] net/mlx4_en: Us... Eric Dumazet
- Re: [net RFC] net/mlx4_en: Us... Saeed Mahameed
- Re: [net RFC] net/mlx4_en: Us... Eric Dumazet
- Re: [net RFC] net/mlx4_en: Us... Saeed Mahameed
- Re: [net RFC] net/mlx4_en: Us... Eric Dumazet