Re: [PATCH net] ptr_ring: use kmalloc_array()

On Fri, Aug 25, 2017 at 11:57:19AM -0700, Eric Dumazet wrote:

On Fri, 2017-08-25 at 21:03 +0300, Michael S. Tsirkin wrote:

quoted

On Wed, Aug 16, 2017 at 10:36:47AM -0700, Eric Dumazet wrote:

quoted

From: Eric Dumazet <redacted>

As found by syzkaller, malicious users can set whatever tx_queue_len
on a tun device and eventually crash the kernel.

Lets remove the ALIGN(XXX, SMP_CACHE_BYTES) thing since a small
ring buffer is not fast anyway.

I'm not sure it's worth changing for small rings.

Does kmalloc_array guarantee cache line alignment for big buffers
then? If the ring is misaligned it will likely cause false sharing
as it's designed to be accessed from two CPUs.

I specifically said that in the changelog :

"since a small ring buffer is not fast anyway."

If one user sets up a pathological small ring buffer, kernel should not
try to fix it.

Yes, I got that point. My question is about big buffers.
Does kmalloc_array give us an aligned array in that case?

E.g. imagine a 100 slot array. Will 800 bytes be allocated?
In that case it uses up 12.5 cache lines. It looks like the
last cache line can become false shared with something else,
causing cache line bounces on each wrap around.

In this case, you would have to setup a ring of 2 or 4 slots to
eventually hit false sharing.

I don't think many people set up such tiny rings so I do not really
think we care what happens in that case. But you need 8 slots to avoid
false sharing I think.

-- 
MST