On 7/19/21 3:22 PM, Shakeel Butt wrote: > 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/ > Ah, alright, never mind than. Thanks, Dmitry
