Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-13 Thread Enrico Weigelt, metux IT consult
On 12.07.19 01:27, Brendan Higgins wrote: > Enrico, want me to CC you on that> and we can continue this discussion there? Yes. But I'd prefer having an own list for it - better for sorting and archiving ;-) > I wonder if that would work for the testing scenario? I don't think it> is >

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-11 Thread Brendan Higgins
On Wed, Jul 3, 2019 at 3:42 PM Luis Chamberlain wrote: > > On Wed, Jul 03, 2019 at 09:31:33PM +0200, Enrico Weigelt, metux IT consult > wrote: > > On 03.07.19 19:35, Luis Chamberlain wrote: > > > > Hi, > > > > >> Okay, but IIRC this will add more boilerplate those modules. > > > > > > Just one

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-11 Thread Luis Chamberlain
On Thu, Jul 11, 2019 at 04:07:27PM -0700, Brendan Higgins wrote: > On Wed, Jul 3, 2019 at 3:25 PM Luis Chamberlain wrote: > > > > On Wed, Jul 03, 2019 at 08:57:58PM +0200, Greg Kroah-Hartman wrote: > > > > But again, this is a separate problem. The one I am addressing, on > > behalf of Cristina,

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-11 Thread Brendan Higgins
On Wed, Jul 3, 2019 at 3:25 PM Luis Chamberlain wrote: > > On Wed, Jul 03, 2019 at 08:57:58PM +0200, Greg Kroah-Hartman wrote: > > On Wed, Jul 03, 2019 at 04:50:20PM +, Luis Chamberlain wrote: > > > On Wed, Jul 03, 2019 at 09:40:48AM +0200, Greg Kroah-Hartman wrote: > > > > On Tue, Jul 02,

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-03 Thread Luis Chamberlain
On Wed, Jul 03, 2019 at 09:31:33PM +0200, Enrico Weigelt, metux IT consult wrote: > On 03.07.19 19:35, Luis Chamberlain wrote: > > Hi, > > >> Okay, but IIRC this will add more boilerplate those modules. > > > > Just one module attribute. > > Yes, but still one per module. This raises the

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-03 Thread Luis Chamberlain
On Wed, Jul 03, 2019 at 08:57:58PM +0200, Greg Kroah-Hartman wrote: > On Wed, Jul 03, 2019 at 04:50:20PM +, Luis Chamberlain wrote: > > On Wed, Jul 03, 2019 at 09:40:48AM +0200, Greg Kroah-Hartman wrote: > > > On Tue, Jul 02, 2019 at 08:51:06PM +, Luis Chamberlain wrote: > > > > On Sat,

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-03 Thread Enrico Weigelt, metux IT consult
On 03.07.19 19:35, Luis Chamberlain wrote: Hi, >> Okay, but IIRC this will add more boilerplate those modules. > > Just one module attribute. Yes, but still one per module. This raises the question whether maintainers are willing to cope w/ tons of tiny patches for just one line - for

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-03 Thread Greg Kroah-Hartman
On Wed, Jul 03, 2019 at 04:50:20PM +, Luis Chamberlain wrote: > On Wed, Jul 03, 2019 at 09:40:48AM +0200, Greg Kroah-Hartman wrote: > > On Tue, Jul 02, 2019 at 08:51:06PM +, Luis Chamberlain wrote: > > > On Sat, Jun 29, 2019 at 10:42:57AM +0200, Greg Kroah-Hartman wrote: > > > > On Fri,

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-03 Thread Luis Chamberlain
On Wed, Jul 03, 2019 at 02:16:44PM +0200, Enrico Weigelt, metux IT consult wrote: > On 28.06.19 20:40, Luis Chamberlain wrote: > > Hi folks, > > > The solution puts forward a mechanism to add a kconfig_symb where > > we are 100% certain we have a direct module --> config mapping. > Okay, but

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-03 Thread Luis Chamberlain
On Wed, Jul 03, 2019 at 09:40:48AM +0200, Greg Kroah-Hartman wrote: > On Tue, Jul 02, 2019 at 08:51:06PM +, Luis Chamberlain wrote: > > On Sat, Jun 29, 2019 at 10:42:57AM +0200, Greg Kroah-Hartman wrote: > > > On Fri, Jun 28, 2019 at 11:40:22AM -0700, Luis Chamberlain wrote: > > > > On Wed,

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-03 Thread Enrico Weigelt, metux IT consult
On 28.06.19 20:40, Luis Chamberlain wrote: Hi folks, > The solution puts forward a mechanism to add a kconfig_symb where we> are > 100% certain we have a direct module --> config mapping. Okay, but IIRC this will add more boilerplate those modules. And I wonder whether target binaries are the

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-03 Thread Greg Kroah-Hartman
On Tue, Jul 02, 2019 at 08:51:06PM +, Luis Chamberlain wrote: > On Sat, Jun 29, 2019 at 10:42:57AM +0200, Greg Kroah-Hartman wrote: > > On Fri, Jun 28, 2019 at 11:40:22AM -0700, Luis Chamberlain wrote: > > > On Wed, Jun 26, 2019 at 9:51 PM Christoph Hellwig wrote: > > > > > > > > On Wed, Jun

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-07-02 Thread Luis Chamberlain
On Sat, Jun 29, 2019 at 10:42:57AM +0200, Greg Kroah-Hartman wrote: > On Fri, Jun 28, 2019 at 11:40:22AM -0700, Luis Chamberlain wrote: > > On Wed, Jun 26, 2019 at 9:51 PM Christoph Hellwig wrote: > > > > > > On Wed, Jun 26, 2019 at 03:21:08PM -0700, Luis Chamberlain wrote: > > > > On Tue, Feb 5,

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-06-29 Thread Greg Kroah-Hartman
On Fri, Jun 28, 2019 at 11:40:22AM -0700, Luis Chamberlain wrote: > On Wed, Jun 26, 2019 at 9:51 PM Christoph Hellwig wrote: > > > > On Wed, Jun 26, 2019 at 03:21:08PM -0700, Luis Chamberlain wrote: > > > On Tue, Feb 5, 2019 at 2:07 PM Luis Chamberlain wrote: > > > > In lieu of no Luke

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-06-28 Thread Luis Chamberlain
On Wed, Jun 26, 2019 at 9:51 PM Christoph Hellwig wrote: > > On Wed, Jun 26, 2019 at 03:21:08PM -0700, Luis Chamberlain wrote: > > On Tue, Feb 5, 2019 at 2:07 PM Luis Chamberlain wrote: > > > In lieu of no Luke Skywalker, if you will, for a large kconfig revamp > > > on this, I'm inclined to

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-06-26 Thread Christoph Hellwig
On Wed, Jun 26, 2019 at 03:21:08PM -0700, Luis Chamberlain wrote: > On Tue, Feb 5, 2019 at 2:07 PM Luis Chamberlain wrote: > > In lieu of no Luke Skywalker, if you will, for a large kconfig revamp > > on this, I'm inclined to believe *at least* having some kconfig_symb > > exposed for some

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-06-26 Thread Luis Chamberlain
On Tue, Feb 5, 2019 at 2:07 PM Luis Chamberlain wrote: > In lieu of no Luke Skywalker, if you will, for a large kconfig revamp > on this, I'm inclined to believe *at least* having some kconfig_symb > exposed for some modules is better than nothing. Christoph are you > totally opposed to this

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2019-02-05 Thread Luis Chamberlain
On Thu, Aug 25, 2016 at 2:19 PM Luis R. Rodriguez wrote: > > On Thu, Aug 25, 2016 at 09:43:13AM +0200, Christoph Hellwig wrote: > > The idea seems useful, but I reallt don't like the 'reverse-engineering' > > approach. > > > > If we want to this properly from the ground up we should just split

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-25 Thread Luis R. Rodriguez
On Thu, Aug 25, 2016 at 09:43:13AM +0200, Christoph Hellwig wrote: > The idea seems useful, but I reallt don't like the 'reverse-engineering' > approach. > > If we want to this properly from the ground up we should just split out > our CONFIG_ SYMBOLS into > > MODULE_* - builds exactly one

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-25 Thread Luis R. Rodriguez
On Thu, Aug 25, 2016 at 09:43:13AM +0200, Christoph Hellwig wrote: > The idea seems useful, but I reallt don't like the 'reverse-engineering' > approach. > > If we want to this properly from the ground up we should just split out > our CONFIG_ SYMBOLS into > > MODULE_* - builds exactly one

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-25 Thread Luis R. Rodriguez
On Thu, Aug 25, 2016 at 10:00:20AM +0200, Johannes Berg wrote: > > > The other nice thing is that we could probably fold most of the > > Makefiles into Kconfig using that methods as well, by listing the > > objectes required for a module, e.g. > > > > module NVME_TARGET > > tristate "NVMe

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-25 Thread Luis R. Rodriguez
On Thu, Aug 25, 2016 at 10:00:20AM +0200, Johannes Berg wrote: > > > The other nice thing is that we could probably fold most of the > > Makefiles into Kconfig using that methods as well, by listing the > > objectes required for a module, e.g. > > > > module NVME_TARGET > > tristate "NVMe

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-25 Thread Michal Marek
On 2016-08-25 09:43, Christoph Hellwig wrote: > The idea seems useful, but I reallt don't like the 'reverse-engineering' > approach. > > If we want to this properly from the ground up we should just split out > our CONFIG_ SYMBOLS into > > MODULE_* - builds exactly one module (tristate, or maybe

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-25 Thread Michal Marek
On 2016-08-25 09:43, Christoph Hellwig wrote: > The idea seems useful, but I reallt don't like the 'reverse-engineering' > approach. > > If we want to this properly from the ground up we should just split out > our CONFIG_ SYMBOLS into > > MODULE_* - builds exactly one module (tristate, or maybe

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-25 Thread Johannes Berg
> The other nice thing is that we could probably fold most of the > Makefiles into Kconfig using that methods as well, by listing the > objectes required for a module, e.g. > > module NVME_TARGET > tristate "NVMe Target support" > depends on BLOCK > depends on CONFIGFS_FS >

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-25 Thread Johannes Berg
> The other nice thing is that we could probably fold most of the > Makefiles into Kconfig using that methods as well, by listing the > objectes required for a module, e.g. > > module NVME_TARGET > tristate "NVMe Target support" > depends on BLOCK > depends on CONFIGFS_FS >

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-25 Thread Christoph Hellwig
The idea seems useful, but I reallt don't like the 'reverse-engineering' approach. If we want to this properly from the ground up we should just split out our CONFIG_ SYMBOLS into MODULE_* - builds exactly one module (tristate, or maybe also as a built-in only one, then like a bool) CONFIG_* -

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-25 Thread Christoph Hellwig
The idea seems useful, but I reallt don't like the 'reverse-engineering' approach. If we want to this properly from the ground up we should just split out our CONFIG_ SYMBOLS into MODULE_* - builds exactly one module (tristate, or maybe also as a built-in only one, then like a bool) CONFIG_* -

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-24 Thread Naveen Kumar
unsubscribe backports On 8/24/16, Luis R. Rodriguez wrote: > On Wed, Aug 24, 2016 at 01:05:04PM +0200, Michal Marek wrote: >> On 2016-08-23 23:32, Luis R. Rodriguez wrote: >> > On Fri, Aug 19, 2016 at 11:07:36AM +0200, Michal Marek wrote: >> >> On 2016-08-18 19:55, Luis R.

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-24 Thread Naveen Kumar
unsubscribe backports On 8/24/16, Luis R. Rodriguez wrote: > On Wed, Aug 24, 2016 at 01:05:04PM +0200, Michal Marek wrote: >> On 2016-08-23 23:32, Luis R. Rodriguez wrote: >> > On Fri, Aug 19, 2016 at 11:07:36AM +0200, Michal Marek wrote: >> >> On 2016-08-18 19:55, Luis R. Rodriguez wrote: >>

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-24 Thread Luis R. Rodriguez
On Wed, Aug 24, 2016 at 01:05:04PM +0200, Michal Marek wrote: > On 2016-08-23 23:32, Luis R. Rodriguez wrote: > > On Fri, Aug 19, 2016 at 11:07:36AM +0200, Michal Marek wrote: > >> On 2016-08-18 19:55, Luis R. Rodriguez wrote: > >>> On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote:

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-24 Thread Luis R. Rodriguez
On Wed, Aug 24, 2016 at 01:05:04PM +0200, Michal Marek wrote: > On 2016-08-23 23:32, Luis R. Rodriguez wrote: > > On Fri, Aug 19, 2016 at 11:07:36AM +0200, Michal Marek wrote: > >> On 2016-08-18 19:55, Luis R. Rodriguez wrote: > >>> On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote:

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-24 Thread Michal Marek
On 2016-08-23 23:32, Luis R. Rodriguez wrote: > On Fri, Aug 19, 2016 at 11:07:36AM +0200, Michal Marek wrote: >> On 2016-08-18 19:55, Luis R. Rodriguez wrote: >>> On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: >>> This patchset implements dynamic pegging of kconfig symbol

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-24 Thread Michal Marek
On 2016-08-23 23:32, Luis R. Rodriguez wrote: > On Fri, Aug 19, 2016 at 11:07:36AM +0200, Michal Marek wrote: >> On 2016-08-18 19:55, Luis R. Rodriguez wrote: >>> On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: >>> This patchset implements dynamic pegging of kconfig symbol

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-23 Thread Luis R. Rodriguez
On Fri, Aug 19, 2016 at 11:07:36AM +0200, Michal Marek wrote: > On 2016-08-18 19:55, Luis R. Rodriguez wrote: > > On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: > > > >> This patchset implements dynamic pegging of kconfig symbol > >> into driver modinfo section > > > > First a

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-23 Thread Luis R. Rodriguez
On Fri, Aug 19, 2016 at 11:07:36AM +0200, Michal Marek wrote: > On 2016-08-18 19:55, Luis R. Rodriguez wrote: > > On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: > > > >> This patchset implements dynamic pegging of kconfig symbol > >> into driver modinfo section > > > > First a

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-23 Thread Luis R. Rodriguez
On Mon, Aug 22, 2016 at 09:35:47PM +0200, Cristina-Gabriela Moraru wrote: > 2016-08-18 19:55 GMT+02:00 Luis R. Rodriguez : > > On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: > > > >> This patchset implements dynamic pegging of kconfig symbol > >> into driver

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-23 Thread Luis R. Rodriguez
On Mon, Aug 22, 2016 at 09:35:47PM +0200, Cristina-Gabriela Moraru wrote: > 2016-08-18 19:55 GMT+02:00 Luis R. Rodriguez : > > On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: > > > >> This patchset implements dynamic pegging of kconfig symbol > >> into driver modinfo section > > >

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-22 Thread Cristina-Gabriela Moraru
2016-08-19 11:07 GMT+02:00 Michal Marek : > On 2016-08-18 19:55, Luis R. Rodriguez wrote: >> On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: >> >>> This patchset implements dynamic pegging of kconfig symbol >>> into driver modinfo section >> >> First a little bit

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-22 Thread Cristina-Gabriela Moraru
2016-08-19 11:07 GMT+02:00 Michal Marek : > On 2016-08-18 19:55, Luis R. Rodriguez wrote: >> On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: >> >>> This patchset implements dynamic pegging of kconfig symbol >>> into driver modinfo section >> >> First a little bit of motivation

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-22 Thread Cristina-Gabriela Moraru
2016-08-18 19:55 GMT+02:00 Luis R. Rodriguez : > On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: > >> This patchset implements dynamic pegging of kconfig symbol >> into driver modinfo section > > First a little bit of motivation here helps, so let me try to >

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-22 Thread Cristina-Gabriela Moraru
2016-08-18 19:55 GMT+02:00 Luis R. Rodriguez : > On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: > >> This patchset implements dynamic pegging of kconfig symbol >> into driver modinfo section > > First a little bit of motivation here helps, so let me try to > help fill in some

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-19 Thread Michal Marek
On 2016-08-18 19:55, Luis R. Rodriguez wrote: > On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: > >> This patchset implements dynamic pegging of kconfig symbol >> into driver modinfo section > > First a little bit of motivation here helps, so let me try to > help fill in some

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-19 Thread Michal Marek
On 2016-08-18 19:55, Luis R. Rodriguez wrote: > On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: > >> This patchset implements dynamic pegging of kconfig symbol >> into driver modinfo section > > First a little bit of motivation here helps, so let me try to > help fill in some

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-18 Thread Luis R. Rodriguez
On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: > This patchset implements dynamic pegging of kconfig symbol > into driver modinfo section First a little bit of motivation here helps, so let me try to help fill in some gaps. This may help explain what you have been working on a

Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute

2016-08-18 Thread Luis R. Rodriguez
On Wed, Aug 17, 2016 at 09:26:58PM +0200, Cristina Moraru wrote: > This patchset implements dynamic pegging of kconfig symbol > into driver modinfo section First a little bit of motivation here helps, so let me try to help fill in some gaps. This may help explain what you have been working on a