Comment 2 for bug 1038225

Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

Raghu, you mean fix for 1015109 perhaps?

This looks like not complete fix for bug 1015109 for the incorrectly merged upstream revision 0.16394.47, the one which eliminated the LRU scan for dropping AHI entries.

The InnoDB buf_flush_dirty_pages() acquires the buffer pool mutex, the XtraDB one acquires the LRU list mutex. The whole of 0.16394.47 needs to be reviewed for mutex usage. Raghu's fix is correct but incomplete: as a minimum, buf_flush_yield() should also release/re-acquire the LRU list mutex instead of the buffer pool mutex.