On Wed, Sep 07, 2011 at 10:29:11AM -0700, Jeremy Fitzhardinge wrote: > On 09/06/2011 10:50 PM, Cihula, Joseph wrote: > >> From: Konrad Rzeszutek Wilk [mailto:konrad.wilk@xxxxxxxxxx] > >> Sent: Wednesday, August 31, 2011 11:31 AM > >> > >> From: Yu Ke <ke.yu@xxxxxxxxx> > >> > >> This patches implements the xen_platform_op hypercall, to pass the parsed ACPI info to hypervisor. > >> > >> Signed-off-by: Yu Ke <ke.yu@xxxxxxxxx> > >> Signed-off-by: Tian Kevin <kevin.tian@xxxxxxxxx> > >> Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@xxxxxxxxxx> > >> [v1: Added DEFINE_GUEST.. in appropiate headers] > >> [v2: Ripped out typedefs] > >> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> > >> --- > >> arch/ia64/include/asm/xen/interface.h | 1 + > >> arch/x86/include/asm/xen/interface.h | 1 + > >> include/xen/interface/platform.h | 320 +++++++++++++++++++++++++++++++++ > >> include/xen/interface/xen.h | 1 + > >> 4 files changed, 323 insertions(+), 0 deletions(-) create mode 100644 > >> include/xen/interface/platform.h > >> > >> diff --git a/arch/ia64/include/asm/xen/interface.h b/arch/ia64/include/asm/xen/interface.h > >> index e951e74..1d2427d 100644 > >> --- a/arch/ia64/include/asm/xen/interface.h > >> +++ b/arch/ia64/include/asm/xen/interface.h > >> @@ -76,6 +76,7 @@ DEFINE_GUEST_HANDLE(char); DEFINE_GUEST_HANDLE(int); > >> DEFINE_GUEST_HANDLE(long); DEFINE_GUEST_HANDLE(void); > >> +DEFINE_GUEST_HANDLE(uint64_t); > >> > >> typedef unsigned long xen_pfn_t; > >> DEFINE_GUEST_HANDLE(xen_pfn_t); > >> diff --git a/arch/x86/include/asm/xen/interface.h b/arch/x86/include/asm/xen/interface.h > >> index 5d4922a..a1f2db5 100644 > >> --- a/arch/x86/include/asm/xen/interface.h > >> +++ b/arch/x86/include/asm/xen/interface.h > >> @@ -55,6 +55,7 @@ DEFINE_GUEST_HANDLE(char); DEFINE_GUEST_HANDLE(int); > >> DEFINE_GUEST_HANDLE(long); DEFINE_GUEST_HANDLE(void); > >> +DEFINE_GUEST_HANDLE(uint64_t); > >> #endif > >> > >> #ifndef HYPERVISOR_VIRT_START > >> diff --git a/include/xen/interface/platform.h b/include/xen/interface/platform.h > >> new file mode 100644 > >> index 0000000..c168468 > >> --- /dev/null > >> +++ b/include/xen/interface/platform.h > > Why are you adding so many new hypercalls that aren't being used in this patchset? > > May as well bring in all the platform-related hypercall definitions at > once rather than be piecemeal. Actually, I think making it piecemeal might be easier. As in 1). Not all of the hypercalls are going to be upstreamed. 2). It gives a nice history/idea of what is actually implemented when seaching the code. 3). git annotate can give precise git commits for when an functionality was added. But maybe I am just overthinking it. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html