Re: [PATCH v20 02/20] mm/memcg: bail early from swap accounting if memcg disabled

From: Alex Shi
Date: Thu Oct 29 2020 - 22:30:06 EST




在 2020/10/29 下午9:46, Johannes Weiner 写道:
>> ? release_pages+0x1ae/0x410
>> shmem_alloc_and_acct_page+0x77/0x1c0
>> shmem_getpage_gfp+0x162/0x910
>> shmem_fault+0x74/0x210
>> ? filemap_map_pages+0x29c/0x410
>> __do_fault+0x37/0x190
>> handle_mm_fault+0x120a/0x1770
>> exc_page_fault+0x251/0x450
>> ? asm_exc_page_fault+0x8/0x30
>> asm_exc_page_fault+0x1e/0x30
>>
>> Signed-off-by: Alex Shi <alex.shi@xxxxxxxxxxxxxxxxx>
>> Reviewed-by: Roman Gushchin <guro@xxxxxx>
>> Acked-by: Michal Hocko <mhocko@xxxxxxxx>
>> Acked-by: Hugh Dickins <hughd@xxxxxxxxxx>
>> Cc: Johannes Weiner <hannes@xxxxxxxxxxx>
>> Cc: Michal Hocko <mhocko@xxxxxxxxxx>
>> Cc: Vladimir Davydov <vdavydov.dev@xxxxxxxxx>
>> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
>> Cc: cgroups@xxxxxxxxxxxxxxx
>> Cc: linux-mm@xxxxxxxxx
>> Cc: linux-kernel@xxxxxxxxxxxxxxx
> Acked-by: Johannes Weiner <hannes@xxxxxxxxxxx>
>
> This should go in before the previous patch that adds the WARN for it.

Right, but than the long ops may not weird. Should I remove the ops and resend the whole patchset?

Which way is convenient for you?

Thanks
Alex