Re: [PATCH] z3fold: limit use of stale list for allocation

From: Andrew Morton
Date: Fri Feb 16 2018 - 19:28:53 EST


On Sat, 10 Feb 2018 12:02:52 +0100 Vitaly Wool <vitalywool@xxxxxxxxx> wrote:

> Currently if z3fold couldn't find an unbuddied page it would first
> try to pull a page off the stale list. The problem with this
> approach is that we can't 100% guarantee that the page is not
> processed by the workqueue thread at the same time unless we run
> cancel_work_sync() on it, which we can't do if we're in an atomic
> context. So let's just limit stale list usage to non-atomic
> contexts only.

This smells like a bugfix. What are the end-user visible effects of
the bug?