From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 494F7C433E6 for ; Thu, 28 Jan 2021 14:24:01 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id F186B64DDD for ; Thu, 28 Jan 2021 14:24:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231552AbhA1OX0 (ORCPT ); Thu, 28 Jan 2021 09:23:26 -0500 Received: from mx2.suse.de ([195.135.220.15]:52686 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231607AbhA1OXL (ORCPT ); Thu, 28 Jan 2021 09:23:11 -0500 X-Virus-Scanned: by amavisd-new at test-mx.suse.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1611843744; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=UTyYfHxVeLzGHWSwOvwq53dZzhDb7zJAaqQ20zEnBQo=; b=tZPG7U2mHgmh4/6Ua8sFwY4yt7lQFbxygTF8BprvvmlP27oDtQXZh768Yg/Z082E1URKI2 T6aM07JvCtuFwwKMu68i5mCLMFi6oNtYXWMBP85PEnh58gWe1DppMZTjN+2kFnbapvgMwz Lu/rvPxkwQW+Nde41FMKFbLhE0rXHUY= Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id 540CCB141; Thu, 28 Jan 2021 14:22:24 +0000 (UTC) Date: Thu, 28 Jan 2021 15:22:22 +0100 From: Michal Hocko To: Shakeel Butt Cc: Roman Gushchin , Matthew Wilcox , Mike Rapoport , Andrew Morton , Alexander Viro , Andy Lutomirski , Arnd Bergmann , Borislav Petkov , Catalin Marinas , Christopher Lameter , Dan Williams , Dave Hansen , David Hildenbrand , Elena Reshetova , "H. Peter Anvin" , Ingo Molnar , James Bottomley , "Kirill A. Shutemov" , Mark Rutland , Mike Rapoport , Michael Kerrisk , Palmer Dabbelt , Paul Walmsley , Peter Zijlstra , Rick Edgecombe , Shuah Khan , Thomas Gleixner , Tycho Andersen , Will Deacon , linux-api@vger.kernel.org, linux-arch@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-fsdevel , Linux MM , LKML , linux-kselftest@vger.kernel.org, linux-nvdimm@lists.01.org, linux-riscv@lists.infradead.org, x86@kernel.org, Hagen Paul Pfeifer , Palmer Dabbelt Subject: Re: [PATCH v16 08/11] secretmem: add memcg accounting Message-ID: References: <20210121122723.3446-1-rppt@kernel.org> <20210121122723.3446-9-rppt@kernel.org> <20210125165451.GT827@dhcp22.suse.cz> <20210125213817.GM6332@kernel.org> <20210126144838.GL308988@casper.infradead.org> <20210126150555.GU827@dhcp22.suse.cz> <20210127184213.GA919963@carbon.dhcp.thefacebook.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-api@vger.kernel.org On Thu 28-01-21 06:05:11, Shakeel Butt wrote: > On Wed, Jan 27, 2021 at 11:59 PM Michal Hocko wrote: > > > > On Wed 27-01-21 10:42:13, Roman Gushchin wrote: > > > On Tue, Jan 26, 2021 at 04:05:55PM +0100, Michal Hocko wrote: > > > > On Tue 26-01-21 14:48:38, Matthew Wilcox wrote: > > > > > On Mon, Jan 25, 2021 at 11:38:17PM +0200, Mike Rapoport wrote: > > > > > > I cannot use __GFP_ACCOUNT because cma_alloc() does not use gfp. > > > > > > Besides, kmem accounting with __GFP_ACCOUNT does not seem > > > > > > to update stats and there was an explicit request for statistics: > > > > > > > > > > > > https://lore.kernel.org/lkml/CALo0P13aq3GsONnZrksZNU9RtfhMsZXGWhK1n=xYJWQizCd4Zw@mail.gmail.com/ > > > > > > > > > > > > As for (ab)using NR_SLAB_UNRECLAIMABLE_B, as it was already discussed here: > > > > > > > > > > > > https://lore.kernel.org/lkml/20201129172625.GD557259@kernel.org/ > > > > > > > > > > > > I think that a dedicated stats counter would be too much at the moment and > > > > > > NR_SLAB_UNRECLAIMABLE_B is the only explicit stat for unreclaimable memory. > > > > > > > > > > That's not true -- Mlocked is also unreclaimable. And doesn't this > > > > > feel more like mlocked memory than unreclaimable slab? It's also > > > > > Unevictable, so could be counted there instead. > > > > > > > > yes, that is indeed true, except the unreclaimable counter is tracking > > > > the unevictable LRUs. These pages are not on any LRU and that can cause > > > > some confusion. Maybe they shouldn't be so special and they should live > > > > on unevistable LRU and get their stats automagically. > > > > > > > > I definitely do agree that this would be a better fit than NR_SLAB > > > > abuse. But considering that this is somehow even more special than mlock > > > > then a dedicated counter sounds as even better fit. > > > > > > I think it depends on how large these areas will be in practice. > > > If they will be measured in single or double digits MBs, a separate entry > > > is hardly a good choice: because of the batching the displayed value > > > will be in the noise range, plus every new vmstat item adds to the > > > struct mem_cgroup size. > > > > > > If it will be measured in GBs, of course, a separate counter is preferred. > > > So I'd suggest to go with NR_SLAB (which should have been named NR_KMEM) > > > as now and conditionally switch to a separate counter later. > > > > I really do not think the overall usage matters when it comes to abusing > > other counters. Changing this in future will be always tricky and there > > always be our favorite "Can this break userspace" question. Yes we dared > > to change meaning of some counters but this is not generally possible. > > Just have a look how accounting shmem as a page cache has turned out > > being much more tricky than many like. > > > > Really if a separate counter is a big deal, for which I do not see any > > big reason, then this should be accounted as unevictable (as suggested > > by Matthew) and ideally pages of those mappings should be sitting in the > > unevictable LRU as well unless there is a strong reason against. > > > > Why not decide based on the movability of these pages? If movable then > unevictable LRU seems like the right way otherwise NR_SLAB. I really do not follow. If the page is unevictable then why movability matters? I also fail to see why NR_SLAB is even considered considering this is completely outside of slab proper. Really what is the point? What are we trying to achieve by stats? Do we want to know how much secret memory is used because that is an interesting/important information or do we just want to make some accounting? Just think at it from a practical point of view. I want to know how much slab memory is used because it can give me an idea whether kernel is consuming unexpected amount of memory. Now I have to subtract _some_ number to get that information. Where do I get that some number? We have been creative with counters and it tends to kick back much more often than it helps. I really do not want this to turn into an endless bike shed but either this should be accounted as a general type of memory (unevictable would be a good fit because that is a userspace memory which is not reclaimable) or it needs its own counter to tell how much of this specific type of memory is used for this purpose. -- Michal Hocko SUSE Labs