Re: [PATCH 1/2] mm: Allow small allocations to fail
From: Tetsuo Handa
Date: Sun Mar 15 2015 - 09:07:09 EST
Michal Hocko wrote:
> On Sun 15-03-15 14:43:37, Tetsuo Handa wrote:
> > If you want to count only those retries which involved OOM killer, you need
> > to do like
> > - nr_retries++;
> > + if (gfp_mask & __GFP_FS)
> > + nr_retries++;
> > in this patch.
> No, we shouldn't create another type of hidden NOFAIL allocation like
> this. I understand that the wording of the changelog might be confusing,
> It says: "This implementation counts only those retries which involved
> OOM killer because we do not want to be too eager to fail the request."
> Would it be more clear if I changed that to?
> "This implemetnation counts only those retries when the system is
> considered OOM because all previous reclaim attempts have resulted
> in no progress because we do not want to be too eager to fail the
> We definitely _want_ to fail GFP_NOFS allocations.
I see. The updated changelog is much more clear.
> Michal Hocko
> SUSE Labs
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/