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. _______________________________________________ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
