[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20180726164304.GP28386@dhcp22.suse.cz>
Date: Thu, 26 Jul 2018 18:43:04 +0200
From: Michal Hocko <mhocko@...nel.org>
To: Pavel Tatashin <pasha.tatashin@...cle.com>
Cc: Oscar Salvador <osalvador@...hadventures.net>,
akpm@...ux-foundation.org, vbabka@...e.cz,
mgorman@...hsingularity.net, aaron.lu@...el.com,
iamjoonsoo.kim@....com, linux-mm@...ck.org,
linux-kernel@...r.kernel.org, dan.j.williams@...el.com,
Oscar Salvador <osalvador@...e.de>
Subject: Re: [PATCH v3 2/5] mm: access zone->node via zone_to_nid() and
zone_set_nid()
On Thu 26-07-18 11:14:20, Pavel Tatashin wrote:
> Hi Oscar,
>
> Below is updated patch, with comment about OpenGrok and Acked-by Michal added.
>
> Thank you,
> Pavel
>
> >From cca1b083d78d0ff99cce6dfaf12f6380d76390c7 Mon Sep 17 00:00:00 2001
> From: Pavel Tatashin <pasha.tatashin@...cle.com>
> Date: Thu, 26 Jul 2018 00:01:41 +0200
> Subject: [PATCH] mm: access zone->node via zone_to_nid() and zone_set_nid()
>
> zone->node is configured only when CONFIG_NUMA=y, so it is a good idea to
> have inline functions to access this field in order to avoid ifdef's in
> c files.
>
> OpenGrok was used to find places where zone->node is accessed. A public one
> is available here: http://src.illumos.org/source/
I assume that tool uses some pattern matching or similar so steps to use
the tool to get your results would be more helpful. This is basically
the same thing as coccinelle generated patches.
--
Michal Hocko
SUSE Labs
Powered by blists - more mailing lists