2019-03-27 16:31:18

by Jagadeesh Pagadala

[permalink] [raw]
Subject: [PATCH] arch:sparc:kernel/uprobes.c : Remove duplicate header

From: Jagadeesh Pagadala <[email protected]>

Remove duplicate header which is included twice.

Signed-off-by: Jagadeesh Pagadala <[email protected]>
---
arch/sparc/kernel/uprobes.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/sparc/kernel/uprobes.c b/arch/sparc/kernel/uprobes.c
index d852ae5..c44bf5b 100644
--- a/arch/sparc/kernel/uprobes.c
+++ b/arch/sparc/kernel/uprobes.c
@@ -29,7 +29,6 @@
#include <linux/kdebug.h>

#include <asm/cacheflush.h>
-#include <linux/uaccess.h>

/* Compute the address of the breakpoint instruction and return it.
*
--
1.8.3.1



2019-03-28 06:36:51

by Mukesh Ojha

[permalink] [raw]
Subject: Re: [PATCH] arch:sparc:kernel/uprobes.c : Remove duplicate header


On 3/28/2019 2:58 AM, [email protected] wrote:
> From: Jagadeesh Pagadala <[email protected]>
>
> Remove duplicate header which is included twice.
>
> Signed-off-by: Jagadeesh Pagadala <[email protected]>

Looks good to me. but did you tested at least the compilation with these
patches ?

Reviewed-by: Mukesh Ojha <[email protected]>

-Mukesh


> ---
> arch/sparc/kernel/uprobes.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/sparc/kernel/uprobes.c b/arch/sparc/kernel/uprobes.c
> index d852ae5..c44bf5b 100644
> --- a/arch/sparc/kernel/uprobes.c
> +++ b/arch/sparc/kernel/uprobes.c
> @@ -29,7 +29,6 @@
> #include <linux/kdebug.h>
>
> #include <asm/cacheflush.h>
> -#include <linux/uaccess.h>
>
> /* Compute the address of the breakpoint instruction and return it.
> *

2019-05-09 00:01:18

by David Miller

[permalink] [raw]
Subject: Re: [PATCH] arch:sparc:kernel/uprobes.c : Remove duplicate header

From: [email protected]
Date: Thu, 28 Mar 2019 02:58:45 +0530

> From: Jagadeesh Pagadala <[email protected]>
>
> Remove duplicate header which is included twice.
>
> Signed-off-by: Jagadeesh Pagadala <[email protected]>

Applied.