Re: [PATCH v2 2/5] mm: access zone->node via zone_to_nid() and zone_set_nid()

2018-07-19 Thread Michal Hocko
On Thu 19-07-18 09:44:09, Pavel Tatashin wrote: > > > On 07/19/2018 09:40 AM, Michal Hocko wrote: > > On Thu 19-07-18 15:27:37, osalva...@techadventures.net wrote: > >> From: Pavel Tatashin > >> > >> zone->node is configured only when CONFIG_NUMA=y, so it is a good idea to > >> have inline funct

Re: [PATCH v2 2/5] mm: access zone->node via zone_to_nid() and zone_set_nid()

2018-07-19 Thread Pavel Tatashin
On 07/19/2018 09:40 AM, Michal Hocko wrote: > On Thu 19-07-18 15:27:37, osalva...@techadventures.net wrote: >> From: Pavel Tatashin >> >> 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 fil

Re: [PATCH v2 2/5] mm: access zone->node via zone_to_nid() and zone_set_nid()

2018-07-19 Thread Michal Hocko
On Thu 19-07-18 15:27:37, osalva...@techadventures.net wrote: > From: Pavel Tatashin > > 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. Is this a manual find & replace or did you use s

[PATCH v2 2/5] mm: access zone->node via zone_to_nid() and zone_set_nid()

2018-07-19 Thread osalvador
From: Pavel Tatashin 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. Signed-off-by: Pavel Tatashin Signed-off-by: Oscar Salvador Reviewed-by: Oscar Salvador --- include/linux/mm.h |