[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] Re: [GIT PULL] Xen time-setting
On 10/25/2011 12:16 PM, Jeremy Fitzhardinge wrote: > Hi Linus, > > This just wires up "set system time" through to Xen so it can set the > actual RTC hardware. The bulk of the patch is a new header to define > the Xen platform interface ABI, which will also get used by other > patches in this merge window (I hope). Ping? > > Thanks, > J > > The following changes since commit 02f8c6aee8df3cdc935e9bdd4f2d020306035dbe: > > Linux 3.0 (2011-07-21 19:17:23 -0700) > > are available in the git repository at: > git://git.kernel.org/pub/scm/linux/kernel/git/jeremy/xen.git > upstream/xen-settime > > Jeremy Fitzhardinge (2): > xen: add dom0_op hypercall > xen/dom0: set wallclock time in Xen > > Yu Ke (1): > xen/acpi: Domain0 acpi parser related platform hypercall > > arch/ia64/include/asm/xen/interface.h | 1 + > arch/x86/include/asm/xen/hypercall.h | 8 + > arch/x86/include/asm/xen/interface.h | 1 + > arch/x86/xen/time.c | 16 ++- > include/xen/interface/platform.h | 320 > +++++++++++++++++++++++++++++++++ > include/xen/interface/xen.h | 1 + > 6 files changed, 346 insertions(+), 1 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/hypercall.h > b/arch/x86/include/asm/xen/hypercall.h > index d240ea9..0c9894e 100644 > --- a/arch/x86/include/asm/xen/hypercall.h > +++ b/arch/x86/include/asm/xen/hypercall.h > @@ -45,6 +45,7 @@ > #include <xen/interface/xen.h> > #include <xen/interface/sched.h> > #include <xen/interface/physdev.h> > +#include <xen/interface/platform.h> > > /* > * The hypercall asms have to meet several constraints: > @@ -299,6 +300,13 @@ HYPERVISOR_set_timer_op(u64 timeout) > } > > static inline int > +HYPERVISOR_dom0_op(struct xen_platform_op *platform_op) > +{ > + platform_op->interface_version = XENPF_INTERFACE_VERSION; > + return _hypercall1(int, dom0_op, platform_op); > +} > + > +static inline int > HYPERVISOR_set_debugreg(int reg, unsigned long value) > { > return _hypercall2(int, set_debugreg, reg, value); > 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/arch/x86/xen/time.c b/arch/x86/xen/time.c > index 5158c50..8c9cdfa 100644 > --- a/arch/x86/xen/time.c > +++ b/arch/x86/xen/time.c > @@ -200,8 +200,22 @@ static unsigned long xen_get_wallclock(void) > > static int xen_set_wallclock(unsigned long now) > { > + struct xen_platform_op op; > + int rc; > + > /* do nothing for domU */ > - return -1; > + if (!xen_initial_domain()) > + return -1; > + > + op.cmd = XENPF_settime; > + op.u.settime.secs = now; > + op.u.settime.nsecs = 0; > + op.u.settime.system_time = xen_clocksource_read(); > + > + rc = HYPERVISOR_dom0_op(&op); > + WARN(rc != 0, "XENPF_settime failed: now=%ld\n", now); > + > + return rc; > } > > static struct clocksource xen_clocksource __read_mostly = { > 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 > @@ -0,0 +1,320 @@ > +/****************************************************************************** > + * platform.h > + * > + * Hardware platform operations. Intended for use by domain-0 kernel. > + * > + * Permission is hereby granted, free of charge, to any person obtaining a > copy > + * of this software and associated documentation files (the "Software"), to > + * deal in the Software without restriction, including without limitation the > + * rights to use, copy, modify, merge, publish, distribute, sublicense, > and/or > + * sell copies of the Software, and to permit persons to whom the Software is > + * furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice shall be included in > + * all copies or substantial portions of the Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > THE > + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER > + * DEALINGS IN THE SOFTWARE. > + * > + * Copyright (c) 2002-2006, K Fraser > + */ > + > +#ifndef __XEN_PUBLIC_PLATFORM_H__ > +#define __XEN_PUBLIC_PLATFORM_H__ > + > +#include "xen.h" > + > +#define XENPF_INTERFACE_VERSION 0x03000001 > + > +/* > + * Set clock such that it would read <secs,nsecs> after 00:00:00 UTC, > + * 1 January, 1970 if the current system time was <system_time>. > + */ > +#define XENPF_settime 17 > +struct xenpf_settime { > + /* IN variables. */ > + uint32_t secs; > + uint32_t nsecs; > + uint64_t system_time; > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_settime_t); > + > +/* > + * Request memory range (@mfn, @mfn+@nr_mfns-1) to have type @type. > + * On x86, @type is an architecture-defined MTRR memory type. > + * On success, returns the MTRR that was used (@reg) and a handle that can > + * be passed to XENPF_DEL_MEMTYPE to accurately tear down the new setting. > + * (x86-specific). > + */ > +#define XENPF_add_memtype 31 > +struct xenpf_add_memtype { > + /* IN variables. */ > + unsigned long mfn; > + uint64_t nr_mfns; > + uint32_t type; > + /* OUT variables. */ > + uint32_t handle; > + uint32_t reg; > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_add_memtype_t); > + > +/* > + * Tear down an existing memory-range type. If @handle is remembered then it > + * should be passed in to accurately tear down the correct setting (in case > + * of overlapping memory regions with differing types). If it is not known > + * then @handle should be set to zero. In all cases @reg must be set. > + * (x86-specific). > + */ > +#define XENPF_del_memtype 32 > +struct xenpf_del_memtype { > + /* IN variables. */ > + uint32_t handle; > + uint32_t reg; > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_del_memtype_t); > + > +/* Read current type of an MTRR (x86-specific). */ > +#define XENPF_read_memtype 33 > +struct xenpf_read_memtype { > + /* IN variables. */ > + uint32_t reg; > + /* OUT variables. */ > + unsigned long mfn; > + uint64_t nr_mfns; > + uint32_t type; > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_read_memtype_t); > + > +#define XENPF_microcode_update 35 > +struct xenpf_microcode_update { > + /* IN variables. */ > + GUEST_HANDLE(void) data; /* Pointer to microcode data */ > + uint32_t length; /* Length of microcode data. */ > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_microcode_update_t); > + > +#define XENPF_platform_quirk 39 > +#define QUIRK_NOIRQBALANCING 1 /* Do not restrict IO-APIC RTE targets */ > +#define QUIRK_IOAPIC_BAD_REGSEL 2 /* IO-APIC REGSEL forgets its value */ > +#define QUIRK_IOAPIC_GOOD_REGSEL 3 /* IO-APIC REGSEL behaves properly */ > +struct xenpf_platform_quirk { > + /* IN variables. */ > + uint32_t quirk_id; > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_platform_quirk_t); > + > +#define XENPF_firmware_info 50 > +#define XEN_FW_DISK_INFO 1 /* from int 13 AH=08/41/48 */ > +#define XEN_FW_DISK_MBR_SIGNATURE 2 /* from MBR offset 0x1b8 */ > +#define XEN_FW_VBEDDC_INFO 3 /* from int 10 AX=4f15 */ > +struct xenpf_firmware_info { > + /* IN variables. */ > + uint32_t type; > + uint32_t index; > + /* OUT variables. */ > + union { > + struct { > + /* Int13, Fn48: Check Extensions Present. */ > + uint8_t device; /* %dl: bios device > number */ > + uint8_t version; /* %ah: major version > */ > + uint16_t interface_support; /* %cx: support > bitmap */ > + /* Int13, Fn08: Legacy Get Device Parameters. */ > + uint16_t legacy_max_cylinder; /* %cl[7:6]:%ch: max > cyl # */ > + uint8_t legacy_max_head; /* %dh: max head # > */ > + uint8_t legacy_sectors_per_track; /* %cl[5:0]: max > sector # */ > + /* Int13, Fn41: Get Device Parameters (as filled into > %ds:%esi). */ > + /* NB. First uint16_t of buffer must be set to buffer > size. */ > + GUEST_HANDLE(void) edd_params; > + } disk_info; /* XEN_FW_DISK_INFO */ > + struct { > + uint8_t device; /* bios device number > */ > + uint32_t mbr_signature; /* offset 0x1b8 in > mbr */ > + } disk_mbr_signature; /* XEN_FW_DISK_MBR_SIGNATURE */ > + struct { > + /* Int10, AX=4F15: Get EDID info. */ > + uint8_t capabilities; > + uint8_t edid_transfer_time; > + /* must refer to 128-byte buffer */ > + GUEST_HANDLE(uchar) edid; > + } vbeddc_info; /* XEN_FW_VBEDDC_INFO */ > + } u; > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_firmware_info_t); > + > +#define XENPF_enter_acpi_sleep 51 > +struct xenpf_enter_acpi_sleep { > + /* IN variables */ > + uint16_t pm1a_cnt_val; /* PM1a control value. */ > + uint16_t pm1b_cnt_val; /* PM1b control value. */ > + uint32_t sleep_state; /* Which state to enter (Sn). */ > + uint32_t flags; /* Must be zero. */ > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_enter_acpi_sleep_t); > + > +#define XENPF_change_freq 52 > +struct xenpf_change_freq { > + /* IN variables */ > + uint32_t flags; /* Must be zero. */ > + uint32_t cpu; /* Physical cpu. */ > + uint64_t freq; /* New frequency (Hz). */ > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_change_freq_t); > + > +/* > + * Get idle times (nanoseconds since boot) for physical CPUs specified in the > + * @cpumap_bitmap with range [0..@cpumap_nr_cpus-1]. The @idletime array is > + * indexed by CPU number; only entries with the corresponding @cpumap_bitmap > + * bit set are written to. On return, @cpumap_bitmap is modified so that any > + * non-existent CPUs are cleared. Such CPUs have their @idletime array entry > + * cleared. > + */ > +#define XENPF_getidletime 53 > +struct xenpf_getidletime { > + /* IN/OUT variables */ > + /* IN: CPUs to interrogate; OUT: subset of IN which are present */ > + GUEST_HANDLE(uchar) cpumap_bitmap; > + /* IN variables */ > + /* Size of cpumap bitmap. */ > + uint32_t cpumap_nr_cpus; > + /* Must be indexable for every cpu in cpumap_bitmap. */ > + GUEST_HANDLE(uint64_t) idletime; > + /* OUT variables */ > + /* System time when the idletime snapshots were taken. */ > + uint64_t now; > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_getidletime_t); > + > +#define XENPF_set_processor_pminfo 54 > + > +/* ability bits */ > +#define XEN_PROCESSOR_PM_CX 1 > +#define XEN_PROCESSOR_PM_PX 2 > +#define XEN_PROCESSOR_PM_TX 4 > + > +/* cmd type */ > +#define XEN_PM_CX 0 > +#define XEN_PM_PX 1 > +#define XEN_PM_TX 2 > + > +/* Px sub info type */ > +#define XEN_PX_PCT 1 > +#define XEN_PX_PSS 2 > +#define XEN_PX_PPC 4 > +#define XEN_PX_PSD 8 > + > +struct xen_power_register { > + uint32_t space_id; > + uint32_t bit_width; > + uint32_t bit_offset; > + uint32_t access_size; > + uint64_t address; > +}; > + > +struct xen_processor_csd { > + uint32_t domain; /* domain number of one dependent group */ > + uint32_t coord_type; /* coordination type */ > + uint32_t num; /* number of processors in same domain */ > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xen_processor_csd); > + > +struct xen_processor_cx { > + struct xen_power_register reg; /* GAS for Cx trigger register */ > + uint8_t type; /* cstate value, c0: 0, c1: 1, ... */ > + uint32_t latency; /* worst latency (ms) to enter/exit this cstate */ > + uint32_t power; /* average power consumption(mW) */ > + uint32_t dpcnt; /* number of dependency entries */ > + GUEST_HANDLE(xen_processor_csd) dp; /* NULL if no dependency */ > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xen_processor_cx); > + > +struct xen_processor_flags { > + uint32_t bm_control:1; > + uint32_t bm_check:1; > + uint32_t has_cst:1; > + uint32_t power_setup_done:1; > + uint32_t bm_rld_set:1; > +}; > + > +struct xen_processor_power { > + uint32_t count; /* number of C state entries in array below */ > + struct xen_processor_flags flags; /* global flags of this processor */ > + GUEST_HANDLE(xen_processor_cx) states; /* supported c states */ > +}; > + > +struct xen_pct_register { > + uint8_t descriptor; > + uint16_t length; > + uint8_t space_id; > + uint8_t bit_width; > + uint8_t bit_offset; > + uint8_t reserved; > + uint64_t address; > +}; > + > +struct xen_processor_px { > + uint64_t core_frequency; /* megahertz */ > + uint64_t power; /* milliWatts */ > + uint64_t transition_latency; /* microseconds */ > + uint64_t bus_master_latency; /* microseconds */ > + uint64_t control; /* control value */ > + uint64_t status; /* success indicator */ > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xen_processor_px); > + > +struct xen_psd_package { > + uint64_t num_entries; > + uint64_t revision; > + uint64_t domain; > + uint64_t coord_type; > + uint64_t num_processors; > +}; > + > +struct xen_processor_performance { > + uint32_t flags; /* flag for Px sub info type */ > + uint32_t platform_limit; /* Platform limitation on freq usage */ > + struct xen_pct_register control_register; > + struct xen_pct_register status_register; > + uint32_t state_count; /* total available performance states */ > + GUEST_HANDLE(xen_processor_px) states; > + struct xen_psd_package domain_info; > + uint32_t shared_type; /* coordination type of this processor */ > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xen_processor_performance); > + > +struct xenpf_set_processor_pminfo { > + /* IN variables */ > + uint32_t id; /* ACPI CPU ID */ > + uint32_t type; /* {XEN_PM_CX, XEN_PM_PX} */ > + union { > + struct xen_processor_power power;/* Cx: _CST/_CSD */ > + struct xen_processor_performance perf; /* Px: > _PPC/_PCT/_PSS/_PSD */ > + }; > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xenpf_set_processor_pminfo); > + > +struct xen_platform_op { > + uint32_t cmd; > + uint32_t interface_version; /* XENPF_INTERFACE_VERSION */ > + union { > + struct xenpf_settime settime; > + struct xenpf_add_memtype add_memtype; > + struct xenpf_del_memtype del_memtype; > + struct xenpf_read_memtype read_memtype; > + struct xenpf_microcode_update microcode; > + struct xenpf_platform_quirk platform_quirk; > + struct xenpf_firmware_info firmware_info; > + struct xenpf_enter_acpi_sleep enter_acpi_sleep; > + struct xenpf_change_freq change_freq; > + struct xenpf_getidletime getidletime; > + struct xenpf_set_processor_pminfo set_pminfo; > + uint8_t pad[128]; > + } u; > +}; > +DEFINE_GUEST_HANDLE_STRUCT(xen_platform_op_t); > + > +#endif /* __XEN_PUBLIC_PLATFORM_H__ */ > diff --git a/include/xen/interface/xen.h b/include/xen/interface/xen.h > index 70213b4..d83cc08 100644 > --- a/include/xen/interface/xen.h > +++ b/include/xen/interface/xen.h > @@ -453,6 +453,7 @@ struct start_info { > /* These flags are passed in the 'flags' field of start_info_t. */ > #define SIF_PRIVILEGED (1<<0) /* Is the domain privileged? */ > #define SIF_INITDOMAIN (1<<1) /* Is this the initial control domain? */ > +#define SIF_PM_MASK (0xFF<<8) /* reserve 1 byte for xen-pm options */ > > typedef uint64_t cpumap_t; > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ > _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |