Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

On Feb 2, 2017 11:43 AM, "Greg Kroah-Hartman" wrote: > On Thu, Feb 02, 2017 at 11:40:32AM +0100, Arnd Bergmann wrote: > > I've done a semi-automated patch to replace CLASSERT() with the respective > > BUILD_BUG_ON() now, but that patch is quite large. > > Should be

Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

On Feb 2, 2017 11:43 AM, "Greg Kroah-Hartman" wrote: > On Thu, Feb 02, 2017 at 11:40:32AM +0100, Arnd Bergmann wrote: > > I've done a semi-automated patch to replace CLASSERT() with the respective > > BUILD_BUG_ON() now, but that patch is quite large. > > Should be easy to script, I missed that

Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

On Thu, Feb 02, 2017 at 11:40:32AM +0100, Arnd Bergmann wrote: > On Thu, Feb 2, 2017 at 10:54 AM, Greg Kroah-Hartman > wrote: > b/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h > >> > index aab15d8112a4..2d5435029185 100644 > >> > ---

Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

On Thu, Feb 02, 2017 at 11:40:32AM +0100, Arnd Bergmann wrote: > On Thu, Feb 2, 2017 at 10:54 AM, Greg Kroah-Hartman > wrote: > b/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h > >> > index aab15d8112a4..2d5435029185 100644 > >> > ---

Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

On Thu, Feb 2, 2017 at 10:54 AM, Greg Kroah-Hartman wrote: b/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h >> > index aab15d8112a4..2d5435029185 100644 >> > --- a/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h >> > +++

Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

On Thu, Feb 2, 2017 at 10:54 AM, Greg Kroah-Hartman wrote: b/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h >> > index aab15d8112a4..2d5435029185 100644 >> > --- a/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h >> > +++

Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

On Thu, Feb 02, 2017 at 07:50:24AM +, Dilger, Andreas wrote: > On Feb 1, 2017, at 09:52, Arnd Bergmann wrote: > > > > lustre uses a fake switch() statement as a compile-time assert, but > > unfortunately > > each use of that causes a warning when building with clang: > > > >

Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

On Thu, Feb 02, 2017 at 07:50:24AM +, Dilger, Andreas wrote: > On Feb 1, 2017, at 09:52, Arnd Bergmann wrote: > > > > lustre uses a fake switch() statement as a compile-time assert, but > > unfortunately > > each use of that causes a warning when building with clang: > > > >

Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

On Feb 1, 2017, at 09:52, Arnd Bergmann wrote: > > lustre uses a fake switch() statement as a compile-time assert, but > unfortunately > each use of that causes a warning when building with clang: > > drivers/staging/lustre/lnet/klnds/socklnd/socklnd.c:2907:2: warning: no case

Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

On Feb 1, 2017, at 09:52, Arnd Bergmann wrote: > > lustre uses a fake switch() statement as a compile-time assert, but > unfortunately > each use of that causes a warning when building with clang: > > drivers/staging/lustre/lnet/klnds/socklnd/socklnd.c:2907:2: warning: no case > matching

[PATCH] staging: lustre: shut up clang warnings on CLASSERT()

lustre uses a fake switch() statement as a compile-time assert, but unfortunately each use of that causes a warning when building with clang: drivers/staging/lustre/lnet/klnds/socklnd/socklnd.c:2907:2: warning: no case matching constant switch condition '42'

[PATCH] staging: lustre: shut up clang warnings on CLASSERT()

lustre uses a fake switch() statement as a compile-time assert, but unfortunately each use of that causes a warning when building with clang: drivers/staging/lustre/lnet/klnds/socklnd/socklnd.c:2907:2: warning: no case matching constant switch condition '42'