Re: [Xen-devel] Regression, host crash with 4.5rc1

2014-11-20 Thread Jan Beulich
On 20.11.14 at 02:23, sfl...@ihonk.com wrote: On 11/17/2014 23:54, Jan Beulich wrote: Another thing - now that serial logging appears to be working for you, did you try whether the host, once hung, still reacts to serial input (perhaps force input to go to Xen right at boot via the

Re: [Xen-devel] [PATCH v10 for-xen-4.5 2/2] dpci: Replace tasklet with an softirq

2014-11-20 Thread Jan Beulich
On 19.11.14 at 17:44, konrad.w...@oracle.com wrote: On Fri, Nov 14, 2014 at 11:11:46AM -0500, Konrad Rzeszutek Wilk wrote: On Fri, Nov 14, 2014 at 03:13:42PM +, Jan Beulich wrote: On 12.11.14 at 03:23, konrad.w...@oracle.com wrote: +static void pt_pirq_softirq_reset(struct

Re: [Xen-devel] dom0 kenrel crashes for openstack + libvirt + libxl

2014-11-20 Thread Xing Lin
Hi Ian, Both of your two points are valid. There is no need to install virt-manager. And the patch to start a qemu process in /etc/init.d/xen seems to be enough for launching instances from horizon. I have updated the wiki page. Please review it at

Re: [Xen-devel] [v7][RFC][PATCH 06/13] hvmloader/ram: check if guest memory is out of reserved device memory maps

2014-11-20 Thread Tian, Kevin
From: Jan Beulich [mailto:jbeul...@suse.com] Sent: Thursday, November 20, 2014 4:04 PM On 20.11.14 at 08:45, kevin.t...@intel.com wrote: Yang and I did some discussion here. We understand your point to avoid introducing new interface if we can leverage existing code. However it's not a

Re: [Xen-devel] Problems accessing passthrough PCI device

2014-11-20 Thread Jan Beulich
On 19.11.14 at 16:12, furryfutt...@gmail.com wrote: This is getting more interesting. It seems that something is overwriting the pci-back configuration data. Starting from a fresh reboot I checked the Dom0 pci configuration and got this: root@smartin-xen:~# lspci -s 00:19.0 -x

Re: [Xen-devel] [PATCH V3] Decouple SnadyBridge quirk form VTd timeout

2014-11-20 Thread Ian Campbell
On Wed, 2014-11-19 at 12:46 -0700, Donald D. Dugger wrote: Currently the quirk code for SandyBridge uses the VTd timeout value when You've got a typo in the subject (SnadyBridge). ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH 0/4 for-4.5] xen: arm: xgene bug fixes + support for McDivitt

2014-11-20 Thread Ian Campbell
On Wed, 2014-11-19 at 16:18 -0500, Konrad Rzeszutek Wilk wrote: On Tue, Nov 18, 2014 at 04:51:42PM +, Ian Campbell wrote: On Tue, 2014-11-18 at 16:44 +, Ian Campbell wrote: These patches: ... which are also at git://xenbits.xen.org/people/ianc/xen.git mcdivitt-v1 I

[Xen-devel] [PATCH] INSTALL: correct EXTRA_CFLAGS handling

2014-11-20 Thread Olaf Hering
The already documented configure patch was not applied. Adjust documentation to describe existing behaviour. Signed-off-by: Olaf Hering o...@aepfle.de --- INSTALL | 19 +-- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/INSTALL b/INSTALL index 6bb9d23..656c90a

Re: [Xen-devel] Number of NICs per VM with qemu-upstream (Was: Re: Re: [Xen-users] libvirt emulator /usr/local/lib/xen/bin/qemu-dm emulator did not work on xen-4.4)

2014-11-20 Thread Ian Campbell
On Mon, 2014-11-17 at 13:00 +, Stefano Stabellini wrote: On Mon, 17 Nov 2014, Ian Campbell wrote: On Sat, 2014-11-15 at 10:16 +0800, hanyandong wrote: By the way, how many NICs can I apply to a VM? On xen-4.4.0, Using qemu-dm, I can apply 8 NIC to a VM, but using

Re: [Xen-devel] dom0 kenrel crashes for openstack + libvirt + libxl

2014-11-20 Thread Ian Campbell
On Wed, 2014-11-19 at 11:57 -0700, Xing Lin wrote: Hi Ian, Both of your two points are valid. There is no need to install virt-manager. And the patch to start a qemu process in /etc/init.d/xen seems to be enough for launching instances from horizon. I have updated the wiki page. Please

[Xen-devel] [PATCH 0/3] x86: XSA-109/110 follow-up (to be considered for 4.5)

2014-11-20 Thread Jan Beulich
1: tighten page table owner checking in do_mmu_update() 2: don't ignore foreigndom input on various MMUEXT ops 3: HVM: don't crash guest upon problems occurring in user mode Reason to request considering this for 4.5: Tightened argument checking (as done in the first two patches) reduces the

[Xen-devel] [PATCH 1/3] x86: tighten page table owner checking in do_mmu_update()

2014-11-20 Thread Jan Beulich
MMU_MACHPHYS_UPDATE, not manipulating page tables, shouldn't ignore a bad page table domain being specified. Also pt_owner can't be NULL when reaching the out label, so the respective check can be dropped. Signed-off-by: Jan Beulich jbeul...@suse.com Acked-by: Tim Deegan t...@xen.org ---

[Xen-devel] [PATCH 2/3] x86: don't ignore foreigndom input on various MMUEXT ops

2014-11-20 Thread Jan Beulich
Instead properly fail requests that shouldn't be issued on foreign domains or - for MMUEXT_{CLEAR,COPY}_PAGE - extend the existing operation to work that way. In the course of doing this the need to always clear okay even when wanting an error code other than -EINVAL became unwieldy, so the

Re: [Xen-devel] Xen-4.5 Testing - Cache Monitoring Technology

2014-11-20 Thread Chao Peng
On Thu, Nov 20, 2014 at 09:58:37AM +, Andrew Cooper wrote: Hi, I have found myself a PSR-capable server and have been having a play with Xen-4.5 At a first pass, I can get some numbers out: [root@blob ~]# xl psr-cmt-attach 0 [root@blob ~]# xl psr-cmt-show cache_occupancy Total

Re: [Xen-devel] [v7][RFC][PATCH 06/13] hvmloader/ram: check if guest memory is out of reserved device memory maps

2014-11-20 Thread Chen, Tiejun
Have a second struct acpi_rmrr_unit pointer, starting out as NULL and getting set to the current one when the callback returns a positive value. Skip further iterations as long as both pointers match. Great! int intel_iommu_get_reserved_device_memory(iommu_grdm_t *func, void *ctxt) {

Re: [Xen-devel] Xen 4.5 random freeze question

2014-11-20 Thread Stefano Stabellini
On Wed, 19 Nov 2014, Andrii Tseglytskyi wrote: 19 лист. 2014 20:32, користувач Stefano Stabellini stefano.stabell...@eu.citrix.com написав: On Wed, 19 Nov 2014, Julien Grall wrote: On 11/19/2014 06:14 PM, Stefano Stabellini wrote: That's right, the maintenance interrupt handler is

Re: [Xen-devel] [PATCH 1/3] x86: tighten page table owner checking in do_mmu_update()

2014-11-20 Thread Andrew Cooper
On 20/11/14 10:11, Jan Beulich wrote: MMU_MACHPHYS_UPDATE, not manipulating page tables, shouldn't ignore a bad page table domain being specified. Also pt_owner can't be NULL when reaching the out label, so the respective check can be dropped. Yes it can. Failing if ( (pg_owner =

Re: [Xen-devel] [PATCH 1/3] x86: tighten page table owner checking in do_mmu_update()

2014-11-20 Thread Andrew Cooper
On 20/11/14 10:29, Andrew Cooper wrote: On 20/11/14 10:11, Jan Beulich wrote: MMU_MACHPHYS_UPDATE, not manipulating page tables, shouldn't ignore a bad page table domain being specified. Also pt_owner can't be NULL when reaching the out label, so the respective check can be dropped. Yes it

Re: [Xen-devel] [for-xen-4.5 PATCH v2 1/2] dpci: Fix list corruption if INTx device is used and an IRQ timeout is invoked.

2014-11-20 Thread Jan Beulich
On 19.11.14 at 23:21, konrad.w...@oracle.com wrote: @@ -97,13 +97,15 @@ bool_t pt_pirq_softirq_active(struct hvm_pirq_dpci *pirq_dpci) } /* - * Reset the pirq_dpci-dom parameter to NULL. + * Cancels an outstanding pirq_dpci (if scheduled). Also if clear is set, + * reset

Re: [Xen-devel] [PATCH 2/3] x86: don't ignore foreigndom input on various MMUEXT ops

2014-11-20 Thread Tim Deegan
At 11:12 +0100 on 20 Nov (1416478351), Jan Beulich wrote: Instead properly fail requests that shouldn't be issued on foreign domains or - for MMUEXT_{CLEAR,COPY}_PAGE - extend the existing operation to work that way. In the course of doing this the need to always clear okay even when

Re: [Xen-devel] [for-xen-4.5 PATCH v2 2/2] dpci: Add ZOMBIE state to allow the softirq to finish with the dpci_pirq.

2014-11-20 Thread Jan Beulich
On 19.11.14 at 23:21, konrad.w...@oracle.com wrote: Leaving aside the question of whether this is the right approach, in case it is a couple of comments: @@ -85,7 +91,7 @@ static void raise_softirq_for(struct hvm_pirq_dpci *pirq_dpci) */ bool_t pt_pirq_softirq_active(struct

Re: [Xen-devel] Xen 4.5 random freeze question

2014-11-20 Thread Julien Grall
On 11/20/2014 10:28 AM, Stefano Stabellini wrote: On Wed, 19 Nov 2014, Andrii Tseglytskyi wrote: 19 лист. 2014 20:32, користувач Stefano Stabellini stefano.stabell...@eu.citrix.com написав: On Wed, 19 Nov 2014, Julien Grall wrote: On 11/19/2014 06:14 PM, Stefano Stabellini wrote: That's

Re: [Xen-devel] [PATCH v2 for-4.5 1/5] xen: arm: Add earlyprintk for McDivitt.

2014-11-20 Thread Julien Grall
Hi Ian, On 11/19/2014 03:28 PM, Ian Campbell wrote: Signed-off-by: Ian Campbell ian.campb...@citrix.com --- v2: Remove pointless/unused baud rate setting. A bunch of other entries have these, but cleaning them up is out of scope here I think. Agreed. Reviewed-by: Julien Grall

Re: [Xen-devel] [PATCH 4/4] x86/xen: use the maximum MFN to calculate the required DMA mask

2014-11-20 Thread Jan Beulich
On 19.11.14 at 17:02, david.vra...@citrix.com wrote: On a Xen PV guest the DMA addresses and physical addresses are not 1:1 (such as Xen PV guests) and the generic dma_get_required_mask() does not return the correct mask (since it uses max_pfn). Some device drivers (such as mptsas, mpt2sas)

Re: [Xen-devel] [PATCH v2 for-4.5 2/5] xen: arm: Drop EARLY_PRINTK_BAUD from entries which don't set ..._INIT_UART

2014-11-20 Thread Julien Grall
Hi Ian, On 11/19/2014 03:28 PM, Ian Campbell wrote: EARLY_PRINTK_BAUD doesn't do anything unless EARLY_PRINTK_INIT_UART is set. Furthermore only the pl011 driver implements the init routine at all, so the entries which use 8250 and specified a BAUD were doubly wrong. NIT: and exynos4210

Re: [Xen-devel] [Spice-devel] screen freezed for 2-3 minutes on spice connect on xen windows 7 domU's with qxl after save/restore

2014-11-20 Thread Fabio Fantoni
Il 13/11/2014 13:22, Fabio Fantoni ha scritto: Il 13/11/2014 11:14, Fabio Fantoni ha scritto: Il 19/09/2014 15:18, Fabio Fantoni ha scritto: Il 12/09/2014 16:46, Fabio Fantoni ha scritto: Il 08/07/2014 12:34, Fabio Fantoni ha scritto: Il 08/07/2014 12:06, Fabio Fantoni ha scritto: Il

Re: [Xen-devel] [PATCH v2 for-4.5 3/5] xen: arm: correct off by one in xgene-storm's map_one_mmio

2014-11-20 Thread Julien Grall
Hi Ian, On 11/19/2014 03:28 PM, Ian Campbell wrote: The callers pass the end as the pfn immediately *after* the last page to be mapped, therefore adding one is incorrect and causes an additional page to be mapped. At the same time correct the printing of the mfn values, zero-padding them to

[Xen-devel] [PATCH v2] xen: use more fixed strings to build the hypervisor

2014-11-20 Thread Olaf Hering
It should be possible to repeatedly build identical sources and get identical binaries, even on different hosts at different build times. This fails for xen.gz and xen.efi because current time and buildhost get included in the binaries. Provide variables XEN_BUILD_DATE, XEN_BUILD_TIME and

Re: [Xen-devel] [PATCH 4/4] x86/xen: use the maximum MFN to calculate the required DMA mask

2014-11-20 Thread David Vrabel
On 19/11/14 17:51, Stefano Stabellini wrote: On Wed, 19 Nov 2014, David Vrabel wrote: +u64 +xen_swiotlb_get_required_mask(struct device *dev) +{ +unsigned long max_mfn; + +max_mfn = HYPERVISOR_memory_op(XENMEM_maximum_ram_page, NULL); As Jan pointed out, I think you need to

Re: [Xen-devel] Number of NICs per VM with qemu-upstream (Was: Re: Re: [Xen-users] libvirt emulator /usr/local/lib/xen/bin/qemu-dm emulator did not work on xen-4.4)

2014-11-20 Thread Ian Campbell
On Thu, 2014-11-20 at 11:39 +, Stefano Stabellini wrote: On Thu, 20 Nov 2014, Ian Campbell wrote: On Mon, 2014-11-17 at 13:00 +, Stefano Stabellini wrote: On Mon, 17 Nov 2014, Ian Campbell wrote: On Sat, 2014-11-15 at 10:16 +0800, hanyandong wrote: By the way, how many NICs

[Xen-devel] [PATCHv4 0/4]: dma, x86, xen: reduce SWIOTLB usage in Xen guests

2014-11-20 Thread David Vrabel
On systems where DMA addresses and physical addresses are not 1:1 (such as Xen PV guests), the generic dma_get_required_mask() will not return the correct mask (since it uses max_pfn). Some device drivers (such as mptsas, mpt2sas) use dma_get_required_mask() to set the device's DMA mask to allow

[Xen-devel] [PATCH 3/4] x86: allow dma_get_required_mask() to be overridden

2014-11-20 Thread David Vrabel
Use dma_ops-get_required_mask() if provided, defaulting to dma_get_requried_mask_from_max_pfn(). This is needed on systems (such as Xen PV guests) where the DMA address and the physical address are not equal. ARCH_HAS_DMA_GET_REQUIRED_MASK is defined in asm/device.h instead of asm/dma-mapping.h

[Xen-devel] [PATCH 1/4] dma: add dma_get_required_mask_from_max_pfn()

2014-11-20 Thread David Vrabel
A generic dma_get_required_mask() is useful even for architectures (such as ia64) that define ARCH_HAS_GET_REQUIRED_MASK. Signed-off-by: David Vrabel david.vra...@citrix.com Reviewed-by: Stefano Stabellini stefano.stabell...@eu.citrix.com --- drivers/base/platform.c | 10 --

[Xen-devel] [PATCH 4/4] x86/xen: assume a 64-bit DMA mask is required

2014-11-20 Thread David Vrabel
On a Xen PV guest the DMA addresses and physical addresses are not 1:1 (such as Xen PV guests) and the generic dma_get_required_mask() does not return the correct mask (since it uses max_pfn). Some device drivers (such as mptsas, mpt2sas) use dma_get_required_mask() to set the device's DMA mask

Re: [Xen-devel] [Vote] Confirm Konrad Rzeszutek Wilk as Xen project Hypervisor Committer (please vote by Nov 30th)

2014-11-20 Thread Ian Jackson
Lars Kurth writes ([Xen-devel] [Vote] Confirm Konrad Rzeszutek Wilk as Xen project Hypervisor Committer (please vote by Nov 30th)): The voting form is at https://docs.google.com/forms/d/ 1Hpoda2VjdMMGDsz1zh01tkHPR1vUsVeUVAR0DWhlgik/viewform but if you want to vote in public feel free to just

Re: [Xen-devel] [Vote] Confirm Konrad Rzeszutek Wilk as Xen project Hypervisor Committer (please vote by Nov 30th)

2014-11-20 Thread Tim Deegan
At 16:26 + on 17 Nov (1416237976), Lars Kurth wrote: last week Ian Jackson nominated Konrad as Xen Project Hypervisor committer. Our governance process requires a formal vote by existing committers to confirm Konrad and for me to set up a voting form. Existing committers are: Keir

Re: [Xen-devel] [PATCH 3/3] x86/HVM: don't crash guest upon problems occurring in user mode

2014-11-20 Thread Jan Beulich
On 20.11.14 at 12:34, t...@xen.org wrote: At 11:13 +0100 on 20 Nov (1416478386), Jan Beulich wrote: This extends commit 5283b310 (x86/HVM: only kill guest when unknown VM exit occurred in guest kernel mode) to further cases, including the failed VM entry one that XSA-110 was needed to be

Re: [Xen-devel] Xen-4.5 Testing - Cache Monitoring Technology

2014-11-20 Thread Andrew Cooper
On 20/11/14 10:23, Andrew Cooper wrote: On 20/11/14 10:15, Chao Peng wrote: On Thu, Nov 20, 2014 at 09:58:37AM +, Andrew Cooper wrote: Hi, I have found myself a PSR-capable server and have been having a play with Xen-4.5 At a first pass, I can get some numbers out: [root@blob ~]# xl

Re: [Xen-devel] [RFC for 4.6] xen: Extend DOMCTL createdomain to support arch configuration

2014-11-20 Thread Jan Beulich
On 18.11.14 at 20:20, julien.gr...@linaro.org wrote: --- a/xen/arch/x86/setup.c +++ b/xen/arch/x86/setup.c @@ -540,6 +540,7 @@ void __init noreturn __start_xen(unsigned long mbi_p) int i, j, e820_warn = 0, bytes = 0; bool_t acpi_boot_table_init_done = 0; struct domain *dom0;

Re: [Xen-devel] [v7][RFC][PATCH 06/13] hvmloader/ram: check if guest memory is out of reserved device memory maps

2014-11-20 Thread Tian, Kevin
From: Tian, Kevin Sent: Thursday, November 20, 2014 4:51 PM From: Jan Beulich [mailto:jbeul...@suse.com] Sent: Thursday, November 20, 2014 4:04 PM On 20.11.14 at 08:45, kevin.t...@intel.com wrote: Yang and I did some discussion here. We understand your point to avoid introducing

Re: [Xen-devel] [v7][RFC][PATCH 06/13] hvmloader/ram: check if guest memory is out of reserved device memory maps

2014-11-20 Thread Jan Beulich
On 20.11.14 at 15:40, kevin.t...@intel.com wrote: From: Tian, Kevin Sent: Thursday, November 20, 2014 4:51 PM From: Jan Beulich [mailto:jbeul...@suse.com] Sent: Thursday, November 20, 2014 4:04 PM On 20.11.14 at 08:45, kevin.t...@intel.com wrote: Current option sounds a reasonable

Re: [Xen-devel] Number of NICs per VM with qemu-upstream (Was: Re: Re: [Xen-users] libvirt emulator /usr/local/lib/xen/bin/qemu-dm emulator did not work on xen-4.4)

2014-11-20 Thread Ian Campbell
On Thu, 2014-11-20 at 14:46 +, Stefano Stabellini wrote: On Thu, 20 Nov 2014, Ian Campbell wrote: There is, it's the romfile option to -device e.g. -device $NICMODEL,vlan=0,romfile=$ROMFILE where NICMODEL is e100, rtl8139, virtio-blah and ROMFILE is e.g. an ipxe

Re: [Xen-devel] Number of NICs per VM with qemu-upstream (Was: Re: Re: [Xen-users] libvirt emulator /usr/local/lib/xen/bin/qemu-dm emulator did not work on xen-4.4)

2014-11-20 Thread Ian Campbell
create ^ title it qemu-upstream: limitation on 4 emulated NICs prevents guest from starting unless PV override is used. thanks ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [RFC for 4.6] xen: Extend DOMCTL createdomain to support arch configuration

2014-11-20 Thread Julien Grall
Hi Jan, On 11/20/2014 02:37 PM, Jan Beulich wrote: On 18.11.14 at 20:20, julien.gr...@linaro.org wrote: --- a/xen/arch/x86/setup.c +++ b/xen/arch/x86/setup.c @@ -540,6 +540,7 @@ void __init noreturn __start_xen(unsigned long mbi_p) int i, j, e820_warn = 0, bytes = 0; bool_t

[Xen-devel] Processed: Re: Number of NICs per VM with qemu-upstream (Was: Re: Re: [Xen-users] libvirt emulator /usr/local/lib/xen/bin/qemu-dm emulator did not work on xen-4.4)

2014-11-20 Thread xen
Processing commands for x...@bugs.xenproject.org: create ^ Created new bug #46 rooted at `1416474814.29243.59.ca...@citrix.com' Title: `Re: [Xen-devel] Number of NICs per VM with qemu-upstream (Was: Re: Re: [Xen-users] libvirt emulator /usr/local/lib/xen/bin/qemu-dm emulator did not work on

[Xen-devel] [PATCH for-4.5 v2] docs/commandline: Fix formatting issues

2014-11-20 Thread Andrew Cooper
For 'dom0_max_vcpus' and 'hvm_debug', markdown was interpreting the text as regular text, and reflowing it as a regular paragraph, leading to a single line as output. Reformat them as code blocks inside blockquote blocks, which causes them to take their precise whitespace layout. For 'psr', the

Re: [Xen-devel] [PATCH 0/2 V3] fix rename: xenstore not fully updated

2014-11-20 Thread Ian Campbell
On Wed, 2014-11-19 at 16:25 -0500, Konrad Rzeszutek Wilk wrote: On Wed, Nov 19, 2014 at 11:26:32AM +, Ian Jackson wrote: Hi Konrad, I have another release ack request: Chunyan Liu writes ([PATCH 0/2 V3] fix rename: xenstore not fully updated): Currently libxl__domain_rename only

Re: [Xen-devel] [PATCH v2] mkdeb: correctly map package architectures for x86 and ARM

2014-11-20 Thread Ian Campbell
On Wed, 2014-11-19 at 14:45 -0500, Konrad Rzeszutek Wilk wrote: On Fri, Nov 14, 2014 at 10:10:58AM +, Ian Campbell wrote: (CCing some more maintainers and the release manager) On Wed, 2014-11-12 at 15:43 +, Ian Campbell wrote: On Wed, 2014-11-12 at 09:38 -0600, Clark Laughlin

Re: [Xen-devel] [PATCH] libxl: Document device parameter of libxl_device_type_add functions

2014-11-20 Thread Ian Campbell
On Tue, 2014-11-18 at 13:19 -0500, Konrad Rzeszutek Wilk wrote: On Tue, Nov 18, 2014 at 05:44:20PM +, Ian Jackson wrote: Euan Harris writes ([PATCH] libxl: Document device parameter of libxl_device_type_add functions): The device parameter of libxl_device_type_add is an in/out

Re: [Xen-devel] [PATCH for-4.5] libxl: remove existence check for PCI device hotplug

2014-11-20 Thread Ian Campbell
On Wed, 2014-11-19 at 16:28 -0500, Konrad Rzeszutek Wilk wrote: On Wed, Nov 19, 2014 at 09:21:23PM +, Wei Liu wrote: On Wed, Nov 19, 2014 at 04:01:54PM -0500, Konrad Rzeszutek Wilk wrote: On Mon, Nov 17, 2014 at 12:10:34PM +, Wei Liu wrote: The existence check is to make sure a

Re: [Xen-devel] [PATCH 0/5 v2 for-4.5] xen: arm: xgene bug fixes + support for McDivitt

2014-11-20 Thread Ian Campbell
On Wed, 2014-11-19 at 16:27 -0500, Konrad Rzeszutek Wilk wrote: On Wed, Nov 19, 2014 at 03:27:48PM +, Ian Campbell wrote: These patches: * fix up an off by one bug in the xgene mapping of additional PCI bus resources, which would cause an additional extra page to be

Re: [Xen-devel] [PATCH for-4.5 v2] docs/commandline: Fix formatting issues

2014-11-20 Thread Ian Campbell
On Thu, 2014-11-20 at 15:22 +, Andrew Cooper wrote: For 'dom0_max_vcpus' and 'hvm_debug', markdown was interpreting the text as regular text, and reflowing it as a regular paragraph, leading to a single line as output. Reformat them as code blocks inside blockquote blocks, which causes

Re: [Xen-devel] [PATCH for-4.5 RFC] pygrub: Fix regression from c/s d1b93ea, attempt 2

2014-11-20 Thread Ian Campbell
On Mon, 2014-11-17 at 15:19 +, Andrew Cooper wrote: c/s d1b93ea causes substantial functional regressions in pygrub's ability to parse bootloader configuration files. c/s d1b93ea itself changed an an interface which previously used exclusively integers, to using strings in the case of a

Re: [Xen-devel] Xen 4.5 random freeze question

2014-11-20 Thread Andrii Tseglytskyi
I think I'll debug this a bit later - unfortunately, now don't have time for this. But I want to get rid of spurious interrupt here. BTW - Stefano are you going to post the patch that we created yesterday ? Will Ian accept it? Regards, Andrii On Thu, Nov 20, 2014 at 1:15 PM, Julien Grall

Re: [Xen-devel] [PATCH for 4.5] xl: Return proper error codes for block-attach and block-detach

2014-11-20 Thread George Dunlap
On Wed, Nov 12, 2014 at 5:31 PM, George Dunlap george.dun...@eu.citrix.com wrote: Return proper error codes on failure so that scripts can tell whether the command completed properly or not. How about changing this to something like: --- Return proper error codes on failure so that scripts can

Re: [Xen-devel] [PATCH v2 for 4.5] xl: Return proper error codes for block-attach and block-detach

2014-11-20 Thread George Dunlap
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 11/20/2014 03:56 PM, Ian Campbell wrote: On Thu, 2014-11-20 at 15:51 +, George Dunlap wrote: On 11/20/2014 03:47 PM, Ian Campbell wrote: On Mon, 2014-11-17 at 12:36 +, George Dunlap wrote: On 11/14/2014 11:12 AM, Ian Campbell wrote:

Re: [Xen-devel] [PATCH for-4.5] scripts/get_maintainer.pl: Correctly CC the maintainers

2014-11-20 Thread Ian Campbell
On Tue, 2014-11-18 at 20:03 +, Julien Grall wrote: By default, the script get_maintainer.pl will remove duplicates email as soon as it appends the list of maintainers of a new file, and therefore override the role of the developper. On complex patch (see [1]), this will result to

Re: [Xen-devel] [PATCH for-4.5 RFC] pygrub: Fix regression from c/s d1b93ea, attempt 2

2014-11-20 Thread Andrew Cooper
On 20/11/14 16:00, Ian Campbell wrote: On Mon, 2014-11-17 at 15:19 +, Andrew Cooper wrote: c/s d1b93ea causes substantial functional regressions in pygrub's ability to parse bootloader configuration files. c/s d1b93ea itself changed an an interface which previously used exclusively

Re: [Xen-devel] Number of NICs per VM with qemu-upstream (Was: Re: Re: [Xen-users] libvirt emulator /usr/local/lib/xen/bin/qemu-dm emulator did not work on xen-4.4)

2014-11-20 Thread Stefano Stabellini
On Thu, 20 Nov 2014, Ian Campbell wrote: On Thu, 2014-11-20 at 14:46 +, Stefano Stabellini wrote: On Thu, 20 Nov 2014, Ian Campbell wrote: There is, it's the romfile option to -device e.g. -device $NICMODEL,vlan=0,romfile=$ROMFILE where NICMODEL is e100,

[Xen-devel] Xen Security Advisory 113 - Guest effectable page reference leak in MMU_MACHPHYS_UPDATE handling

2014-11-20 Thread Xen . org security team
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Xen Security Advisory XSA-113 Guest effectable page reference leak in MMU_MACHPHYS_UPDATE handling ISSUE DESCRIPTION = An error handling path in the processing of MMU_MACHPHYS_UPDATE failed to drop a page

Re: [Xen-devel] [PATCH for-4.5] scripts/get_maintainer.pl: Correctly CC the maintainers

2014-11-20 Thread Ian Campbell
On Thu, 2014-11-20 at 16:21 +, Julien Grall wrote: On 11/20/2014 04:15 PM, Julien Grall wrote: Hi Ian, On 11/20/2014 04:08 PM, Ian Campbell wrote: On Tue, 2014-11-18 at 20:03 +, Julien Grall wrote: By default, the script get_maintainer.pl will remove duplicates email as soon

Re: [Xen-devel] [PATCH for-4.5] scripts/get_maintainer.pl: Correctly CC the maintainers

2014-11-20 Thread Ian Campbell
On Thu, 2014-11-20 at 16:15 +, Julien Grall wrote: Hi Ian, On 11/20/2014 04:08 PM, Ian Campbell wrote: On Tue, 2014-11-18 at 20:03 +, Julien Grall wrote: By default, the script get_maintainer.pl will remove duplicates email as soon as it appends the list of maintainers of a

Re: [Xen-devel] [PATCH for-4.5 RFC] pygrub: Fix regression from c/s d1b93ea, attempt 2

2014-11-20 Thread Boris Ostrovsky
On 11/20/2014 11:15 AM, Ian Campbell wrote: On Thu, 2014-11-20 at 16:08 +, Andrew Cooper wrote: On 20/11/14 16:00, Ian Campbell wrote: On Mon, 2014-11-17 at 15:19 +, Andrew Cooper wrote: c/s d1b93ea causes substantial functional regressions in pygrub's ability to parse bootloader

Re: [Xen-devel] Xen 4.5 random freeze question

2014-11-20 Thread Andrii Tseglytskyi
OK - I see. thanks a lot. On Thu, Nov 20, 2014 at 6:15 PM, Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: Already posted: http://marc.info/?l=xen-develm=141648092100568 Ian hasn't provided any feedback yet. On Thu, 20 Nov 2014, Andrii Tseglytskyi wrote: I think I'll debug

Re: [Xen-devel] [PATCH v2 for-4.5] xen/arm: clear UIE on hypervisor entry

2014-11-20 Thread Julien Grall
Hi Stefano, On 11/20/2014 03:54 PM, Stefano Stabellini wrote: On Thu, 20 Nov 2014, Julien Grall wrote: On 11/20/2014 11:02 AM, Julien Grall wrote: Hi Stefano, On 11/20/2014 10:53 AM, Stefano Stabellini wrote: UIE being set can cause maintenance interrupts to occur when Xen writes to one or

Re: [Xen-devel] [PATCH v2 for-4.5] xen/arm: clear UIE on hypervisor entry

2014-11-20 Thread Ian Campbell
On Thu, 2014-11-20 at 10:53 +, Stefano Stabellini wrote: UIE being set can cause maintenance interrupts to occur when Xen writes to one or more LR registers. The effect is a busy loop around the interrupt handler in Xen (http://marc.info/?l=xen-develm=141597517132682): everything gets

Re: [Xen-devel] [PATCH for-4.5] scripts/get_maintainer.pl: Correctly CC the maintainers

2014-11-20 Thread Ian Campbell
On Thu, 2014-11-20 at 16:43 +, Julien Grall wrote: On 11/20/2014 04:29 PM, Ian Campbell wrote: Forgot to add, the example above show the difference without and with the patch. The list is correct because both ARM and x86 maintainers should be CC. Because of this all THE REST maintainers

Re: [Xen-devel] [PATCH for-4.5] scripts/get_maintainer.pl: Correctly CC the maintainers

2014-11-20 Thread Ian Campbell
On Thu, 2014-11-20 at 16:52 +, Ian Campbell wrote: On Thu, 2014-11-20 at 16:43 +, Julien Grall wrote: On 11/20/2014 04:29 PM, Ian Campbell wrote: Forgot to add, the example above show the difference without and with the patch. The list is correct because both ARM and x86

Re: [Xen-devel] [PATCH for-4.5 v2] libxc: don't leak buffer containing the uncompressed PV kernel

2014-11-20 Thread Frediano Ziglio
2014-11-20 15:48 GMT+00:00 Ian Campbell ian.campb...@citrix.com: The libxc xc_dom_* infrastructure uses a very simple malloc memory pool which is freed by xc_dom_release. However the various xc_try_*_decode routines (other than the gzip one) just use plain malloc/realloc and therefore the

Re: [Xen-devel] [OSSTEST PATCH] linux-next tests: Use correct branch for baseline

2014-11-20 Thread Ian Campbell
On Thu, 2014-11-20 at 13:51 +, Ian Jackson wrote: Make cr-daily-branch honour an environment or setting variable EXTRA_SGR_ARGS. In branch-settings.linux-next set it appropriately to arrange that the linux-next test reports consider linux-linus tests as interesting as well as just

[Xen-devel] [OSSTEST PATCH] README.planner: Document the resource planning system

2014-11-20 Thread Ian Jackson
Signed-off-by: Ian Jackson ian.jack...@eu.citrix.com --- README.planner | 181 +++- 1 file changed, 180 insertions(+), 1 deletion(-) diff --git a/README.planner b/README.planner index de8b962..ec4dce8 100644 --- a/README.planner +++

[Xen-devel] Buggy interaction of live migration and p2m updates

2014-11-20 Thread Andrew Cooper
Hello, Tim, David and I were discussing this over lunch. This email is a (hopefully accurate) account of our findings, and potential solutions. (If I have messed up, please shout.) Currently, correct live migration of PV domains relies on the toolstack (which has a live mapping of the guests

Re: [Xen-devel] [PATCH v2 for 4.5] xl: Return proper error codes for block-attach and block-detach

2014-11-20 Thread Konrad Rzeszutek Wilk
On Thu, Nov 20, 2014 at 03:47:04PM +, Ian Campbell wrote: On Mon, 2014-11-17 at 12:36 +, George Dunlap wrote: On 11/14/2014 11:12 AM, Ian Campbell wrote: On Thu, 2014-11-13 at 19:04 +, George Dunlap wrote: Return proper error codes on failure so that scripts can tell whether

Re: [Xen-devel] [for-xen-4.5 PATCH v2 2/2] dpci: Add ZOMBIE state to allow the softirq to finish with the dpci_pirq.

2014-11-20 Thread Konrad Rzeszutek Wilk
On Thu, Nov 20, 2014 at 11:55:43AM +0100, Jan Beulich wrote: On 19.11.14 at 23:21, konrad.w...@oracle.com wrote: Leaving aside the question of whether this is the right approach, in case it is a couple of comments: @@ -85,7 +91,7 @@ static void raise_softirq_for(struct hvm_pirq_dpci

Re: [Xen-devel] Regression, host crash with 4.5rc1

2014-11-20 Thread Steve Freitas
Hi Jan, Thanks for all your help so far! Here's my latest update. On 11/17/2014 23:54, Jan Beulich wrote: Plus, without said adjustment, first just disable the MWAIT CPU idle driver (mwait-idle=0) and then, if that didn't make a difference, use of C states altogether (cpuidle=0). If any of

Re: [Xen-devel] [PATCH v2 for-4.5] xen/arm: clear UIE on hypervisor entry

2014-11-20 Thread Konrad Rzeszutek Wilk
On Thu, Nov 20, 2014 at 04:47:42PM +, Ian Campbell wrote: On Thu, 2014-11-20 at 10:53 +, Stefano Stabellini wrote: UIE being set can cause maintenance interrupts to occur when Xen writes to one or more LR registers. The effect is a busy loop around the interrupt handler in Xen

Re: [Xen-devel] [for-xen-4.5 PATCH v2 2/2] dpci: Add ZOMBIE state to allow the softirq to finish with the dpci_pirq.

2014-11-20 Thread Sander Eikelenboom
Thursday, November 20, 2014, 8:51:33 PM, you wrote: Ah crud. So a simple fix could be to seperate the 'state' to only deal with the raise_softirq and softirq_dpci. And then add a new (old) 'masked' to deal between hvm_dirq_assist, pt_irq_guest_eoi and hvm_do_IRQ_dpci. From

Re: [Xen-devel] [PATCH for-4.5 v2] libxc: don't leak buffer containing the uncompressed PV kernel

2014-11-20 Thread Konrad Rzeszutek Wilk
On Thu, Nov 20, 2014 at 03:48:47PM +, Ian Campbell wrote: The libxc xc_dom_* infrastructure uses a very simple malloc memory pool which is freed by xc_dom_release. However the various xc_try_*_decode routines (other than the gzip one) just use plain malloc/realloc and therefore the buffer

Re: [Xen-devel] [PATCH v2 for 4.5] scripts/get_maintainer.pl: Correctly CC the maintainers

2014-11-20 Thread Konrad Rzeszutek Wilk
On Thu, Nov 20, 2014 at 05:36:03PM +, Julien Grall wrote: The current script is setting $email_remove_duplicates to 1 by default, on complex patch (see [1]), this will result to ommitting randomly some maintainers. One could see that as feature - the emails about bugs or patches to review

Re: [Xen-devel] [PATCH v9 12/13] swiotlb-xen: pass dev_addr to xen_dma_unmap_page and xen_dma_sync_single_for_cpu

2014-11-20 Thread Konrad Rzeszutek Wilk
On Thu, Nov 20, 2014 at 10:47:51AM +, Stefano Stabellini wrote: On Thu, 20 Nov 2014, Stefano Stabellini wrote: On Wed, 19 Nov 2014, Konrad Rzeszutek Wilk wrote: On Wed, Nov 12, 2014 at 11:40:53AM +, Stefano Stabellini wrote: xen_dma_unmap_page and xen_dma_sync_single_for_cpu take

[Xen-devel] [PATCH V4] Decouple SandyBridge quirk from VTd timeout

2014-11-20 Thread Donald D. Dugger
Currently the quirk code for SandyBridge uses the VTd timeout value when writing to an IGD register. This is the wrong timeout to use and, at 1000 msec., is also much too large. This patch changes the quirk code to use a timeout that is specific to the IGD device and allows the user control of

Re: [Xen-devel] [v7][RFC][PATCH 04/13] hvmloader/util: get reserved device memory maps

2014-11-20 Thread Chen, Tiejun
On 2014/11/3 18:02, Jan Beulich wrote: On 03.11.14 at 10:55, tiejun.c...@intel.com wrote: On 2014/11/3 17:45, Jan Beulich wrote: On 03.11.14 at 10:32, tiejun.c...@intel.com wrote: On 2014/11/3 16:53, Jan Beulich wrote: On 03.11.14 at 03:22, tiejun.c...@intel.com wrote: On 2014/10/31 16:14,

Re: [Xen-devel] [for-xen-4.5 PATCH v2 2/2] dpci: Add ZOMBIE state to allow the softirq to finish with the dpci_pirq.

2014-11-20 Thread Jan Beulich
On 20.11.14 at 20:51, konrad.w...@oracle.com wrote: @@ -669,7 +670,7 @@ static void hvm_dirq_assist(struct domain *d, struct hvm_pirq_dpci *pirq_dpci) ASSERT(d-arch.hvm_domain.irq.dpci); spin_lock(d-event_lock); -if ( pirq_dpci-state ) +if (

Re: [Xen-devel] [v7][RFC][PATCH 04/13] hvmloader/util: get reserved device memory maps

2014-11-20 Thread Jan Beulich
On 21.11.14 at 08:43, kevin.t...@intel.com wrote: From: Chen, Tiejun Sent: Friday, November 21, 2014 2:26 PM On 2014/11/3 18:02, Jan Beulich wrote: On 03.11.14 at 10:55, tiejun.c...@intel.com wrote: On 2014/11/3 17:45, Jan Beulich wrote: On 03.11.14 at 10:32, tiejun.c...@intel.com