Re: [PATCH] modpost: Check the section flags, not name, to catch missing "ax"/"aw"
- From: "H. Peter Anvin" <hpa@xxxxxxxxx>
- Date: Fri, 27 Feb 2009 04:35:25 -0800
Anders Kaseorg wrote:
On Tue, 24 Feb 2009, H. Peter Anvin wrote:
This is a pretty critical set of warnings, that often represent real
bugs. I don't think it's acceptable to lose them. However, you already
have (prior in the sequence) changed those section names to not conflict
with the automatically generated ones, so it seems to me that this
should be fixable without too much pain.
Actually, the problem arises from a conflict between two classes of
automatically generated section names, one from ld, and one from gcc.
Instead of basing the warning on section names (the existing heuristic
also fails to warn on some cases where it should, as it turns out), what
do you think about this patch? I've tested that it still triggers the
warning when an "ax" or "aw" is incorrectly removed, and it should handle
even more cases than before.
Seems reasonable to me.
-hpa
--
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel. I don't speak on their behalf.
--
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/
- References:
- [PATCH v2 0/4] Ksplice: Rebootless kernel updates
- From: Tim Abbott
- [PATCH v2 1/4] Make section names compatible with -ffunction-sections -fdata-sections
- From: Tim Abbott
- [PATCH v2 2/4] x86: Add an option to compile with -ffunction-sections -fdata-sections
- From: Tim Abbott
- Re: [PATCH v2 2/4] x86: Add an option to compile with -ffunction-sections -fdata-sections
- From: H. Peter Anvin
- [PATCH] modpost: Check the section flags, not name, to catch missing "ax"/"aw"
- From: Anders Kaseorg
- [PATCH v2 0/4] Ksplice: Rebootless kernel updates
- Prev by Date: E-mail Account Maintainace
- Next by Date: Re: [BUGFIX] dm-crypt: Fix a bug of async cryption complete function
- Previous by thread: [PATCH] modpost: Check the section flags, not name, to catch missing "ax"/"aw"
- Next by thread: [patch/RESEND 2.6.29-rc6] NAND: fix "raw" reads with ECC syndrome layouts
- Index(es):
Relevant Pages
|
Loading