Re: [Xen-devel] vTPM ownership problem

2015-09-15 Thread Xu, Quan
1. For xen 4.3.0, you’d better enable XSM security module. 2. share your vtpmmgr / vtpm instance log. 3. Clear vtpm / vtpmmgr disk, do it again. Quan From: xen-devel-boun...@lists.xen.org [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of Zhenyang Feng Sent: Wednesday, September 09, 2015

[Xen-devel] [PATCH] pci-attach: fix assertation

2015-09-15 Thread Chunyan Liu
Run "xl pci-attach ", the 2nd time fails: xl: libxl_xshelp.c:209: libxl__xs_transaction_start: Assertion `!*t' failed. Aborted To fix that, initialize xs_transaction to avoid libxl__xs_transaction_start assertation error. Signed-off-by: Chunyan Liu --- tools/libxl/libxl_pci.c | 2 +- 1 file ch

[Xen-devel] [xen-4.1-testing baseline-only test] 37935: tolerable FAIL

2015-09-15 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 37935 xen-4.1-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/37935/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-migrupgrade 1 build-check(1) b

[Xen-devel] [PATCH v2 0/4] Fix tools/xen-mceinj to handle >=4GB guest memory

2015-09-15 Thread Haozhong Zhang
The existing xen-mceinj can not inject MCE through MSR_MCI_ADDR to a domain w/ more than 4GB memory, e.g. if domain 0 has more than 4GB memory, the execution of the command xen-mceinj -d 0 -t 0 -p 0x2721a900 will fail w/ a message "Failed to get pfn list : Operation not supported".

[Xen-devel] [PATCH v2 4/4] tools/xen-mceinj: Pass in GPA when injecting through MSR_MCI_ADDR

2015-09-15 Thread Haozhong Zhang
This patch removes the address translation in xen-mceinj which translates the guest physical address passed-in through the argument of '-p' to the host machine address. Instead, xen-mceinj now passes a flag MC_MSRINJ_F_GPADDR to ask do_mca() in the hypervisor to do this translation. Signed-off-by:

[Xen-devel] [PATCH v2 2/4] tools/xen-mceinj: Fix code style

2015-09-15 Thread Haozhong Zhang
Remove trailing whitespaces in xen-mceinj.c. Signed-off-by: Haozhong Zhang --- tools/tests/mce-test/tools/xen-mceinj.c | 66 - 1 file changed, 33 insertions(+), 33 deletions(-) diff --git a/tools/tests/mce-test/tools/xen-mceinj.c b/tools/tests/mce-test/tools/xen

[Xen-devel] [PATCH v2 1/4] x86/mce: Fix code style

2015-09-15 Thread Haozhong Zhang
Remove trailing whitespaces and fix indentations in mce.c and xen_mca.h. Signed-off-by: Haozhong Zhang --- xen/arch/x86/cpu/mcheck/mce.c | 10 +- xen/include/public/arch-x86/xen-mca.h | 30 +++--- 2 files changed, 20 insertions(+), 20 deletions(-) diff --

[Xen-devel] [PATCH v2 3/4] x86/mce: Translate passed-in GPA to host machine address

2015-09-15 Thread Haozhong Zhang
This patch adds a new flag MC_MSRINJ_F_GPADDR to xen_mc_msrinject.mcinj_flags, and makes do_mca() to translate the guest physical address passed-in through xen_mc_msrinject.mcinj_msr[i].value to the host machine address if this flag is present. Signed-off-by: Haozhong Zhang --- xen/arch/x86/cpu/

[Xen-devel] [xen-4.3-testing test] 61961: regressions - FAIL

2015-09-15 Thread osstest service owner
flight 61961 xen-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/61961/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-raw 9 debian-di-install fail REGR. vs. 60742 Tests which are f

[Xen-devel] [Patch RFC 09/13] vt-d: Put the page in Queued Invalidation(QI) interrupt handler if

2015-09-15 Thread Quan Xu
the Device-TLB flush is completed. Signed-off-by: Quan Xu --- xen/drivers/passthrough/vtd/iommu.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/xen/drivers/passthrough/vtd/iommu.c b/xen/drivers/passthrough/vtd/iommu.c index 5c03e41..1297dea 100644 --- a/xen/drivers/passthrough

[Xen-devel] [Patch RFC 12/13] vt-d: For gnttab_transfer, If the Device-TLB flush is still

2015-09-15 Thread Quan Xu
not completed when to map the transferring page to a remote domain, schedule and wait on a waitqueue until the Device-TLB flush is completed. Signed-off-by: Quan Xu --- xen/common/grant_table.c | 16 1 file changed, 16 insertions(+) diff --git a/xen/common/grant_table.c b/xen/c

[Xen-devel] [Patch RFC 08/13] vt-d: Held on the freed page until the Device-TLB flush is completed.

2015-09-15 Thread Quan Xu
The page freed from the domain should be on held, until the Device-TLB flush is completed. The page previously associated with the freed portion of GPA should not be reallocated for another purpose until the appropriate invalidations have been performed. Otherwise, the original page owner can still

[Xen-devel] [Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device

2015-09-15 Thread Quan Xu
Introduction VT-d code currently has a number of cases where completion of certain operations is being waited for by way of spinning. The majority of instances use that variable indirectly through IOMMU_WAIT_OP() macro , allowing for loops of up to 1 second (DMAR_OPERATION_TIMEOU

[Xen-devel] [Patch RFC 03/13] vt-d: Track the Device-TLB invalidation status in an invalidation table.

2015-09-15 Thread Quan Xu
Update invalidation table's count of in-flight Device-TLB invalidation request and assign the address of global polling parameter per domain in the Status Address of each invalidation wait descriptor, when submit Device-TLB invalidation requests. Signed-off-by: Quan Xu --- xen/drivers/passthroug

[Xen-devel] [Patch RFC 01/13] vt-d: Redefine iommu_set_interrupt() for registering MSI interrupt

2015-09-15 Thread Quan Xu
Signed-off-by: Quan Xu --- xen/drivers/passthrough/vtd/iommu.c | 21 - 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/xen/drivers/passthrough/vtd/iommu.c b/xen/drivers/passthrough/vtd/iommu.c index 1dffc40..17bfb76 100644 --- a/xen/drivers/passthrough/vtd/iomm

[Xen-devel] [Patch RFC 11/13] vt-d: If the Device-TLB flush is still not completed when

2015-09-15 Thread Quan Xu
to destroy virtual machine, schedule and wait on a waitqueue until the Device-TLB flush is completed. Signed-off-by: Quan Xu --- xen/common/domain.c | 10 ++ xen/drivers/passthrough/vtd/iommu.c | 9 + xen/include/xen/hvm/iommu.h | 6 ++ 3 files chang

[Xen-devel] [Patch RFC 10/13] vt-d: Held on the removed page until the Device-TLB flush is completed.

2015-09-15 Thread Quan Xu
Signed-off-by: Quan Xu --- xen/common/memory.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/xen/common/memory.c b/xen/common/memory.c index 61bb94c..4b2def5 100644 --- a/xen/common/memory.c +++ b/xen/common/memory.c @@ -253,7 +253,21 @@ int guest_remove_pag

[Xen-devel] [Patch RFC 02/13] vt-d: Register MSI for async invalidation completion interrupt.

2015-09-15 Thread Quan Xu
Signed-off-by: Quan Xu --- xen/drivers/passthrough/vtd/iommu.c | 133 xen/drivers/passthrough/vtd/iommu.h | 10 +++ 2 files changed, 143 insertions(+) diff --git a/xen/drivers/passthrough/vtd/iommu.c b/xen/drivers/passthrough/vtd/iommu.c index 17bfb76..db6e

[Xen-devel] [Patch RFC 13/13] vt-d: Set the IF bit in Invalidation Wait Descriptor When submit Device-TLB

2015-09-15 Thread Quan Xu
invalidataion requests. If the IF bit is Set, the interrupt based mechanism will be used to track the Device-TLB invalidation requests. Do not do polling to detect whether hardware completes the Device-TLB invalidation during Device- TLB invalidation. Signed-off-by: Quan Xu --- xen/drivers/passt

[Xen-devel] [Patch RFC 05/13] vt-d: Clear the IWC field of Invalidation Event Control Register in

2015-09-15 Thread Quan Xu
QI interrupt handler and QI startup. If the IWC field was already Set at the time of setting this field, it is not treated as a new interrupt conditions. In QI interrupt handler, Check IP field of Invalidation Event Control register after scan domain status. if IP field is Set, scan agian, instead

[Xen-devel] [Patch RFC 06/13] vt-d: Introduce a new per-domain flag - qi_flag.

2015-09-15 Thread Quan Xu
The qi_flag is Set when submit Device-TLB invalidation requests. The qi_flag will be Clear in QI interrupt handler if there are no in-flight Device-TLB invalidation requests. Signed-off-by: Quan Xu --- xen/drivers/passthrough/vtd/iommu.c | 1 + xen/drivers/passthrough/vtd/qinval.c | 1 + xen/in

[Xen-devel] [Patch RFC 04/13] vt-d: Clear invalidation table in invaidation interrupt handler

2015-09-15 Thread Quan Xu
if it has no in-flight Device-TLB invalidation request. Signed-off-by: Quan Xu --- xen/drivers/passthrough/vtd/iommu.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/xen/drivers/passthrough/vtd/iommu.c b/xen/drivers/passthrough/vtd/iommu.c index db6e3a2..0e912fb 100

[Xen-devel] [Patch RFC 07/13] vt-d: If the qi_flag is Set, the domain's vCPUs are not allowed to

2015-09-15 Thread Quan Xu
entry guest mode and put into the SCHEDOP_yield list. Signed-off-by: Quan Xu --- xen/arch/x86/hvm/vmx/entry.S | 10 ++ xen/arch/x86/x86_64/asm-offsets.c | 1 + xen/common/domain.c | 5 + xen/include/xen/hvm/iommu.h | 2 ++ 4 files changed, 18 insertions(+)

[Xen-devel] [xen-4.2-testing test] 61955: tolerable FAIL - PUSHED

2015-09-15 Thread osstest service owner
flight 61955 xen-4.2-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/61955/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-i386-libvirt-vhd 17 guest-start/debian.repeat fail in 61789 pass in 61955 test-amd64-amd64-xl-qemu

Re: [Xen-devel] [PULL 0/19] xen-2015-09-08-tag

2015-09-15 Thread Chen, Tiejun
On 9/15/2015 7:00 PM, Paolo Bonzini wrote: On 15/09/2015 11:55, Stefano Stabellini wrote: On Mon, 14 Sep 2015, Paolo Bonzini wrote: > On 10/09/2015 12:29, Stefano Stabellini wrote: > > +if (lseek(config_fd, pos, SEEK_SET) != pos) { > > +return -errno; > > +} > > do { > >

[Xen-devel] [PATCH] vtd/iommu: correct loglevel when check group divices

2015-09-15 Thread Tiejun Chen
Since commit 3848058e7dd6 (vtd/iommu: permit group devices to passthrough in relaxed mode) is introduced, we always print message as XENLOG_G_WARNING but its not correct in the case of strict mode. So here is making this message depending on the specific mode. CC: Yang Zhang CC: Kevin Tian CC: J

Re: [Xen-devel] [PATCH 3/4] x86/mce: Translate passed-in GPA to host machine address

2015-09-15 Thread Haozhong Zhang
On Tue, Sep 15, 2015 at 02:50:27PM +0100, Andrew Cooper wrote: > On 15/09/15 14:42, Haozhong Zhang wrote: > > > >>> +mfn = mfn_x(get_gfn(d, gpfn, &t)); > >>> + > >>> +if (mfn == INVALID_MFN) { > >>> +put_domain(d); > >>> +retur

[Xen-devel] [qemu-upstream-4.6-testing baseline-only test] 37930: tolerable trouble: broken/fail/pass

2015-09-15 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 37930 qemu-upstream-4.6-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/37930/ Failures :-/ but no regressions. Tests which did not succeed, including tests which could not be run: test-amd64-i386-xl-qemuu-winxpsp3

[Xen-devel] [PATCH] xen/mcfg: Call PHYSDEVOP_pci_mmcfg_reserved before PCI enumeration

2015-09-15 Thread Ed Swierk
On systems where the ACPI DSDT advertises the PCI MMCONFIG area but the E820 table does not reserve it, it's up to Dom0 to inform Xen via PHYSDEVOP_pci_mmcfg_reserved. This needs to happen before Xen tries to access extended capabilities like SRIOV in pci_add_device(), which is triggered when Linu

[Xen-devel] [linux-4.1 test] 61947: regressions - FAIL

2015-09-15 Thread osstest service owner
flight 61947 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/61947/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-credit2 16 guest-start/debian.repeat fail REGR. vs. 60785 Regressions which are r

[Xen-devel] Patch "x86/ldt: Make modify_ldt synchronous" has been added to the 4.1-stable tree

2015-09-15 Thread gregkh
This is a note to let you know that I've just added the patch titled x86/ldt: Make modify_ldt synchronous to the 4.1-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: x86-ldt-make-modify

[Xen-devel] [linux-3.16 test] 61939: regressions - trouble: blocked/broken/fail/pass

2015-09-15 Thread osstest service owner
flight 61939 linux-3.16 real [real] http://logs.test-lab.xenproject.org/osstest/logs/61939/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-pvops 4 host-build-prep fail REGR. vs. 60836 Regressions which are

[Xen-devel] [xen-4.6-testing baseline-only test] 37929: tolerable FAIL

2015-09-15 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 37929 xen-4.6-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/37929/ Failures :-/ but no regressions. Tests which did not succeed, including tests which could not be run: test-armhf-armhf-xl-vhd 9 debian-di-in

Re: [Xen-devel] [Patch V1 2/3] xen/usb: add capability for passing through isoc jobs to host devices

2015-09-15 Thread Konrad Rzeszutek Wilk
On Thu, Sep 03, 2015 at 12:45:12PM +0200, Juergen Gross wrote: > When Xen is using the qemu usb framework for pure passthrough of I/Os > to host devices the handling of isoc jobs is rather complicated if > multiple isoc frames are transferred with one call. > > Instead of calling the framework wit

Re: [Xen-devel] [PATCH] KVM: arm64: add workaround for Cortex-A57 erratum #852523

2015-09-15 Thread Christoffer Dall
On Mon, Sep 14, 2015 at 04:46:28PM +0100, Marc Zyngier wrote: > On 14/09/15 16:06, Will Deacon wrote: > > When restoring the system register state for an AArch32 guest at EL2, > > writes to DACR32_EL2 may not be correctly synchronised by Cortex-A57, > > which can lead to the guest effectively runni

Re: [Xen-devel] PCI Pass-through in Xen ARM: Draft 4

2015-09-15 Thread Jaggi, Manish
On Thursday 10 September 2015 06:42 AM, Julien Grall wrote: > Hi Manish, Hi Julien, sorry for late response.. > > On 13/08/2015 10:42, Manish Jaggi wrote: >> 3.2.Mapping between streamID - deviceID - pci sbdf - requesterID >> -

Re: [Xen-devel] [PATCH] xsplice: Use ld-embedded build-ids

2015-09-15 Thread Konrad Rzeszutek Wilk
On Fri, Aug 14, 2015 at 9:57 AM, Martin Pohlack wrote: > On 14.08.2015 15:54, Jan Beulich wrote: > On 14.08.15 at 14:59, wrote: >>> On 11.08.2015 16:12, Jan Beulich wrote: >>> On 05.08.15 at 16:09, wrote: > Todo: > * Should be moved to sysctl to only allow Dom0 access

Re: [Xen-devel] [PATCH v2 for-4.6] libxl: handle read-only drives with qemu-xen

2015-09-15 Thread M A Young
On Tue, 15 Sep 2015, Stefano Stabellini wrote: > } else { > +if (!disks[i].readwrite) { > +LIBXL__LOG(ctx, LIBXL__LOG_ERROR, "qemu-xen doesn't > support read-only disk drivers"); > +return ERROR_INVAL; > +} > +

Re: [Xen-devel] [libvirt] [PATCH] libxl: open libxl log stream with libvirtd log_level

2015-09-15 Thread Jim Fehlig
Daniel P. Berrange wrote: > On Tue, Sep 15, 2015 at 10:57:50AM -0600, Jim Fehlig wrote: > >> Daniel P. Berrange wrote: >> >>> On Tue, Sep 15, 2015 at 09:26:23AM -0600, Jim Fehlig wrote: >>> >>> Instead of a hardcoded DEBUG log level, use the overall daemon log level spec

[Xen-devel] [xen-4.4-testing test] 61925: regressions - FAIL

2015-09-15 Thread osstest service owner
flight 61925 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/61925/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qcow2 9 debian-di-install fail REGR. vs. 60727 test-amd64-i386-x

Re: [Xen-devel] [PATCH OSSTEST 0/4] Avoid running Linux on hosts for the given version lacks drivers

2015-09-15 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST 0/4] Avoid running Linux on hosts for the given version lacks drivers"): > Before applying this the following new host properties should be added: ... > The full diff of ./standalone-generate-dump-flight-runvars is huge, before > and after are attached (compres

Re: [Xen-devel] [PATCH OSSTEST 4/4] make-flight: Add a minimum linux version requirement to all linux-* branches

2015-09-15 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST 4/4] make-flight: Add a minimum linux version requirement to all linux-* branches"): > We have some hosts in the colo which are not supported by older Linux > versions. ... > + case "$branch" in > +linux-[3-9].[0-9]*) Acked-by: Ian Jackson _

Re: [Xen-devel] [PATCH OSSTEST 3/4] Add support for selecting resources based on their properties.

2015-09-15 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST 3/4] Add support for selecting resources based on their properties."): > In particular for allocating hosts based on host properties. > > To do this we extend the hostflags syntax with "condition:arg1:arg2". > This specifies that the candidate host must pass t

Re: [Xen-devel] [PATCH RFC] xen: if on Xen, "flatten" the scheduling domain hierarchy

2015-09-15 Thread Dario Faggioli
On Wed, 2015-09-02 at 16:30 +0200, Juergen Gross wrote: > On 09/02/2015 04:08 PM, Boris Ostrovsky wrote: > > On 09/02/2015 07:58 AM, Juergen Gross wrote: > >> On 08/31/2015 06:12 PM, Boris Ostrovsky wrote: > >>> If set_cpu_sibling_map()'s has_mp is false, wouldn't we effectively have > >>> both of

Re: [Xen-devel] [PATCHv1 net] xen-netback: require fewer guest Rx slots when not using GSO

2015-09-15 Thread David Miller
From: David Vrabel Date: Tue, 15 Sep 2015 11:30:17 +0100 > This is causing regressions with certain frontend drivers. Can you > drop it, please? All patches applied to my tree are part of the permanent record, and this one even made it's way to Linus already I think. So what you need to do is

Re: [Xen-devel] [libvirt] [PATCH] libxl: open libxl log stream with libvirtd log_level

2015-09-15 Thread Daniel P. Berrange
On Tue, Sep 15, 2015 at 10:57:50AM -0600, Jim Fehlig wrote: > Daniel P. Berrange wrote: > > On Tue, Sep 15, 2015 at 09:26:23AM -0600, Jim Fehlig wrote: > > > >> Instead of a hardcoded DEBUG log level, use the overall > >> daemon log level specified in libvirtd.conf when opening > >> a log stream

Re: [Xen-devel] [PATCH OSSTEST 2/4] ts-hosts-allocate-Executive: add a label to loop over candidates

2015-09-15 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST 2/4] ts-hosts-allocate-Executive: add a label to loop over candidates"): > I'm going to want to "next CANDIDATE" from within a nested loop. I'm assuming you've checked that these are the only `next's in that loop. Acked-by: Ian Jackson Ian. ___

Re: [Xen-devel] [PATCH OSSTEST 1/4] ts-hosts-allocate-Executive: Allow dry-run

2015-09-15 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST 1/4] ts-hosts-allocate-Executive: Allow dry-run"): > Provide a new -n command line option which causes no allocations to be > done, for debugging. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xe

Re: [Xen-devel] [PATCH 0/4] libxl: use LOG() macro where appropriate

2015-09-15 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH 0/4] libxl: use LOG() macro where appropriate"): > In terms of applying it seems like the sort of thing which would cause a > lot of pain for backports to 4.6. Shall we hold off or do we think the > inflow of backports has slowed sufficiently now? I would like to

Re: [Xen-devel] [PATCH 1/4] libxl: convert to use LOG() macro

2015-09-15 Thread Ian Jackson
Andrew Cooper writes ("Re: [Xen-devel] [PATCH 1/4] libxl: convert to use LOG() macro"): > There are a number of entries like this where the number of lines could > be reduced. Indeed, but that's IMO no reason to block this patch which is a big improvement. > > -if (ret==0 || xcinfo.domain !=

Re: [Xen-devel] [libvirt] [PATCH] libxl: open libxl log stream with libvirtd log_level

2015-09-15 Thread Jim Fehlig
Daniel P. Berrange wrote: > On Tue, Sep 15, 2015 at 09:26:23AM -0600, Jim Fehlig wrote: > >> Instead of a hardcoded DEBUG log level, use the overall >> daemon log level specified in libvirtd.conf when opening >> a log stream with libxl. libxl is very verbose when DEBUG >> log level is set, resul

Re: [Xen-devel] [PATCH RFC] xen: if on Xen, "flatten" the scheduling domain hierarchy

2015-09-15 Thread Dario Faggioli
On Thu, 2015-08-20 at 20:16 +0200, Juergen Groß wrote: > On 08/18/2015 05:55 PM, Dario Faggioli wrote: > > Hey everyone, > > > > So, as a followup of what we were discussing in this thread: > > > > [Xen-devel] PV-vNUMA issue: topology is misinterpreted by the guest > > http://lists.xenproject.o

Re: [Xen-devel] DomU: kernel BUG at arch/x86/xen/enlighten.c:425

2015-09-15 Thread Andrew Cooper
On 15/09/15 17:28, Thomas DEBESSE wrote: > 2015-09-15 18:09 GMT+02:00 Andrew Cooper >: > > The instantiation of HYPERVISOR_update_va_mapping() in > set_aliased_prot() has always been buggy in pvops kernels. > > This bug should be fixed by c/s 0b0e55 "x

Re: [Xen-devel] DomU: kernel BUG at arch/x86/xen/enlighten.c:425

2015-09-15 Thread Thomas DEBESSE
2015-09-15 18:09 GMT+02:00 Andrew Cooper : > The instantiation of HYPERVISOR_update_va_mapping() in set_aliased_prot() > has always been buggy in pvops kernels. > > This bug should be fixed by c/s 0b0e55 "x86/xen: Probe target addresses in > set_aliased_prot before the hypercall" which is in the p

[Xen-devel] DRAFT XSA 142 - libxl fails to honour readonly flag on disks with qemu-xen

2015-09-15 Thread Xen . org security team
* DRAFT DRAFT DRAFT * Xen Security Advisory XSA-142 libxl fails to honour readonly flag on disks with qemu-xen ISSUE DESCRIPTION = Callers of libxl can specify that a disk should be read-only to the guest. However, there is no code in libxl t

Re: [Xen-devel] [PATCH v2 for-4.6] libxl: handle read-only drives with qemu-xen

2015-09-15 Thread Ian Jackson
Stefano Stabellini writes ("[PATCH v2 for-4.6] libxl: handle read-only drives with qemu-xen"): > The current libxl code doesn't deal with read-only drives at all. > > Upstream QEMU and qemu-xen only support read-only cdrom drives: make > sure to specify "readonly=on" for cdrom drives and return e

Re: [Xen-devel] DomU: kernel BUG at arch/x86/xen/enlighten.c:425

2015-09-15 Thread Andrew Cooper
On 15/09/15 17:03, Thomas DEBESSE wrote: > Hi, I'm replying to this thread from 2013: > http://lists.xen.org/archives/html/xen-devel/2013-03/threads.html#00649 > > Like James Sinclair, all I could find is a closed Debian bug from Dec > 2010 with no resolution: > http://bugs.debian.org/cgi-bin/bugre

[Xen-devel] [PATCH OSSTEST 3/4] Add support for selecting resources based on their properties.

2015-09-15 Thread Ian Campbell
In particular for allocating hosts based on host properties. To do this we extend the hostflags syntax with "condition:arg1:arg2". This specifies that the candidate host must pass the condition given the arguments. Each "condition" is a new module in the Osstest::ResourceCondition namespace. For

[Xen-devel] [PATCH OSSTEST 4/4] make-flight: Add a minimum linux version requirement to all linux-* branches

2015-09-15 Thread Ian Campbell
We have some hosts in the colo which are not supported by older Linux versions. Add a suitable hostflag using the new resource conditions syntax to cause this to occur. Signed-off-by: Ian Campbell --- mfi-common | 11 +++ 1 file changed, 11 insertions(+) diff --git a/mfi-common b/mfi-c

[Xen-devel] [PATCH OSSTEST 1/4] ts-hosts-allocate-Executive: Allow dry-run

2015-09-15 Thread Ian Campbell
Provide a new -n command line option which causes no allocations to be done, for debugging. Signed-off-by: Ian Campbell --- ts-hosts-allocate-Executive | 10 ++ 1 file changed, 10 insertions(+) diff --git a/ts-hosts-allocate-Executive b/ts-hosts-allocate-Executive index 781ddaf..c9aae88

[Xen-devel] [PATCH OSSTEST 2/4] ts-hosts-allocate-Executive: add a label to loop over candidates

2015-09-15 Thread Ian Campbell
I'm going to want to "next CANDIDATE" from within a nested loop. Signed-off-by: Ian Campbell --- ts-hosts-allocate-Executive | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ts-hosts-allocate-Executive b/ts-hosts-allocate-Executive index c9aae88..294395d 100755 --- a/ts-h

Re: [Xen-devel] DomU: kernel BUG at arch/x86/xen/enlighten.c:425

2015-09-15 Thread Thomas DEBESSE
Hi, I'm replying to this thread from 2013: http://lists.xen.org/archives/html/xen-devel/2013-03/threads.html#00649 Like James Sinclair, all I could find is a closed Debian bug from Dec 2010 with no resolution: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=60770 Do you have some news about this

Re: [Xen-devel] [libvirt] [PATCH] libxl: open libxl log stream with libvirtd log_level

2015-09-15 Thread Daniel P. Berrange
On Tue, Sep 15, 2015 at 09:26:23AM -0600, Jim Fehlig wrote: > Instead of a hardcoded DEBUG log level, use the overall > daemon log level specified in libvirtd.conf when opening > a log stream with libxl. libxl is very verbose when DEBUG > log level is set, resulting in huge log files that can > pot

Re: [Xen-devel] [PATCH] libxl: open libxl log stream with libvirtd log_level

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 09:26 -0600, Jim Fehlig wrote: > Instead of a hardcoded DEBUG log level, use the overall > daemon log level specified in libvirtd.conf when opening > a log stream with libxl. libxl is very verbose when DEBUG > log level is set, resulting in huge log files that can > potentiall

Re: [Xen-devel] When NUMA dynamic memory migration will available?

2015-09-15 Thread Dario Faggioli
On Tue, 2015-09-01 at 10:00 +0800, yvxi...@zju.edu.cn wrote: > Hi, Dario: > > > I see the Xen NUMA Roadmap and notice there are some works on dynamic > memory migration. I think it's a very cool feature. > As said on another thread. The roadmap page is slightly outdated. I'll update it ASAP. >

Re: [Xen-devel] [PATCH 1/7] tools/hotplug: remove SELinux options from var-lib-xenstored.mount

2015-09-15 Thread George Dunlap
On 09/15/2015 04:12 PM, Konrad Rzeszutek Wilk wrote: > On Tue, Sep 15, 2015 at 03:01:31PM +0100, George Dunlap wrote: >> On 09/15/2015 02:58 PM, Konrad Rzeszutek Wilk wrote: >>> On Tue, Sep 15, 2015 at 01:55:15PM +0100, George Dunlap wrote: On Tue, Sep 15, 2015 at 1:48 PM, Olaf Hering wrote:

Re: [Xen-devel] Fwd: Question about the status of vNUMA in Xen

2015-09-15 Thread Dario Faggioli
On Tue, 2015-09-08 at 10:50 +0100, Wei Liu wrote: > On Tue, Sep 08, 2015 at 01:21:04AM +, 甘清甜 wrote: > > > So far, I have read the Xen NUMA Roadmap page and watched the > > > video about vNUMA in Xen on Youtube. I have known that some work > > > has been done for vNUMA in Xen. Since my benchma

Re: [Xen-devel] [PATCH 1/2] xen, libxc: Introduced XEN_DOMCTL_emulate_each_rep

2015-09-15 Thread Razvan Cojocaru
On 09/15/2015 06:36 PM, Julien Grall wrote: > Hi Razvan, > > On 15/09/15 10:19, Razvan Cojocaru wrote: >> diff --git a/xen/common/domctl.c b/xen/common/domctl.c >> index 9e0fef5..214a22a 100644 >> --- a/xen/common/domctl.c >> +++ b/xen/common/domctl.c >> @@ -1180,6 +1180,11 @@ long do_domctl(XEN_G

Re: [Xen-devel] [PATCH 0/4] libxl: use LOG() macro where appropriate

2015-09-15 Thread Wei Liu
On Tue, Sep 15, 2015 at 04:16:58PM +0100, Ian Campbell wrote: > On Tue, 2015-09-15 at 15:50 +0100, Ian Jackson wrote: > > Wei Liu writes ("[PATCH 0/4] libxl: use LOG() macro where appropriate"): > > > There are mixed usage of different logging macros. Ideally we only use > > > one > > > style to av

Re: [Xen-devel] [PATCH 1/2] xen, libxc: Introduced XEN_DOMCTL_emulate_each_rep

2015-09-15 Thread Julien Grall
Hi Razvan, On 15/09/15 10:19, Razvan Cojocaru wrote: > diff --git a/xen/common/domctl.c b/xen/common/domctl.c > index 9e0fef5..214a22a 100644 > --- a/xen/common/domctl.c > +++ b/xen/common/domctl.c > @@ -1180,6 +1180,11 @@ long do_domctl(XEN_GUEST_HANDLE_PARAM(xen_domctl_t) > u_domctl) >

Re: [Xen-devel] [PATCH 1/4] libxl: convert to use LOG() macro

2015-09-15 Thread Wei Liu
On Tue, Sep 15, 2015 at 04:12:57PM +0100, Andrew Cooper wrote: > On 15/09/15 10:41, Wei Liu wrote: > > @@ -440,15 +436,18 @@ int libxl__domain_rename(libxl__gc *gc, uint32_t > > domid, > > if (old_name) { > > got_old_name = xs_read(ctx->xsh, trans, name_path, &got_old_len); > >

[Xen-devel] [PATCH] libxl: open libxl log stream with libvirtd log_level

2015-09-15 Thread Jim Fehlig
Instead of a hardcoded DEBUG log level, use the overall daemon log level specified in libvirtd.conf when opening a log stream with libxl. libxl is very verbose when DEBUG log level is set, resulting in huge log files that can potentially fill a disk. Control of libxl verbosity should be placed in t

Re: [Xen-devel] [PATCH 0/4] libxl: use LOG() macro where appropriate

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 15:50 +0100, Ian Jackson wrote: > Wei Liu writes ("[PATCH 0/4] libxl: use LOG() macro where appropriate"): > > There are mixed usage of different logging macros. Ideally we only use > > one > > style to avoid confusion. > > All four > > Acked-by: Ian Jackson In terms of ap

Re: [Xen-devel] [PATCH 1/4] libxl: convert to use LOG() macro

2015-09-15 Thread Andrew Cooper
On 15/09/15 10:41, Wei Liu wrote: > @@ -440,15 +436,18 @@ int libxl__domain_rename(libxl__gc *gc, uint32_t domid, > if (old_name) { > got_old_name = xs_read(ctx->xsh, trans, name_path, &got_old_len); > if (!got_old_name) { > -LIBXL__LOG_ERRNOVAL(ctx, LIBXL__LOG_ER

Re: [Xen-devel] [PATCH 1/7] tools/hotplug: remove SELinux options from var-lib-xenstored.mount

2015-09-15 Thread Konrad Rzeszutek Wilk
On Tue, Sep 15, 2015 at 03:01:31PM +0100, George Dunlap wrote: > On 09/15/2015 02:58 PM, Konrad Rzeszutek Wilk wrote: > > On Tue, Sep 15, 2015 at 01:55:15PM +0100, George Dunlap wrote: > >> On Tue, Sep 15, 2015 at 1:48 PM, Olaf Hering wrote: > >>> On Tue, Sep 15, George Dunlap wrote: > >>> >

[Xen-devel] [PATCH net-next] xen-netfront: always set num queues if possible

2015-09-15 Thread Charles (Chas) Williams
If netfront connects with two (or more) queues and then reconnects with only one queue it fails to delete or rewrite the multi-queue-num-queues key and netback will try to use the wrong number of queues. Always write the num-queues field if the backend has multi-queue support. Signed-off-by: Chas

Re: [Xen-devel] [PATCH 0/4] libxl: use LOG() macro where appropriate

2015-09-15 Thread Ian Jackson
Wei Liu writes ("[PATCH 0/4] libxl: use LOG() macro where appropriate"): > There are mixed usage of different logging macros. Ideally we only use one > style to avoid confusion. All four Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists

Re: [Xen-devel] [PATCH net-next] xen-netfront: always set num queues if possible

2015-09-15 Thread Charles (Chas) Williams
On Tue, 2015-09-15 at 11:59 +0100, David Vrabel wrote: > On 14/09/15 22:28, Charles (Chas) Williams wrote: > > The xen store preserves this information across module invocations. > > If you insmod netfront with two queues and later insmod again with one > > queue, the backend will still believe you

Re: [Xen-devel] [PATCH OSSTEST RFC 1/2] cs-adjust-flight: add recipe-set to adjust the recipe for a set of jobs

2015-09-15 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST RFC 1/2] cs-adjust-flight: add recipe-set to adjust the recipe for a set of jobs"): > When constructing an adhoc test it may be useful to copy an existing > job's configuration but run it with a custom recipe. Acked-by: Ian Jackson __

Re: [Xen-devel] [PATCH OSSTEST RFC 2/2] sg-run-job: support dropping in adhoc test recipes

2015-09-15 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST RFC 2/2] sg-run-job: support dropping in adhoc test recipes"): > By reading sg-run-job-adhoc if it exists. ... > +if {[file exists sg-run-job-adhoc]} { > +source sg-run-job-adhoc > +} > + Acked-by: Ian Jackson

Re: [Xen-devel] [PATCH RFC] xen: if on Xen, "flatten" the scheduling domain hierarchy

2015-09-15 Thread Dario Faggioli
On Thu, 2015-08-27 at 11:24 +0100, George Dunlap wrote: > On 08/18/2015 04:55 PM, Dario Faggioli wrote: > > *** Intel(R) Xeon(R) X5650 @ 2.67GHz > > *** pCPUs 48DOM0 vCPUS 16 > > *** RAM393138 MB DOM0 Memory 9955 MB > > *** NUMA nodes 2 > > ===

Re: [Xen-devel] Dynamically adjust nr_pirqs (aka extra_domU_irqs) from 32 default to an value based on MSI-X options, etc.

2015-09-15 Thread Jan Beulich
>>> On 15.09.15 at 15:42, wrote: > We are running into issues where our guests immediately exhaust > the amount of PIRQs. Using the 'extra_guest_irqs' of course fixes > it but it would be nice to have this be dynamic. > > One idea I had been toying with was that when an device is inserted > into

[Xen-devel] [linux-linus test] 61911: regressions - trouble: broken/fail/pass

2015-09-15 Thread osstest service owner
flight 61911 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/61911/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-cubietruck 6 xen-bootfail REGR. vs. 59254 test-armhf-armhf-xl-a

[Xen-devel] [PATCH 1/2] docs: Template for feature documents

2015-09-15 Thread Andrew Cooper
Signed-off-by: Andrew Cooper --- v3: Added #Testing and #Reference sections v2: Added %Revision and #History table --- docs/Makefile |2 +- docs/features/template.pandoc | 75 + 2 files changed, 76 insertions(+), 1 deletion(-) create

[Xen-devel] [PATCH v3 for-4.6 0/2] In-tree feature documentation

2015-09-15 Thread Andrew Cooper
An issue which Xen has is an uncertain support statement for features. Given the success seen with docs/misc/xen-command-line.markdown, and in particular keeping it up to date, introduce a similar system for features. Patch 1 introduces a proposed template (and a makefile tweak to include the new

Re: [Xen-devel] [PATCH 1/7] tools/hotplug: remove SELinux options from var-lib-xenstored.mount

2015-09-15 Thread George Dunlap
On 09/15/2015 02:58 PM, Konrad Rzeszutek Wilk wrote: > On Tue, Sep 15, 2015 at 01:55:15PM +0100, George Dunlap wrote: >> On Tue, Sep 15, 2015 at 1:48 PM, Olaf Hering wrote: >>> On Tue, Sep 15, George Dunlap wrote: >>> It's very reasonable for you to expect it to be fixed on non-SELinux s

Re: [Xen-devel] [PATCH 1/7] tools/hotplug: remove SELinux options from var-lib-xenstored.mount

2015-09-15 Thread Konrad Rzeszutek Wilk
On Tue, Sep 15, 2015 at 01:55:15PM +0100, George Dunlap wrote: > On Tue, Sep 15, 2015 at 1:48 PM, Olaf Hering wrote: > > On Tue, Sep 15, George Dunlap wrote: > > > >> It's very reasonable for you to expect it to be fixed on non-SELinux > >> systems. But what you did is fix it for non-SELinux syst

Re: [Xen-devel] [PATCH 1/7] tools/hotplug: remove SELinux options from var-lib-xenstored.mount

2015-09-15 Thread Konrad Rzeszutek Wilk
On Tue, Sep 15, 2015 at 02:48:57PM +0200, Olaf Hering wrote: > On Tue, Sep 15, George Dunlap wrote: > > > It's very reasonable for you to expect it to be fixed on non-SELinux > > systems. But what you did is fix it for non-SELinux systems by simply > > breaking it on SELinux systems -- that's not

Re: [Xen-devel] [PATCH 3/4] x86/mce: Translate passed-in GPA to host machine address

2015-09-15 Thread Andrew Cooper
On 15/09/15 14:42, Haozhong Zhang wrote: > >>> +mfn = mfn_x(get_gfn(d, gpfn, &t)); >>> + >>> +if (mfn == INVALID_MFN) { >>> +put_domain(d); >>> +return x86_mcerr("do_mca inject: illegal MSR value", >>> +

Re: [Xen-devel] [PULL 21/29] xen/pt: Sync up the dev.config and data values.

2015-09-15 Thread Konrad Rzeszutek Wilk
On Tue, Sep 15, 2015 at 02:28:51PM +0100, Stefano Stabellini wrote: > On Tue, 15 Sep 2015, Konrad Rzeszutek Wilk wrote: > > On Tue, Sep 15, 2015 at 11:07:02AM +0100, Stefano Stabellini wrote: > > > CC Konrad > > > > > > On Mon, 14 Sep 2015, Paolo Bonzini wrote: > > > > On 10/09/2015 19:15, Stefano

[Xen-devel] [PATCH 2/2] docs: Migration feature document

2015-09-15 Thread Andrew Cooper
Signed-off-by: Andrew Cooper --- v3: * Corrections, and include LIBXL_HAVE_*, * #Testing and #Reference, following template review v2: * %Revision and #History, following template review * Grammar fixes --- docs/features/migration.pandoc | 123 1 file

Re: [Xen-devel] [PATCH 3/4] x86/mce: Translate passed-in GPA to host machine address

2015-09-15 Thread Haozhong Zhang
On Tue, Sep 15, 2015 at 07:47:49AM -0600, Jan Beulich wrote: > >>> On 15.09.15 at 15:42, wrote: > > On Tue, Sep 15, 2015 at 07:24:34AM -0600, Jan Beulich wrote: > >> >>> On 15.09.15 at 10:29, wrote: > >> > +mfn = mfn_x(get_gfn(d, gpfn, &t)); > >> > + > >> > +if (mf

Re: [Xen-devel] Dynamically adjust nr_pirqs (aka extra_domU_irqs) from 32 default to an value based on MSI-X options, etc.

2015-09-15 Thread Konrad Rzeszutek Wilk
On Tue, Sep 15, 2015 at 09:42:40AM -0400, Konrad Rzeszutek Wilk wrote: > Hey, CC-ing Xen-devel which I forgot. > > We are running into issues where our guests immediately exhaust > the amount of PIRQs. Using the 'extra_guest_irqs' of course fixes > it but it would be nice to have this be dynamic.

Re: [Xen-devel] [PATCH 3/4] x86/mce: Translate passed-in GPA to host machine address

2015-09-15 Thread Jan Beulich
>>> On 15.09.15 at 15:42, wrote: > On Tue, Sep 15, 2015 at 07:24:34AM -0600, Jan Beulich wrote: >> >>> On 15.09.15 at 10:29, wrote: >> > +mfn = mfn_x(get_gfn(d, gpfn, &t)); >> > + >> > +if (mfn == INVALID_MFN) { >> > +put_domain(d); >> > +

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

2015-09-15 Thread osstest service owner
flight 61892 linux-3.4 real [real] http://logs.test-lab.xenproject.org/osstest/logs/61892/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-amd64 6 xen-boot fail REGR. vs. 30511 test-amd64-i386-xl-qemu

Re: [Xen-devel] [PATCH 3/4] x86/mce: Translate passed-in GPA to host machine address

2015-09-15 Thread Haozhong Zhang
On Tue, Sep 15, 2015 at 07:24:34AM -0600, Jan Beulich wrote: > >>> On 15.09.15 at 10:29, wrote: > > @@ -1422,6 +1423,38 @@ long do_mca(XEN_GUEST_HANDLE_PARAM(xen_mc_t) > > u_xen_mc) > > if (mc_msrinject->mcinj_count == 0) > > return 0; > > > > +if (mc_msrinject->mcin

Re: [Xen-devel] [PULL 21/29] xen/pt: Sync up the dev.config and data values.

2015-09-15 Thread Paolo Bonzini
On 15/09/2015 15:28, Stefano Stabellini wrote: > >> > 2). Send an follow up patch to change this to abort()? (and wherever else >> > I used >> > assert(..)? > Yes, that would be good. > > >> > 3). Wait till Paolo is done going through the patchset and then revisit >> > 1) or 2)? > I d

Re: [Xen-devel] [PULL 21/29] xen/pt: Sync up the dev.config and data values.

2015-09-15 Thread Stefano Stabellini
On Tue, 15 Sep 2015, Konrad Rzeszutek Wilk wrote: > On Tue, Sep 15, 2015 at 11:07:02AM +0100, Stefano Stabellini wrote: > > CC Konrad > > > > On Mon, 14 Sep 2015, Paolo Bonzini wrote: > > > On 10/09/2015 19:15, Stefano Stabellini wrote: > > > > + > > > > +switch (reg->size) { > > > > +

Re: [Xen-devel] [PULL 21/29] xen/pt: Sync up the dev.config and data values.

2015-09-15 Thread Konrad Rzeszutek Wilk
On Tue, Sep 15, 2015 at 11:07:02AM +0100, Stefano Stabellini wrote: > CC Konrad > > On Mon, 14 Sep 2015, Paolo Bonzini wrote: > > On 10/09/2015 19:15, Stefano Stabellini wrote: > > > + > > > +switch (reg->size) { > > > +case 1: rc = xen_host_pci_get_byte(&s->real_device, offset, >

Re: [Xen-devel] [PATCH 3/4] x86/mce: Translate passed-in GPA to host machine address

2015-09-15 Thread Jan Beulich
>>> On 15.09.15 at 10:29, wrote: > @@ -1422,6 +1423,38 @@ long do_mca(XEN_GUEST_HANDLE_PARAM(xen_mc_t) u_xen_mc) > if (mc_msrinject->mcinj_count == 0) > return 0; > > +if (mc_msrinject->mcinj_flags & MC_MSRINJ_F_GPADDR) { Perhaps you earlier patch fixing coding sty

  1   2   >