On Fri, Feb 13, 2009 at 08:50:18PM -0500, Theodore Tso wrote: > > Patch from Aneesh, un-whitespace-mangled. > > > > Ted, can you push this out? Works great. :) We might want to ask > > the other reporter of something similar (next-20090206: deadlock on > > ext4) to test it too. I'll ping him. > > Do we completely understand the root cause, in terms of which commit > broken the mm/page-writeback.c code we were depending on? And if so, > what of the code in mm/page-writeback.c? Does anyone else use it? > Can anyone sanely use it? AFAIU we need the changes even for older kernels. The reasoning is, with delayed allocation we cannot allow to retry with lower page index in write_cache_pages. We do retry even in older version of kernel. What made it so easy to reproduce it on later kernels is that we were doing a retry even if nr_to_write was zero. This got fixed on mainline by 3a4c6800f31ea8395628af5e7e490270ee5d0585. So with that change we are logically back to 2.6.28 state, But still the possibility of deadlock remain. > > And am I right in assuming that this only applies to 2.6.29-rcX > kernels, and is not needed for 2.6.28 or earlier kernels? I guess the hang can happen on 2.6.28 or earlier kernels. > > I hadn't yet pushed it out because I needed time to understand all of > these issues, hence these questions.... -aneesh -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
