Re: [PATCH v4 6/8] create/adjust generated/autoksyms.h

Hi Nicolas, sorry for the late review. Dne 29.2.2016 v 04:48 Nicolas Pitre napsal(a): > +set -e > + > +cur_ksyms_file="include/generated/autoksyms.h" > +new_ksyms_file="include/generated/autoksyms.h.tmpnew" > + > +info() { [ "$quiet" != "silent_" ] && printf " %-7s %s\n" "$1" "$2"; } Using &&

Re: [PATCH v4 6/8] create/adjust generated/autoksyms.h

Hi Nicolas, sorry for the late review. Dne 29.2.2016 v 04:48 Nicolas Pitre napsal(a): > +set -e > + > +cur_ksyms_file="include/generated/autoksyms.h" > +new_ksyms_file="include/generated/autoksyms.h.tmpnew" > + > +info() { [ "$quiet" != "silent_" ] && printf " %-7s %s\n" "$1" "$2"; } Using &&

[PATCH v3] treewide: Remove newlines inside DEFINE_PER_CPU() macros

quot;\1" ctags: Warning: net/ipv4/syncookies.c:53: null expansion of name pattern "\1" This patch removes the newlines in the DEFINE_PER_CPU, so that the respective per-cpu variable definitions can be indexed. Signed-off-by: Michal Marek <mma...@suse.com> --- v2->v3: Reb

[PATCH v3] treewide: Remove newlines inside DEFINE_PER_CPU() macros

quot;\1" ctags: Warning: net/ipv4/syncookies.c:53: null expansion of name pattern "\1" This patch removes the newlines in the DEFINE_PER_CPU, so that the respective per-cpu variable definitions can be indexed. Signed-off-by: Michal Marek --- v2->v3: Rebased and improved chang

Re: [PATCH] lz4c: shut up lz4c output

On 2016-02-29 13:40, Arnd Bergmann wrote: > When KERNEL_LZ4 is set, we always get an output even when building > with 'make -s': > > *** LZ4 Compression CLI , by Yann Collet (May 31 2013) *** > ! Generating compressed LZ4 using Legacy format (deprecated !) ! > Using stdin for input >

Re: [PATCH] lz4c: shut up lz4c output

On 2016-02-29 13:40, Arnd Bergmann wrote: > When KERNEL_LZ4 is set, we always get an output even when building > with 'make -s': > > *** LZ4 Compression CLI , by Yann Collet (May 31 2013) *** > ! Generating compressed LZ4 using Legacy format (deprecated !) ! > Using stdin for input >

Re: [PATCH] kbuild/mkspec: fix grub2 installkernel issue

On 2016-02-26 16:15, Jiri Kosina wrote: > mkspec is copying built kernel to temporrary location > > /boot/vmlinuz-$KERNELRELEASE-rpm > > and runs installkernel on it. This however directly leads to grub2 > menuentry for this suffixed binary being generated as well during the run > of

Re: [PATCH] kbuild/mkspec: fix grub2 installkernel issue

On 2016-02-26 16:15, Jiri Kosina wrote: > mkspec is copying built kernel to temporrary location > > /boot/vmlinuz-$KERNELRELEASE-rpm > > and runs installkernel on it. This however directly leads to grub2 > menuentry for this suffixed binary being generated as well during the run > of

Re: [PATCH] X.509: Fix test for self-signed certificate

On 2016-02-24 15:54, David Howells wrote: > Hi Michal, > > I have the attached patch already in my queue. > > David > --- > commit d19fcb825912c67e09e0575b95accaa42899e07f > Author: David Howells > Date: Wed Feb 24 14:37:54 2016 + > > X.509: Don't treat

Re: [PATCH] X.509: Fix test for self-signed certificate

On 2016-02-24 15:54, David Howells wrote: > Hi Michal, > > I have the attached patch already in my queue. > > David > --- > commit d19fcb825912c67e09e0575b95accaa42899e07f > Author: David Howells > Date: Wed Feb 24 14:37:54 2016 + > > X.509: Don't treat self-signed keys specially Hi

Re: [PATCH] arm64, kbuild: make "make install" not depend on vmlinux

Dne 19.2.2016 v 18:29 Masahiro Yamada napsal(a): > Just for clarification: > > This patch has been applied to arm64 tree. > > Michal, > Just ignore this patch. Sure. I assumed that a patch touching solely arch/arm* will go through the respective arm tree. Michal

Re: [PATCH] arm64, kbuild: make "make install" not depend on vmlinux

Dne 19.2.2016 v 18:29 Masahiro Yamada napsal(a): > Just for clarification: > > This patch has been applied to arm64 tree. > > Michal, > Just ignore this patch. Sure. I assumed that a patch touching solely arch/arm* will go through the respective arm tree. Michal

Re: [PATCH] scripts/coccinelle: modernize

On 2016-02-18 00:47, Nishanth Menon wrote: > On 02/17/2016 05:16 PM, Julia Lawall wrote: >> & is no longer allowed in column 0, since Coccinelle 1.0.4. >> >> Signed-off-by: Julia Lawall [...] > Verified with: > spatch --version > spatch byte-code version

Re: [PATCH] scripts/coccinelle: modernize

On 2016-02-18 00:47, Nishanth Menon wrote: > On 02/17/2016 05:16 PM, Julia Lawall wrote: >> & is no longer allowed in column 0, since Coccinelle 1.0.4. >> >> Signed-off-by: Julia Lawall [...] > Verified with: > spatch --version > spatch byte-code version 1.0.4-00118-gc7cf750d1c44 compiled with

Re: [PATCH] coccinelle: bugon: reduce rule applicability

Dne 3.2.2016 v 19:05 Julia Lawall napsal(a): > Rule r is only use in org or report mode, so only execute it in those > cases. I applied this and the previous three patches to kbuild.git#misc. Thanks, Michal

Re: [PATCH] coccinelle: bugon: reduce rule applicability

Dne 3.2.2016 v 19:05 Julia Lawall napsal(a): > Rule r is only use in org or report mode, so only execute it in those > cases. I applied this and the previous three patches to kbuild.git#misc. Thanks, Michal

Re: [PATCH] scripts/tags.sh: add regex to map kprobe helpers

On Thu, Feb 18, 2016 at 08:56:31PM +0530, Naveen N. Rao wrote: > Add regex for [get|free]_[insn|optinsn|dmainsn]_slot() functions. > > Signed-off-by: Naveen N. Rao Applied to kbuild.git#misc, thanks. Michal

Re: [PATCH] scripts/tags.sh: add regex to map kprobe helpers

On Thu, Feb 18, 2016 at 08:56:31PM +0530, Naveen N. Rao wrote: > Add regex for [get|free]_[insn|optinsn|dmainsn]_slot() functions. > > Signed-off-by: Naveen N. Rao Applied to kbuild.git#misc, thanks. Michal

Re: [PATCH] kbuild: disable Android-specific compiler features

Dne 18.2.2016 v 00:03 Kees Cook napsal(a): > On Fri, Feb 5, 2016 at 4:44 PM, Kees Cook wrote: >> The Android compilers enable some non-standard features by default. While >> most Android build systems inject the needed "-mno-android" option via >> KCFLAGS, it happens too

Re: [PATCH] kbuild: disable Android-specific compiler features

Dne 18.2.2016 v 00:03 Kees Cook napsal(a): > On Fri, Feb 5, 2016 at 4:44 PM, Kees Cook wrote: >> The Android compilers enable some non-standard features by default. While >> most Android build systems inject the needed "-mno-android" option via >> KCFLAGS, it happens too late (at least on

Re: [PATCH] kbuild: fixdep: Check fstat(2) return value

On Tue, Feb 16, 2016 at 10:32:15AM -0500, Tom Rini wrote: > Ping? Thanks! Applied now, thanks. Michal

Re: [PATCH] kbuild: fixdep: Check fstat(2) return value

On Tue, Feb 16, 2016 at 10:32:15AM -0500, Tom Rini wrote: > Ping? Thanks! Applied now, thanks. Michal

[PATCH] MPI: Endianness fix

: 0f74fbf77d45 ("MPI: Fix mpi_read_buffer") Signed-off-by: Michal Marek <mma...@suse.com> --- lib/mpi/mpicoder.c | 39 +-- 1 file changed, 21 insertions(+), 18 deletions(-) diff --git a/lib/mpi/mpicoder.c b/lib/mpi/mpicoder.c index ec533a6c77b5..eb1

[PATCH] MPI: Endianness fix

: 0f74fbf77d45 ("MPI: Fix mpi_read_buffer") Signed-off-by: Michal Marek --- lib/mpi/mpicoder.c | 39 +-- 1 file changed, 21 insertions(+), 18 deletions(-) diff --git a/lib/mpi/mpicoder.c b/lib/mpi/mpicoder.c index ec533a6c77b5..eb15e7dc7b65 100644 --- a/lib/mpi/

Re: [PATCH] ld-version: Drop the 4th and 5th version components

On 2016-02-11 15:25, Maciej W. Rozycki wrote: > ... making upstream development binutils snapshots work as expected, > e.g.: > > $ mips64el-linux-ld --version > GNU ld (GNU Binutils) 2.20.1.20100303 > [...] > $ > > Signed-off-by: Maciej W. Rozycki fwiw, Acke

[PATCH] X.509: Fix test for self-signed certificate

form AuthorityKeyIdentifier") Signed-off-by: Michal Marek --- crypto/asymmetric_keys/x509_public_key.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/crypto/asymmetric_keys/x509_public_key.c b/crypto/asymmetric_keys/x509_public_key.c index 7092d5cbb5d3..2c46e022a2a3 100644 --

[PATCH] X.509: Fix test for self-signed certificate

form AuthorityKeyIdentifier") Signed-off-by: Michal Marek <mma...@suse.com> --- crypto/asymmetric_keys/x509_public_key.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/crypto/asymmetric_keys/x509_public_key.c b/crypto/asymmetric_keys/x509_public_key.c index 7092d5cbb5d3..2c4

Re: [PATCH] ld-version: Drop the 4th and 5th version components

On 2016-02-11 15:25, Maciej W. Rozycki wrote: > ... making upstream development binutils snapshots work as expected, > e.g.: > > $ mips64el-linux-ld --version > GNU ld (GNU Binutils) 2.20.1.20100303 > [...] > $ > > Signed-off-by: Maciej W. Rozycki <ma...@imgte

Re: [PATCH] Kbuild: provide a __UNIQUE_ID for clang

On Mon, Feb 08, 2016 at 03:38:32PM +0100, Arnd Bergmann wrote: > The default __UNIQUE_ID macro in compiler.h fails to work for some drivers: > > drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c:615:1: error: > redefinition of > '__UNIQUE_ID_firmware615' > BRCMF_FW_NVRAM_DEF(4354,

Re: [PATCH] Kbuild: provide a __UNIQUE_ID for clang

On Mon, Feb 08, 2016 at 03:38:32PM +0100, Arnd Bergmann wrote: > The default __UNIQUE_ID macro in compiler.h fails to work for some drivers: > > drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c:615:1: error: > redefinition of > '__UNIQUE_ID_firmware615' > BRCMF_FW_NVRAM_DEF(4354,

Re: [PATCH] scripts/link-vmlinux.sh: force error on kallsyms failure

On 2016-02-04 18:09, Ard Biesheuvel wrote: > - ${NM} -n ${1} | \ > - scripts/kallsyms ${kallsymopt} | \ > - ${CC} ${aflags} -c -o ${2} -x assembler-with-cpp - > + # capture the return code of scripts/kallsyms in $RC > + local RC=`(${NM} -n ${1} | \ > +

Re: [PATCH] scripts/link-vmlinux.sh: force error on kallsyms failure

On 2016-02-04 18:09, Ard Biesheuvel wrote: > - ${NM} -n ${1} | \ > - scripts/kallsyms ${kallsymopt} | \ > - ${CC} ${aflags} -c -o ${2} -x assembler-with-cpp - > + # capture the return code of scripts/kallsyms in $RC > + local RC=`(${NM} -n ${1} | \ > +

Re: [PATCH v2 01/13] [kbuild] handle exports in lib-y objects reliably

arch/ia64/lib into lib-* and don't bother with arch/ia64/lib/built-in.o > anymore. > > Signed-off-by: Al Viro > --- > arch/ia64/hp/sim/boot/Makefile | 2 +- > arch/ia64/lib/Makefile | 8 +++- > scripts/Makefile.build | 20 ++++ > 3 fil

Re: [PATCH v2 01/13] [kbuild] handle exports in lib-y objects reliably

/Makefile.build | 20 > 3 files changed, 24 insertions(+), 6 deletions(-) Acked-by: Michal Marek <mma...@suse.cz> Sorry for the delay. > --- a/scripts/Makefile.build > +++ b/scripts/Makefile.build > @@ -81,6 +81,7 @@ endif > > ifneq ($(str

Re: Dependency issue through subdir's

On 2016-01-25 17:22, Kieran Bingham wrote: > I've got a working patch now for the $(MAKE) version, > however I've had to use $(build)=scripts/gdb/linux build_constants to > make it work. $(build)= is the correct syntax, sorry for the confusion. Michal

Re: Dependency issue through subdir's

On 2016-01-25 12:33, Kieran Bingham wrote: > This is functional on a make -j1 build, however at -j2 and above (from a > clean build), kbuild attempts to generate my file before > include/generated/timeconst.h resulting in the following error: > > --- 8< --- > GEN

Re: Dependency issue through subdir's

On 2016-01-25 12:33, Kieran Bingham wrote: > This is functional on a make -j1 build, however at -j2 and above (from a > clean build), kbuild attempts to generate my file before > include/generated/timeconst.h resulting in the following error: > > --- 8< --- > GEN

Re: Dependency issue through subdir's

On 2016-01-25 17:22, Kieran Bingham wrote: > I've got a working patch now for the $(MAKE) version, > however I've had to use $(build)=scripts/gdb/linux build_constants to > make it work. $(build)= is the correct syntax, sorry for the confusion. Michal

Re: tags: Unify emacs and exuberant rules

1a9709 > > Parent: a1ccdb63b5535dc3446b0a9efc6d97aca82c72ef > > Refname:refs/heads/master > > Author: Michal Marek > > AuthorDate: Wed Oct 14 11:48:06 2015 +0200 > > Committer: Michal Marek > > CommitDate: Tue Jan 5 22:18:48 2016 +0100 > > > > tags: Unify

Re: tags: Unify emacs and exuberant rules

1a9709 > > Parent: a1ccdb63b5535dc3446b0a9efc6d97aca82c72ef > > Refname:refs/heads/master > > Author: Michal Marek <mma...@suse.com> > > AuthorDate: Wed Oct 14 11:48:06 2015 +0200 > > Committer: Michal Marek <mma...@suse.com> > > CommitDate: Tue Jan 5 22:18:

Re: linux-next: manual merge of the akpm-current tree with the kbuild tree

On 2016-01-06 07:54, Stephen Rothwell wrote: > Hi Andrew, > > Today's linux-next merge of the akpm-current tree got a conflict in: > > scripts/tags.sh > > between commit: > > 93209d65c1d3 ("tags: Unify emacs and exuberant rules") > > from the kbuild tree and commit: > > ef0e95208c33

Re: linux-next: manual merge of the akpm-current tree with the kbuild tree

On 2016-01-06 07:54, Stephen Rothwell wrote: > Hi Andrew, > > Today's linux-next merge of the akpm-current tree got a conflict in: > > scripts/tags.sh > > between commit: > > 93209d65c1d3 ("tags: Unify emacs and exuberant rules") > > from the kbuild tree and commit: > > ef0e95208c33

Re: [PATCH v2 7/8] tags: Unify emacs and exuberant rules

Dne 4.12.2015 v 16:26 Michal Marek napsal(a): > The emacs rules were constantly lagging behind the exuberant ones. Use a > single set of rules for both, to make the script easier to maintain. > The language understood by both tools is basic regular expression with > some limita

Re: [PATCH -next] MIPS: VDSO: Fix build error with binutils 2.24 and earlier

On 2015-12-24 13:57, James Hogan wrote: > On Thu, Dec 24, 2015 at 12:48:12PM +, James Hogan wrote: >> Hi Guenter, >> >> On Wed, Dec 23, 2015 at 09:04:31PM -0800, Guenter Roeck wrote: >>> Commit 2a037f310bab ("MIPS: VDSO: Fix build error") tries to fix a build >>> error seen with binutils 2.24

Re: [PATCH -next] MIPS: VDSO: Fix build error with binutils 2.24 and earlier

On 2015-12-24 13:57, James Hogan wrote: > On Thu, Dec 24, 2015 at 12:48:12PM +, James Hogan wrote: >> Hi Guenter, >> >> On Wed, Dec 23, 2015 at 09:04:31PM -0800, Guenter Roeck wrote: >>> Commit 2a037f310bab ("MIPS: VDSO: Fix build error") tries to fix a build >>> error seen with binutils 2.24

Re: [PATCH v2 7/8] tags: Unify emacs and exuberant rules

Dne 4.12.2015 v 16:26 Michal Marek napsal(a): > The emacs rules were constantly lagging behind the exuberant ones. Use a > single set of rules for both, to make the script easier to maintain. > The language understood by both tools is basic regular expression with > some limita

Re: Thoughts about introducing OPTIMIZATION_CFLAG

Dne 4.1.2016 v 12:47 Sedat Dilek napsal(a): > But I think you did not get my problem - to have two different > optimization-levels for a compiler in *one* make-line makes no sense > to me. That we sometimes have -O2 ... -Os on the command line is not a problem, since any same unix tool parses its

Re: Thoughts about introducing OPTIMIZATION_CFLAG

Dne 4.1.2016 v 12:47 Sedat Dilek napsal(a): > But I think you did not get my problem - to have two different > optimization-levels for a compiler in *one* make-line makes no sense > to me. That we sometimes have -O2 ... -Os on the command line is not a problem, since any same unix tool parses its

Re: [PATCH] kbuild: support make dir/file.i for *.S

Dne 14.12.2015 v 17:14 Wang YanQing napsal(a): > On Sat, Dec 12, 2015 at 02:13:03PM -0500, Brian Gerst wrote: >> On Thu, Dec 10, 2015 at 11:33 AM, Wang YanQing wrote: >>> This feature could reduce a little time to expand all >>> the macros in *.S for reading. >>> >>> Signed-off-by: Wang YanQing

Re: [PATCH] kbuild: support make dir/file.i for *.S

Dne 14.12.2015 v 17:14 Wang YanQing napsal(a): > On Sat, Dec 12, 2015 at 02:13:03PM -0500, Brian Gerst wrote: >> On Thu, Dec 10, 2015 at 11:33 AM, Wang YanQing wrote: >>> This feature could reduce a little time to expand all >>> the macros in *.S for reading. >>> >>>

Re: [PATCH] Makefile: revert "Makefile: Document ability to make file.lst and file.S" partially

Dne 10.12.2015 v 17:35 Wang YanQing napsal(a): > Commit 627189797807 ("Makefile: Document ability to make file.lst > and file.S") document ability to make file.S, but there isn't such > ability in kbuild, so revert it. > > Signed-off-by: Wang YanQing Thanks, applied to kbuild.git#kbuild.

Re: [PATCH] kbuild: support make dir/file.i for *.S

Dne 10.12.2015 v 17:33 Wang YanQing napsal(a): > This feature could reduce a little time to expand all > the macros in *.S for reading. > > Signed-off-by: Wang YanQing Makes sense. Applied to kbuild.git#kbuild. Michal -- To unsubscribe from this list: send the line "unsubscribe linux-kernel"

Re: [PATCH] kbuild: support make dir/file.i for *.S

Dne 10.12.2015 v 17:33 Wang YanQing napsal(a): > This feature could reduce a little time to expand all > the macros in *.S for reading. > > Signed-off-by: Wang YanQing Makes sense. Applied to kbuild.git#kbuild. Michal -- To unsubscribe from this list: send the line

Re: [PATCH] Makefile: revert "Makefile: Document ability to make file.lst and file.S" partially

Dne 10.12.2015 v 17:35 Wang YanQing napsal(a): > Commit 627189797807 ("Makefile: Document ability to make file.lst > and file.S") document ability to make file.S, but there isn't such > ability in kbuild, so revert it. > > Signed-off-by: Wang YanQing Thanks, applied to

Re: [PATCH] kbuild: Do not run modules_install and install in paralel

Dne 9.12.2015 v 22:34 Andy Lutomirski napsal(a): > On Fri, Jun 13, 2014 at 2:39 AM, Michal Marek wrote: >> Based on a x86-only patch by Andy Lutomirski >> >> With modular kernels, 'make install' is going to need the installed >> modules at some point to generate the

Re: [RESEND PATCH] kconfig: allow kconfig to handle longer path names

On 2015-12-09 23:56, Markus Mayer wrote: > The current (arbitrary) limit of 128 characters for path names has > proven too short for Android builds, as longer path names are used > there. > > Change conf.c, so it can handle path lengths up to PATH_MAX characters. > > Signed-off-by: Markus Mayer

Re: [PATCH] genksyms: Handle string literals with spaces in reference files

Dne 9.12.2015 v 16:06 Michal Marek napsal(a): > The reference files use spaces to separate tokens, however, we must > preserve spaces inside string literals. Currently the only case in the > tree is struct edac_raw_error_desc in : > > $ KBUILD_SYMTYPES=1 make -s drivers/edac/amd6

Re: [RESEND PATCH] kconfig: allow kconfig to handle longer path names

On 2015-12-09 23:56, Markus Mayer wrote: > The current (arbitrary) limit of 128 characters for path names has > proven too short for Android builds, as longer path names are used > there. > > Change conf.c, so it can handle path lengths up to PATH_MAX characters. > > Signed-off-by: Markus Mayer

Re: [PATCH] genksyms: Handle string literals with spaces in reference files

Dne 9.12.2015 v 16:06 Michal Marek napsal(a): > The reference files use spaces to separate tokens, however, we must > preserve spaces inside string literals. Currently the only case in the > tree is struct edac_raw_error_desc in : > > $ KBUILD_SYMTYPES=1 make -s drivers/edac/amd6

Re: [PATCH] kbuild: Do not run modules_install and install in paralel

Dne 9.12.2015 v 22:34 Andy Lutomirski napsal(a): > On Fri, Jun 13, 2014 at 2:39 AM, Michal Marek <mma...@suse.cz> wrote: >> Based on a x86-only patch by Andy Lutomirski <l...@amacapital.net> >> >> With modular kernels, 'make install' is going to need the i

[PATCH] genksyms: Handle string literals with spaces in reference files

} $ KBUILD_SYMTYPES=1 make -s drivers/edac/amd64_edac.symtypes drivers/edac/amd64_edac.c:527: warning: amd64_get_dram_hole_info: modversion changed because of changes in struct edac_raw_error_desc Signed-off-by: Michal Marek --- scripts/genksyms/genksyms.c | 6 -- 1 file changed, 4 insertions(+), 2

[PATCH] genksyms: Handle string literals with spaces in reference files

} $ KBUILD_SYMTYPES=1 make -s drivers/edac/amd64_edac.symtypes drivers/edac/amd64_edac.c:527: warning: amd64_get_dram_hole_info: modversion changed because of changes in struct edac_raw_error_desc Signed-off-by: Michal Marek <mma...@suse.com> --- scripts/genksyms/genksyms.c | 6 -- 1 file chan

Re: [PATCH v2 8/8] treewide: Remove newlines inside DEFINE_PER_CPU() macros

On 2015-12-07 18:04, Joe Perches wrote: > On Mon, 2015-12-07 at 17:53 +0100, Michal Marek wrote: >> On 2015-12-07 17:33, David Laight wrote: >>> From: Michal Marek >>>> Sent: 04 December 2015 15:26 >>>> Otherwise make tags can't parse them: >>>>

Re: [PATCH v2 8/8] treewide: Remove newlines inside DEFINE_PER_CPU() macros

On 2015-12-07 17:33, David Laight wrote: > From: Michal Marek >> Sent: 04 December 2015 15:26 >> Otherwise make tags can't parse them: >> >> ctags: Warning: arch/ia64/kernel/smp.c:60: null expansion of name pattern >> "\1" > ... > > Seems to

Re: [PATCH] fixdep: constify strrcmp arguments

On 2015-12-05 10:43, Nicolas Iooss wrote: > Hello, > I sent the path below a few weeks ago and did not have any feedback. > Could you please review it? Applied to kbuild.git#kbuild now. Michal -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [PATCH v2 8/8] treewide: Remove newlines inside DEFINE_PER_CPU() macros

On 2015-12-07 18:04, Joe Perches wrote: > On Mon, 2015-12-07 at 17:53 +0100, Michal Marek wrote: >> On 2015-12-07 17:33, David Laight wrote: >>> From: Michal Marek >>>> Sent: 04 December 2015 15:26 >>>> Otherwise make tags can't parse them: >>>>

Re: [PATCH v2 8/8] treewide: Remove newlines inside DEFINE_PER_CPU() macros

On 2015-12-07 17:33, David Laight wrote: > From: Michal Marek >> Sent: 04 December 2015 15:26 >> Otherwise make tags can't parse them: >> >> ctags: Warning: arch/ia64/kernel/smp.c:60: null expansion of name pattern >> "\1" > ... > > Seems to

Re: [PATCH] fixdep: constify strrcmp arguments

On 2015-12-05 10:43, Nicolas Iooss wrote: > Hello, > I sent the path below a few weeks ago and did not have any feedback. > Could you please review it? Applied to kbuild.git#kbuild now. Michal -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

[PATCH v2 6/8] tags: Drop the _PE rule

We are not indexing the userspace tools, so the rules only match some false positives in the kernel code. Signed-off-by: Michal Marek --- v2: Rebased onto current linux-next scripts/tags.sh | 2 -- 1 file changed, 2 deletions(-) diff --git a/scripts/tags.sh b/scripts/tags.sh index

[PATCH v2 5/8] tags: Do not try to index defconfigs

The defconfig files are in predictable locations, so there is no need to index them. Plus, the script was only looking for files named 'defconfig', which only works on a few architectures nowadays. Signed-off-by: Michal Marek --- v2: No change scripts/tags.sh | 12 1 file changed

[PATCH v2 1/8] tags: Treat header files as C code

macros in the code and the C and C++ parsers fail and recover at different points. Also, qconf.h is parsed as C, but that's a negligible regression. Signed-off-by: Michal Marek --- v2: Rebased on current linux-next scripts/tags.sh | 64 - 1

[PATCH v2 8/8] treewide: Remove newlines inside DEFINE_PER_CPU() macros

okies.c:44: null expansion of name pattern "\1" Cc: linux-i...@vger.kernel.org Cc: xen-de...@lists.xenproject.org Cc: linux-a...@vger.kernel.org Cc: rds-de...@oss.oracle.com Cc: net...@vger.kernel.org Signed-off-by: Michal Marek --- v2: No change arch/ia64/kernel/smp.c | 3 +-- drivers/acpi/p

[PATCH v2 2/8] aic7xxx: Avoid name collision with

expansion of name pattern "\3" Cc: linux-s...@vger.kernel.org Signed-off-by: Michal Marek --- v2: No change drivers/scsi/aic7xxx/aic79xx.h | 4 ++-- drivers/scsi/aic7xxx/aic79xx_osm.h | 5 - drivers/scsi/aic7xxx/aic7xxx.h | 2 +- drivers/scsi/aic7xxx/aic7xxx_osm.h | 5 -

[PATCH v2 7/8] tags: Unify emacs and exuberant rules

with these versions of the tools: Exuberant Ctags 5.8, Copyright (C) 1996-2009 Darren Hiebert etags (GNU Emacs 24.5) Signed-off-by: Michal Marek --- v2: Rebased onto current linux-next scripts/tags.sh | 199 ++-- 1 file changed, 108 insertions

[PATCH v2 3/8] tags: Fix erroneous pattern match in a comment

l a better solution is found. Signed-off-by: Michal Marek --- v2: No change scripts/tags.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/tags.sh b/scripts/tags.sh index abcf43eed874..c845b3e2e7b0 100755 --- a/scripts/tags.sh +++ b/scripts/tags.sh @@ -204,7 +204,7 @@

[PATCH v2 0/8] scripts/tags.sh cleanup

-next, ARM KVM patch got merged. Michal Marek (8): tags: Treat header files as C code aic7xxx: Avoid name collision with tags: Fix erroneous pattern match in a comment tags: Process Kconfig files in a single pass tags: Do not try to index defconfigs tags: Drop the _PE rule tags: Unify

[PATCH v2 4/8] tags: Process Kconfig files in a single pass

Signed-off-by: Michal Marek --- v2: No change scripts/tags.sh | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/scripts/tags.sh b/scripts/tags.sh index c845b3e2e7b0..283f772133fe 100755 --- a/scripts/tags.sh +++ b/scripts/tags.sh @@ -219,10 +219,7 @@ exuberant

[PATCH v2 2/8] aic7xxx: Avoid name collision with

expansion of name pattern "\3" Cc: linux-s...@vger.kernel.org Signed-off-by: Michal Marek <mma...@suse.com> --- v2: No change drivers/scsi/aic7xxx/aic79xx.h | 4 ++-- drivers/scsi/aic7xxx/aic79xx_osm.h | 5 - drivers/scsi/aic7xxx/aic7xxx.h | 2 +- drivers/scsi

[PATCH v2 1/8] tags: Treat header files as C code

macros in the code and the C and C++ parsers fail and recover at different points. Also, qconf.h is parsed as C, but that's a negligible regression. Signed-off-by: Michal Marek <mma...@suse.com> --- v2: Rebased on current linux-next scripts/tags.s

[PATCH v2 6/8] tags: Drop the _PE rule

We are not indexing the userspace tools, so the rules only match some false positives in the kernel code. Signed-off-by: Michal Marek <mma...@suse.com> --- v2: Rebased onto current linux-next scripts/tags.sh | 2 -- 1 file changed, 2 deletions(-) diff --git a/scripts/tags.sh b/scripts/t

[PATCH v2 0/8] scripts/tags.sh cleanup

-next, ARM KVM patch got merged. Michal Marek (8): tags: Treat header files as C code aic7xxx: Avoid name collision with tags: Fix erroneous pattern match in a comment tags: Process Kconfig files in a single pass tags: Do not try to index defconfigs tags: Drop the _PE rule tags: Unify

[PATCH v2 4/8] tags: Process Kconfig files in a single pass

Signed-off-by: Michal Marek <mma...@suse.com> --- v2: No change scripts/tags.sh | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/scripts/tags.sh b/scripts/tags.sh index c845b3e2e7b0..283f772133fe 100755 --- a/scripts/tags.sh +++ b/scripts/tags.sh @@ -219,10

[PATCH v2 5/8] tags: Do not try to index defconfigs

The defconfig files are in predictable locations, so there is no need to index them. Plus, the script was only looking for files named 'defconfig', which only works on a few architectures nowadays. Signed-off-by: Michal Marek <mma...@suse.com> --- v2: No change scripts/tags.s

[PATCH v2 3/8] tags: Fix erroneous pattern match in a comment

l a better solution is found. Signed-off-by: Michal Marek <mma...@suse.com> --- v2: No change scripts/tags.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/tags.sh b/scripts/tags.sh index abcf43eed874..c845b3e2e7b0 100755 --- a/scripts/tags.sh +++ b/scripts/tags.sh

[PATCH v2 7/8] tags: Unify emacs and exuberant rules

with these versions of the tools: Exuberant Ctags 5.8, Copyright (C) 1996-2009 Darren Hiebert etags (GNU Emacs 24.5) Signed-off-by: Michal Marek <mma...@suse.com> --- v2: Rebased onto current linux-next scripts/tags.sh | 199 ++--

[PATCH v2 8/8] treewide: Remove newlines inside DEFINE_PER_CPU() macros

okies.c:44: null expansion of name pattern "\1" Cc: linux-i...@vger.kernel.org Cc: xen-de...@lists.xenproject.org Cc: linux-a...@vger.kernel.org Cc: rds-de...@oss.oracle.com Cc: net...@vger.kernel.org Signed-off-by: Michal Marek <mma...@suse.com> --- v2: No change arch/ia64/kernel/smp.c

Re: [PATCH v2 1/2] scripts: Add a recorduidiv program

Dne 1.12.2015 v 19:16 Russell King - ARM Linux napsal(a): > On Tue, Dec 01, 2015 at 12:22:12PM -0500, Steven Rostedt wrote: >> I guess another solution is to do a copy instead of modifying in place >> if it detects the multiple hard link? > > That would be the "transparent" solution. If you

Re: [PATCH v2 1/2] scripts: Add a recorduidiv program

On 2015-12-01 17:19, Russell King - ARM Linux wrote: > On Tue, Dec 01, 2015 at 05:07:05PM +0100, Michal Marek wrote: >> So, both recordmcount and the new recordudiv program are idempotent. >> They check if the to-be-added section is already present and do nothing. > > Th

Re: [PATCH v2 1/2] scripts: Add a recorduidiv program

On 2015-11-30 16:40, Michal Marek wrote: > On 2015-11-30 16:32, Russell King - ARM Linux wrote: >> On Mon, Nov 30, 2015 at 04:11:16PM +0100, Michal Marek wrote: >>> On 2015-11-26 00:47, Russell King - ARM Linux wrote: >>>> Do we have any resolution on these pr

Re: [PATCH v2 1/2] scripts: Add a recorduidiv program

Dne 1.12.2015 v 19:16 Russell King - ARM Linux napsal(a): > On Tue, Dec 01, 2015 at 12:22:12PM -0500, Steven Rostedt wrote: >> I guess another solution is to do a copy instead of modifying in place >> if it detects the multiple hard link? > > That would be the "transparent" solution. If you

Re: [PATCH v2 1/2] scripts: Add a recorduidiv program

On 2015-12-01 17:19, Russell King - ARM Linux wrote: > On Tue, Dec 01, 2015 at 05:07:05PM +0100, Michal Marek wrote: >> So, both recordmcount and the new recordudiv program are idempotent. >> They check if the to-be-added section is already present and do nothing. > > Th

Re: [PATCH v2 1/2] scripts: Add a recorduidiv program

On 2015-11-30 16:40, Michal Marek wrote: > On 2015-11-30 16:32, Russell King - ARM Linux wrote: >> On Mon, Nov 30, 2015 at 04:11:16PM +0100, Michal Marek wrote: >>> On 2015-11-26 00:47, Russell King - ARM Linux wrote: >>>> Do we have any resolution on these pr

Re: [PATCH v2 1/2] scripts: Add a recorduidiv program

On 2015-11-30 16:32, Russell King - ARM Linux wrote: > On Mon, Nov 30, 2015 at 04:11:16PM +0100, Michal Marek wrote: >> On 2015-11-26 00:47, Russell King - ARM Linux wrote: >>> On Wed, Nov 25, 2015 at 01:51:03PM -0800, Stephen Boyd wrote: >>>> The ARM compiler

Re: [PATCH v2 1/2] scripts: Add a recorduidiv program

On 2015-11-26 00:47, Russell King - ARM Linux wrote: > On Wed, Nov 25, 2015 at 01:51:03PM -0800, Stephen Boyd wrote: >> The ARM compiler inserts calls to __aeabi_uidiv() and >> __aeabi_idiv() when it needs to perform division on signed and >> unsigned integers. If a processor has support for the

Re: [PATCH 1/2] kbuild: add AFLAGS_REMOVE_(basename).o option

On 2015-11-30 09:17, Heiko Carstens wrote: > On Sat, Nov 21, 2015 at 11:59:58AM +0100, Heiko Carstens wrote: >> It is already possible to remove CFLAGS with the CFLAGS_REMOVE option >> that was introduced with commit 656ee82cc855 ("kbuild: create new >> CFLAGS_REMOVE_(basename).o option").

Re: [PATCH v2 1/2] scripts: Add a recorduidiv program

On 2015-11-26 00:47, Russell King - ARM Linux wrote: > On Wed, Nov 25, 2015 at 01:51:03PM -0800, Stephen Boyd wrote: >> The ARM compiler inserts calls to __aeabi_uidiv() and >> __aeabi_idiv() when it needs to perform division on signed and >> unsigned integers. If a processor has support for the

Re: [PATCH v2 1/2] scripts: Add a recorduidiv program

On 2015-11-30 16:32, Russell King - ARM Linux wrote: > On Mon, Nov 30, 2015 at 04:11:16PM +0100, Michal Marek wrote: >> On 2015-11-26 00:47, Russell King - ARM Linux wrote: >>> On Wed, Nov 25, 2015 at 01:51:03PM -0800, Stephen Boyd wrote: >>>> The ARM compiler

Re: [PATCH 1/2] kbuild: add AFLAGS_REMOVE_(basename).o option

On 2015-11-30 09:17, Heiko Carstens wrote: > On Sat, Nov 21, 2015 at 11:59:58AM +0100, Heiko Carstens wrote: >> It is already possible to remove CFLAGS with the CFLAGS_REMOVE option >> that was introduced with commit 656ee82cc855 ("kbuild: create new >> CFLAGS_REMOVE_(basename).o option").

Re: [PATCH] logfs: clarify MTD dependency

On 2015-11-27 15:30, Arnd Bergmann wrote: > On Friday 27 November 2015 15:14:06 Michal Marek wrote: >> >> Hi Arnd, >> >> I hit this as well and was about to submit a slightly different fix. Can >> you try the logfs portion of the below patch? Proper changel

Re: [PATCH] logfs: clarify MTD dependency

) evaluates to 1 if CONFIG_FOO=y or we are building a module and CONFIG_FOO=m. Michal >From 63e7ab41e3958e753277728acbe15faf35a5374e Mon Sep 17 00:00:00 2001 From: Michal Marek Date: Fri, 27 Nov 2015 15:08:29 +0100 Subject: [PATCH] kbuild: Use IS_REACHABLE to handle objname-m composite ob

<    1   2   3   4   5   6   7   8   9   10   >