Re: [Xen-devel] [PATCH 4/4] iommu: add rmrr Xen command line option for extra rmrrs

2015-06-01 Thread Tian, Kevin
From: elena.ufimts...@oracle.com [mailto:elena.ufimts...@oracle.com] Sent: Saturday, May 30, 2015 5:39 AM From: Elena Ufimtseva elena.ufimts...@oracle.com On some platforms RMRR regions may be not specified in ACPI and thus will not be mapped 1:1 in dom0. This causes IO Page Faults and

Re: [Xen-devel] [PATCH V8] xen/vm_event: Clean up control-register-write vm_events and add XCR0 event

2015-06-01 Thread Tian, Kevin
From: Razvan Cojocaru [mailto:rcojoc...@bitdefender.com] Sent: Friday, May 29, 2015 9:46 PM As suggested by Andrew Cooper, this patch attempts to remove some redundancy and allow for an easier time when adding vm_events for new control registers in the future, by having a single

[Xen-devel] Ping: [PATCH] xen/pass-through: fold host PCI command register writes

2015-06-01 Thread Jan Beulich
Ping? On 15.05.15 at 14:46, jbeul...@suse.com wrote: The code introduced to address XSA-126 allows simplification of other code in xen_pt_initfn(): All we need to do is update cmd suitably, as it'll be written back to the host register near the end of the function anyway. Signed-off-by:

[Xen-devel] Ping: [PATCH] xen/pass-through: ROM BAR handling adjustments

2015-06-01 Thread Jan Beulich
Ping? On 15.05.15 at 14:41, jbeul...@suse.com wrote: Expecting the ROM BAR to be written with an all ones value when sizing the region is wrong - the low bit has another meaning (enable/disable) and bits 1..10 are reserved. The PCI spec also mandates writing all ones to just the address

Re: [Xen-devel] [PATCH v3 1/2] efi: Fix allocation problems if ExitBootServices() fails

2015-06-01 Thread Jan Beulich
On 01.06.15 at 12:17, ross.lagerw...@citrix.com wrote: If calling ExitBootServices() fails, the required memory map size may have increased. When initially allocating the memory map, allocate a slightly larger buffer (by an arbitrary 8 entries) to fix this. The ARM code path was already

Re: [Xen-devel] [Draft C] Xen on ARM vITS Handling

2015-06-01 Thread Julien Grall
On 01/06/15 13:11, Ian Campbell wrote: ### Device ID (`ID`) This parameter is used by commands which manage a specific device and the interrupts associated with that device. Checking if a device is present and retrieving the data structure must be fast. The device identifiers may not be

[Xen-devel] [PATCH V6 02/10] xen/arm: Add functions of mapping between vCPUID and virtual affinity

2015-06-01 Thread Chen Baozi
From: Chen Baozi baoz...@gmail.com GICv3 restricts that the maximum number of CPUs in affinity 0 (one cluster) is 16. That is to say the upper 4 bits of affinity 0 is unused. Current implementation considers that AFF0 is equal to vCPUID, which makes all vCPUs in one cluster, limiting its number

[Xen-devel] [PATCH V6 01/10] xen/arm: gic-v3: Increase the size of GICR in address space for guest

2015-06-01 Thread Chen Baozi
From: Chen Baozi baoz...@gmail.com Currently it only supports up to 8 vCPUs. Increase the region to hold up to 128 vCPUs, which is the maximum number that GIC-500 supports. Signed-off-by: Chen Baozi baoz...@gmail.com Reviewed-by: Julien Grall julien.gr...@citrix.com ---

[Xen-devel] [PATCH V6 03/10] xen/arm: Use the new functions for vCPUID/vaffinity transformation

2015-06-01 Thread Chen Baozi
From: Chen Baozi baoz...@gmail.com There are 3 places to change: * Initialise vMPIDR value in vcpu_initialise() * Find the vCPU from vMPIDR affinity information when accessing GICD registers in vGIC * Find the vCPU from vMPIDR affinity information when booting with vPSCI in vGIC - Also

Re: [Xen-devel] [PATCH v2] xen: netback: fix printf format string warning

2015-06-01 Thread Wei Liu
On Mon, Jun 01, 2015 at 11:30:04AM +0100, Ian Campbell wrote: drivers/net/xen-netback/netback.c: In function ‘xenvif_tx_build_gops’: drivers/net/xen-netback/netback.c:1253:8: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 5 has type ‘int’ [-Wformat=]

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 12:13 +0100, Julien Grall wrote: Hi, On 18/05/15 14:36, Zoltan Kiss wrote: On 15/05/15 22:08, Julien Grall wrote: Hi Zoltan, On 07/05/2015 13:37, Zoltan Kiss wrote: On 07/05/15 10:32, Ian Campbell wrote: On Thu, 2015-05-07 at 09:52 +0100, Zoltan Kiss

[Xen-devel] [rumpuserxen test] 57717: regressions - FAIL

2015-06-01 Thread osstest service user
flight 57717 rumpuserxen real [real] http://logs.test-lab.xenproject.org/osstest/logs/57717/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-rumpuserxen 5 rumpuserxen-build fail REGR. vs. 33866

[Xen-devel] [PATCH V6 08/10] xen: Add arch_domain_preinit to initialise vGIC before evtchn_init

2015-06-01 Thread Chen Baozi
From: Chen Baozi baoz...@gmail.com evtchn_init will call domain_max_vcpus to allocate poll_mask. On arm/arm64 platform, this number is determined by the vGIC the guest is going to use, which won't be initialised until arch_domain_create is called in current implementation. However, moving

[Xen-devel] [PATCH V6 04/10] xen/arm: Use cpumask_t type for vcpu_mask in vgic_to_sgi

2015-06-01 Thread Chen Baozi
From: Chen Baozi baoz...@gmail.com Use cpumask_t instead of unsigned long which can only express 64 cpus at the most. Add the {gicv2|gicv3}_sgir_to_cpumask in corresponding vGICs to translate GICD_SGIR/ICC_SGI1R_EL1 to vcpu_mask for vgic_to_sgi. Signed-off-by: Chen Baozi baoz...@gmail.com ---

[Xen-devel] [PATCH V6 06/10] tools/libxl: Set 'reg' of cpu node equal to MPIDR affinity for domU

2015-06-01 Thread Chen Baozi
From: Chen Baozi baoz...@gmail.com According to ARM CPUs bindings, the reg field should match the MPIDR's affinity bits. We will use AFF0 and AFF1 when constructing the reg value of the guest at the moment, for it is enough for the current max vcpu number. Signed-off-by: Chen Baozi

[Xen-devel] [PATCH V6 05/10] xen/arm64: gicv3: Use AFF1 when translating ICC_SGI1R_EL1 to cpumask

2015-06-01 Thread Chen Baozi
From: Chen Baozi baoz...@gmail.com To support more than 16 vCPUs, we have to calculate cpumask with AFF1 field value in ICC_SGI1R_EL1. Signed-off-by: Chen Baozi baoz...@gmail.com --- xen/arch/arm/vgic-v3.c| 30 ++ xen/include/asm-arm/gic_v3_defs.h | 2 ++

[Xen-devel] [PATCH V6 07/10] xen/arm: Set 'reg' of cpu node for dom0 to match MPIDR's affinity

2015-06-01 Thread Chen Baozi
From: Chen Baozi baoz...@gmail.com According to ARM CPUs bindings, the reg field should match the MPIDR's affinity bits. We will use AFF0 and AFF1 when constructing the reg value of the guest at the moment, for it is enough for the current max vcpu number. Signed-off-by: Chen Baozi

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-01 Thread Julien Grall
Hi, On 18/05/15 14:36, Zoltan Kiss wrote: On 15/05/15 22:08, Julien Grall wrote: Hi Zoltan, On 07/05/2015 13:37, Zoltan Kiss wrote: On 07/05/15 10:32, Ian Campbell wrote: On Thu, 2015-05-07 at 09:52 +0100, Zoltan Kiss wrote: Looks good at first glance, let me try it on a board. On

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-01 Thread Zoltan Kiss
Hi, Yes, we managed to test it, and it works. Then only thing I've found is this bit: +/* Only 1020 interrupts are supported */ +gicv2_info.nr_lines = min(1020U, nr_lines); This interrupt controller only supports 511, so 1020 should be replaced. We had such checking in the code in

Re: [Xen-devel] [Draft C] Xen on ARM vITS Handling

2015-06-01 Thread Ian Campbell
On Wed, 2015-05-27 at 22:14 +0530, Vijay Kilari wrote: ## pITS Scheduling A pITS scheduling pass is attempted: * On write to any virtual `CWRITER` iff that write results in there being new outstanding requests for that vits; You mean, scheduling pass (softirq trigger) is

[Xen-devel] [PATCH V6 09/10] xen/arm: make domain_max_vcpus return value from vgic_ops

2015-06-01 Thread Chen Baozi
From: Chen Baozi baoz...@gmail.com When a guest uses vGICv2, the maximum number of vCPU it can support should not be as many as MAX_VIRT_CPUS, which will be more than 8 when GICv3 is used on arm64. So the domain_max_vcpus should return the value according to the vGIC the domain uses. We didn't

[Xen-devel] [PATCH V6 10/10] xen/arm64: increase MAX_VIRT_CPUS to 128 on arm64

2015-06-01 Thread Chen Baozi
From: Chen Baozi baoz...@gmail.com After we have increased the size of GICR in address space for guest and made use of both AFF0 and AFF1 in (v)MPIDR, we are now able to support up to 4096 vCPUs in theory. However, it will cost 512M address space for GICR region, which is not necessary big at the

Re: [Xen-devel] [PATCH] tools: link executables with libtinfo explicitly

2015-06-01 Thread Wei Liu
On Sat, May 30, 2015 at 12:07:28AM +0200, Daniel Kiper wrote: binutils 2.22 changed ld default from --copy-dt-needed-entries to -no-copy-dt-needed-entries. This revealed that some objects are linked implicitly with libtinfo and newer ld fails to build relevant executables. Below is short

Re: [Xen-devel] [PATCH v2 net] xen: netback: read hotplug script once at start of day.

2015-06-01 Thread Wei Liu
On Mon, Jun 01, 2015 at 11:30:24AM +0100, Ian Campbell wrote: When we come to tear things down in netback_remove() and generate the uevent it is possible that the xenstore directory has already been removed (details below). In such cases netback_uevent() won't be able to read the hotplug

Re: [Xen-devel] [PATCH v3 1/2] efi: Fix allocation problems if ExitBootServices() fails

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 12:10 +0100, Jan Beulich wrote: On 01.06.15 at 12:17, ross.lagerw...@citrix.com wrote: If calling ExitBootServices() fails, the required memory map size may have increased. When initially allocating the memory map, allocate a slightly larger buffer (by an arbitrary 8

Re: [Xen-devel] [xen-4.2-testing test] 57630: regressions - FAIL

2015-06-01 Thread Jan Beulich
On 01.06.15 at 00:57, osst...@xenbits.xen.org wrote: flight 57630 xen-4.2-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/57630/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xend-winxpsp3 16

[Xen-devel] [Patch v3 27/36] x86, irq: Use access helper irq_data_get_affinity_mask()

2015-06-01 Thread Jiang Liu
Use access helper irq_data_get_affinity_mask() to hide implementation details of struct irq_desc. Signed-off-by: Jiang Liu jiang@linux.intel.com --- arch/x86/kernel/apic/io_apic.c |2 +- arch/x86/kernel/apic/vector.c|5 ++--- arch/x86/kernel/irq.c|5 +++--

Re: [Xen-devel] [PATCH V5 10/10] xen/arm64: increase MAX_VIRT_CPUS to 128 on arm64

2015-06-01 Thread Julien Grall
Hi Chen, On 01/06/2015 01:56, Chen Baozi wrote: For instance, with your series a cluster can use up to 16 cores but the GIC-500 is only supporting up to 8 cores... Well, if 4096 is a acceptable value, it will cost 512M address space for GICR_* (We need to chagne patch #1 too). Although we do

Re: [Xen-devel] [PATCH v6 2/4] iommu VT-d: separate rmrr addition function

2015-06-01 Thread Jan Beulich
On 29.05.15 at 23:38, elena.ufimts...@oracle.com wrote: In preparation for auxiliary RMRR data provided on Xen command line, make RMRR adding a separate function. Also free memery for rmrr device scope in error path. No changes since v5. Certainly there is. (And the statement wouldn't belong

Re: [Xen-devel] [PATCH net] xen: netback: read hotplug script once at start of day.

2015-06-01 Thread Ian Campbell
On Fri, 2015-05-29 at 18:38 +0100, Wei Liu wrote: On Fri, May 29, 2015 at 05:24:53PM +0100, Ian Campbell wrote: [...] if (be-vif != NULL) return 0; @@ -417,12 +409,23 @@ static int backend_create_xenvif(struct backend_info *be) return (err 0) ? err :

Re: [Xen-devel] [PATCH v3 2/2] efi: Avoid calling boot services after ExitBootServices()

2015-06-01 Thread Andrew Cooper
On 01/06/15 11:17, Ross Lagerwall wrote: After the first call to ExitBootServices(), avoid calling any boot services by setting setting efi_bs to NULL and entering an infinite loop in blexit(). Signed-off-by: Ross Lagerwall ross.lagerw...@citrix.com --- xen/common/efi/boot.c | 16

Re: [Xen-devel] [PATCH v3 2/2] efi: Avoid calling boot services after ExitBootServices()

2015-06-01 Thread Jan Beulich
On 01.06.15 at 12:26, andrew.coop...@citrix.com wrote: On 01/06/15 11:17, Ross Lagerwall wrote: --- a/xen/common/efi/boot.c +++ b/xen/common/efi/boot.c @@ -216,6 +216,12 @@ static void __init noreturn blexit(const CHAR16 *str) PrintStr((CHAR16 *)str); PrintStr(newline); +

Re: [Xen-devel] [PATCH v23 10/15] x86/VPMU: Use pre-computed masks when checking validity of MSRs

2015-06-01 Thread Jan Beulich
On 29.05.15 at 20:42, boris.ostrov...@oracle.com wrote: No need to compute those masks on every MSR access. Also, when checking MSR_P6_EVNTSELx registers make sure that bit 21 is not set. I had hoped you would save me and other readers to look up what this bit means by giving a reason why

[Xen-devel] [linux-linus test] 57671: trouble: blocked/broken/fail/pass

2015-06-01 Thread osstest service user
flight 57671 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/57671/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 3 host-install(3) broken REGR. vs. 57442

Re: [Xen-devel] [PATCH] drivers: xen-blkfront: blkif_recover: recheck feature-persistent

2015-06-01 Thread Roger Pau Monné
El 26/05/15 a les 2.11, Bob Liu ha escrit: When migrate from !feature-persistent host to feature-persistent host, domU still think new host/backend don't support persistent. Dmesg like: backed has not unmapped grant: 839 backed has not unmapped grant: 773 backed has not unmapped grant: 773

Re: [Xen-devel] [PATCH v8 03/13] x86: maintain COS to CBM mapping for each socket

2015-06-01 Thread Chao Peng
On Fri, May 29, 2015 at 04:38:31PM +0800, Chao Peng wrote: On Fri, May 29, 2015 at 09:06:46AM +0100, Jan Beulich wrote: On 29.05.15 at 04:43, chao.p.p...@linux.intel.com wrote: On Thu, May 28, 2015 at 02:17:54PM +0100, Jan Beulich wrote: On 21.05.15 at 10:41, chao.p.p...@linux.intel.com

Re: [Xen-devel] [PATCH] xen: netback: fix error printf format string.

2015-06-01 Thread Ian Campbell
On Sun, 2015-05-31 at 21:26 -0700, David Miller wrote: From: Ian Campbell ian.campb...@citrix.com Date: Fri, 29 May 2015 17:22:04 +0100 drivers/net/xen-netback/netback.c: In function ‘xenvif_tx_build_gops’: drivers/net/xen-netback/netback.c:1253:8: warning: format ‘%lu’ expects argument

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/4] Enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-06-01 Thread George Dunlap
On 05/29/2015 03:40 PM, Meng Xu wrote: 2015-05-29 4:15 GMT-07:00 Dario Faggioli dario.faggi...@citrix.com: On Tue, 2015-05-26 at 09:59 +0100, Ian Campbell wrote: On Mon, 2015-05-25 at 18:59 -0500, Chong Li wrote: This series arrived in my mailbox as 5 distinct mails. Please use git

Re: [Xen-devel] [PATCH net] xen: netback: read hotplug script once at start of day.

2015-06-01 Thread Wei Liu
On Mon, Jun 01, 2015 at 09:52:45AM +0100, Ian Campbell wrote: On Fri, 2015-05-29 at 18:38 +0100, Wei Liu wrote: On Fri, May 29, 2015 at 05:24:53PM +0100, Ian Campbell wrote: [...] if (be-vif != NULL) return 0; @@ -417,12 +409,23 @@ static int backend_create_xenvif(struct

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/4] Enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-06-01 Thread Andrew Cooper
On 01/06/15 09:58, Jan Beulich wrote: On 01.06.15 at 10:50, george.dun...@eu.citrix.com wrote: On 06/01/2015 09:48 AM, Ian Campbell wrote: On Mon, 2015-06-01 at 09:36 +0100, George Dunlap wrote: Signed-off-by: George Dunlap george.dun...@eu.citrix.com --- CC: Ian Campbell

[Xen-devel] The size of memory is wrong inside of virtual machine(VM) when using OVMF

2015-06-01 Thread Maoming
Hi all: I encountered a troublesome problem about OVMF. I used OVMF.fd as a BIOS of virtual machine(VM). 1、my environment: xen_version: 4.6-unstable I git clone xen from git://xenbits.xen.org/xen.git. configure it and parameters as below:

Re: [Xen-devel] [xen-unstable test] 57644: regressions - trouble: blocked/broken/fail/pass

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 03:40 +, osstest service user wrote: flight 57644 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/57644/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run:

[Xen-devel] [qemu-mainline test] 57666: regressions - trouble: broken/fail/pass

2015-06-01 Thread osstest service user
flight 57666 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/57666/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-multivcpu 17 leak-check/check fail REGR. vs. 56831 Tests which are

Re: [Xen-devel] [xen-4.4-testing test] 57474: tolerable trouble: blocked/broken/fail/pass - PUSHED

2015-06-01 Thread Jan Beulich
On 29.05.15 at 20:51, osst...@xenbits.xen.org wrote: flight 57474 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/57474/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armhf-armhf-xl-multivcpu 3 host-install(3)

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/4] Enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-06-01 Thread Jan Beulich
On 01.06.15 at 10:50, george.dun...@eu.citrix.com wrote: On 06/01/2015 09:48 AM, Ian Campbell wrote: On Mon, 2015-06-01 at 09:36 +0100, George Dunlap wrote: Signed-off-by: George Dunlap george.dun...@eu.citrix.com --- CC: Ian Campbell ian.campb...@citrix.com CC: Wei Liu wei@citrix.com

Re: [Xen-devel] [PATCH v8 03/13] x86: maintain COS to CBM mapping for each socket

2015-06-01 Thread Chao Peng
On Mon, Jun 01, 2015 at 09:36:15AM +0100, Jan Beulich wrote: On 01.06.15 at 10:05, chao.p.p...@linux.intel.com wrote: 2) Unfeasible to allocate memory first and do initialization later in cpu hotplug notifications. My former approach is performing both the allocation and initialization in

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/4] Enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 09:50 +0100, George Dunlap wrote: On 06/01/2015 09:48 AM, Ian Campbell wrote: On Mon, 2015-06-01 at 09:36 +0100, George Dunlap wrote: Signed-off-by: George Dunlap george.dun...@eu.citrix.com --- CC: Ian Campbell ian.campb...@citrix.com CC: Wei Liu

Re: [Xen-devel] [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL)

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 09:34 +0100, Ian Campbell wrote: From aa79bd958fe42da2876fd76e61468c183a7fabdc Mon Sep 17 00:00:00 2001 From: Ian Campbell ian.campb...@citrix.com Date: Mon, 1 Jun 2015 09:32:37 +0100 Subject: [PATCH] Toolstack: Do not pass -F to xm shutdown This is a feature of xl

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/4] Enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 09:36 +0100, George Dunlap wrote: Signed-off-by: George Dunlap george.dun...@eu.citrix.com --- CC: Ian Campbell ian.campb...@citrix.com CC: Wei Liu wei@citrix.com Most people put the Cc about the cut (---) which is fine too. It means the history ends up recording who

[Xen-devel] [linux-3.4 test] 57661: regressions - FAIL

2015-06-01 Thread osstest service user
flight 57661 linux-3.4 real [real] http://logs.test-lab.xenproject.org/osstest/logs/57661/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl 9 debian-install fail REGR. vs. 52209-bisect test-amd64-amd64-pair

Re: [Xen-devel] [PATCH 4/4] iommu: add rmrr Xen command line option for extra rmrrs

2015-06-01 Thread Jan Beulich
On 01.06.15 at 08:30, kevin.t...@intel.com wrote: From: elena.ufimts...@oracle.com [mailto:elena.ufimts...@oracle.com] +rmrrn = xzalloc(struct acpi_rmrr_unit); +if ( !rmrrn ) +return; + +rmrrn-scope.devices = xmalloc_array(u16, +

[Xen-devel] [Formal Vote] Changes to Xen Project Security Vulnerability Process - Open until June 8th, 2015

2015-06-01 Thread Lars Kurth
Hi, in accordance with the project's governance, I would like to put the following text changes to a committer vote (committers are on the TO list). The discussion leading to the changes can be found at http://lists.xenproject.org/archives/html/xen-devel/2015-05/msg02881.html

Re: [Xen-devel] [PATCH v2 6/9] x86/intel_pstate: the main boby of the intel_pstate driver

2015-06-01 Thread Jan Beulich
On 01.06.15 at 11:12, wei.w.w...@intel.com wrote: On 29/05/2015 16:46, Jan Beulich wrote On 29.05.15 at 10:19, wei.w.w...@intel.com wrote: On 26/05/2015 21:58, Jan Beulich wrote On 13.05.16 at 09:50, wei.w.w...@intel.com wrote: +static int intel_pstate_verify_policy(struct

[Xen-devel] [PATCH v2] xen: netback: fix printf format string warning

2015-06-01 Thread Ian Campbell
drivers/net/xen-netback/netback.c: In function ‘xenvif_tx_build_gops’: drivers/net/xen-netback/netback.c:1253:8: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 5 has type ‘int’ [-Wformat=] (txreq.offset~PAGE_MASK) + txreq.size); ^ PAGE_MASK's

Re: [Xen-devel] [PATCH] drivers: xen-blkback: delay pending_req allocation to connect_ring

2015-06-01 Thread Bob Liu
On 06/01/2015 04:36 PM, Roger Pau Monné wrote: El 26/05/15 a les 2.06, Bob Liu ha escrit: In connect_ring, we can know exactly how many pages are used for the shared ring and also whether feature-persistent is enabled, delay pending_req allocation here so that we won't waste too much memory.

Re: [Xen-devel] [PATCH v2 6/9] x86/intel_pstate: the main boby of the intel_pstate driver

2015-06-01 Thread Wang, Wei W
On 29/05/2015 16:46, Jan Beulich wrote On 29.05.15 at 10:19, wei.w.w...@intel.com wrote: On 26/05/2015 21:58, Jan Beulich wrote On 13.05.16 at 09:50, wei.w.w...@intel.com wrote: +static int intel_pstate_verify_policy(struct cpufreq_policy +*policy) { +

Re: [Xen-devel] [PATCH] libxl: Don't insert PCI device into xenstore for HVM guests

2015-06-01 Thread George Dunlap
On Fri, May 29, 2015 at 8:59 AM, Ross Lagerwall ross.lagerw...@citrix.com wrote: When doing passthrough of a PCI device for an HVM guest, don't insert the device into xenstore, otherwise pciback attempts to use it which conflicts with QEMU. This manifests itself such that the first time a

Re: [Xen-devel] Xen BUG at page_alloc.c:1738 (Xen 4.5)

2015-06-01 Thread M A Young
On Mon, 1 Jun 2015, Jan Beulich wrote: On 31.05.15 at 00:43, andrew.coop...@citrix.com wrote: On 30/05/2015 23:07, M A Young wrote: On Fri, 29 May 2015, Andrew Cooper wrote: FC22 is miscompiling the C to: struct page_info *page = mfn_to_page(mfn); struct domain *owner =

Re: [Xen-devel] [xen-4.3-testing test] 57600: regressions - trouble: blocked/broken/fail/pass

2015-06-01 Thread Jan Beulich
On 31.05.15 at 19:18, osst...@xenbits.xen.org wrote: flight 57600 xen-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/57600/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run:

Re: [Xen-devel] [PATCH] drivers: xen-blkfront: blkif_recover: recheck feature-persistent

2015-06-01 Thread Bob Liu
On 06/01/2015 03:50 PM, Roger Pau Monné wrote: El 26/05/15 a les 2.11, Bob Liu ha escrit: When migrate from !feature-persistent host to feature-persistent host, domU still think new host/backend don't support persistent. Dmesg like: backed has not unmapped grant: 839 backed has not unmapped

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/4] Enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-06-01 Thread George Dunlap
On 06/01/2015 09:48 AM, Ian Campbell wrote: On Mon, 2015-06-01 at 09:36 +0100, George Dunlap wrote: Signed-off-by: George Dunlap george.dun...@eu.citrix.com --- CC: Ian Campbell ian.campb...@citrix.com CC: Wei Liu wei@citrix.com Most people put the Cc about the cut (---) which is fine

[Xen-devel] [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL)

2015-06-01 Thread Jan Beulich
On 01.06.15 at 10:34, ian.campb...@citrix.com wrote: Subject: [PATCH] Toolstack: Do not pass -F to xm shutdown This is a feature of xl only. Signed-off-by: Ian Campbell ian.campb...@citrix.com --- Osstest/Toolstack/xl.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/4] Enabling XL to set per-VCPU parameters of a domain for RTDS scheduler

2015-06-01 Thread George Dunlap
On 06/01/2015 10:06 AM, Andrew Cooper wrote: On 01/06/15 09:58, Jan Beulich wrote: On 01.06.15 at 10:50, george.dun...@eu.citrix.com wrote: On 06/01/2015 09:48 AM, Ian Campbell wrote: On Mon, 2015-06-01 at 09:36 +0100, George Dunlap wrote: Signed-off-by: George Dunlap

Re: [Xen-devel] [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL)

2015-06-01 Thread Jan Beulich
On 01.06.15 at 11:42, ian.campb...@citrix.com wrote: On Mon, 2015-06-01 at 10:21 +0100, Jan Beulich wrote: On 01.06.15 at 10:34, ian.campb...@citrix.com wrote: Subject: [PATCH] Toolstack: Do not pass -F to xm shutdown This is a feature of xl only. Signed-off-by: Ian Campbell

Re: [Xen-devel] Running 3.9 kernel in domU with 3.11 kernel in dom0

2015-06-01 Thread Julien Grall
On 30/05/15 14:43, Gautam Malu wrote: Hi, Hello, It's not necessary to send the same mail twice at one day of interval. The mail will be answered as soon as we can... The xen-arm mailing list has been archived few months ago and this question should be asked on xen-users given it's more a

Re: [Xen-devel] [PATCH v8 03/13] x86: maintain COS to CBM mapping for each socket

2015-06-01 Thread Jan Beulich
On 01.06.15 at 10:05, chao.p.p...@linux.intel.com wrote: 2) Unfeasible to allocate memory first and do initialization later in cpu hotplug notifications. My former approach is performing both the allocation and initialization in the CPU_STARTING, which is not a good idea indicated by Jan.

Re: [Xen-devel] [PATCH] drivers: xen-blkback: delay pending_req allocation to connect_ring

2015-06-01 Thread Roger Pau Monné
El 26/05/15 a les 2.06, Bob Liu ha escrit: In connect_ring, we can know exactly how many pages are used for the shared ring and also whether feature-persistent is enabled, delay pending_req allocation here so that we won't waste too much memory. I would very much prefer for this to be a

Re: [Xen-devel] [PATCH v2] dmar: device scope mem leak fix

2015-06-01 Thread Jan Beulich
On 01.06.15 at 06:47, kevin.t...@intel.com wrote: From: Tian, Kevin Sent: Monday, June 01, 2015 12:43 PM and looks you dropped earlier changes to acpi_parse_one_rmrr. any elaboration why it's not required in this version? Never mind this one. Seems you have it in RMRR patch set. No -

Re: [Xen-devel] [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL)

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 10:21 +0100, Jan Beulich wrote: On 01.06.15 at 10:34, ian.campb...@citrix.com wrote: Subject: [PATCH] Toolstack: Do not pass -F to xm shutdown This is a feature of xl only. Signed-off-by: Ian Campbell ian.campb...@citrix.com --- Osstest/Toolstack/xl.pm | 2

Re: [Xen-devel] [xen-unstable test] 57488: regressions - trouble: broken/fail/pass

2015-06-01 Thread Jan Beulich
On 30.05.15 at 01:24, osst...@xenbits.xen.org wrote: flight 57488 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/57488/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-freebsd10-amd64 14

[Xen-devel] [PATCH v2 net] xen: netback: read hotplug script once at start of day.

2015-06-01 Thread Ian Campbell
When we come to tear things down in netback_remove() and generate the uevent it is possible that the xenstore directory has already been removed (details below). In such cases netback_uevent() won't be able to read the hotplug script and will write a xenstore error node. A recent change to the

Re: [Xen-devel] Xen BUG at page_alloc.c:1738 (Xen 4.5)

2015-06-01 Thread Jan Beulich
On 31.05.15 at 00:43, andrew.coop...@citrix.com wrote: On 30/05/2015 23:07, M A Young wrote: On Fri, 29 May 2015, Andrew Cooper wrote: FC22 is miscompiling the C to: struct page_info *page = mfn_to_page(mfn); struct domain *owner = page_get_owner_and_reference(page); if ( owner )

[Xen-devel] [PATCH OSSTEST] Toolstack: Do not pass -F to xm shutdown (Was: Re: [xen-4.2-testing test] 57630: regressions - FAIL)

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 08:56 +0100, Jan Beulich wrote: On 01.06.15 at 00:57, osst...@xenbits.xen.org wrote: flight 57630 xen-4.2-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/57630/ Regressions :-( Tests which did not succeed and are blocking, including

Re: [Xen-devel] [PATCH v6 3/4] pci: add wrapper for parse_pci

2015-06-01 Thread Jan Beulich
On 29.05.15 at 23:38, elena.ufimts...@oracle.com wrote: For sbdf'si parsing in rmrr command line add __parse_pci with addtional parameter def_seg. __parse_pci will help to identify if segment was found in string being parsed or default segment was used. Make a wrapper parse_pci so the rest

Re: [Xen-devel] [PATCH 4/4] iommu: add rmrr Xen command line option for extra rmrrs

2015-06-01 Thread Jan Beulich
On 01.06.15 at 08:30, kevin.t...@intel.com wrote: From: elena.ufimts...@oracle.com [mailto:elena.ufimts...@oracle.com] --- a/docs/misc/xen-command-line.markdown +++ b/docs/misc/xen-command-line.markdown @@ -1185,6 +1185,19 @@ Specify the host reboot method. 'efi' instructs Xen to reboot

[Xen-devel] [PATCH v3 1/2] efi: Fix allocation problems if ExitBootServices() fails

2015-06-01 Thread Ross Lagerwall
If calling ExitBootServices() fails, the required memory map size may have increased. When initially allocating the memory map, allocate a slightly larger buffer (by an arbitrary 8 entries) to fix this. The ARM code path was already allocating a larger buffer than required, so this moves the code

[Xen-devel] [PATCH v3 2/4] libxc: print more error messages when failed

2015-06-01 Thread Wei Liu
No functional changes introduced. Signed-off-by: Wei Liu wei.l...@citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com --- tools/libxc/xc_hvm_build_x86.c | 30 +++--- 1 file changed, 27 insertions(+), 3 deletions(-) diff --git

[Xen-devel] [PATCH v3 3/4] libxc: rework vnuma bits in setup_guest

2015-06-01 Thread Wei Liu
Make the setup process similar to PV counterpart. That is, to allocate a P2M array that covers the whole memory range and start from there. This is clearer than using an array with no holes in it. Also the dummy layout should take MMIO hole into consideration. We might end up having two

[Xen-devel] [PATCH v3 0/4] Fix HVM vNUMA

2015-06-01 Thread Wei Liu
Boris discovered that HVM vNUMA didn't actually work. This patch series fixes that. The first patch is a prerequisite patch for the actual fixes. The second patch is to help debugging. The fixes are in the last two patches, which can be squashed into one if necessary. Patch 1 and 3 are updated

[Xen-devel] [PATCH v3 4/4] libxl: fix HVM vNUMA

2015-06-01 Thread Wei Liu
This patch does two thing: The original code erroneously fills in xc_hvm_build_args before generating vmemranges. The effect is that guest memory is populated without vNUMA information. Move the hunk to right place to fix this. Move the subtraction of video ram to

[Xen-devel] [PATCH v3 1/4] libxc/libxl: fill xc_hvm_build_args in libxl

2015-06-01 Thread Wei Liu
When building HVM guests, originally some fields of xc_hvm_build_args are filled in xc_hvm_build (and buried in the wrong function), some are set in libxl__build_hvm before passing xc_hvm_build_args to xc_hvm_build. This is fragile. After examining the code in xc_hvm_build that sets those fields,

[Xen-devel] [PATCH v3 2/2] efi: Avoid calling boot services after ExitBootServices()

2015-06-01 Thread Ross Lagerwall
After the first call to ExitBootServices(), avoid calling any boot services by setting setting efi_bs to NULL and entering an infinite loop in blexit(). Signed-off-by: Ross Lagerwall ross.lagerw...@citrix.com --- xen/common/efi/boot.c | 16 +--- 1 file changed, 13 insertions(+), 3

Re: [Xen-devel] [PATCH 4/4] iommu: add rmrr Xen command line option for extra rmrrs

2015-06-01 Thread Jan Beulich
On 29.05.15 at 23:38, elena.ufimts...@oracle.com wrote: --- a/xen/drivers/passthrough/vtd/dmar.c +++ b/xen/drivers/passthrough/vtd/dmar.c @@ -866,6 +866,106 @@ out: return ret; } +#define MAX_EXTRA_RMRR_PAGES 16 +#define MAX_EXTRA_RMRR 10 + +/* RMRR units derived from command

[Xen-devel] [PATCH OSSTEST] Stubdom test case

2015-06-01 Thread Wei Liu
Currently only QEMU traditional supports stubdom, so we only create test-amd64-amd64-xl-qemut-stubdom-debianhvm-amd64 test-amd64-amd64-xl-qemut-stubdom-debianhvm-amd64-xsm Note that stubdom only supports serial='pty'. Piping serial to stderr causes stubdom to exit abnormally. Signed-off-by:

Re: [Xen-devel] [PATCH] libxl: Don't insert PCI device into xenstore for HVM guests

2015-06-01 Thread Konrad Rzeszutek Wilk
On Fri, May 29, 2015 at 08:59:45AM +0100, Ross Lagerwall wrote: When doing passthrough of a PCI device for an HVM guest, don't insert the device into xenstore, otherwise pciback attempts to use it which conflicts with QEMU. How does it conflict? This manifests itself such that the first

Re: [Xen-devel] [PATCH] libxl: Don't insert PCI device into xenstore for HVM guests

2015-06-01 Thread Sander Eikelenboom
Monday, June 1, 2015, 5:43:53 PM, you wrote: On 06/01/2015 04:26 PM, Konrad Rzeszutek Wilk wrote: On Fri, May 29, 2015 at 08:59:45AM +0100, Ross Lagerwall wrote: When doing passthrough of a PCI device for an HVM guest, don't insert the device into xenstore, otherwise pciback attempts to use

Re: [Xen-devel] [PATCH RFC] hotplug/Linux: Add --wait to iptables calls.

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 17:09 +0100, Anthony PERARD wrote: On Mon, Jun 01, 2015 at 04:17:51PM +0100, Ian Campbell wrote: On Mon, 2015-06-01 at 16:08 +0100, Jan Beulich wrote: On 01.06.15 at 16:59, anthony.per...@citrix.com wrote: --- a/tools/hotplug/Linux/vif-common.sh +++

Re: [Xen-devel] [PATCH v6 2/4] iommu VT-d: separate rmrr addition function

2015-06-01 Thread Elena Ufimtseva
On Mon, Jun 01, 2015 at 04:51:55AM +, Tian, Kevin wrote: From: elena.ufimts...@oracle.com [mailto:elena.ufimts...@oracle.com] Sent: Saturday, May 30, 2015 5:39 AM From: Elena Ufimtseva elena.ufimts...@oracle.com In preparation for auxiliary RMRR data provided on Xen command

Re: [Xen-devel] [PATCH v6 2/4] iommu VT-d: separate rmrr addition function

2015-06-01 Thread Elena Ufimtseva
On Mon, Jun 01, 2015 at 09:53:55AM +0100, Jan Beulich wrote: On 29.05.15 at 23:38, elena.ufimts...@oracle.com wrote: In preparation for auxiliary RMRR data provided on Xen command line, make RMRR adding a separate function. Also free memery for rmrr device scope in error path. No changes

Re: [Xen-devel] [PATCH v3] run QEMU as non-root

2015-06-01 Thread Stefano Stabellini
On Fri, 29 May 2015, Ian Campbell wrote: On Fri, 2015-05-29 at 14:47 +0100, Stefano Stabellini wrote: Try to use xen-qemudepriv-$domname first, then xen-qemudepriv-base + domid, finally xen-qemudepriv-shared and root if everything else fails. The uids need to be manually created by the

Re: [Xen-devel] [PATCH RFC] hotplug/Linux: Add --wait to iptables calls.

2015-06-01 Thread Jan Beulich
On 01.06.15 at 16:59, anthony.per...@citrix.com wrote: --- a/tools/hotplug/Linux/vif-common.sh +++ b/tools/hotplug/Linux/vif-common.sh @@ -130,9 +130,9 @@ frob_iptable() local c=-D fi - iptables $c FORWARD -m physdev --physdev-is-bridged --physdev-in $dev \ + iptables --wait

Re: [Xen-devel] [PATCH RFC] hotplug/Linux: Add --wait to iptables calls.

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 16:08 +0100, Jan Beulich wrote: On 01.06.15 at 16:59, anthony.per...@citrix.com wrote: --- a/tools/hotplug/Linux/vif-common.sh +++ b/tools/hotplug/Linux/vif-common.sh @@ -130,9 +130,9 @@ frob_iptable() local c=-D fi - iptables $c FORWARD -m physdev

Re: [Xen-devel] [PATCH] x86/HVM: Avoid cache flush operations during hvm_load

2015-06-01 Thread Jan Beulich
On 01.06.15 at 17:26, ross.lagerw...@citrix.com wrote: --- a/xen/common/hvm/save.c +++ b/xen/common/hvm/save.c This being an x86-specific problem I would think this would better be addressed in x86-specific code. If it was to stay here, a proper arch hook should be introduced (even if ARM

Re: [Xen-devel] [PATCH RFC] hotplug/Linux: Add --wait to iptables calls.

2015-06-01 Thread Anthony PERARD
On Mon, Jun 01, 2015 at 04:17:51PM +0100, Ian Campbell wrote: On Mon, 2015-06-01 at 16:08 +0100, Jan Beulich wrote: On 01.06.15 at 16:59, anthony.per...@citrix.com wrote: --- a/tools/hotplug/Linux/vif-common.sh +++ b/tools/hotplug/Linux/vif-common.sh @@ -130,9 +130,9 @@ frob_iptable()

[Xen-devel] [PATCH 1/2] xen: separate the xenstore_record_dm_state calls for pv and hvm machines

2015-06-01 Thread Stefano Stabellini
The following patch will introduce a new option to restrict the privilege of the xenstore connection. In that case, we do not want to use multiple xenstore connections, but just the one, with lower privileges. For this reason, split the xenstore_record_dm_state calls for pv and hvm machines, so

Re: [Xen-devel] [PATCH] libxl: Don't insert PCI device into xenstore for HVM guests

2015-06-01 Thread Ross Lagerwall
On 06/01/2015 04:26 PM, Konrad Rzeszutek Wilk wrote: On Fri, May 29, 2015 at 08:59:45AM +0100, Ross Lagerwall wrote: When doing passthrough of a PCI device for an HVM guest, don't insert the device into xenstore, otherwise pciback attempts to use it which conflicts with QEMU. How does it

[Xen-devel] [PATCH] x86/HVM: Avoid cache flush operations during hvm_load

2015-06-01 Thread Ross Lagerwall
An MTRR record is processed for each vCPU during hvm_load. Each MTRR record sets several mtrrs, each of which flushes the cache on all pCPUs. This can take some time and trip the watchdog for HVM guests with many CPUs. To fix this, introduce a flag which prevents flushing the cache when doing

Re: [Xen-devel] [Draft C] Xen on ARM vITS Handling

2015-06-01 Thread Julien Grall
On 01/06/15 14:12, Ian Campbell wrote: On Fri, 2015-05-29 at 14:40 +0100, Julien Grall wrote: Hi Ian, Hi Ian, NIT: You used my Linaro email which I think is de-activated now :). I keep finding new address books with that address in them! ## ITS Translation Table Message signalled

[Xen-devel] [PATCH 0/2] restrict the privilege of the xenstore connection

2015-06-01 Thread Stefano Stabellini
Hi all, this patch series introduces a new command line option to restrict the privilege of the xenstore connection. Used together with -runas, can help secure the execution of QEMU in Dom0. Stefano Stabellini (2): xen: separate the xenstore_record_dm_state calls for pv and hvm machines

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-01 Thread Julien Grall
On 01/06/15 14:18, Zoltan Kiss wrote: Sure. Btw. do you have them in a public repo somewhere? It would make it a little bit easier to apply and test. It's based on the latest staging:

  1   2   >