Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1466818
| From | Cristina-Gabriela Moraru <cristina.moraru09@gmail.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [RFC PATCH 2/5] Add CONFIG symbol to module as compilation parameter |
| Date | 2016-08-20 17:20 +0200 |
| Message-ID | <s8gdX-4FQ-13@gated-at.bofh.it> (permalink) |
| References | <s7eHf-5l0-9@gated-at.bofh.it> <s7eHg-5l0-27@gated-at.bofh.it> <s7Gkb-6XF-71@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
2016-08-18 20:10 GMT+02:00 Luis R. Rodriguez <mcgrof@kernel.org>: > On Wed, Aug 17, 2016 at 09:27:00PM +0200, Cristina Moraru wrote: >> Add CONFIG symbol to kernel modules as a define via -D > > Perhaps better worded as: > > When modules have a direct Kconfig CONFIG_ symbol associated with > we want to be able to make it available to the build system when we > are building the module. > > You can then describe you do this with -D. > >> compilation parameter. The CONFIG_FOO symbol for each >> module is determined by the module name, using the >> associations from Module.ksymb. This file is loaded >> using the 'include' directive, thus run like a regular >> makefile. The format of the content of the file is the >> following: >> >> foo_KCONF=CONFIG_FOO >> >> creating a set of Makefile variables foo_KCONF with the >> CONFIG_FOO as values. >> >> The Makefile then adds this value in the compilation >> command with -DKBUILD_KCONF='"CONFIG_FOO"'. > > Very nice. What would it mean if it lacks this ? > This should be explained on the commit log. > If we lack it then KBUILD_KCONF="" and in /sys the module has the kconfig_symbol attribute but with the empty string as content: prompt:/sys$ cat ./module/mptbase/kconfig_symbol prompt:/sys$ I will add it into the commit log. >> This patch is part of a research project within >> Google Summer of Code of porting 'make localmodconfig' >> for backported drivers. The goal is to enable each >> module to expose in /sys its corresponding CONFIG_* option. >> The value of this attribute will be dynamically pegged by >> modpost without requiring extra work from the driver developers. >> Further, this information will be used by a hardware interogation >> tool to extract build information about the existing devices. > > You can leave this out as its part of the cover letter and > not relevant to the patch. You can however mention you will > use this later to make the define more useful in userspace. > >> Signed-off-by: Cristina Moraru <cristina.moraru09@gmail.com> >> --- >> scripts/Makefile.lib | 9 ++++++++- >> 1 file changed, 8 insertions(+), 1 deletion(-) >> >> diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib >> index e7df0f5..8ae9b7f 100644 >> --- a/scripts/Makefile.lib >> +++ b/scripts/Makefile.lib >> @@ -89,6 +89,10 @@ multi-objs-m := $(addprefix $(obj)/,$(multi-objs-m)) >> subdir-ym := $(addprefix $(obj)/,$(subdir-ym)) >> obj-dirs := $(addprefix $(obj)/,$(obj-dirs)) >> >> +# Include Module.ksymb which contains the associations of modules' names >> +# and corresponding CONFIG_* options >> +include Module.ksymb > > Is this file always generated? If not prefixing the include call with - would > be better. If we are going to make this optional, then an ifdef wrapper would > suffice as we know it'd be expected only if the new feature was enabled. Yes. This file is always generated. In case a 'git pull' happened between two 'make' commands, the associations should be updated. Ok. I will add a ifdef. > >> + >> # These flags are needed for modversions and compiling, so we define them here >> # already >> # $(modname_flags) #defines KBUILD_MODNAME as the name of the module it will >> @@ -100,6 +104,9 @@ name-fix = $(squote)$(quote)$(subst $(comma),_,$(subst -,_,$1))$(quote)$(squote) >> basename_flags = -DKBUILD_BASENAME=$(call name-fix,$(basetarget)) >> modname_flags = $(if $(filter 1,$(words $(modname))),\ >> -DKBUILD_MODNAME=$(call name-fix,$(modname))) >> +ksym-fix = $(squote)$(quote)$($(subst $(comma),_,$(subst -,_,$1))_KCONF)$(quote)$(squote) >> +ksymb_flags = $(if $(filter 1,$(words $(modname))),\ >> + -DKBUILD_KSYMB=$(call ksym-fix, $(modname))) > > Are clashes possible with this formula? Can we end up with two results for instance? > If not what prevents current konfig logic and namespace from a clash ? If we do > not have anything to prevent a clash, what can we do to help make such clash not > possible ? I think the fact that modname is unique prevents from having clashes. KBUILD_KSYMB is found according to modname. Currently there is no clash because I enforced the Module.ksymb to have 1-to-1 mapping. The only potential clash I can imagine right now it having the same module name but architecture specific CONFIG_* symbol. If there is a 1-to-1 mapping in Module.ksymb there should be no clash in the makefile. > >> orig_c_flags = $(KBUILD_CPPFLAGS) $(KBUILD_CFLAGS) $(KBUILD_SUBDIR_CCFLAGS) \ >> $(ccflags-y) $(CFLAGS_$(basetarget).o) >> @@ -162,7 +169,7 @@ endif >> >> c_flags = -Wp,-MD,$(depfile) $(NOSTDINC_FLAGS) $(LINUXINCLUDE) \ >> $(__c_flags) $(modkern_cflags) \ >> - $(basename_flags) $(modname_flags) >> + $(basename_flags) $(modname_flags) $(ksymb_flags) >> >> a_flags = -Wp,-MD,$(depfile) $(NOSTDINC_FLAGS) $(LINUXINCLUDE) \ >> $(__a_flags) $(modkern_aflags) >> -- > > Other than that looks good! > > Luis
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[RFC PATCH 0/5] Add CONFIG symbol as module attribute Cristina Moraru <cristina.moraru09@gmail.com> - 2016-08-17 21:30 +0200
[RFC PATCH 2/5] Add CONFIG symbol to module as compilation parameter Cristina Moraru <cristina.moraru09@gmail.com> - 2016-08-17 21:30 +0200
Re: [RFC PATCH 2/5] Add CONFIG symbol to module as compilation parameter "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-08-19 03:00 +0200
Re: [RFC PATCH 2/5] Add CONFIG symbol to module as compilation parameter "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-08-19 03:40 +0200
Re: [RFC PATCH 2/5] Add CONFIG symbol to module as compilation parameter Cristina-Gabriela Moraru <cristina.moraru09@gmail.com> - 2016-08-20 17:20 +0200
Re: [RFC PATCH 2/5] Add CONFIG symbol to module as compilation parameter "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-08-23 21:20 +0200
[RFC PATCH 4/5] Set KCONFIG_KSYMB as value for kconfig_ksymb module attribute Cristina Moraru <cristina.moraru09@gmail.com> - 2016-08-17 21:30 +0200
Re: [RFC PATCH 4/5] Set KCONFIG_KSYMB as value for kconfig_ksymb module attribute "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-08-19 04:10 +0200
Re: [RFC PATCH 4/5] Set KCONFIG_KSYMB as value for kconfig_ksymb module attribute Cristina-Gabriela Moraru <cristina.moraru09@gmail.com> - 2016-08-20 17:20 +0200
Re: [RFC PATCH 4/5] Set KCONFIG_KSYMB as value for kconfig_ksymb module attribute "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-08-23 21:20 +0200
[RFC PATCH 3/5] Trigger Module.ksymb generation in Makefile Cristina Moraru <cristina.moraru09@gmail.com> - 2016-08-17 21:30 +0200
Re: [RFC PATCH 3/5] Trigger Module.ksymb generation in Makefile "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-08-18 21:50 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-08-19 04:10 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute Michal Marek <mmarek@suse.com> - 2016-08-19 11:10 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute Cristina-Gabriela Moraru <cristina.moraru09@gmail.com> - 2016-08-22 21:50 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-08-23 23:40 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute Michal Marek <mmarek@suse.com> - 2016-08-24 13:10 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-08-24 18:40 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute Naveen Kumar <nk1383886@gmail.com> - 2016-08-24 19:40 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute Cristina-Gabriela Moraru <cristina.moraru09@gmail.com> - 2016-08-22 21:40 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-08-23 21:20 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute Christoph Hellwig <hch@lst.de> - 2016-08-25 09:50 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute Johannes Berg <johannes@sipsolutions.net> - 2016-08-25 10:40 +0200
Re: [RFC PATCH 0/5] Add CONFIG symbol as module attribute Michal Marek <mmarek@suse.com> - 2016-08-25 10:50 +0200
csiph-web