On Sat 12-05-18 23:18:24, Tetsuo Handa wrote:
[...]
> @@ -4241,6 +4240,12 @@ bool gfp_pfmemalloc_allowed(gfp_t gfp_mask)
> /* Retry as long as the OOM killer is making progress */
> if (did_some_progress) {
> no_progress_loops = 0;
> + /*
> + * This schedule_timeout_*() serves as a guaranteed sleep for
> + * PF_WQ_WORKER threads when __zone_watermark_ok() == false.
> + */
> + if (!tsk_is_oom_victim(current))
> + schedule_timeout_uninterruptible(1);
> goto retry;
We already do have that sleep for PF_WQ_WORKER in should_reclaim_retry.
Why do we need it here as well?
--
Michal Hocko
SUSE Labs
