lkml.org 
[lkml]   [2012]   [Jul]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] extend_brk and fixes to users of extend_brk (v1).
On 07/31/2012 03:42 AM, Konrad Rzeszutek Wilk wrote:
>
> [PATCH 3/3] x86: Let extend_brk return a NULL pointer instead of
>
> makes extend_brk not do BUG_ON and instead just return NULL. This allows
> us to handle the bootup issues more gracefully.
>

NAK. The whole point of the brk allocator is that users specify the
upper limit on what they may need and stick to it. Hence it is a fatal
code bug if that is ever exceeded. We want to catch those errors, not
"handle" them.

This means you're either abusing the brk allocator to do something it is
not meant to do... which may mean you can a failure in *other* code, or
you have a bug in your code that you haven't fixed.

-hpa

--
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel. I don't speak on their behalf.



\
 
 \ /
  Last update: 2012-07-31 18:22    [W:0.053 / U:0.120 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site