2018-11-19 14:02:16

by Jürgen Groß

Subject: [PATCH] x86/xen: cleanup includes in arch/x86/xen/spinlock.c

arch/x86/xen/spinlock.c includes several headers which are not needed.
Remove the #includes.

Signed-off-by: Juergen Gross <[email protected]>
---
arch/x86/xen/spinlock.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/arch/x86/xen/spinlock.c b/arch/x86/xen/spinlock.c
index 1c8a8816a402..3776122c87cc 100644
--- a/arch/x86/xen/spinlock.c
+++ b/arch/x86/xen/spinlock.c
@@ -3,22 +3,17 @@
* Split spinlock implementation out into its own file, so it can be
* compiled in a FTRACE-compatible way.
*/
-#include <linux/kernel_stat.h>
+#include <linux/kernel.h>
#include <linux/spinlock.h>
-#include <linux/debugfs.h>
-#include <linux/log2.h>
-#include <linux/gfp.h>
#include <linux/slab.h>
#include <linux/atomic.h>

#include <asm/paravirt.h>
#include <asm/qspinlock.h>

-#include <xen/interface/xen.h>
#include <xen/events.h>

#include "xen-ops.h"
-#include "debugfs.h"

static DEFINE_PER_CPU(int, lock_kicker_irq) = -1;
static DEFINE_PER_CPU(char *, irq_name);
--
2.16.4

2018-11-19 15:33:42

by Boris Ostrovsky

Subject: Re: [PATCH] x86/xen: cleanup includes in arch/x86/xen/spinlock.c

On 11/19/18 8:59 AM, Juergen Gross wrote:
> arch/x86/xen/spinlock.c includes several headers which are not needed.
> Remove the #includes.
>
> Signed-off-by: Juergen Gross <[email protected]>
> ---

Reviewed-by: Boris Ostrovsky <[email protected]>