diff options
author | Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com> | 2017-04-13 15:18:21 -0400 |
---|---|---|
committer | Michael Ellerman <mpe@ellerman.id.au> | 2017-04-19 06:00:18 -0400 |
commit | be77e999e3937322b7e15274b8fc7da309a040a0 (patch) | |
tree | e850b22d9ee7ae37a8e5b8ceb6f16fad8ae1b7e3 /arch/powerpc/mm/slice.c | |
parent | 8d1b48ef580097e111c2644e6fc6041b9784d218 (diff) |
powerpc/mm/radix: Use mm->task_size for boundary checking instead of addr_limit
We don't init addr_limit correctly for 32 bit applications. So default to using
mm->task_size for boundary condition checking. We use addr_limit to only control
free space search. This makes sure that we do the right thing with 32 bit
applications.
We should consolidate the usage of TASK_SIZE/mm->task_size and
mm->context.addr_limit later.
This partially reverts commit fbfef9027c2a7ad (powerpc/mm: Switch some
TASK_SIZE checks to use mm_context addr_limit).
Fixes: fbfef9027c2a ("powerpc/mm: Switch some TASK_SIZE checks to use mm_context addr_limit")
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Diffstat (limited to 'arch/powerpc/mm/slice.c')
-rw-r--r-- | arch/powerpc/mm/slice.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/powerpc/mm/slice.c b/arch/powerpc/mm/slice.c index 251b6bae7023..ade66c3ecdce 100644 --- a/arch/powerpc/mm/slice.c +++ b/arch/powerpc/mm/slice.c | |||
@@ -292,8 +292,8 @@ static unsigned long slice_find_area_bottomup(struct mm_struct *mm, | |||
292 | * Check if we need to reduce the range, or if we can | 292 | * Check if we need to reduce the range, or if we can |
293 | * extend it to cover the next available slice. | 293 | * extend it to cover the next available slice. |
294 | */ | 294 | */ |
295 | if (addr >= mm->context.addr_limit) | 295 | if (addr >= high_limit) |
296 | addr = mm->context.addr_limit; | 296 | addr = high_limit; |
297 | else if (slice_scan_available(addr, available, 1, &next_end)) { | 297 | else if (slice_scan_available(addr, available, 1, &next_end)) { |
298 | addr = next_end; | 298 | addr = next_end; |
299 | goto next_slice; | 299 | goto next_slice; |