On Tue, Jul 06, 2010 at 12:11:56PM -0400, Mike Snitzer wrote: > Filesystems can call sb_issue_discard on a memory reclaim path > (e.g. ext4 calls sb_issue_discard during journal commit). > > Use GFP_NOFS in sb_issue_discard to avoid recursing back into the FS. > > Reported-by: Mikulas Patocka <mpatocka@xxxxxxxxxx> > Signed-off-by: Mike Snitzer <snitzer@xxxxxxxxxx> Hi Jens, I never saw an ack from you on this patch. Are you ok with it, and have you grabbed it for your tree? Do you want me to include this in the ext4 tree, even though it's a patch to include/linux/blkdev.h? Thanks, - Ted > --- > include/linux/blkdev.h | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h > index baf5258..dbb510c 100644 > --- a/include/linux/blkdev.h > +++ b/include/linux/blkdev.h > @@ -933,7 +933,7 @@ static inline int sb_issue_discard(struct super_block *sb, > { > block <<= (sb->s_blocksize_bits - 9); > nr_blocks <<= (sb->s_blocksize_bits - 9); > - return blkdev_issue_discard(sb->s_bdev, block, nr_blocks, GFP_KERNEL, > + return blkdev_issue_discard(sb->s_bdev, block, nr_blocks, GFP_NOFS, > BLKDEV_IFL_WAIT | BLKDEV_IFL_BARRIER); > } > > -- > 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 -- 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