Commit eed63519e3e74d515d2007ecd895338d0ba2a85c
Committed by
Ingo Molnar
1 parent
c967da6a0b
Exists in
master
and in
7 other branches
x86: Do not free zero sized per cpu areas
This avoids an infinite loop in free_early_partial(). Add a warning to free_early_partial() to catch future problems. -v5: put back start > end back into WARN_ONCE() -v6: use one line for warning, suggested by Linus -v7: more tests -v8: remove the function name as suggested by Johannes WARN_ONCE() will print out that function name. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Signed-off-by: Yinghai Lu <yinghai@kernel.org> Tested-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Tested-by: Joel Becker <joel.becker@oracle.com> Tested-by: Stanislaw Gruszka <sgruszka@redhat.com> Acked-by: Johannes Weiner <hannes@cmpxchg.org> Cc: Peter Zijlstra <peterz@infradead.org> Cc: David Miller <davem@davemloft.net> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> Cc: Linus Torvalds <torvalds@linux-foundation.org> LKML-Reference: <1269830604-26214-4-git-send-email-yinghai@kernel.org> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Showing 1 changed file with 6 additions and 0 deletions Side-by-side Diff
kernel/early_res.c
... | ... | @@ -333,6 +333,12 @@ |
333 | 333 | struct early_res *r; |
334 | 334 | int i; |
335 | 335 | |
336 | + if (start == end) | |
337 | + return; | |
338 | + | |
339 | + if (WARN_ONCE(start > end, " wrong range [%#llx, %#llx]\n", start, end)) | |
340 | + return; | |
341 | + | |
336 | 342 | try_next: |
337 | 343 | i = find_overlapped_early(start, end); |
338 | 344 | if (i >= max_early_res) |