On Wed, 5 Nov 2014 09:56:20 -0500 Benjamin LaHaise <bcrl@xxxxxxxxx> wrote: > On Wed, Nov 05, 2014 at 05:53:11PM +0800, Gu Zheng wrote: > > ping... > > I need someone a bit more familiar with this area of code to chime in on > reviewing this. Andrew, can you provide any feedback on this fix? That looks like a sensible solution and the changelog is good. A couple of suggested modifications: - Don't use the term "pinned" in the changelog without explaining it - it's a terribly ambiguous term. I assume here you mean "pinned by elevating the page's refcount"? - Could we have just one wee little code comment, explaining why aio_fs_backing_dev_info exists, what function it is serving? Put yourself in the position of a reader wondering "why is this here". -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html