linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] fs: compat: Remove warning from COMPATIBLE_IOCTL
@ 2017-04-04 18:07 Matthias Kaehlcke
  2017-04-19 18:14 ` Matthias Kaehlcke
  0 siblings, 1 reply; 6+ messages in thread
From: Matthias Kaehlcke @ 2017-04-04 18:07 UTC (permalink / raw)
  To: Alexander Viro, Mark Charlebois, Behan Webster, Arnd Bergmann
  Cc: linux-kernel, linux-fsdevel, Grant Grundler, Michael Davidson,
	Greg Hackmann, Peter Foley, Matthias Kaehlcke

From: Mark Charlebois <charlebm@gmail.com>

cmd in COMPATIBLE_IOCTL is always a u32, so cast it so there isn't a
warning about an overflow in XFORM.

From: Mark Charlebois <charlebm@gmail.com>
Signed-off-by: Mark Charlebois <charlebm@gmail.com>
Signed-off-by: Behan Webster <behanw@converseincode.com>
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
---
Resending https://patchwork.kernel.org/patch/4961631/

 fs/compat_ioctl.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/compat_ioctl.c b/fs/compat_ioctl.c
index 11d087b2b28e..6116d5275a3e 100644
--- a/fs/compat_ioctl.c
+++ b/fs/compat_ioctl.c
@@ -833,7 +833,7 @@ static int compat_ioctl_preallocate(struct file *file,
  */
 #define XFORM(i) (((i) ^ ((i) << 27) ^ ((i) << 17)) & 0xffffffff)
 
-#define COMPATIBLE_IOCTL(cmd) XFORM(cmd),
+#define COMPATIBLE_IOCTL(cmd) XFORM((u32)cmd),
 /* ioctl should not be warned about even if it's not implemented.
    Valid reasons to use this:
    - It is implemented with ->compat_ioctl on some device, but programs
-- 
2.12.2.715.g7642488e1d-goog

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

* Re: [PATCH] fs: compat: Remove warning from COMPATIBLE_IOCTL
  2017-04-04 18:07 [PATCH] fs: compat: Remove warning from COMPATIBLE_IOCTL Matthias Kaehlcke
@ 2017-04-19 18:14 ` Matthias Kaehlcke
  2017-04-19 20:48   ` Arnd Bergmann
  0 siblings, 1 reply; 6+ messages in thread
From: Matthias Kaehlcke @ 2017-04-19 18:14 UTC (permalink / raw)
  To: Alexander Viro, Mark Charlebois, Behan Webster, Arnd Bergmann
  Cc: linux-kernel, linux-fsdevel, Grant Grundler, Michael Davidson,
	Greg Hackmann, Peter Foley

El Tue, Apr 04, 2017 at 11:07:20AM -0700 Matthias Kaehlcke ha dit:

> From: Mark Charlebois <charlebm@gmail.com>
> 
> cmd in COMPATIBLE_IOCTL is always a u32, so cast it so there isn't a
> warning about an overflow in XFORM.
> 
> From: Mark Charlebois <charlebm@gmail.com>
> Signed-off-by: Mark Charlebois <charlebm@gmail.com>
> Signed-off-by: Behan Webster <behanw@converseincode.com>
> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
> Acked-by: Arnd Bergmann <arnd@arndb.de>
> ---
> Resending https://patchwork.kernel.org/patch/4961631/
> 
>  fs/compat_ioctl.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fs/compat_ioctl.c b/fs/compat_ioctl.c
> index 11d087b2b28e..6116d5275a3e 100644
> --- a/fs/compat_ioctl.c
> +++ b/fs/compat_ioctl.c
> @@ -833,7 +833,7 @@ static int compat_ioctl_preallocate(struct file *file,
>   */
>  #define XFORM(i) (((i) ^ ((i) << 27) ^ ((i) << 17)) & 0xffffffff)
>  
> -#define COMPATIBLE_IOCTL(cmd) XFORM(cmd),
> +#define COMPATIBLE_IOCTL(cmd) XFORM((u32)cmd),
>  /* ioctl should not be warned about even if it's not implemented.
>     Valid reasons to use this:
>     - It is implemented with ->compat_ioctl on some device, but programs

Ping, any feedback on this change?

Thanks

Matthias

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

* Re: [PATCH] fs: compat: Remove warning from COMPATIBLE_IOCTL
  2017-04-19 18:14 ` Matthias Kaehlcke
@ 2017-04-19 20:48   ` Arnd Bergmann
  2017-04-19 21:37     ` Matthias Kaehlcke
  0 siblings, 1 reply; 6+ messages in thread
From: Arnd Bergmann @ 2017-04-19 20:48 UTC (permalink / raw)
  To: Matthias Kaehlcke
  Cc: Alexander Viro, Mark Charlebois, Behan Webster,
	Linux Kernel Mailing List, Linux FS-devel Mailing List,
	Grant Grundler, Michael Davidson, Greg Hackmann, Peter Foley

On Wed, Apr 19, 2017 at 8:14 PM, Matthias Kaehlcke <mka@chromium.org> wrote:
> El Tue, Apr 04, 2017 at 11:07:20AM -0700 Matthias Kaehlcke ha dit:
>
>> From: Mark Charlebois <charlebm@gmail.com>
>>
>> cmd in COMPATIBLE_IOCTL is always a u32, so cast it so there isn't a
>> warning about an overflow in XFORM.
>>
>> From: Mark Charlebois <charlebm@gmail.com>
>> Signed-off-by: Mark Charlebois <charlebm@gmail.com>
>> Signed-off-by: Behan Webster <behanw@converseincode.com>
>> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
>> Acked-by: Arnd Bergmann <arnd@arndb.de>
>> ---
>> Resending https://patchwork.kernel.org/patch/4961631/

The patch looks correct to me,

Acked-by: Arnd Bergmann <arnd@arndb.de>

>>  fs/compat_ioctl.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/fs/compat_ioctl.c b/fs/compat_ioctl.c
>> index 11d087b2b28e..6116d5275a3e 100644
>> --- a/fs/compat_ioctl.c
>> +++ b/fs/compat_ioctl.c
>> @@ -833,7 +833,7 @@ static int compat_ioctl_preallocate(struct file *file,
>>   */
>>  #define XFORM(i) (((i) ^ ((i) << 27) ^ ((i) << 17)) & 0xffffffff)
>>
>> -#define COMPATIBLE_IOCTL(cmd) XFORM(cmd),
>> +#define COMPATIBLE_IOCTL(cmd) XFORM((u32)cmd),
>>  /* ioctl should not be warned about even if it's not implemented.
>>     Valid reasons to use this:
>>     - It is implemented with ->compat_ioctl on some device, but programs
>
> Ping, any feedback on this change?

One minor comment on the patch: when you address a warning in a patch,
it helps to put the compiler warning output into the changelog.

Aside from that, I see that you are upstreaming a number of clang
related patches. I actually have a longer series of clang patches that
I took from llvmlinux and hacked up to the point where I could
build ARM randconfig kernels without any warnings or errors.
If you are interested, I can separate the clang patches from my normal
randconfig build tree and upload the git tree for you to look at and
cherry-pick further patches.

      Arnd

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

* Re: [PATCH] fs: compat: Remove warning from COMPATIBLE_IOCTL
  2017-04-19 20:48   ` Arnd Bergmann
@ 2017-04-19 21:37     ` Matthias Kaehlcke
  2017-04-20  8:45       ` Arnd Bergmann
  0 siblings, 1 reply; 6+ messages in thread
From: Matthias Kaehlcke @ 2017-04-19 21:37 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Alexander Viro, Mark Charlebois, Behan Webster,
	Linux Kernel Mailing List, Linux FS-devel Mailing List,
	Grant Grundler, Michael Davidson, Greg Hackmann, Peter Foley

Hi Arnd,

El Wed, Apr 19, 2017 at 10:48:47PM +0200 Arnd Bergmann ha dit:

> On Wed, Apr 19, 2017 at 8:14 PM, Matthias Kaehlcke <mka@chromium.org> wrote:
> > El Tue, Apr 04, 2017 at 11:07:20AM -0700 Matthias Kaehlcke ha dit:
> >
> >> From: Mark Charlebois <charlebm@gmail.com>
> >>
> >> cmd in COMPATIBLE_IOCTL is always a u32, so cast it so there isn't a
> >> warning about an overflow in XFORM.
> >>
> >> From: Mark Charlebois <charlebm@gmail.com>
> >> Signed-off-by: Mark Charlebois <charlebm@gmail.com>
> >> Signed-off-by: Behan Webster <behanw@converseincode.com>
> >> Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
> >> Acked-by: Arnd Bergmann <arnd@arndb.de>
> >> ---
> >> Resending https://patchwork.kernel.org/patch/4961631/
> 
> The patch looks correct to me,
> 
> Acked-by: Arnd Bergmann <arnd@arndb.de>

Thanks for the review!

> >>  fs/compat_ioctl.c | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/fs/compat_ioctl.c b/fs/compat_ioctl.c
> >> index 11d087b2b28e..6116d5275a3e 100644
> >> --- a/fs/compat_ioctl.c
> >> +++ b/fs/compat_ioctl.c
> >> @@ -833,7 +833,7 @@ static int compat_ioctl_preallocate(struct file *file,
> >>   */
> >>  #define XFORM(i) (((i) ^ ((i) << 27) ^ ((i) << 17)) & 0xffffffff)
> >>
> >> -#define COMPATIBLE_IOCTL(cmd) XFORM(cmd),
> >> +#define COMPATIBLE_IOCTL(cmd) XFORM((u32)cmd),
> >>  /* ioctl should not be warned about even if it's not implemented.
> >>     Valid reasons to use this:
> >>     - It is implemented with ->compat_ioctl on some device, but programs
> >
> > Ping, any feedback on this change?
> 
> One minor comment on the patch: when you address a warning in a patch,
> it helps to put the compiler warning output into the changelog.

Thanks, I added it initially and then wondered if it is too much
noise. Will take it into account in the future.

> Aside from that, I see that you are upstreaming a number of clang
> related patches. I actually have a longer series of clang patches that
> I took from llvmlinux and hacked up to the point where I could
> build ARM randconfig kernels without any warnings or errors.
> If you are interested, I can separate the clang patches from my normal
> randconfig build tree and upload the git tree for you to look at and
> cherry-pick further patches.

Sure, that would be interesting, though I won't promise to take up
everything :)

Cheers

Matthias

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

* Re: [PATCH] fs: compat: Remove warning from COMPATIBLE_IOCTL
  2017-04-19 21:37     ` Matthias Kaehlcke
@ 2017-04-20  8:45       ` Arnd Bergmann
  2017-04-20 19:56         ` Matthias Kaehlcke
  0 siblings, 1 reply; 6+ messages in thread
From: Arnd Bergmann @ 2017-04-20  8:45 UTC (permalink / raw)
  To: Matthias Kaehlcke
  Cc: Alexander Viro, Mark Charlebois, Behan Webster,
	Linux Kernel Mailing List, Linux FS-devel Mailing List,
	Grant Grundler, Michael Davidson, Greg Hackmann, Peter Foley

On Wed, Apr 19, 2017 at 11:37 PM, Matthias Kaehlcke <mka@chromium.org> wrote:
> El Wed, Apr 19, 2017 at 10:48:47PM +0200 Arnd Bergmann ha dit:
>> On Wed, Apr 19, 2017 at 8:14 PM, Matthias Kaehlcke <mka@chromium.org> wrote:
>> > El Tue, Apr 04, 2017 at 11:07:20AM -0700 Matthias Kaehlcke ha dit:

>> Aside from that, I see that you are upstreaming a number of clang
>> related patches. I actually have a longer series of clang patches that
>> I took from llvmlinux and hacked up to the point where I could
>> build ARM randconfig kernels without any warnings or errors.
>> If you are interested, I can separate the clang patches from my normal
>> randconfig build tree and upload the git tree for you to look at and
>> cherry-pick further patches.
>
> Sure, that would be interesting, though I won't promise to take up
> everything :)

I see now that almost all the patches I did have been picked up in the
meantime, so I'm sure you already have most of the remaining contents
as they come from the old llvmlinux project. For reference, I've uploaded
my set to

git://git.kernel.org/pub/scm/linux/kernel/git/arnd/playground.git
next-20170420+llvmlinux

This is rebased from the middle of a larger series, so there might be
problems I introduce. Just have a look to see what you can use.

      Arnd

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

* Re: [PATCH] fs: compat: Remove warning from COMPATIBLE_IOCTL
  2017-04-20  8:45       ` Arnd Bergmann
@ 2017-04-20 19:56         ` Matthias Kaehlcke
  0 siblings, 0 replies; 6+ messages in thread
From: Matthias Kaehlcke @ 2017-04-20 19:56 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Alexander Viro, Mark Charlebois, Behan Webster,
	Linux Kernel Mailing List, Linux FS-devel Mailing List,
	Grant Grundler, Michael Davidson, Greg Hackmann, Peter Foley

El Thu, Apr 20, 2017 at 10:45:47AM +0200 Arnd Bergmann ha dit:

> On Wed, Apr 19, 2017 at 11:37 PM, Matthias Kaehlcke <mka@chromium.org> wrote:
> > El Wed, Apr 19, 2017 at 10:48:47PM +0200 Arnd Bergmann ha dit:
> >> On Wed, Apr 19, 2017 at 8:14 PM, Matthias Kaehlcke <mka@chromium.org> wrote:
> >> > El Tue, Apr 04, 2017 at 11:07:20AM -0700 Matthias Kaehlcke ha dit:
> 
> >> Aside from that, I see that you are upstreaming a number of clang
> >> related patches. I actually have a longer series of clang patches that
> >> I took from llvmlinux and hacked up to the point where I could
> >> build ARM randconfig kernels without any warnings or errors.
> >> If you are interested, I can separate the clang patches from my normal
> >> randconfig build tree and upload the git tree for you to look at and
> >> cherry-pick further patches.
> >
> > Sure, that would be interesting, though I won't promise to take up
> > everything :)
> 
> I see now that almost all the patches I did have been picked up in the
> meantime, so I'm sure you already have most of the remaining contents
> as they come from the old llvmlinux project.

Yeah, I picked a subset of the llvmlinux patches from different
sources.

> For reference, I've uploaded my set to
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/arnd/playground.git
> next-20170420+llvmlinux
> 
> This is rebased from the middle of a larger series, so there might be
> problems I introduce. Just have a look to see what you can use.

Thanks!

Matthias

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

end of thread, other threads:[~2017-04-20 19:56 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-04 18:07 [PATCH] fs: compat: Remove warning from COMPATIBLE_IOCTL Matthias Kaehlcke
2017-04-19 18:14 ` Matthias Kaehlcke
2017-04-19 20:48   ` Arnd Bergmann
2017-04-19 21:37     ` Matthias Kaehlcke
2017-04-20  8:45       ` Arnd Bergmann
2017-04-20 19:56         ` Matthias Kaehlcke

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).