Re: [PATCH] mm: code improvement of check_stack_guard_page

From: jovi zhang
Date: Sun Aug 15 2010 - 21:32:10 EST


On Mon, Aug 16, 2010 at 1:58 AM, Linus Torvalds
<torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:
> On Sat, Aug 14, 2010 at 10:15 PM, jovi zhang <bookjovi@xxxxxxxxx> wrote:
>>
>> little code improvement of check_stack_guard_page function.
>> this commit is on top of commit "mm: keep a guard page below a
>> grow-down stack segment" of linus.
>
> I don't think your patch is in any way wrong, but I just checked, and
> it makes absolutely zero difference to the code generation at least
> for my version of gcc. The resulting assembly is identical except for
> the line numbers that changed (BUG_ON() line/file information and
> debug information).
>
> So I think I prefer the shallower indentation.
>
> Â Â Â Â Â Â Â Â Â Â Â Linus
>

That's fine. I havn't consider gcc optimization.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/