Re: [PATCH] Discardable strings for init and exit sections
From: Maciej W. Rozycki
Date: Tue Oct 23 2007 - 12:57:57 EST
On Fri, 12 Oct 2007, Adrian Bunk wrote:
> - Most of the string annotations are (naturally) dev{init,exit}
> annotations, and bugs there are therefore in configurations that have
> only extremely low testing coverage during -rc.
> - I'm counting 22 annotations in the driver Maciej converted as an
> example. When estimating the number of possible annotations based
> on the number of C files in the kernel I'm getting a six digit
> number.
>
> No matter how hard it would be to teach gcc about it, when thinking of
> the amount of __*init*/__*exit* bugs we already have I simply can't
> imagine the string annotations as a maintainable solution.
Well, it is up to the maintainer of code in question to get it right if
interested. Otherwise having no annotation and leaving the relevant
strings resident in the memory throughout the lifespan of the system is a
valid and perfectly working option.
If you worry about the reverse case, where an annotation should be
removed because the containing function is no longer __*init*/__*exit*,
then I think `modpost' does a reasonably good job finding such places.
Maciej
-
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/