Re: [PATCH] scripts/package: binrpm-pkg do not build source tar file

2014-11-27 Thread Michal Marek
: Michal Marek mma...@suse.cz Cc: Linus Torvalds torva...@linux-foundation.org --- scripts/package/mkspec | 2 ++ 1 file changed, 2 insertions(+) diff --git a/scripts/package/mkspec b/scripts/package/mkspec index 1395760..b6f5e31 100755 --- a/scripts/package/mkspec +++ b/scripts/package

[PATCH] kbuild: Automatically remove stale linux/version.h file

2014-11-27 Thread Michal Marek
this, provided the user does make distclean between builds. Better not rely on the user and delete the stale file each time make is invoked. Cc: Paul Gortmaker paul.gortma...@windriver.com Cc: David Howells dhowe...@redhat.com Signed-off-by: Michal Marek mma...@suse.cz --- Makefile | 4 +++- 1 file

[PATCH] kbuild: Remove duplicate $(cmd) definition in Makefile.clean

2014-11-26 Thread Michal Marek
Makefile.clean includes Kbuild.include since commit 371fdc77 (kbuild: collect shorthands into scripts/Kbuild.include), so there is no need for a local copy. Signed-off-by: Michal Marek --- scripts/Makefile.clean | 5 - 1 file changed, 5 deletions(-) diff --git a/scripts/Makefile.clean b

Re: [PATCH] kbuild: collect shorthands into scripts/Kbuild.include

2014-11-26 Thread Michal Marek
On Wed, Nov 26, 2014 at 07:31:13PM +0900, Masahiro Yamada wrote: > The shorthand "clean" is defined in both the top Makefile and > scripts/Makefile.clean. Likewise, the "hdr-inst" is defined in > both the top Makefile and scripts/Makefile.headersinst. > > To reduce code duplication, this commit

Re: [PATCH] ARM: kbuild: Fix forced rebuild after 'make dtbs'

2014-11-26 Thread Michal Marek
On 2014-11-26 00:51, Russell King - ARM Linux wrote: > On Tue, Nov 25, 2014 at 10:41:35PM +0100, Michal Marek wrote: >> Dne 2.11.2014 v 21:52 Jason Cooper napsal(a): >>> After this patch: >>> >>> f4d4ffc03efc kbuild: dtbs_install: new make target >>>

Re: [PATCH] ARM: kbuild: Fix forced rebuild after 'make dtbs'

2014-11-26 Thread Michal Marek
On 2014-11-26 00:51, Russell King - ARM Linux wrote: On Tue, Nov 25, 2014 at 10:41:35PM +0100, Michal Marek wrote: Dne 2.11.2014 v 21:52 Jason Cooper napsal(a): After this patch: f4d4ffc03efc kbuild: dtbs_install: new make target was added the kernel tree, Linus Walleij noticed that 'make

Re: [PATCH] kbuild: collect shorthands into scripts/Kbuild.include

2014-11-26 Thread Michal Marek
On Wed, Nov 26, 2014 at 07:31:13PM +0900, Masahiro Yamada wrote: The shorthand clean is defined in both the top Makefile and scripts/Makefile.clean. Likewise, the hdr-inst is defined in both the top Makefile and scripts/Makefile.headersinst. To reduce code duplication, this commit collects

[PATCH] kbuild: Remove duplicate $(cmd) definition in Makefile.clean

2014-11-26 Thread Michal Marek
Makefile.clean includes Kbuild.include since commit 371fdc77 (kbuild: collect shorthands into scripts/Kbuild.include), so there is no need for a local copy. Signed-off-by: Michal Marek mma...@suse.cz --- scripts/Makefile.clean | 5 - 1 file changed, 5 deletions(-) diff --git a/scripts

Re: [PATCH] ARM: kbuild: Fix forced rebuild after 'make dtbs'

2014-11-25 Thread Michal Marek
Dne 2.11.2014 v 21:52 Jason Cooper napsal(a): > After this patch: > > f4d4ffc03efc kbuild: dtbs_install: new make target > > was added the kernel tree, Linus Walleij noticed that 'make dtbs' forced > a following 'make zImage' to rebuild the entire tree, even though > nothing had changed. His

Re: [PATCH] kbuild: collect shorthands into the top Makefile

2014-11-25 Thread Michal Marek
Dne 31.10.2014 v 03:12 Masahiro Yamada napsal(a): > Hi Sam, > > On Wed, 29 Oct 2014 19:59:07 +0100 > Sam Ravnborg wrote: > >> On Wed, Oct 29, 2014 at 04:27:31PM +0900, Masahiro Yamada wrote: >>> The motivation of this commit is to avoid duplicated definitions >>> of "clean" and "hdr-inst"

Re: [PATCH RESEND] Kbuild: Add an option to enable GCC VTA

2014-11-25 Thread Michal Marek
Dne 25.11.2014 v 00:46 Josh Stone napsal(a): > On 11/24/2014 01:46 PM, Michal Marek wrote: >> Dne 21.11.2014 v 19:40 Josh Stone napsal(a): >>> Due to recent codegen issues, gcc -fvar-tracking-assignments was >>> unconditionally disabled in commit 2062afb4f804a ("

Re: [PATCH] .gitignore: Add Kdevelop4 project files

2014-11-25 Thread Michal Marek
On Tue, Nov 25, 2014 at 06:42:54PM +0200, Boaz Harrosh wrote: > From: Boaz Harrosh > > I'm not sure what is the costume with such IDE project files. > Most might be dot-files. It is kind of annoying for the Kdevelop4 user. > > So please consider adding *.kdev4 files to be ignored? Sure, why

Re: [PATCH] .gitignore: Add Kdevelop4 project files

2014-11-25 Thread Michal Marek
On Tue, Nov 25, 2014 at 06:42:54PM +0200, Boaz Harrosh wrote: From: Boaz Harrosh b...@plexistor.com I'm not sure what is the costume with such IDE project files. Most might be dot-files. It is kind of annoying for the Kdevelop4 user. So please consider adding *.kdev4 files to be ignored?

Re: [PATCH RESEND] Kbuild: Add an option to enable GCC VTA

2014-11-25 Thread Michal Marek
Dne 25.11.2014 v 00:46 Josh Stone napsal(a): On 11/24/2014 01:46 PM, Michal Marek wrote: Dne 21.11.2014 v 19:40 Josh Stone napsal(a): Due to recent codegen issues, gcc -fvar-tracking-assignments was unconditionally disabled in commit 2062afb4f804a (Fix gcc-4.9.0 miscompilation of load_balance

Re: [PATCH] kbuild: collect shorthands into the top Makefile

2014-11-25 Thread Michal Marek
Dne 31.10.2014 v 03:12 Masahiro Yamada napsal(a): Hi Sam, On Wed, 29 Oct 2014 19:59:07 +0100 Sam Ravnborg s...@ravnborg.org wrote: On Wed, Oct 29, 2014 at 04:27:31PM +0900, Masahiro Yamada wrote: The motivation of this commit is to avoid duplicated definitions of clean and hdr-inst

Re: [PATCH] ARM: kbuild: Fix forced rebuild after 'make dtbs'

2014-11-25 Thread Michal Marek
Dne 2.11.2014 v 21:52 Jason Cooper napsal(a): After this patch: f4d4ffc03efc kbuild: dtbs_install: new make target was added the kernel tree, Linus Walleij noticed that 'make dtbs' forced a following 'make zImage' to rebuild the entire tree, even though nothing had changed. His report:

Re: [PATCH RESEND] Kbuild: Add an option to enable GCC VTA

2014-11-24 Thread Michal Marek
Dne 21.11.2014 v 19:40 Josh Stone napsal(a): > Due to recent codegen issues, gcc -fvar-tracking-assignments was > unconditionally disabled in commit 2062afb4f804a ("Fix gcc-4.9.0 > miscompilation of load_balance() in scheduler"). However, this reduces [...] > With gcc-4.9.2-1.fc22, I can now

Re: [Cocci] bugon.cocci: fix Options at the macro

2014-11-24 Thread Michal Marek
Dne 21.11.2014 v 19:07 Wolfram Sang napsal(a): > On Fri, Nov 07, 2014 at 06:51:05PM +0100, Wolfram Sang wrote: >> On Tue, Oct 28, 2014 at 03:18:24PM -0200, Mauro Carvalho Chehab wrote: >>> The comma after --no-includes makes coccinelle to not run the script: >>> >>> /usr/bin/spatch -D report

Re: [Cocci] bugon.cocci: fix Options at the macro

2014-11-24 Thread Michal Marek
Dne 21.11.2014 v 19:07 Wolfram Sang napsal(a): On Fri, Nov 07, 2014 at 06:51:05PM +0100, Wolfram Sang wrote: On Tue, Oct 28, 2014 at 03:18:24PM -0200, Mauro Carvalho Chehab wrote: The comma after --no-includes makes coccinelle to not run the script: /usr/bin/spatch -D report --very-quiet

Re: [PATCH RESEND] Kbuild: Add an option to enable GCC VTA

2014-11-24 Thread Michal Marek
Dne 21.11.2014 v 19:40 Josh Stone napsal(a): Due to recent codegen issues, gcc -fvar-tracking-assignments was unconditionally disabled in commit 2062afb4f804a (Fix gcc-4.9.0 miscompilation of load_balance() in scheduler). However, this reduces [...] With gcc-4.9.2-1.fc22, I can now build

Re: [PATCH] kbuild: Fix output of make kernelrelease

2014-10-23 Thread Michal Marek
On 2014-10-23 03:11, Steven Rostedt wrote: > On Wed, 22 Oct 2014 21:44:08 +0200 > Michal Marek wrote: > >> Dne 22.10.2014 v 16:19 Steven Rostedt napsal(a): >>> >>> Commit 7ff525712acf "kbuild: fake the "Entering directory ..." message >>&g

Re: [PATCH] kbuild: Fix output of make kernelrelease

2014-10-23 Thread Michal Marek
On 2014-10-22 22:35, Steven Rostedt wrote: > On Wed, 22 Oct 2014 21:44:08 +0200 > Michal Marek wrote: > >> Dne 22.10.2014 v 16:19 Steven Rostedt napsal(a): >>> >>> Commit 7ff525712acf "kbuild: fake the "Entering directory ..." message >>&g

Re: [PATCH] kbuild: Fix output of make kernelrelease

2014-10-23 Thread Michal Marek
On 2014-10-22 22:35, Steven Rostedt wrote: On Wed, 22 Oct 2014 21:44:08 +0200 Michal Marek mma...@suse.cz wrote: Dne 22.10.2014 v 16:19 Steven Rostedt napsal(a): Commit 7ff525712acf kbuild: fake the Entering directory ... message more simply changed the output of make kernelrelease

Re: [PATCH] kbuild: Fix output of make kernelrelease

2014-10-23 Thread Michal Marek
On 2014-10-23 03:11, Steven Rostedt wrote: On Wed, 22 Oct 2014 21:44:08 +0200 Michal Marek mma...@suse.cz wrote: Dne 22.10.2014 v 16:19 Steven Rostedt napsal(a): Commit 7ff525712acf kbuild: fake the Entering directory ... message more simply changed the output of make kernelrelease

Re: [PATCH] kbuild: Fix output of make kernelrelease

2014-10-22 Thread Michal Marek
was ktest.pl which resides in the kernel. Can you please apply this patch? Thanks, Michal >From c660b235e25eee053337e0e6c952e87f39839c63 Mon Sep 17 00:00:00 2001 From: Michal Marek Date: Wed, 22 Oct 2014 21:25:39 +0200 Subject: [PATCH] ktest: Use make -s kernelrelease The previous tail -

Re: [PATCH] kbuild: Fix output of make kernelrelease

2014-10-22 Thread Michal Marek
this patch? Thanks, Michal From c660b235e25eee053337e0e6c952e87f39839c63 Mon Sep 17 00:00:00 2001 From: Michal Marek mma...@suse.cz Date: Wed, 22 Oct 2014 21:25:39 +0200 Subject: [PATCH] ktest: Use make -s kernelrelease The previous tail -1 broke with commit 7ff525712acf (kbuild: fake the Entering

[GIT PULL] kbuild misc changes for v3.17-rc1

2014-10-13 Thread Michal Marek
Michal Marek (1): builddeb: put the dbg files into the correct directory Documentation/kbuild/makefiles.txt | 94 ++- scripts/coccinelle/misc/simple_return.cocci | 180 scripts/package/builddeb| 26 ++-- scripts/tags.sh

Re: [GIT PULL] kbuild changes for v3.18-rc1

2014-10-13 Thread Michal Marek
Dne 13.10.2014 v 21:55 Michal Marek napsal(a): > Hi Linus, > > please pull the following for v3.18-rc1: > > * Fix for handling dependencies of *-objs targets by Masahiro Yamada > * Lots of cleanups in the kbuild machinery, also by Masahiro > * Fixes for the kconfig build to

[GIT PULL] kbuild changes for v3.18-rc1

2014-10-13 Thread Michal Marek
obj- := dummy.o" trick kbuild: remove unnecessary variable initializaions kbuild: remove obj-n and lib-n handling kbuild: arm: Do not define "comma" twice kbuild: simplify build, clean, modbuiltin shorthands Michal Marek (2): kbuild: Make scripts executable

[GIT PULL] kbuild changes for v3.18-rc1

2014-10-13 Thread Michal Marek
kbuild: remove unnecessary variable initializaions kbuild: remove obj-n and lib-n handling kbuild: arm: Do not define comma twice kbuild: simplify build, clean, modbuiltin shorthands Michal Marek (2): kbuild: Make scripts executable kbuild: Do not reference *-n

Re: [GIT PULL] kbuild changes for v3.18-rc1

2014-10-13 Thread Michal Marek
Dne 13.10.2014 v 21:55 Michal Marek napsal(a): Hi Linus, please pull the following for v3.18-rc1: * Fix for handling dependencies of *-objs targets by Masahiro Yamada * Lots of cleanups in the kbuild machinery, also by Masahiro * Fixes for the kconfig build to use an UTF-8 capable ncurses

[GIT PULL] kbuild misc changes for v3.17-rc1

2014-10-13 Thread Michal Marek
Michal Marek (1): builddeb: put the dbg files into the correct directory Documentation/kbuild/makefiles.txt | 94 ++- scripts/coccinelle/misc/simple_return.cocci | 180 scripts/package/builddeb| 26 ++-- scripts/tags.sh

Re: [PATCH v8] checkkconfigsymbols.sh: reimplementation in python

2014-10-08 Thread Michal Marek
On 2014-10-04 11:29, Valentin Rothberg wrote: > On Wed, Oct 1, 2014 at 4:58 PM, Michal Marek wrote: >> On 2014-09-29 19:05, Valentin Rothberg wrote: >>> The scripts/checkkconfigsymbols.sh script searches Kconfig features >>> in the source code that are not defined i

[PATCH] X.509: Fix certificate gathering again

2014-10-08 Thread Michal Marek
. Instead of prepending the absolute path and stripping it again, just check if $(srctree) and $(objtree) are different directories and add $(srctree)/*.x509 only in such case. Fixes: 7e1c047 (kbuild: Use relative path for $(objtree)) Cc: sta...@vger.kernel.org Signed-off-by: Michal Marek --- kernel

[PATCH] X.509: Fix certificate gathering again

2014-10-08 Thread Michal Marek
. Instead of prepending the absolute path and stripping it again, just check if $(srctree) and $(objtree) are different directories and add $(srctree)/*.x509 only in such case. Fixes: 7e1c047 (kbuild: Use relative path for $(objtree)) Cc: sta...@vger.kernel.org Signed-off-by: Michal Marek mma

Re: [PATCH v8] checkkconfigsymbols.sh: reimplementation in python

2014-10-08 Thread Michal Marek
On 2014-10-04 11:29, Valentin Rothberg wrote: On Wed, Oct 1, 2014 at 4:58 PM, Michal Marek mma...@suse.cz wrote: On 2014-09-29 19:05, Valentin Rothberg wrote: The scripts/checkkconfigsymbols.sh script searches Kconfig features in the source code that are not defined in Kconfig

Re: [PATCH] Kbuild: Check for CONFIG_READABLE_ASM when building .s targets

2014-10-07 Thread Michal Marek
On 2014-10-07 14:27, Borislav Petkov wrote: > On Tue, Oct 07, 2014 at 02:13:56PM +0200, Michal Marek wrote: >> This violates the principle of least surprise: >> >> make $file.s >> as -o $file.o $file.s >> >> should be equivalent to >> >> make

Re: [PATCH] Kbuild: Check for CONFIG_READABLE_ASM when building .s targets

2014-10-07 Thread Michal Marek
On 2014-10-05 17:58, Borislav Petkov wrote: > On Sun, Oct 05, 2014 at 05:32:34PM +0200, Borislav Petkov wrote: >> From: Borislav Petkov >> >> We do need to look at the asm output of c files for various reasons. In >> order to do so, we do make .s. >> >> However, it can happen that the Kconfig

Re: [PATCH] deb-pkg: remove obsolete -isp option to dpkg-gencontrol

2014-10-07 Thread Michal Marek
On 2014-10-05 19:43, Asbjoern Sloth Toennesen wrote: > The -isp option has been deprecated, after it became the default > behaviour back in 2006. > > Since dpkg 1.17.11, dpkg-gencontrol emits a warning on -isp usage. > > References: https://bugs.debian.org/215233 > Signed-off-by: Asbjoern Sloth

Re: [PATCH] deb-pkg: remove obsolete -isp option to dpkg-gencontrol

2014-10-07 Thread Michal Marek
On 2014-10-05 19:43, Asbjoern Sloth Toennesen wrote: The -isp option has been deprecated, after it became the default behaviour back in 2006. Since dpkg 1.17.11, dpkg-gencontrol emits a warning on -isp usage. References: https://bugs.debian.org/215233 Signed-off-by: Asbjoern Sloth

Re: [PATCH] Kbuild: Check for CONFIG_READABLE_ASM when building .s targets

2014-10-07 Thread Michal Marek
On 2014-10-05 17:58, Borislav Petkov wrote: On Sun, Oct 05, 2014 at 05:32:34PM +0200, Borislav Petkov wrote: From: Borislav Petkov b...@suse.de We do need to look at the asm output of c files for various reasons. In order to do so, we do make path-to-filename.s. However, it can happen that

Re: [PATCH] Kbuild: Check for CONFIG_READABLE_ASM when building .s targets

2014-10-07 Thread Michal Marek
On 2014-10-07 14:27, Borislav Petkov wrote: On Tue, Oct 07, 2014 at 02:13:56PM +0200, Michal Marek wrote: This violates the principle of least surprise: make $file.s as -o $file.o $file.s should be equivalent to make $file.o I know but we need to enable -g for .s targets so that we get

[PATCH] powerpc/boot: Makefile cleanup

2014-10-02 Thread Michal Marek
The $(image-n) variable will never exist, because unset Kconfig options are '' and not 'n'. Signed-off-by: Michal Marek --- arch/powerpc/boot/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile index ccc25ed

[PATCH] IXP4xx: Makefile cleanup

2014-10-02 Thread Michal Marek
obj-pci-n will never be assigned or used. obj-pci-y does not need to be initialized, because each Kbuild file is processed separately. Signed-off-by: Michal Marek --- arch/arm/mach-ixp4xx/Makefile | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/arm/mach-ixp4xx/Makefile b/arch/arm

Re: [PATCH] kbuild: Do not reference *-n variables in the Makefile

2014-10-02 Thread Michal Marek
On 2014-10-02 15:33, Michal Marek wrote: > Kconfig options are either 'y', 'm', or undefined. > > Signed-off-by: Michal Marek > --- > Makefile | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/Makefile b/Makefile > index efbd2cb..106f

[PATCH] kbuild: Do not reference *-n variables in the Makefile

2014-10-02 Thread Michal Marek
Kconfig options are either 'y', 'm', or undefined. Signed-off-by: Michal Marek --- Makefile | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/Makefile b/Makefile index efbd2cb..106f300 100644 --- a/Makefile +++ b/Makefile @@ -868,9 +868,7 @@ vmlinux-dirs

Re: [PATCH] kbuild: simplify build, clean, modbuiltin shorthands

2014-10-02 Thread Michal Marek
On 2014-09-09 13:03, Masahiro Yamada wrote: > $(if $(KBUILD_SRC),$(srctree)/) was a useful strategy > to omit a long absolute path for in-source-tree build > prior to commit 890676c65d699db3ad82e70cf8fb449031af > (kbuild: Use relative path when building in the source tree). I think it was

Re: [PATCH 3/3] kbuild: remove obj-n and lib-n handling

2014-10-02 Thread Michal Marek
On 2014-09-09 12:26, Masahiro Yamada wrote: > Kconfig never defines CONFIG_* as 'n'. > Now obj-n is only used in firmware/Makefile and it can be > replaced with obj-. No makefile uses lib-n. BTW, there are a few more instances of *-n, none of which seem useful: $ git --no-pager grep -e

Re: [PATCH 3/3] kbuild: remove obj-n and lib-n handling

2014-10-02 Thread Michal Marek
On 2014-09-09 12:26, Masahiro Yamada wrote: > Kconfig never defines CONFIG_* as 'n'. > Now obj-n is only used in firmware/Makefile and it can be > replaced with obj-. No makefile uses lib-n. A possible usecase would be overriding CONFIG_ options on the commandline: make CONFIG_FOOBAR=n but

Re: [PATCH 3/3] kbuild: remove obj-n and lib-n handling

2014-10-02 Thread Michal Marek
On 2014-09-09 12:26, Masahiro Yamada wrote: Kconfig never defines CONFIG_* as 'n'. Now obj-n is only used in firmware/Makefile and it can be replaced with obj-. No makefile uses lib-n. A possible usecase would be overriding CONFIG_ options on the commandline: make CONFIG_FOOBAR=n but

Re: [PATCH 3/3] kbuild: remove obj-n and lib-n handling

2014-10-02 Thread Michal Marek
On 2014-09-09 12:26, Masahiro Yamada wrote: Kconfig never defines CONFIG_* as 'n'. Now obj-n is only used in firmware/Makefile and it can be replaced with obj-. No makefile uses lib-n. BTW, there are a few more instances of *-n, none of which seem useful: $ git --no-pager grep -e '[a-z]-n\'

Re: [PATCH] kbuild: simplify build, clean, modbuiltin shorthands

2014-10-02 Thread Michal Marek
On 2014-09-09 13:03, Masahiro Yamada wrote: $(if $(KBUILD_SRC),$(srctree)/) was a useful strategy to omit a long absolute path for in-source-tree build prior to commit 890676c65d699db3ad82e70cf8fb449031af (kbuild: Use relative path when building in the source tree). I think it was quite

[PATCH] kbuild: Do not reference *-n variables in the Makefile

2014-10-02 Thread Michal Marek
Kconfig options are either 'y', 'm', or undefined. Signed-off-by: Michal Marek mma...@suse.cz --- Makefile | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/Makefile b/Makefile index efbd2cb..106f300 100644 --- a/Makefile +++ b/Makefile @@ -868,9 +868,7 @@ vmlinux-dirs

Re: [PATCH] kbuild: Do not reference *-n variables in the Makefile

2014-10-02 Thread Michal Marek
On 2014-10-02 15:33, Michal Marek wrote: Kconfig options are either 'y', 'm', or undefined. Signed-off-by: Michal Marek mma...@suse.cz --- Makefile | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/Makefile b/Makefile index efbd2cb..106f300 100644 --- a/Makefile

[PATCH] IXP4xx: Makefile cleanup

2014-10-02 Thread Michal Marek
obj-pci-n will never be assigned or used. obj-pci-y does not need to be initialized, because each Kbuild file is processed separately. Signed-off-by: Michal Marek mma...@suse.cz --- arch/arm/mach-ixp4xx/Makefile | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/arm/mach-ixp4xx/Makefile

[PATCH] powerpc/boot: Makefile cleanup

2014-10-02 Thread Michal Marek
The $(image-n) variable will never exist, because unset Kconfig options are '' and not 'n'. Signed-off-by: Michal Marek mma...@suse.cz --- arch/powerpc/boot/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile index

Re: [PATCH 3/3] kbuild: handle C=... and M=... after entering into build directory

2014-10-01 Thread Michal Marek
Dne 9.9.2014 13:02, Masahiro Yamada napsal(a): > sub-make: FORCE > $(Q)$(MAKE) -C $(KBUILD_OUTPUT) KBUILD_SRC=$(CURDIR) \ > - KBUILD_EXTMOD="$(KBUILD_EXTMOD)" -f $(CURDIR)/Makefile \ > - $(filter-out _all sub-make,$(MAKECMDGOALS)) > + -f $(CURDIR)/Makefile $(filter-out _all

Re: [PATCH 1/3] kbuild: fake the "Entering directory ..." message more simply

2014-10-01 Thread Michal Marek
Dne 9.9.2014 13:02, Masahiro Yamada napsal(a): > Commit c2e28dc975ea87feed84415006ae143424912ac7 > (kbuild: Print the name of the build directory) > added a gimmick to show the "Entering directory ...". > > Instead of echoing the hard-coded message (that is, we need to know > the exact message),

Re: [PATCH v8] checkkconfigsymbols.sh: reimplementation in python

2014-10-01 Thread Michal Marek
On 2014-09-29 19:05, Valentin Rothberg wrote: > The scripts/checkkconfigsymbols.sh script searches Kconfig features > in the source code that are not defined in Kconfig. Such identifiers > always evaluate to false and are the source of various kinds of bugs. > However, the shell script is slow and

Re: [PATCH 2/3] limit _shipped to c,h and S files

2014-10-01 Thread Michal Marek
On 2014-10-01 15:36, Nicholas Mc Guire wrote: > > limit _shipped to c,h and S files > > _shipped targets are intended for generated source files which should > not permit .o binary blobs which is currently the case. This patch > simply restricts _shipped targets to .c, .h and .S files. Patches

Re: [PATCH 2/3] limit _shipped to c,h and S files

2014-10-01 Thread Michal Marek
On 2014-10-01 15:36, Nicholas Mc Guire wrote: limit _shipped to c,h and S files _shipped targets are intended for generated source files which should not permit .o binary blobs which is currently the case. This patch simply restricts _shipped targets to .c, .h and .S files. Patches 1/3

Re: [PATCH v8] checkkconfigsymbols.sh: reimplementation in python

2014-10-01 Thread Michal Marek
On 2014-09-29 19:05, Valentin Rothberg wrote: The scripts/checkkconfigsymbols.sh script searches Kconfig features in the source code that are not defined in Kconfig. Such identifiers always evaluate to false and are the source of various kinds of bugs. However, the shell script is slow and it

Re: [PATCH 1/3] kbuild: fake the Entering directory ... message more simply

2014-10-01 Thread Michal Marek
Dne 9.9.2014 13:02, Masahiro Yamada napsal(a): Commit c2e28dc975ea87feed84415006ae143424912ac7 (kbuild: Print the name of the build directory) added a gimmick to show the Entering directory Instead of echoing the hard-coded message (that is, we need to know the exact message), moving

Re: [PATCH 3/3] kbuild: handle C=... and M=... after entering into build directory

2014-10-01 Thread Michal Marek
Dne 9.9.2014 13:02, Masahiro Yamada napsal(a): sub-make: FORCE $(Q)$(MAKE) -C $(KBUILD_OUTPUT) KBUILD_SRC=$(CURDIR) \ - KBUILD_EXTMOD=$(KBUILD_EXTMOD) -f $(CURDIR)/Makefile \ - $(filter-out _all sub-make,$(MAKECMDGOALS)) + -f $(CURDIR)/Makefile $(filter-out _all

Re: [PATCH v7] checkkconfigsymbols.sh: reimplementation in python

2014-09-29 Thread Michal Marek
On 2014-09-29 12:28, Paul Bolle wrote: >> +STMT = r"^\s*(?:if|select|depends\s+on)\s+" + EXPR > > Could please make that "depends on"? Yes, it seems the yacc grammar > accepts any amount of whitespace, but that doesn't make it right to use > anything other than a single space. But then lines

Re: [PATCH 1/1] [RFC] kconfig: warn if an unknown symbol is selected

2014-09-29 Thread Michal Marek
ncies. Feel free to submit as a regular patch with Acked-by: Michal Marek Michal -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH 1/1] [RFC] kconfig: warn if an unknown symbol is selected

2014-09-29 Thread Michal Marek
patch with Acked-by: Michal Marek mma...@suse.cz Michal -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH v7] checkkconfigsymbols.sh: reimplementation in python

2014-09-29 Thread Michal Marek
On 2014-09-29 12:28, Paul Bolle wrote: +STMT = r^\s*(?:if|select|depends\s+on)\s+ + EXPR Could please make that depends on? Yes, it seems the yacc grammar accepts any amount of whitespace, but that doesn't make it right to use anything other than a single space. But then lines that violate

Re: [PATCH v2] coccinelle: misc: semantic patch to delete overly complex return code processing

2014-09-26 Thread Michal Marek
On Sat, Aug 23, 2014 at 07:34:45AM +0200, Julia Lawall wrote: > From: Julia Lawall > > This semantic patch simplifies cases where the effect of the processing of > a function call's return code is just to return the result of the function > directly. It may also delete a local return flag

Re: [PATCH v2] coccinelle: misc: semantic patch to delete overly complex return code processing

2014-09-26 Thread Michal Marek
On Sat, Aug 23, 2014 at 07:34:45AM +0200, Julia Lawall wrote: From: Julia Lawall julia.law...@lip6.fr This semantic patch simplifies cases where the effect of the processing of a function call's return code is just to return the result of the function directly. It may also delete a local

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-25 Thread Michal Marek
On 2014-09-24 20:50, Behan Webster wrote: > On 09/24/14 05:07, Michal Marek wrote: >> On 2014-09-23 21:28, beh...@converseincode.com wrote: >>> From: Mark Charlebois >>> >>> Clang will warn about unknown warnings but will not return false >> You mean un

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-25 Thread Michal Marek
On 2014-09-24 20:50, Behan Webster wrote: On 09/24/14 05:07, Michal Marek wrote: On 2014-09-23 21:28, beh...@converseincode.com wrote: From: Mark Charlebois charl...@gmail.com Clang will warn about unknown warnings but will not return false You mean unknown options, right? 2 kinds

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-24 Thread Michal Marek
On 2014-09-23 21:28, beh...@converseincode.com wrote: > From: Mark Charlebois > > Clang will warn about unknown warnings but will not return false You mean unknown options, right? > unless -Werror is set. GCC will return false if an unknown > warning is passed. > > Adding -Werror make both

Re: [PATCH] kbuild, LLVMLinux: Add -Werror to cc-option to support clang

2014-09-24 Thread Michal Marek
On 2014-09-23 21:28, beh...@converseincode.com wrote: From: Mark Charlebois charl...@gmail.com Clang will warn about unknown warnings but will not return false You mean unknown options, right? unless -Werror is set. GCC will return false if an unknown warning is passed. Adding -Werror

Re: [PATCH 1/5] mips: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
On 2014-09-23 18:02, David Miller wrote: > From: Michal Marek > Date: Tue, 23 Sep 2014 17:44:00 +0200 > >> Commit 5d6be6a5 ("scsi_netlink : Make SCSI_NETLINK dependent on NET >> instead of selecting NET") removed what happened to be the only instance >>

[PATCH 5/5] sparc: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
kernel.org Signed-off-by: Michal Marek --- arch/sparc/configs/sparc64_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/sparc/configs/sparc64_defconfig b/arch/sparc/configs/sparc64_defconfig index 9d8521b..6b68f12 100644 --- a/arch/sparc/configs/sparc64_defconfig +++ b/arch/spa

[PATCH 3/5] powerpc: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
ozlabs.org Signed-off-by: Michal Marek --- arch/powerpc/configs/pmac32_defconfig | 1 + arch/powerpc/configs/ppc64_defconfig | 1 + arch/powerpc/configs/ppc64e_defconfig | 1 + arch/powerpc/configs/pseries_defconfig| 1 + arch/powerpc/configs/pseries_le_defconfig | 1 + 5 files

[PATCH 4/5] sh: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
org Signed-off-by: Michal Marek --- arch/sh/configs/sdk7780_defconfig | 1 + arch/sh/configs/sh2007_defconfig | 1 + 2 files changed, 2 insertions(+) diff --git a/arch/sh/configs/sdk7780_defconfig b/arch/sh/configs/sdk7780_defconfig index 6a96b9a..bbd4c22 100644 --- a/arch/sh/configs/sdk7780

[PATCH 1/5] mips: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
org Signed-off-by: Michal Marek --- arch/mips/configs/gpr_defconfig | 1 + arch/mips/configs/ip27_defconfig| 1 + arch/mips/configs/jazz_defconfig| 1 + arch/mips/configs/loongson3_defconfig | 1 + arch/mips/configs/malta_defconfig | 1 + arch/mi

[PATCH 2/5] parisc: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
kernel.org Signed-off-by: Michal Marek --- arch/parisc/configs/c8000_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/parisc/configs/c8000_defconfig b/arch/parisc/configs/c8000_defconfig index 8249ac9..269c23d 100644 --- a/arch/parisc/configs/c8000_defconfig +++ b/arch/pari

Re: linux-next: build warning after merge of the net tree

2014-09-23 Thread Michal Marek
On 2014-09-18 02:35, Randy Dunlap wrote: > On 09/17/14 17:32, Stephen Rothwell wrote: >> Hi all, >> >> After merging the net tree, today's linux-next build (powerpc >> ppc64_defconfig) produced these warnings: >> >> warning: (PPC_CELL_NATIVE && BLUESTONE && CANYONLANDS && GLACIER && EIGER && >>

Re: [PATCH] x86 generated files in gitignore

2014-09-23 Thread Michal Marek
(please do not top-post) On 2014-09-23 14:57, Matteo Croce wrote: > Didn't know, thanks. > > What about arch/x86/purgatory? That part of the patch seems fine, but it should be sent to the x86 maintainers. Michal -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the

Re: [PATCH 01/15] kbuild: add support of custom paths for "auto.conf*" files

2014-09-23 Thread Michal Marek
On 2014-09-22 23:53, Alexis Berlemont wrote: > +#define PATH_MAXLENGTH 4096 > +const char *conf_get_autoconfigdep_name(void) > +{ > + static char res_value[PATH_MAXLENGTH]; Please use the PATH_MAX macro instead. Michal -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH 01/15] kbuild: add support of custom paths for "auto.conf*" files

2014-09-23 Thread Michal Marek
On 2014-09-22 23:53, Alexis Berlemont wrote: > In the scripts/Makefile.build, use KCONFIG_AUTOCONFIG to include a > custom path for the "auto.conf" file. > > The "*conf" programs were modified so as to work with the environment > variable KCONFIG_AUTOCONFIG. > --- > scripts/Makefile.build |

Re: [PATCH] x86 generated files in gitignore

2014-09-23 Thread Michal Marek
On Wed, Sep 03, 2014 at 10:42:04AM +0200, Matteo Croce wrote: > @@ -10,3 +10,4 @@ recordmcount > docproc > sortextable > asn1_compiler > +bin2c bin2c has been moved to scripts/basic, what you see is probably a leftover from building an older kernel. Michal -- To unsubscribe from this list:

Re: [PATCH] x86 generated files in gitignore

2014-09-23 Thread Michal Marek
On Wed, Sep 03, 2014 at 10:42:04AM +0200, Matteo Croce wrote: @@ -10,3 +10,4 @@ recordmcount docproc sortextable asn1_compiler +bin2c bin2c has been moved to scripts/basic, what you see is probably a leftover from building an older kernel. Michal -- To unsubscribe from this list: send

Re: [PATCH 01/15] kbuild: add support of custom paths for auto.conf* files

2014-09-23 Thread Michal Marek
On 2014-09-22 23:53, Alexis Berlemont wrote: In the scripts/Makefile.build, use KCONFIG_AUTOCONFIG to include a custom path for the auto.conf file. The *conf programs were modified so as to work with the environment variable KCONFIG_AUTOCONFIG. --- scripts/Makefile.build | 3 ++-

Re: [PATCH 01/15] kbuild: add support of custom paths for auto.conf* files

2014-09-23 Thread Michal Marek
On 2014-09-22 23:53, Alexis Berlemont wrote: +#define PATH_MAXLENGTH 4096 +const char *conf_get_autoconfigdep_name(void) +{ + static char res_value[PATH_MAXLENGTH]; Please use the PATH_MAX macro instead. Michal -- To unsubscribe from this list: send the line unsubscribe linux-kernel in

Re: [PATCH] x86 generated files in gitignore

2014-09-23 Thread Michal Marek
(please do not top-post) On 2014-09-23 14:57, Matteo Croce wrote: Didn't know, thanks. What about arch/x86/purgatory? That part of the patch seems fine, but it should be sent to the x86 maintainers. Michal -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body

Re: linux-next: build warning after merge of the net tree

2014-09-23 Thread Michal Marek
On 2014-09-18 02:35, Randy Dunlap wrote: On 09/17/14 17:32, Stephen Rothwell wrote: Hi all, After merging the net tree, today's linux-next build (powerpc ppc64_defconfig) produced these warnings: warning: (PPC_CELL_NATIVE BLUESTONE CANYONLANDS GLACIER EIGER 440EPX 440GRX 440GX

[PATCH 2/5] parisc: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
...@vger.kernel.org Signed-off-by: Michal Marek mma...@suse.cz --- arch/parisc/configs/c8000_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/parisc/configs/c8000_defconfig b/arch/parisc/configs/c8000_defconfig index 8249ac9..269c23d 100644 --- a/arch/parisc/configs/c8000_defconfig

[PATCH 1/5] mips: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
...@linux-mips.org Signed-off-by: Michal Marek mma...@suse.cz --- arch/mips/configs/gpr_defconfig | 1 + arch/mips/configs/ip27_defconfig| 1 + arch/mips/configs/jazz_defconfig| 1 + arch/mips/configs/loongson3_defconfig | 1 + arch/mips/configs/malta_defconfig

[PATCH 4/5] sh: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
...@vger.kernel.org Signed-off-by: Michal Marek mma...@suse.cz --- arch/sh/configs/sdk7780_defconfig | 1 + arch/sh/configs/sh2007_defconfig | 1 + 2 files changed, 2 insertions(+) diff --git a/arch/sh/configs/sdk7780_defconfig b/arch/sh/configs/sdk7780_defconfig index 6a96b9a..bbd4c22 100644 --- a/arch

[PATCH 3/5] powerpc: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
-...@lists.ozlabs.org Signed-off-by: Michal Marek mma...@suse.cz --- arch/powerpc/configs/pmac32_defconfig | 1 + arch/powerpc/configs/ppc64_defconfig | 1 + arch/powerpc/configs/ppc64e_defconfig | 1 + arch/powerpc/configs/pseries_defconfig| 1 + arch/powerpc/configs

[PATCH 5/5] sparc: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
...@vger.kernel.org Signed-off-by: Michal Marek mma...@suse.cz --- arch/sparc/configs/sparc64_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/sparc/configs/sparc64_defconfig b/arch/sparc/configs/sparc64_defconfig index 9d8521b..6b68f12 100644 --- a/arch/sparc/configs/sparc64_defconfig

Re: [PATCH 1/5] mips: Set CONFIG_NET=y in defconfigs

2014-09-23 Thread Michal Marek
On 2014-09-23 18:02, David Miller wrote: From: Michal Marek mma...@suse.cz Date: Tue, 23 Sep 2014 17:44:00 +0200 Commit 5d6be6a5 (scsi_netlink : Make SCSI_NETLINK dependent on NET instead of selecting NET) removed what happened to be the only instance of 'select NET'. Defconfigs that were

Re: [PATCH 0/3] Implement /proc/built-in file similar to /proc/modules

2014-09-15 Thread Michal Marek
On 2014-09-14 19:31, Kirill Tkhai wrote: > For example, when I compile "loop" module built-in and "loop" is in > /etc/modules, init script warns about this module is not present and > can't be autoloaded. The script does not store CONFIG_XXX <-> module_xxx > conformity. And nobody stores it. You

Re: [PATCH 0/3] Implement /proc/built-in file similar to /proc/modules

2014-09-15 Thread Michal Marek
On 2014-09-14 17:38, Greg KH wrote: > On Sun, Sep 14, 2014 at 02:18:13PM +0400, Kirill Tkhai wrote: >> This series implements a possibility to show the list of built-in drivers >> to userspace. The names of drivers will be the same as when they are modules. > > Have you looked at /sys/modules/ ?

Re: [PATCH 0/3] Implement /proc/built-in file similar to /proc/modules

2014-09-15 Thread Michal Marek
On 2014-09-14 17:38, Greg KH wrote: On Sun, Sep 14, 2014 at 02:18:13PM +0400, Kirill Tkhai wrote: This series implements a possibility to show the list of built-in drivers to userspace. The names of drivers will be the same as when they are modules. Have you looked at /sys/modules/ ?

Re: [PATCH 0/3] Implement /proc/built-in file similar to /proc/modules

2014-09-15 Thread Michal Marek
On 2014-09-14 19:31, Kirill Tkhai wrote: For example, when I compile loop module built-in and loop is in /etc/modules, init script warns about this module is not present and can't be autoloaded. The script does not store CONFIG_XXX - module_xxx conformity. And nobody stores it. You must be

<    6   7   8   9   10   11   12   13   14   15   >