public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Sasha Levin <alexander.levin@microsoft.com>, peterz@infradead.org
Cc: kbuild-all@lists.01.org, mingo@kernel.org,
	linux-kernel@vger.kernel.org, tglx@linutronix.de,
	jolsa@redhat.com, alexey.budankov@linux.intel.com,
	songliubraving@fb.com, acme@redhat.com, allison@lohutok.net,
	sashal@kernel.org
Subject: Re: [PATCH v3 12/14] tools/kernel.h: hide noinstr
Date: Tue, 23 Jun 2020 11:12:30 +0800	[thread overview]
Message-ID: <202006231125.ATxaHHdo%lkp@intel.com> (raw)
In-Reply-To: <20200622224258.1208588-13-sashal@kernel.org>

[-- Attachment #1: Type: text/plain, Size: 8077 bytes --]

Hi Sasha,

I love your patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v5.8-rc2 next-20200622]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use  as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Sasha-Levin/Fix-up-liblockdep-for-5-8-rc/20200623-064735
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git dd0d718152e4c65b173070d48ea9dfc06894c3e5
config: x86_64-rhel (attached as .config)
compiler: gcc-9 (Debian 9.3.0-13) 9.3.0
reproduce (this is a W=1 build):
        # save the attached .config to linux build tree
        make W=1 ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   In file included from objtool.h:13,
                    from weak.c:10:
>> elf.h:42:37: error: expected identifier or '(' before ';' token
      42 |  bool changed, text, rodata, noinstr;
         |                                     ^
   elf.h:43:1: error: no semicolon at end of struct or union [-Werror]
      43 | };
         | ^
   cc1: all warnings being treated as errors
   In file included from objtool.h:13,
                    from orc_dump.c:8:
>> elf.h:42:37: error: expected identifier or '(' before ';' token
      42 |  bool changed, text, rodata, noinstr;
         |                                     ^
   elf.h:43:1: error: no semicolon at end of struct or union [-Werror]
      43 | };
         | ^
   In file included from objtool.h:13,
                    from builtin-orc.c:17:
>> elf.h:42:37: error: expected identifier or '(' before ';' token
      42 |  bool changed, text, rodata, noinstr;
         |                                     ^
   elf.h:43:1: error: no semicolon at end of struct or union [-Werror]
      43 | };
         | ^
   In file included from objtool.h:13,
                    from arch.h:11,
                    from check.h:11,
                    from orc_gen.c:9:
>> elf.h:42:37: error: expected identifier or '(' before ';' token
      42 |  bool changed, text, rodata, noinstr;
         |                                     ^
   elf.h:43:1: error: no semicolon at end of struct or union [-Werror]
      43 | };
         | ^
   In file included from orc_gen.c:9:
   check.h:18:14: error: declaration does not declare anything [-Werror]
      18 |  bool noinstr;
         |              ^
   cc1: all warnings being treated as errors
   mv: cannot stat 'tools/objtool/.weak.o.tmp': No such file or directory
   make[4]: *** [tools/build/Makefile.build:97: tools/objtool/weak.o] Error 1
   make[4]: *** Waiting for unfinished jobs....
   In file included from objtool.h:13,
                    from builtin-check.c:19:
>> elf.h:42:37: error: expected identifier or '(' before ';' token
      42 |  bool changed, text, rodata, noinstr;
         |                                     ^
   elf.h:43:1: error: no semicolon at end of struct or union [-Werror]
      43 | };
         | ^
   In file included from objtool.h:13,
                    from arch.h:11,
                    from check.c:11:
>> elf.h:42:37: error: expected identifier or '(' before ';' token
      42 |  bool changed, text, rodata, noinstr;
         |                                     ^
   elf.h:43:1: error: no semicolon at end of struct or union [-Werror]
      43 | };
         | ^
   In file included from special.h:10,
                    from special.c:15:
>> elf.h:42:37: error: expected identifier or '(' before ';' token
      42 |  bool changed, text, rodata, noinstr;
         |                                     ^
   elf.h:43:1: error: no semicolon at end of struct or union [-Werror]
      43 | };
         | ^
   In file included from check.c:12:
   check.h:18:14: error: declaration does not declare anything [-Werror]
      18 |  bool noinstr;
         |              ^
   cc1: all warnings being treated as errors
   mv: cannot stat 'tools/objtool/.builtin-orc.o.tmp': No such file or directory
   In file included from elf.c:20:
>> elf.h:42:37: error: expected identifier or '(' before ';' token
      42 |  bool changed, text, rodata, noinstr;
         |                                     ^
   elf.h:43:1: error: no semicolon at end of struct or union [-Werror]
      43 | };
         | ^
   make[4]: *** [tools/build/Makefile.build:97: tools/objtool/builtin-orc.o] Error 1
   cc1: all warnings being treated as errors
   mv: cannot stat 'tools/objtool/.builtin-check.o.tmp': No such file or directory
   make[4]: *** [tools/build/Makefile.build:97: tools/objtool/builtin-check.o] Error 1
   mv: cannot stat 'tools/objtool/.orc_gen.o.tmp': No such file or directory
   cc1: all warnings being treated as errors
   make[4]: *** [tools/build/Makefile.build:97: tools/objtool/orc_gen.o] Error 1
   cc1: all warnings being treated as errors
   mv: cannot stat 'tools/objtool/.orc_dump.o.tmp': No such file or directory
   mv: cannot stat 'tools/objtool/.special.o.tmp': No such file or directory
   make[4]: *** [tools/build/Makefile.build:97: tools/objtool/orc_dump.o] Error 1
   make[4]: *** [tools/build/Makefile.build:97: tools/objtool/special.o] Error 1
   cc1: all warnings being treated as errors
   check.c: In function 'init_insn_state':
>> check.c:254:18: error: expected identifier before '=' token
     254 |   state->noinstr = sec->noinstr;
         |                  ^
   check.c: In function 'decode_instructions':
   check.c:282:17: error: expected identifier before '=' token
     282 |    sec->noinstr = true;
         |                 ^
   mv: cannot stat 'tools/objtool/.elf.o.tmp': No such file or directory
   make[4]: *** [tools/build/Makefile.build:97: tools/objtool/elf.o] Error 1
   check.c: In function 'validate_call':
   check.c:2195:21: error: expected identifier before '&&' token
    2195 |  if (state->noinstr && state->instr <= 0 &&
         |                     ^~
   check.c: In function 'validate_return':
   check.c:2230:21: error: expected identifier before '&&' token
    2230 |  if (state->noinstr && state->instr > 0) {
         |                     ^~
   check.c: In function 'validate_branch':
>> check.c:2334:20: error: expected identifier before ')' token
    2334 |   if (state.noinstr)
         |                    ^
   In file included from arch/x86/../../objtool.h:13,
                    from arch/x86/../../arch.h:11,
                    from arch/x86/../../check.h:11,
                    from arch/x86/decode.c:14:
>> arch/x86/../../elf.h:42:37: error: expected identifier or '(' before ';' token
      42 |  bool changed, text, rodata, noinstr;
         |                                     ^
   arch/x86/../../elf.h:43:1: error: no semicolon at end of struct or union [-Werror]
      43 | };
         | ^
   In file included from arch/x86/decode.c:14:
   arch/x86/../../check.h:18:14: error: declaration does not declare anything [-Werror]
      18 |  bool noinstr;
         |              ^
   cc1: all warnings being treated as errors
   cc1: all warnings being treated as errors
   mv: cannot stat 'tools/objtool/.check.o.tmp': No such file or directory
   mv: cannot stat 'tools/objtool/arch/x86/.decode.o.tmp': No such file or directory
   make[4]: *** [tools/build/Makefile.build:96: tools/objtool/check.o] Error 1
   make[5]: *** [tools/build/Makefile.build:97: tools/objtool/arch/x86/decode.o] Error 1
   make[4]: *** [tools/build/Makefile.build:139: arch/x86] Error 2
   make[3]: *** [Makefile:64: tools/objtool/objtool-in.o] Error 2
   make[3]: *** Waiting for unfinished jobs....
   make[2]: *** [Makefile:68: objtool] Error 2
   make[1]: *** [Makefile:1858: tools/objtool] Error 2
   make[1]: Target 'prepare' not remade because of errors.
   make: *** [Makefile:185: __sub-make] Error 2
   make: Target 'prepare' not remade because of errors.

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 45449 bytes --]

  reply	other threads:[~2020-06-23  3:49 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-22 22:42 [PATCH v3 00/14] Fix up liblockdep for 5.8-rc Sasha Levin
2020-06-22 22:42 ` [PATCH v3 01/14] tools headers: Add kprobes.h header Sasha Levin
2020-06-22 22:42 ` [PATCH v3 02/14] tools headers: Add rcupdate.h header Sasha Levin
2020-06-22 22:42 ` [PATCH v3 03/14] tools/kernel.h: extend with dummy RCU functions Sasha Levin
2020-06-22 22:42 ` [PATCH v3 04/14] tools bitmap: add bitmap_andnot definition Sasha Levin
2020-06-22 22:42 ` [PATCH v3 05/14] tools/lib/lockdep: add definition required for IRQ flag tracing Sasha Levin
2020-06-22 22:42 ` [PATCH v3 06/14] tools bitmap: add bitmap_clear definition Sasha Levin
2020-06-22 22:42 ` [PATCH v3 07/14] tools/lib/lockdep: Hook up vsprintf, find_bit, hweight libraries Sasha Levin
2020-06-22 22:42 ` [PATCH v3 08/14] tools/lib/lockdep: Enable building with CONFIG_TRACE_IRQFLAGS Sasha Levin
2020-06-22 22:42 ` [PATCH v3 09/14] tools/lib/lockdep: New stacktrace API Sasha Levin
2020-06-22 22:42 ` [PATCH v3 10/14] tools/lib/lockdep: call lockdep_init_task on init Sasha Levin
2020-06-22 22:42 ` [PATCH v3 11/14] tools/lib/lockdep: switch to using lockdep_init_map_waits Sasha Levin
2020-06-22 22:42 ` [PATCH v3 12/14] tools/kernel.h: hide noinstr Sasha Levin
2020-06-23  3:12   ` kernel test robot [this message]
2020-06-22 22:42 ` [PATCH v3 13/14] tools/lib/lockdep: explicitly declare lockdep_init_task() Sasha Levin
2020-06-22 22:42 ` [PATCH v3 14/14] tools/kernel.h: hide task_struct.hardirq_chain_key Sasha Levin

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=202006231125.ATxaHHdo%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=acme@redhat.com \
    --cc=alexander.levin@microsoft.com \
    --cc=alexey.budankov@linux.intel.com \
    --cc=allison@lohutok.net \
    --cc=jolsa@redhat.com \
    --cc=kbuild-all@lists.01.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@kernel.org \
    --cc=peterz@infradead.org \
    --cc=sashal@kernel.org \
    --cc=songliubraving@fb.com \
    --cc=tglx@linutronix.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox