Re: [PATCH 1/2] mm/memory_hotplug: Change pfn_to_section_nr/section_nr_to_pfn macro to inline function

2017-09-17 Thread Michal Hocko
On Fri 15-09-17 22:52:20, YASUAKI ISHIMATSU wrote: > pfn_to_section_nr() and section_nr_to_pfn() are defined as macro. > pfn_to_section_nr() has no issue even if it is defined as macro. > But section_nr_to_pfn() has overflow issue if sec is defined as int. > > section_nr_to_pfn() just shifts sec b

[PATCH 1/2] mm/memory_hotplug: Change pfn_to_section_nr/section_nr_to_pfn macro to inline function

2017-09-15 Thread YASUAKI ISHIMATSU
pfn_to_section_nr() and section_nr_to_pfn() are defined as macro. pfn_to_section_nr() has no issue even if it is defined as macro. But section_nr_to_pfn() has overflow issue if sec is defined as int. section_nr_to_pfn() just shifts sec by PFN_SECTION_SHIFT. If sec is defined as unsigned long, sect