Linux-rt-users archive on lore.kernel.org
 help / color / Atom feed
* [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)
  To: unlisted-recipients:; (no To-header on input)
  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	[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-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  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 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

* 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

end of thread, back to index

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

Linux-rt-users archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-rt-users/0 linux-rt-users/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-rt-users linux-rt-users/ https://lore.kernel.org/linux-rt-users \
		linux-rt-users@vger.kernel.org linux-rt-users@archiver.kernel.org
	public-inbox-index linux-rt-users

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-rt-users


AGPL code for this site: git clone https://public-inbox.org/ public-inbox