Re: [PATCH 02/10] x86: Convert BUG() to use unreachable()

From: H. Peter Anvin
Date: Thu Sep 10 2009 - 20:57:53 EST


On 09/10/2009 04:56 PM, David Daney wrote:
> Use the new unreachable() macro instead of for(;;);. When
> allyesconfig is built with a GCC-4.5 snapshot on i686 the size of the
> text segment is reduced by 3987 bytes (from 6827019 to 6823032).
>
> Signed-off-by: David Daney <ddaney@xxxxxxxxxxxxxxxxxx>
> CC: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> CC: Ingo Molnar <mingo@xxxxxxxxxx>
> CC: "H. Peter Anvin" <hpa@xxxxxxxxx>
> CC: x86@xxxxxxxxxx

Acked-by: H. Peter Anvin <hpa@xxxxxxxxx>

... although of course this clashes with Roland McGrath's patchset for
the same thing which I applied earlier. I have to say I like
unreachable() in lower case better though...

-hpa
--
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/