Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1330231 > unrolled thread
| Started by | Christoph Hellwig <hch@infradead.org> |
|---|---|
| First post | 2016-02-09 14:30 +0100 |
| Last post | 2016-02-10 00:40 +0100 |
| Articles | 4 — 4 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH 6/6] kconfig option for TRIM_UNUSED_EXPSYMS Christoph Hellwig <hch@infradead.org> - 2016-02-09 14:30 +0100
Re: [PATCH 6/6] kconfig option for TRIM_UNUSED_EXPSYMS "Austin S. Hemmelgarn" <ahferroin7@gmail.com> - 2016-02-09 14:40 +0100
Re: [PATCH 6/6] kconfig option for TRIM_UNUSED_EXPSYMS Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-02-09 18:10 +0100
Re: [PATCH 6/6] kconfig option for TRIM_UNUSED_EXPSYMS Rusty Russell <rusty@rustcorp.com.au> - 2016-02-10 00:40 +0100
| From | Christoph Hellwig <hch@infradead.org> |
|---|---|
| Date | 2016-02-09 14:30 +0100 |
| Subject | Re: [PATCH 6/6] kconfig option for TRIM_UNUSED_EXPSYMS |
| Message-ID | <r0gwG-4Ai-33@gated-at.bofh.it> |
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..
[toc] | [next] | [standalone]
| From | "Austin S. Hemmelgarn" <ahferroin7@gmail.com> |
|---|---|
| Date | 2016-02-09 14:40 +0100 |
| Message-ID | <r0gGl-4Dt-7@gated-at.bofh.it> |
| In reply to | #1330231 |
On 2016-02-09 08:28, 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.. > I doubt we care too much about third party modules, but enabling this by default will likely cause at least some of them to stop working. At the very least, we need the config option, even if it's enabled by default.
[toc] | [prev] | [next] | [standalone]
| From | Nicolas Pitre <nicolas.pitre@linaro.org> |
|---|---|
| Date | 2016-02-09 18:10 +0100 |
| Message-ID | <r0jXC-76q-43@gated-at.bofh.it> |
| In reply to | #1330231 |
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. Nicolas
[toc] | [prev] | [next] | [standalone]
| From | Rusty Russell <rusty@rustcorp.com.au> |
|---|---|
| Date | 2016-02-10 00:40 +0100 |
| Message-ID | <r0q2Z-2zi-5@gated-at.bofh.it> |
| In reply to | #1330491 |
Nicolas Pitre <nicolas.pitre@linaro.org> 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.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web