From: Jason Baron <jbaron@akamai.com>
To: kbuild test robot <fengguang.wu@intel.com>,
Andrew Morton <akpm@linux-foundation.org>
Cc: kbuild-all@01.org, linux-mm@kvack.org, Michal Hocko <mhocko@suse.com>
Subject: Re: [memcg:since-4.6 827/827] arch/s390/include/asm/jump_label.h:17:32: error: expected ':' before '__stringify'
Date: Wed, 13 Jul 2016 13:43:44 -0400 [thread overview]
Message-ID: <57867DD0.9080801@akamai.com> (raw)
In-Reply-To: <201607140156.2OxakZPq%fengguang.wu@intel.com>
Hi,
This is likely due to the fact that the s390 bits
bits were not pulled into -mm here:
http://lkml.iu.edu/hypermail/linux/kernel/1607.0/03114.html
However, I do see them in linux-next, I think from
the s390 tree. So perhaps, that patch can be pulled
in here as well?
Thanks,
-Jason
On 07/13/2016 01:19 PM, kbuild test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/mhocko/mm.git since-4.6
> head: 92b971babd4ca0b796a247752839f82b6f77a5e7
> commit: 92b971babd4ca0b796a247752839f82b6f77a5e7 [827/827] jump_label: remove bug.h, atomic.h dependencies for HAVE_JUMP_LABEL
> config: s390-default_defconfig (attached as .config)
> compiler: s390x-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205
> reproduce:
> wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> git checkout 92b971babd4ca0b796a247752839f82b6f77a5e7
> # save the attached .config to linux build tree
> make.cross ARCH=s390
>
> All error/warnings (new ones prefixed by >>):
>
> In file included from include/linux/compiler.h:60:0,
> from include/uapi/linux/stddef.h:1,
> from include/linux/stddef.h:4,
> from include/uapi/linux/posix_types.h:4,
> from include/uapi/linux/types.h:13,
> from include/linux/types.h:5,
> from include/linux/jump_label.h:77,
> from arch/s390/lib/uaccess.c:10:
> arch/s390/include/asm/jump_label.h: In function 'arch_static_branch':
>>> arch/s390/include/asm/jump_label.h:17:32: error: expected ':' before '__stringify'
> asm_volatile_goto("0: brcl 0,"__stringify(JUMP_LABEL_NOP_OFFSET)"\n"
> ^
> include/linux/compiler-gcc.h:243:47: note: in definition of macro 'asm_volatile_goto'
> #define asm_volatile_goto(x...) do { asm goto(x); asm (""); } while (0)
> ^
> In file included from include/linux/jump_label.h:105:0,
> from arch/s390/lib/uaccess.c:10:
>>> arch/s390/include/asm/jump_label.h:25:1: warning: label 'label' defined but not used [-Wunused-label]
> label:
> ^
>
> vim +17 arch/s390/include/asm/jump_label.h
>
> d5caa4db Heiko Carstens 2015-01-29 11 /*
> d5caa4db Heiko Carstens 2015-01-29 12 * We use a brcl 0,2 instruction for jump labels at compile time so it
> d5caa4db Heiko Carstens 2015-01-29 13 * can be easily distinguished from a hotpatch generated instruction.
> d5caa4db Heiko Carstens 2015-01-29 14 */
> 11276d53 Peter Zijlstra 2015-07-24 15 static __always_inline bool arch_static_branch(struct static_key *key, bool branch)
> 5373db88 Jan Glauber 2011-03-16 16 {
> d5caa4db Heiko Carstens 2015-01-29 @17 asm_volatile_goto("0: brcl 0,"__stringify(JUMP_LABEL_NOP_OFFSET)"\n"
> 5373db88 Jan Glauber 2011-03-16 18 ".pushsection __jump_table, \"aw\"\n"
> 5a79859a Heiko Carstens 2015-02-12 19 ".balign 8\n"
> 5a79859a Heiko Carstens 2015-02-12 20 ".quad 0b, %l[label], %0\n"
> 5373db88 Jan Glauber 2011-03-16 21 ".popsection\n"
> 11276d53 Peter Zijlstra 2015-07-24 22 : : "X" (&((char *)key)[branch]) : : label);
> 11276d53 Peter Zijlstra 2015-07-24 23
> 11276d53 Peter Zijlstra 2015-07-24 24 return false;
> 11276d53 Peter Zijlstra 2015-07-24 @25 label:
> 11276d53 Peter Zijlstra 2015-07-24 26 return true;
> 11276d53 Peter Zijlstra 2015-07-24 27 }
> 11276d53 Peter Zijlstra 2015-07-24 28
>
> :::::: The code at line 17 was first introduced by commit
> :::::: d5caa4dbf9bd2ad8cd7f6be0ca76722be947182b s390/jump label: use different nop instruction
>
> :::::: TO: Heiko Carstens <heiko.carstens@de.ibm.com>
> :::::: CC: Martin Schwidefsky <schwidefsky@de.ibm.com>
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2016-07-13 17:43 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-13 17:19 [memcg:since-4.6 827/827] arch/s390/include/asm/jump_label.h:17:32: error: expected ':' before '__stringify' kbuild test robot
2016-07-13 17:43 ` Jason Baron [this message]
2016-07-13 18:05 ` Andrew Morton
2016-07-13 18:16 ` Jason Baron
2016-07-14 7:19 ` Michal Hocko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=57867DD0.9080801@akamai.com \
--to=jbaron@akamai.com \
--cc=akpm@linux-foundation.org \
--cc=fengguang.wu@intel.com \
--cc=kbuild-all@01.org \
--cc=linux-mm@kvack.org \
--cc=mhocko@suse.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.