The patch titled JBD slab cleanups 3 has been added to the -mm tree. Its filename is jbd-slab-cleanups-3.patch *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find out what to do about this ------------------------------------------------------ Subject: JBD slab cleanups 3 From: Mingming Cao <cmm@xxxxxxxxxx> Here is the patch to clean up __GFP_NOFAIL flag in jbd/jbd2. In all cases except one handles memory allocation failure so I get rid of those GFP_NOFAIL flags. Signed-off-by: Mingming Cao <cmm@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- fs/jbd/journal.c | 2 +- fs/jbd/transaction.c | 3 +-- fs/jbd2/journal.c | 2 +- fs/jbd2/transaction.c | 3 +-- 4 files changed, 4 insertions(+), 6 deletions(-) diff -puN fs/jbd2/journal.c~jbd-slab-cleanups-3 fs/jbd2/journal.c --- a/fs/jbd2/journal.c~jbd-slab-cleanups-3 +++ a/fs/jbd2/journal.c @@ -654,7 +654,7 @@ static journal_t * journal_init_common ( journal_t *journal; int err; - journal = kmalloc(sizeof(*journal), GFP_KERNEL|__GFP_NOFAIL); + journal = kmalloc(sizeof(*journal), GFP_KERNEL); if (!journal) goto fail; memset(journal, 0, sizeof(*journal)); diff -puN fs/jbd2/transaction.c~jbd-slab-cleanups-3 fs/jbd2/transaction.c --- a/fs/jbd2/transaction.c~jbd-slab-cleanups-3 +++ a/fs/jbd2/transaction.c @@ -96,8 +96,7 @@ static int start_this_handle(journal_t * alloc_transaction: if (!journal->j_running_transaction) { - new_transaction = kmalloc(sizeof(*new_transaction), - GFP_NOFS|__GFP_NOFAIL); + new_transaction = kmalloc(sizeof(*new_transaction), GFP_NOFS); if (!new_transaction) { ret = -ENOMEM; goto out; diff -puN fs/jbd/journal.c~jbd-slab-cleanups-3 fs/jbd/journal.c --- a/fs/jbd/journal.c~jbd-slab-cleanups-3 +++ a/fs/jbd/journal.c @@ -653,7 +653,7 @@ static journal_t * journal_init_common ( journal_t *journal; int err; - journal = kmalloc(sizeof(*journal), GFP_KERNEL|__GFP_NOFAIL); + journal = kmalloc(sizeof(*journal), GFP_KERNEL); if (!journal) goto fail; memset(journal, 0, sizeof(*journal)); diff -puN fs/jbd/transaction.c~jbd-slab-cleanups-3 fs/jbd/transaction.c --- a/fs/jbd/transaction.c~jbd-slab-cleanups-3 +++ a/fs/jbd/transaction.c @@ -96,8 +96,7 @@ static int start_this_handle(journal_t * alloc_transaction: if (!journal->j_running_transaction) { - new_transaction = kmalloc(sizeof(*new_transaction), - GFP_NOFS|__GFP_NOFAIL); + new_transaction = kmalloc(sizeof(*new_transaction), GFP_NOFS); if (!new_transaction) { ret = -ENOMEM; goto out; _ Patches currently in -mm which might be from cmm@xxxxxxxxxx are convert-ill-defined-log2-to-ilog2.patch ext2-ext3-ext4-add-block-bitmap-validation.patch ext2-ext3-ext4-add-block-bitmap-validation-fix.patch jbd-slab-cleanups.patch jbd-slab-cleanups-2.patch jbd-slab-cleanups-3.patch ext2-reservations.patch fix-for-ext2-reservation.patch ext2-balloc-use-io_error-label.patch ext4-uninitialized-block-groups.patch - To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html