Currently if an index page is not in the cache the critical section is entered again to add it.  If two threads request the same index page at the same time they will both load it.  It is worth investigating how often this happens, and potentially refactoring the code to avoid it.