Messages in this thread | | | Date | Mon, 15 Jun 2015 21:56:22 +0200 | From | Michal Hocko <> | Subject | Re: [PATCH] jbd2: get rid of open coded allocation retry loop |
| |
On Mon 15-06-15 15:47:04, Theodore Ts'o wrote: > On Mon, Jun 15, 2015 at 04:17:34PM +0200, Michal Hocko wrote: > > insert_revoke_hash does an open coded endless allocation loop if > > journal_oom_retry is true. It doesn't implement any allocation fallback > > strategy between the retries, though. The memory allocator doesn't know > > about the never fail requirement so it cannot potentially help to move > > on with the allocation (e.g. use memory reserves). > > > > Get rid of the retry loop and use __GFP_NOFAIL instead. We will lose the > > debugging message but I am not sure it is anyhow helpful. > > > > Do the same for journal_alloc_journal_head which is doing a similar > > thing. > > > > Signed-off-by: Michal Hocko <mhocko@suse.cz> > > Thanks, applied.
Thanks! -- Michal Hocko SUSE Labs
| |