All of lore.kernel.org
 help / color / mirror / Atom feed
* [next:master 10653/11539] arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case' statement
@ 2014-12-08 18:40 kbuild test robot
  2014-12-08 21:03 ` Andrew Morton
  0 siblings, 1 reply; 5+ messages in thread
From: kbuild test robot @ 2014-12-08 18:40 UTC (permalink / raw)
  To: David Drysdale; +Cc: kbuild-all, Andrew Morton, Linux Memory Management List

tree:   git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   cf12164be498180dc466ef97194ca7755ea39f3b
commit: b4baa9e36be0651f7eb15077af5e0eff53b7691b [10653/11539] x86: hook up execveat system call
reproduce:
  # apt-get install sparse
  git checkout b4baa9e36be0651f7eb15077af5e0eff53b7691b
  make ARCH=x86_64 allmodconfig
  make C=1 CF=-D__CHECK_ENDIAN__


sparse warnings: (new ones prefixed by >>)

   arch/x86/ia32/audit.c:38:14: sparse: undefined identifier '__NR_execveat'
>> arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case' statement
   arch/x86/ia32/audit.c:38:14: sparse: Expected constant expression in case statement
   arch/x86/ia32/audit.c: In function 'ia32_classify_syscall':
   arch/x86/ia32/audit.c:38:7: error: '__NR_execveat' undeclared (first use in this function)
     case __NR_execveat:
          ^
   arch/x86/ia32/audit.c:38:7: note: each undeclared identifier is reported only once for each function it appears in
--
   arch/x86/kernel/audit_64.c:53:14: sparse: undefined identifier '__NR_execveat'
>> arch/x86/kernel/audit_64.c:53:14: sparse: incompatible types for 'case' statement
   arch/x86/kernel/audit_64.c:53:14: sparse: Expected constant expression in case statement
   arch/x86/kernel/audit_64.c: In function 'audit_classify_syscall':
   arch/x86/kernel/audit_64.c:53:7: error: '__NR_execveat' undeclared (first use in this function)
     case __NR_execveat:
          ^
   arch/x86/kernel/audit_64.c:53:7: note: each undeclared identifier is reported only once for each function it appears in

vim +/case +38 arch/x86/ia32/audit.c

     1	#include <asm/unistd_32.h>
     2	
   > 3	unsigned ia32_dir_class[] = {
     4	#include <asm-generic/audit_dir_write.h>
     5	~0U
     6	};
     7	
     8	unsigned ia32_chattr_class[] = {
     9	#include <asm-generic/audit_change_attr.h>
    10	~0U
    11	};
    12	
    13	unsigned ia32_write_class[] = {
    14	#include <asm-generic/audit_write.h>
    15	~0U
    16	};
    17	
    18	unsigned ia32_read_class[] = {
    19	#include <asm-generic/audit_read.h>
    20	~0U
    21	};
    22	
    23	unsigned ia32_signal_class[] = {
    24	#include <asm-generic/audit_signal.h>
    25	~0U
    26	};
    27	
    28	int ia32_classify_syscall(unsigned syscall)
    29	{
    30		switch (syscall) {
    31		case __NR_open:
    32			return 2;
    33		case __NR_openat:
    34			return 3;
    35		case __NR_socketcall:
    36			return 4;
    37		case __NR_execve:
  > 38		case __NR_execveat:
    39			return 5;
    40		default:
    41			return 1;

---
0-DAY kernel test infrastructure                Open Source Technology Center
http://lists.01.org/mailman/listinfo/kbuild                 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>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [next:master 10653/11539] arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case' statement
  2014-12-08 18:40 [next:master 10653/11539] arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case' statement kbuild test robot
@ 2014-12-08 21:03 ` Andrew Morton
  2014-12-09  9:02   ` David Drysdale
  0 siblings, 1 reply; 5+ messages in thread
From: Andrew Morton @ 2014-12-08 21:03 UTC (permalink / raw)
  To: kbuild test robot
  Cc: David Drysdale, kbuild-all, Linux Memory Management List

On Tue, 9 Dec 2014 02:40:09 +0800 kbuild test robot <fengguang.wu@intel.com> wrote:

> tree:   git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head:   cf12164be498180dc466ef97194ca7755ea39f3b
> commit: b4baa9e36be0651f7eb15077af5e0eff53b7691b [10653/11539] x86: hook up execveat system call
> reproduce:
>   # apt-get install sparse
>   git checkout b4baa9e36be0651f7eb15077af5e0eff53b7691b
>   make ARCH=x86_64 allmodconfig
>   make C=1 CF=-D__CHECK_ENDIAN__
> 
> 
> sparse warnings: (new ones prefixed by >>)
> 
>    arch/x86/ia32/audit.c:38:14: sparse: undefined identifier '__NR_execveat'
> >> arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case' statement
>    arch/x86/ia32/audit.c:38:14: sparse: Expected constant expression in case statement
>    arch/x86/ia32/audit.c: In function 'ia32_classify_syscall':
>    arch/x86/ia32/audit.c:38:7: error: '__NR_execveat' undeclared (first use in this function)
>      case __NR_execveat:
>           ^
>    arch/x86/ia32/audit.c:38:7: note: each undeclared identifier is reported only once for each function it appears in
> --

Confused. This makes no sense and I can't reproduce it.

--
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>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [next:master 10653/11539] arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case' statement
  2014-12-08 21:03 ` Andrew Morton
@ 2014-12-09  9:02   ` David Drysdale
  2014-12-09 19:27     ` Fengguang Wu
  0 siblings, 1 reply; 5+ messages in thread
From: David Drysdale @ 2014-12-09  9:02 UTC (permalink / raw)
  To: Andrew Morton; +Cc: kbuild test robot, kbuild-all, Linux Memory Management List

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

On Mon, Dec 8, 2014 at 9:03 PM, Andrew Morton <akpm@linux-foundation.org>
wrote:

> On Tue, 9 Dec 2014 02:40:09 +0800 kbuild test robot <
> fengguang.wu@intel.com> wrote:
>
> > tree:   git://
> git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > head:   cf12164be498180dc466ef97194ca7755ea39f3b
> > commit: b4baa9e36be0651f7eb15077af5e0eff53b7691b [10653/11539] x86: hook
> up execveat system call
> > reproduce:
> >   # apt-get install sparse
> >   git checkout b4baa9e36be0651f7eb15077af5e0eff53b7691b
> >   make ARCH=x86_64 allmodconfig
> >   make C=1 CF=-D__CHECK_ENDIAN__
> >
> >
> > sparse warnings: (new ones prefixed by >>)
> >
> >    arch/x86/ia32/audit.c:38:14: sparse: undefined identifier
> '__NR_execveat'
> > >> arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case'
> statement
> >    arch/x86/ia32/audit.c:38:14: sparse: Expected constant expression in
> case statement
> >    arch/x86/ia32/audit.c: In function 'ia32_classify_syscall':
> >    arch/x86/ia32/audit.c:38:7: error: '__NR_execveat' undeclared (first
> use in this function)
> >      case __NR_execveat:
> >           ^
> >    arch/x86/ia32/audit.c:38:7: note: each undeclared identifier is
> reported only once for each function it appears in
> > --
>
> Confused. This makes no sense and I can't reproduce it.
>

Ditto.

Someone else did previously[1] have a build problem from a stale copy of
arch/x86/include/generated/asm/unistd_32.h in their tree, but I don't know
how that could happen.

[1] https://lkml.org/lkml/2014/11/25/542

[-- Attachment #2: Type: text/html, Size: 2768 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [next:master 10653/11539] arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case' statement
  2014-12-09  9:02   ` David Drysdale
@ 2014-12-09 19:27     ` Fengguang Wu
  2014-12-09 20:08       ` Fengguang Wu
  0 siblings, 1 reply; 5+ messages in thread
From: Fengguang Wu @ 2014-12-09 19:27 UTC (permalink / raw)
  To: David Drysdale; +Cc: Andrew Morton, kbuild-all, Linux Memory Management List

On Tue, Dec 09, 2014 at 09:02:02AM +0000, David Drysdale wrote:
> On Mon, Dec 8, 2014 at 9:03 PM, Andrew Morton <akpm@linux-foundation.org>
> wrote:
> 
> > On Tue, 9 Dec 2014 02:40:09 +0800 kbuild test robot <
> > fengguang.wu@intel.com> wrote:
> >
> > > tree:   git://
> > git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > > head:   cf12164be498180dc466ef97194ca7755ea39f3b
> > > commit: b4baa9e36be0651f7eb15077af5e0eff53b7691b [10653/11539] x86: hook
> > up execveat system call
> > > reproduce:
> > >   # apt-get install sparse
> > >   git checkout b4baa9e36be0651f7eb15077af5e0eff53b7691b
> > >   make ARCH=x86_64 allmodconfig
> > >   make C=1 CF=-D__CHECK_ENDIAN__
> > >
> > >
> > > sparse warnings: (new ones prefixed by >>)
> > >
> > >    arch/x86/ia32/audit.c:38:14: sparse: undefined identifier
> > '__NR_execveat'
> > > >> arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case'
> > statement
> > >    arch/x86/ia32/audit.c:38:14: sparse: Expected constant expression in
> > case statement
> > >    arch/x86/ia32/audit.c: In function 'ia32_classify_syscall':
> > >    arch/x86/ia32/audit.c:38:7: error: '__NR_execveat' undeclared (first
> > use in this function)
> > >      case __NR_execveat:
> > >           ^
> > >    arch/x86/ia32/audit.c:38:7: note: each undeclared identifier is
> > reported only once for each function it appears in
> > > --
> >
> > Confused. This makes no sense and I can't reproduce it.
> >
> 
> Ditto.

Sorry I cannot reproduce the issue, too. I've tried upgrading sparse.

> Someone else did previously[1] have a build problem from a stale copy of
> arch/x86/include/generated/asm/unistd_32.h in their tree, but I don't know
> how that could happen.
> 
> [1] https://lkml.org/lkml/2014/11/25/542

Since I'm doing incremental builds, it could happen that some left
over generated files lead to interesting errors.

Thanks,
Fengguang

--
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>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [next:master 10653/11539] arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case' statement
  2014-12-09 19:27     ` Fengguang Wu
@ 2014-12-09 20:08       ` Fengguang Wu
  0 siblings, 0 replies; 5+ messages in thread
From: Fengguang Wu @ 2014-12-09 20:08 UTC (permalink / raw)
  To: David Drysdale
  Cc: Andrew Morton, kbuild-all, Linux Memory Management List, Dan Carpenter

On Tue, Dec 09, 2014 at 11:27:51AM -0800, Fengguang Wu wrote:
> On Tue, Dec 09, 2014 at 09:02:02AM +0000, David Drysdale wrote:
> > On Mon, Dec 8, 2014 at 9:03 PM, Andrew Morton <akpm@linux-foundation.org>
> > wrote:
> > 
> > > On Tue, 9 Dec 2014 02:40:09 +0800 kbuild test robot <
> > > fengguang.wu@intel.com> wrote:
> > >
> > > > tree:   git://
> > > git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > > > head:   cf12164be498180dc466ef97194ca7755ea39f3b
> > > > commit: b4baa9e36be0651f7eb15077af5e0eff53b7691b [10653/11539] x86: hook
> > > up execveat system call
> > > > reproduce:
> > > >   # apt-get install sparse
> > > >   git checkout b4baa9e36be0651f7eb15077af5e0eff53b7691b
> > > >   make ARCH=x86_64 allmodconfig
> > > >   make C=1 CF=-D__CHECK_ENDIAN__
> > > >
> > > >
> > > > sparse warnings: (new ones prefixed by >>)
> > > >
> > > >    arch/x86/ia32/audit.c:38:14: sparse: undefined identifier
> > > '__NR_execveat'
> > > > >> arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case'
> > > statement
> > > >    arch/x86/ia32/audit.c:38:14: sparse: Expected constant expression in
> > > case statement
> > > >    arch/x86/ia32/audit.c: In function 'ia32_classify_syscall':
> > > >    arch/x86/ia32/audit.c:38:7: error: '__NR_execveat' undeclared (first
> > > use in this function)
> > > >      case __NR_execveat:
> > > >           ^
> > > >    arch/x86/ia32/audit.c:38:7: note: each undeclared identifier is
> > > reported only once for each function it appears in
> > > > --
> > >
> > > Confused. This makes no sense and I can't reproduce it.
> > >
> > 
> > Ditto.
> 
> Sorry I cannot reproduce the issue, too. I've tried upgrading sparse.
> 
> > Someone else did previously[1] have a build problem from a stale copy of
> > arch/x86/include/generated/asm/unistd_32.h in their tree, but I don't know
> > how that could happen.
> > 
> > [1] https://lkml.org/lkml/2014/11/25/542
> 
> Since I'm doing incremental builds, it could happen that some left
> over generated files lead to interesting errors.

I'll run this during bisect, hopefully it will avoid such kind of
false errors in future.

        rm -fr include/generated/ arch/*/include/generated/

Thanks,
Fengguang

--
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>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-12-09 20:09 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-08 18:40 [next:master 10653/11539] arch/x86/ia32/audit.c:38:14: sparse: incompatible types for 'case' statement kbuild test robot
2014-12-08 21:03 ` Andrew Morton
2014-12-09  9:02   ` David Drysdale
2014-12-09 19:27     ` Fengguang Wu
2014-12-09 20:08       ` Fengguang Wu

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.