Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1449881 > unrolled thread

[PATCH] xen/x86: Define stubs for xen_smp_intr_init/xen_smp_intr_free

Back to article view | Back to linux.kernel

Contents

  [PATCH] xen/x86: Define stubs for xen_smp_intr_init/xen_smp_intr_free Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2016-07-25 23:20 +0200
    Re: [PATCH] xen/x86: Define stubs for  xen_smp_intr_init/xen_smp_intr_free Juergen Gross <jgross@suse.com> - 2016-07-26 07:40 +0200
    [tip:smp/hotplug] xen/x86: Define stubs for  xen_smp_intr_init/xen_smp_intr_free tip-bot for Boris Ostrovsky <tipbot@zytor.com> - 2016-07-28 17:40 +0200

#1449881 — [PATCH] xen/x86: Define stubs for xen_smp_intr_init/xen_smp_intr_free

xen_smp_intr_init() and xen_smp_intr_free() are now called from
enlighten.c and therefore not guaranteed to have CONFIG_SMP.

Instead of adding multiple ifdefs there provide stubs in smp.h

Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
---

The patch is against tip:smp/hotplug which is broken by 
https://lists.xenproject.org/archives/html/xen-devel/2016-03/msg02301.html


 arch/x86/xen/smp.h | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/arch/x86/xen/smp.h b/arch/x86/xen/smp.h
index 45faaf3..a1918d8 100644
--- a/arch/x86/xen/smp.h
+++ b/arch/x86/xen/smp.h
@@ -1,5 +1,6 @@
 #ifndef _XEN_SMP_H
 
+#ifdef CONFIG_SMP
 extern void xen_send_IPI_mask(const struct cpumask *mask,
 			      int vector);
 extern void xen_send_IPI_mask_allbutself(const struct cpumask *mask,
@@ -19,4 +20,12 @@ static inline void xen_pvh_early_cpu_init(int cpu, bool entry)
 }
 #endif
 
+#else /* CONFIG_SMP */
+static inline int xen_smp_intr_init(unsigned int cpu)
+{
+	return 0;
+}
+static inline void xen_smp_intr_free(unsigned int cpu) {}
+#endif /* CONFIG_SMP */
+
 #endif
-- 
2.7.4

[toc] | [next] | [standalone]

#1450384 — Re: [PATCH] xen/x86: Define stubs for xen_smp_intr_init/xen_smp_intr_free

On 25/07/16 23:14, Boris Ostrovsky wrote:
> xen_smp_intr_init() and xen_smp_intr_free() are now called from
> enlighten.c and therefore not guaranteed to have CONFIG_SMP.
> 
> Instead of adding multiple ifdefs there provide stubs in smp.h
> 
> Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>

Reviewed-by: Juergen Gross <jgross@suse.com>


Juergen

[toc] | [prev] | [next] | [standalone]

#1451978 — [tip:smp/hotplug] xen/x86: Define stubs for xen_smp_intr_init/xen_smp_intr_free

Commit-ID:  854e9fa5a56a9771fad4701a427e4844d2cbade1
Gitweb:     http://git.kernel.org/tip/854e9fa5a56a9771fad4701a427e4844d2cbade1
Author:     Boris Ostrovsky <boris.ostrovsky@oracle.com>
AuthorDate: Mon, 25 Jul 2016 17:14:48 -0400
Committer:  Thomas Gleixner <tglx@linutronix.de>
CommitDate: Wed, 27 Jul 2016 12:38:08 +0200

xen/x86: Define stubs for xen_smp_intr_init/xen_smp_intr_free

xen_smp_intr_init() and xen_smp_intr_free() are now called from
enlighten.c and therefore not guaranteed to have CONFIG_SMP.

Instead of adding multiple ifdefs there provide stubs in smp.h

Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Reviewed-by: jgross@suse.com
Cc: xen-devel@lists.xenproject.org
Cc: david.vrabel@citrix.com
Link: http://lkml.kernel.org/r/1469481288-2378-1-git-send-email-boris.ostrovsky@oracle.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>

---
 arch/x86/xen/smp.h | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/arch/x86/xen/smp.h b/arch/x86/xen/smp.h
index 45faaf3..a1918d8 100644
--- a/arch/x86/xen/smp.h
+++ b/arch/x86/xen/smp.h
@@ -1,5 +1,6 @@
 #ifndef _XEN_SMP_H
 
+#ifdef CONFIG_SMP
 extern void xen_send_IPI_mask(const struct cpumask *mask,
 			      int vector);
 extern void xen_send_IPI_mask_allbutself(const struct cpumask *mask,
@@ -19,4 +20,12 @@ static inline void xen_pvh_early_cpu_init(int cpu, bool entry)
 }
 #endif
 
+#else /* CONFIG_SMP */
+static inline int xen_smp_intr_init(unsigned int cpu)
+{
+	return 0;
+}
+static inline void xen_smp_intr_free(unsigned int cpu) {}
+#endif /* CONFIG_SMP */
+
 #endif

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web