Re: [PATCH 6/6] kconfig option for TRIM_UNUSED_EXPSYMS

From: Rusty Russell
Date: Tue Feb 09 2016 - 18:37:13 EST


Nicolas Pitre <nicolas.pitre@xxxxxxxxxx> writes:
> On Tue, 9 Feb 2016, Christoph Hellwig wrote:
>
>> On Mon, Feb 08, 2016 at 03:28:35PM -0500, Nicolas Pitre wrote:
>> > The config option to enable it all.
>>
>> Just enable it by default..
>
> :-)
>
> Viro took the opposite view with this patch:
>
> http://lkml.org/lkml/2016/2/3/1068
>
> where he's ensuring that no exports are dropped, even if obviously
> they're not used (otherwise some complaints would have come about). So
> I'm not sure if having this turned on by default is a good thing.
>
> But I don't mind being convinced otherwise.

People do build out-of-tree modules. Dropping support for that would
simplify our lives, but seems fairly controversial :)

Cheers,
Rusty.