[infinispan-dev] Faster LRU

[Vladimir Blagojevic]

On 11-07-08 5:44 AM, Dan Berindei wrote:
> I haven't run the tests with concurrency level 512, as the total
> number of threads is only 100, but I suspect the old LRU still won't
> catch up with the new LRU's performance.
>
> It's interesting that in the writeOnMiss test the new LRU performance
> dropped when I increased the concurrency level from 32 to 128. I think
> it might be because the eviction.thresholdExpired() check in
> BCHM.attemptEviction() is done without a lock and so it could return
> true simultaneously for multiple threads - which will all proceed to
> wait on the segment lock and attempt eviction at the same time.
>

I am not sure about this Dan. I looked at this code for hours! I do not 
see how two threads can call eviction#execute() concurrently.