Re: [patch V3] percpu_counter: scalability works

[Eric Dumazet]

Le lundi 16 mai 2011 Ã 15:15 +0800, Shaohua Li a Ãcrit :

> I can, but you can't prevent me to optimize percpu_counter.
> 

Well, I have the right to say you're wrong.

Your last patch is not good, sorry. Please take the time to read it
again and fix obvious problems. And also give us numbers if one process
does the mmap()/munmap() loop, before and after your patch.

A percpu_counter is already a beast as is, you're suggesting to double
its size, for a pathological case.

Its absolutely not clear to me why vm_committed_as is using the default
percpu_counter_batch. 

By the way could you make sure percpu_counter_batch has the right value
on your 24 cpus machine ?

Your 128Mbyte mmap threshold sounds like percpu_counter_batch=32 instead
of 48



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/