powerpc/pseries: Pass CPPR value to H_XIRR hcall

Message ID 201002011712.58684.markn@au1.ibm.com
State Accepted, archived
Delegated to: Benjamin Herrenschmidt
Headers show

Commit Message

Mark Nelson Feb. 1, 2010, 6:12 a.m.
Now that we properly keep track of the CPPR value (since
49bd3647134ea47420067aea8d1401e722bf2aac, "powerpc/pseries: Track previous
CPPR values to correctly EOI interrupts") we can pass it to the
H_XIRR hcall.

This is needed because the Partition Adjunct Option of new versions of
pHyp extend the H_XIRR hcall to include the CPPR as an input parameter.
Earlier versions not supporting this option just disregard the extra
input parameter, so this doesn't cause any problems for existing systems.

The Partition Adjunct Option is required for future systems that will
support SR-IOV capable devices.

Signed-off-by: Mark Nelson <markn@au1.ibm.com>
 arch/powerpc/platforms/pseries/plpar_wrappers.h |    4 ++--
 arch/powerpc/platforms/pseries/xics.c           |    7 ++++---
 2 files changed, 6 insertions(+), 5 deletions(-)


Index: upstream/arch/powerpc/platforms/pseries/plpar_wrappers.h
--- upstream.orig/arch/powerpc/platforms/pseries/plpar_wrappers.h
+++ upstream/arch/powerpc/platforms/pseries/plpar_wrappers.h
@@ -259,12 +259,12 @@  static inline long plpar_ipi(unsigned lo
 	return plpar_hcall_norets(H_IPI, servernum, mfrr);
-static inline long plpar_xirr(unsigned long *xirr_ret)
+static inline long plpar_xirr(unsigned long *xirr_ret, unsigned char cppr)
 	long rc;
 	unsigned long retbuf[PLPAR_HCALL_BUFSIZE];
-	rc = plpar_hcall(H_XIRR, retbuf);
+	rc = plpar_hcall(H_XIRR, retbuf, cppr);
 	*xirr_ret = retbuf[0];
Index: upstream/arch/powerpc/platforms/pseries/xics.c
--- upstream.orig/arch/powerpc/platforms/pseries/xics.c
+++ upstream/arch/powerpc/platforms/pseries/xics.c
@@ -120,12 +120,12 @@  static inline void direct_qirr_info(int 
 /* LPAR low level accessors */
-static inline unsigned int lpar_xirr_info_get(void)
+static inline unsigned int lpar_xirr_info_get(unsigned char cppr)
 	unsigned long lpar_rc;
 	unsigned long return_value;
-	lpar_rc = plpar_xirr(&return_value);
+	lpar_rc = plpar_xirr(&return_value, cppr);
 	if (lpar_rc != H_SUCCESS)
 		panic(" bad return code xirr - rc = %lx \n", lpar_rc);
 	return (unsigned int)return_value;
@@ -335,7 +335,8 @@  static unsigned int xics_get_irq_direct(
 static unsigned int xics_get_irq_lpar(void)
-	unsigned int xirr = lpar_xirr_info_get();
+	struct xics_cppr *os_cppr = &__get_cpu_var(xics_cppr);
+	unsigned int xirr = lpar_xirr_info_get(os_cppr->stack[os_cppr->index]);
 	unsigned int vec = xics_xirr_vector(xirr);
 	unsigned int irq;