On Tue 18-08-15 12:39:03, Michal Hocko wrote:
[...]
> @@ -992,9 +992,8 @@ static int ext4_mb_get_buddy_page_lock(struct super_block 
> *sb,
>       block = group * 2;
>       pnum = block / blocks_per_page;
>       poff = block % blocks_per_page;
> -     page = find_or_create_page(inode->i_mapping, pnum, GFP_NOFS);
> -     if (!page)
> -             return -ENOMEM;
> +     page = find_or_create_page(inode->i_mapping, pnum,
> +                                GFP_NOFS|__GFP_NOFAIL);

Scratch this one. find_or_create_page is allowed to return NULL. The
patch is bogus. I was overly eager to turn all places to not check the
return value.

-- 
Michal Hocko
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to