On Mon, Jul 19, 2021 at 7:00 AM Dmitry Safonov <0x7f454c46@xxxxxxxxx> wrote: > > Hi Vasily, > > On Mon, 19 Jul 2021 at 11:45, Vasily Averin <vvs@xxxxxxxxxxxxx> wrote: > [..] > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > > index ae1f5d0..1bbf239 100644 > > --- a/mm/memcontrol.c > > +++ b/mm/memcontrol.c > > @@ -968,7 +968,7 @@ static __always_inline bool memcg_kmem_bypass(void) > > return false; > > > > /* Memcg to charge can't be determined. */ > > - if (in_interrupt() || !current->mm || (current->flags & PF_KTHREAD)) > > + if (!in_task() || !current->mm || (current->flags & PF_KTHREAD)) > > return true; > > This seems to do two separate things in one patch. > Probably, it's better to separate them. > (I may miss how route changes are related to more generic > __alloc_pages() change) > It was requested to squash them together in some previous versions. https://lore.kernel.org/linux-mm/YEiUIf0old+AZssa@xxxxxxxxxxxxxx/
