Re: [PATCH v3 1/2] memory_hotplug: Add a bounds check to check_hotplug_memory_range()

2019-09-24 Thread Michal Hocko
On Tue 24-09-19 11:13:31, David Hildenbrand wrote: > On 24.09.19 11:09, Michal Hocko wrote: > > On Tue 24-09-19 11:31:05, Alastair D'Silva wrote: > >> On Mon, 2019-09-23 at 14:25 +0200, Michal Hocko wrote: [...] > >>> This will result in a silent failure (unlike misaligned case). Is > >>> this > >>

Re: [PATCH v3 1/2] memory_hotplug: Add a bounds check to check_hotplug_memory_range()

2019-09-24 Thread David Hildenbrand
On 24.09.19 11:09, Michal Hocko wrote: > On Tue 24-09-19 11:31:05, Alastair D'Silva wrote: >> On Mon, 2019-09-23 at 14:25 +0200, Michal Hocko wrote: >>> On Tue 17-09-19 11:07:47, Alastair D'Silva wrote: From: Alastair D'Silva On PowerPC, the address ranges allocated to OpenCAPI LPC

Re: [PATCH v3 1/2] memory_hotplug: Add a bounds check to check_hotplug_memory_range()

2019-09-24 Thread Michal Hocko
On Tue 24-09-19 11:31:05, Alastair D'Silva wrote: > On Mon, 2019-09-23 at 14:25 +0200, Michal Hocko wrote: > > On Tue 17-09-19 11:07:47, Alastair D'Silva wrote: > > > From: Alastair D'Silva > > > > > > On PowerPC, the address ranges allocated to OpenCAPI LPC memory > > > are allocated from firmwa

Re: [PATCH v3 1/2] memory_hotplug: Add a bounds check to check_hotplug_memory_range()

2019-09-23 Thread Alastair D'Silva
On Mon, 2019-09-23 at 14:25 +0200, Michal Hocko wrote: > On Tue 17-09-19 11:07:47, Alastair D'Silva wrote: > > From: Alastair D'Silva > > > > On PowerPC, the address ranges allocated to OpenCAPI LPC memory > > are allocated from firmware. These address ranges may be higher > > than what older ker

Re: [PATCH v3 1/2] memory_hotplug: Add a bounds check to check_hotplug_memory_range()

2019-09-23 Thread Michal Hocko
On Tue 17-09-19 11:07:47, Alastair D'Silva wrote: > From: Alastair D'Silva > > On PowerPC, the address ranges allocated to OpenCAPI LPC memory > are allocated from firmware. These address ranges may be higher > than what older kernels permit, as we increased the maximum > permissable address in c

Re: [PATCH v3 1/2] memory_hotplug: Add a bounds check to check_hotplug_memory_range()

2019-09-17 Thread David Hildenbrand
On 17.09.19 03:07, Alastair D'Silva wrote: > From: Alastair D'Silva > > On PowerPC, the address ranges allocated to OpenCAPI LPC memory > are allocated from firmware. These address ranges may be higher > than what older kernels permit, as we increased the maximum > permissable address in commit 4