Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly

2019-09-17 Thread John Kacur
On Tue, 17 Sep 2019, Sultan Alsawaf wrote: > On Mon, Sep 16, 2019 at 11:57:32PM +0200, John Kacur wrote: > > Signed-off-by: John Kacur > > But please in the future > > 1. Don't cc lkml on this > > 2. Include the maintainers in your patch > > Hi, > > Thanks for the sign-off. I was following

Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly

2019-09-17 Thread John Kacur
On Tue, 17 Sep 2019, Sebastian Andrzej Siewior wrote: > On 2019-09-16 23:57:32 [+0200], John Kacur wrote: > > Signed-off-by: John Kacur > > Hmmm. I remember this thing came up years ago in the Debian BTS and then > that backfire module got removed from the Debian package because there > was

Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly

2019-09-17 Thread Kurt Kanzenbach
Hi, On Tue, Sep 17, 2019 at 09:15:46AM +0200, Sultan Alsawaf wrote: > On Mon, Sep 16, 2019 at 11:57:32PM +0200, John Kacur wrote: > > Signed-off-by: John Kacur > > But please in the future > > 1. Don't cc lkml on this > > 2. Include the maintainers in your patch > > Hi, > > Thanks for the

Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly

2019-09-17 Thread Sebastian Andrzej Siewior
On 2019-09-16 23:57:32 [+0200], John Kacur wrote: > Signed-off-by: John Kacur Hmmm. I remember this thing came up years ago in the Debian BTS and then that backfire module got removed from the Debian package because there was no need for it. Just to clarify: is there any need to keep this module

Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly

2019-09-17 Thread Sultan Alsawaf
On Mon, Sep 16, 2019 at 11:57:32PM +0200, John Kacur wrote: > Signed-off-by: John Kacur > But please in the future > 1. Don't cc lkml on this > 2. Include the maintainers in your patch Hi, Thanks for the sign-off. I was following the instructions listed here:

Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly

2019-09-16 Thread John Kacur
On Tue, 3 Sep 2019, Sultan Alsawaf wrote: > From: Sultan Alsawaf > > Architecture-specific uaccess.h headers can have dependencies on > linux/uaccess.h (i.e., VERIFY_WRITE), so it cannot be included directly. > Since linux/uaccess.h includes asm/uaccess.h, just do that instead. > > This

[PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly

2019-09-03 Thread Sultan Alsawaf
From: Sultan Alsawaf Architecture-specific uaccess.h headers can have dependencies on linux/uaccess.h (i.e., VERIFY_WRITE), so it cannot be included directly. Since linux/uaccess.h includes asm/uaccess.h, just do that instead. This fixes compile errors with certain kernels and architectures.