* [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly
@ 2019-09-03 19:13 Sultan Alsawaf
2019-09-16 21:57 ` John Kacur
0 siblings, 1 reply; 7+ messages in thread
From: Sultan Alsawaf @ 2019-09-03 19:13 UTC (permalink / raw)
Cc: sultan, linux-kernel, linux-rt-users, sebastian, tglx, rostedt
From: Sultan Alsawaf <sultan@kerneltoast.com>
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.
Signed-off-by: Sultan Alsawaf <sultan@kerneltoast.com>
---
src/backfire/backfire.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/backfire/backfire.c b/src/backfire/backfire.c
index aaf9c4a..a8ac9f5 100644
--- a/src/backfire/backfire.c
+++ b/src/backfire/backfire.c
@@ -30,8 +30,8 @@
#include <linux/miscdevice.h>
#include <linux/proc_fs.h>
#include <linux/spinlock.h>
+#include <linux/uaccess.h>
-#include <asm/uaccess.h>
#include <asm/system.h>
#define BACKFIRE_MINOR MISC_DYNAMIC_MINOR
--
2.23.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly
2019-09-03 19:13 [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly Sultan Alsawaf
@ 2019-09-16 21:57 ` John Kacur
2019-09-17 7:15 ` Sultan Alsawaf
2019-09-17 10:24 ` Sebastian Andrzej Siewior
0 siblings, 2 replies; 7+ messages in thread
From: John Kacur @ 2019-09-16 21:57 UTC (permalink / raw)
To: Sultan Alsawaf; +Cc: linux-kernel, linux-rt-users, sebastian, tglx, rostedt
On Tue, 3 Sep 2019, Sultan Alsawaf wrote:
> From: Sultan Alsawaf <sultan@kerneltoast.com>
>
> 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.
>
> Signed-off-by: Sultan Alsawaf <sultan@kerneltoast.com>
> ---
> src/backfire/backfire.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/backfire/backfire.c b/src/backfire/backfire.c
> index aaf9c4a..a8ac9f5 100644
> --- a/src/backfire/backfire.c
> +++ b/src/backfire/backfire.c
> @@ -30,8 +30,8 @@
> #include <linux/miscdevice.h>
> #include <linux/proc_fs.h>
> #include <linux/spinlock.h>
> +#include <linux/uaccess.h>
>
> -#include <asm/uaccess.h>
> #include <asm/system.h>
>
> #define BACKFIRE_MINOR MISC_DYNAMIC_MINOR
> --
> 2.23.0
>
>
Signed-off-by: John Kacur <jkacur@redhat.com>
But please in the future
1. Don't cc lkml on this
2. Include the maintainers in your patch
Thanks
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly
2019-09-16 21:57 ` John Kacur
@ 2019-09-17 7:15 ` Sultan Alsawaf
2019-09-17 10:28 ` Kurt Kanzenbach
2019-09-17 14:14 ` John Kacur
2019-09-17 10:24 ` Sebastian Andrzej Siewior
1 sibling, 2 replies; 7+ messages in thread
From: Sultan Alsawaf @ 2019-09-17 7:15 UTC (permalink / raw)
To: John Kacur; +Cc: linux-kernel, linux-rt-users, sebastian, tglx, rostedt
On Mon, Sep 16, 2019 at 11:57:32PM +0200, John Kacur wrote:
> Signed-off-by: John Kacur <jkacur@redhat.com>
> 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:
https://wiki.linuxfoundation.org/realtime/communication/send_rt_patches
I couldn't find any documentation of how to send patches for rt-tests. Is there
a different set of patch submission instructions on a wiki somewhere I missed?
Thanks,
Sultan
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly
2019-09-17 7:15 ` Sultan Alsawaf
@ 2019-09-17 10:28 ` Kurt Kanzenbach
2019-09-17 14:14 ` John Kacur
1 sibling, 0 replies; 7+ messages in thread
From: Kurt Kanzenbach @ 2019-09-17 10:28 UTC (permalink / raw)
To: Sultan Alsawaf
Cc: John Kacur, linux-kernel, linux-rt-users, sebastian, tglx, rostedt
[-- Attachment #1: Type: text/plain, Size: 758 bytes --]
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 <jkacur@redhat.com>
> > 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:
> https://wiki.linuxfoundation.org/realtime/communication/send_rt_patches
I guess, that's for rt kernel patches.
>
> I couldn't find any documentation of how to send patches for
> rt-tests. Is there a different set of patch submission instructions on
> a wiki somewhere I missed?
For rt-tests see the top level MAINTAINERS file on how to send patches.
>
> Thanks,
> Sultan
Thanks,
Kurt
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly
2019-09-17 7:15 ` Sultan Alsawaf
2019-09-17 10:28 ` Kurt Kanzenbach
@ 2019-09-17 14:14 ` John Kacur
1 sibling, 0 replies; 7+ messages in thread
From: John Kacur @ 2019-09-17 14:14 UTC (permalink / raw)
To: Sultan Alsawaf; +Cc: linux-kernel, linux-rt-users, sebastian, tglx, rostedt
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 <jkacur@redhat.com>
> > 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:
> https://wiki.linuxfoundation.org/realtime/communication/send_rt_patches
Those are the instructions for rt kernel code.
rt-tests is a users space program to test the performance of the rt
kernel.
Always just check if there is a MAINTAINERS file (there is)
when you clone the git repo.
Thanks
John
>
> I couldn't find any documentation of how to send patches for rt-tests. Is there
> a different set of patch submission instructions on a wiki somewhere I missed?
>
> Thanks,
> Sultan
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly
2019-09-16 21:57 ` John Kacur
2019-09-17 7:15 ` Sultan Alsawaf
@ 2019-09-17 10:24 ` Sebastian Andrzej Siewior
2019-09-17 14:12 ` John Kacur
1 sibling, 1 reply; 7+ messages in thread
From: Sebastian Andrzej Siewior @ 2019-09-17 10:24 UTC (permalink / raw)
To: John Kacur
Cc: Sultan Alsawaf, linux-kernel, linux-rt-users, sebastian, tglx, rostedt
On 2019-09-16 23:57:32 [+0200], John Kacur wrote:
> Signed-off-by: John Kacur <jkacur@redhat.com>
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 or do I mix up
things?
Sebastian
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly
2019-09-17 10:24 ` Sebastian Andrzej Siewior
@ 2019-09-17 14:12 ` John Kacur
0 siblings, 0 replies; 7+ messages in thread
From: John Kacur @ 2019-09-17 14:12 UTC (permalink / raw)
To: Sebastian Andrzej Siewior
Cc: Sultan Alsawaf, linux-kernel, linux-rt-users, sebastian, tglx, rostedt
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 <jkacur@redhat.com>
>
> 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 or do I mix up
> things?
>
> Sebastian
>
The cost of carrying the code should anyone wish to revive this is very
small, and it wouldn't take much effort to get it going again.
It could go either way, I'm fine with carrying it for now.
John
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-09-17 14:14 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-03 19:13 [PATCH] rt-tests: backfire: Don't include asm/uaccess.h directly Sultan Alsawaf
2019-09-16 21:57 ` John Kacur
2019-09-17 7:15 ` Sultan Alsawaf
2019-09-17 10:28 ` Kurt Kanzenbach
2019-09-17 14:14 ` John Kacur
2019-09-17 10:24 ` Sebastian Andrzej Siewior
2019-09-17 14:12 ` John Kacur
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).