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: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <25017BF213203E48912DB000DE5F5E1E6B70EA3C@SHSMSX101.ccr.corp.intel.com>
Date:   Thu, 30 Nov 2017 11:06:51 +0000
From:   "Wang, Kemi" <kemi.wang@...el.com>
To:     Michal Hocko <mhocko@...nel.org>
CC:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Vlastimil Babka <vbabka@...e.cz>,
        Mel Gorman <mgorman@...hsingularity.net>,
        Johannes Weiner <hannes@...xchg.org>,
        Christopher Lameter <cl@...ux.com>,
        YASUAKI ISHIMATSU <yasu.isimatu@...il.com>,
        Andrey Ryabinin <aryabinin@...tuozzo.com>,
        "Nikolay Borisov" <nborisov@...e.com>,
        Pavel Tatashin <pasha.tatashin@...cle.com>,
        David Rientjes <rientjes@...gle.com>,
        Sebastian Andrzej Siewior <bigeasy@...utronix.de>,
        Dave <dave.hansen@...ux.intel.com>,
        "Kleen, Andi" <andi.kleen@...el.com>,
        "Chen, Tim C" <tim.c.chen@...el.com>,
        "Jesper Dangaard Brouer" <brouer@...hat.com>,
        "Huang, Ying" <ying.huang@...el.com>,
        "Lu, Aaron" <aaron.lu@...el.com>,
        "Li, Aubrey" <aubrey.li@...el.com>, Linux MM <linux-mm@...ck.org>,
        Linux Kernel <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 1/2] mm: NUMA stats code cleanup and enhancement

Of course, we should do that AFAP. Thanks for your comments :)

-----Original Message-----
From: owner-linux-mm@...ck.org [mailto:owner-linux-mm@...ck.org] On Behalf Of Michal Hocko
Sent: Thursday, November 30, 2017 5:45 PM
To: Wang, Kemi <kemi.wang@...el.com>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>; Andrew Morton <akpm@...ux-foundation.org>; Vlastimil Babka <vbabka@...e.cz>; Mel Gorman <mgorman@...hsingularity.net>; Johannes Weiner <hannes@...xchg.org>; Christopher Lameter <cl@...ux.com>; YASUAKI ISHIMATSU <yasu.isimatu@...il.com>; Andrey Ryabinin <aryabinin@...tuozzo.com>; Nikolay Borisov <nborisov@...e.com>; Pavel Tatashin <pasha.tatashin@...cle.com>; David Rientjes <rientjes@...gle.com>; Sebastian Andrzej Siewior <bigeasy@...utronix.de>; Dave <dave.hansen@...ux.intel.com>; Kleen, Andi <andi.kleen@...el.com>; Chen, Tim C <tim.c.chen@...el.com>; Jesper Dangaard Brouer <brouer@...hat.com>; Huang, Ying <ying.huang@...el.com>; Lu, Aaron <aaron.lu@...el.com>; Li, Aubrey <aubrey.li@...el.com>; Linux MM <linux-mm@...ck.org>; Linux Kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/2] mm: NUMA stats code cleanup and enhancement

On Thu 30-11-17 17:32:08, kemi wrote:
[...]
> Your patch saves more code than mine because the node stats framework 
> is reused for numa stats. But it has a performance regression because 
> of the limitation of threshold size (125 at most, see 
> calculate_normal_threshold() in vmstat.c) in inc_node_state().

But this "regression" would be visible only on those workloads which really need to squeeze every single cycle out of the allocation hot path and those are supposed to disable the accounting altogether. Or is this visible on a wider variety of workloads.

Do not get me wrong. If we want to make per-node stats more optimal, then by all means let's do that. But having 3 sets of counters is just way to much.

--
Michal Hocko
SUSE Labs

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@...ck.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@...ck.org"> email@...ck.org </a>

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ