linux-kernel - Re: [PATCH v2 1/2] memcg: switch lruvec stats to rstat

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <877di549aq.fsf@yhuang6-desk2.ccr.corp.intel.com>
Date:   Mon, 05 Jul 2021 13:30:21 +0800
From:   "Huang, Ying" <ying.huang@...el.com>
To:     Shakeel Butt <shakeelb@...gle.com>
Cc:     Tejun Heo <tj@...nel.org>, Johannes Weiner <hannes@...xchg.org>,
        Muchun Song <songmuchun@...edance.com>,
        Michal Hocko <mhocko@...nel.org>, Roman Gushchin <guro@...com>,
        Michal Koutný <mkoutny@...e.com>,
        Andrew Morton <akpm@...ux-foundation.org>,
        cgroups@...r.kernel.org, linux-mm@...ck.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/2] memcg: switch lruvec stats to rstat

Hi, Shakeel,

Shakeel Butt <shakeelb@...gle.com> writes:

> The commit 2d146aa3aa84 ("mm: memcontrol: switch to rstat") but skipped
> the conversion of the lruvec stats as such stats are read in the
> performance critical code paths and flushing stats may have impacted the
> performances of the applications. This patch converts the lruvec stats
> to rstat and later patch adds the periodic flushing of the stats and
> thus remove the need to synchronously flushing the stats in the
> performance critical code paths.
>
> The rstat conversion comes with the price i.e. memory cost. Effectively
> this patch reverts the savings done by the commit f3344adf38bd ("mm:
> memcontrol: optimize per-lruvec stats counter memory usage"). However
> this cost is justified due to negative impact of the inaccurate lruvec
> stats on many heuristics. One such case is reported in [1].
>
> The memory reclaim code is filled with plethora of heuristics and many
> of those heuristics reads the lruvec stats. So, inaccurate stats can
> make such heuristics ineffective. [1] reports the impact of inaccurate
> lruvec stats on the "cache trim mode" heuristic. Inaccurate lruvec stats
> can impact the deactivation and aging anon heuristics as well.
>
> [1] https://lore.kernel.org/linux-mm/20210311004449.1170308-1-ying.huang@intel.com/

Sorry for late reply.  I have just verified that this patchset can
resolve the problem as in the above link for me.  Thanks!

Best Regards,
Huang, Ying

> Signed-off-by: Shakeel Butt <shakeelb@...gle.com>
> ---
> Changes since v1:
> - no changes
>