On Thu, Jun 30, 2011 at 10:12:25AM +0800, Steven Liu wrote: > > + bio = bio_alloc(GFP_NOIO, min(nvecs, BIO_MAX_PAGES)); > Is this needn't to check if bio_alloc success? No, that was the point of this patch. See the commit description. Also see the comment for bio_alloc() in fs/bio.c, and note that the __GFP_WAIT bit is set in GFP_NOFS. Regards, - Ted -- 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