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

2015-09-15 Thread Charles (Chas) Williams
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 asked for two queues. Signed-off-by: Chas Williams <3ch...@gmail.com> --- drivers/net/xen-netfront.c | 12 +++---

Re: [Xen-devel] [Draft C] Boot ABI for HVM guests without a device-model

2015-09-15 Thread Roger Pau Monné
El 07/09/15 a les 12.05, Jan Beulich ha escrit: On 07.09.15 at 11:34, wrote: >> So AFAICS we have 3 options: >> >> 1. Overload VCPUOP_initialise like it's done in the current series (v6). >> For PV guests the hypercall parameter is of type vcpu_guest_context, >> while for HVM guests the param

[Xen-devel] [PATCH 0/4] x86/p2m: use large pages for MMIO mappings

2015-09-15 Thread Jan Beulich
... where possible. This is in response to http://lists.xenproject.org/archives/html/xen-devel/2015-09/msg01502.html, albeit only the first three patches are really needed for that (the fourth one just leverages what the first two do in another way). The third (main) patch is RFC for a number of re

Re: [Xen-devel] [Draft C] Boot ABI for HVM guests without a device-model

2015-09-15 Thread Jan Beulich
>>> On 15.09.15 at 09:08, wrote: > El 07/09/15 a les 12.05, Jan Beulich ha escrit: > On 07.09.15 at 11:34, wrote: >>> So AFAICS we have 3 options: >>> >>> 1. Overload VCPUOP_initialise like it's done in the current series (v6). >>> For PV guests the hypercall parameter is of type vcpu_guest_c

[Xen-devel] [PATCH 2/4] x86/NPT: always return proper order value from p2m_pt_get_entry()

2015-09-15 Thread Jan Beulich
>>> On 15.09.15 at 09:13, wrote: > ... where possible. This is in response to > http://lists.xenproject.org/archives/html/xen-devel/2015-09/msg01502.html, > albeit only the first three patches are really needed for that (the > fourth one just leverages what the first two do in another way). > The

[Xen-devel] [PATCH 1/4] x86/EPT: always return proper order value from ept_get_entry()

2015-09-15 Thread Jan Beulich
This is so that callers can determine what range of address space would get altered by a corresponding "set". Signed-off-by: Jan Beulich --- a/xen/arch/x86/mm/p2m-ept.c +++ b/xen/arch/x86/mm/p2m-ept.c @@ -879,7 +879,13 @@ static mfn_t ept_get_entry(struct p2m_do /* This pfn is higher than

[Xen-devel] [PATCH 2/4] x86/NPT: always return proper order value from p2m_pt_get_entry()

2015-09-15 Thread Jan Beulich
This is so that callers can determine what range of address space would get altered by a corresponding "set". Signed-off-by: Jan Beulich --- a/xen/arch/x86/mm/p2m-pt.c +++ b/xen/arch/x86/mm/p2m-pt.c @@ -715,14 +715,26 @@ p2m_pt_get_entry(struct p2m_domain *p2m, *a = p2m_access_rwx;

[Xen-devel] [PATCH 3/4 RFC] x86/p2m: use large pages for MMIO mappings

2015-09-15 Thread Jan Beulich
When mapping large BARs (e.g. the frame buffer of a graphics card) the overhead or establishing such mappings using onle 4k pages has, particularly after the XSA-125 fix, become unacceptable. Alter the XEN_DOMCTL_memory_mapping semantics once again, so that there's no longer a fixed amount of guest

Re: [Xen-devel] [PATCH 2/4] x86/NPT: always return proper order value from p2m_pt_get_entry()

2015-09-15 Thread Jan Beulich
Please disregard this - hit "send" too early. >>> On 15.09.15 at 09:31, wrote: On 15.09.15 at 09:13, wrote: >> ... where possible. This is in response to >> http://lists.xenproject.org/archives/html/xen-devel/2015-09/msg01502.html, >> albeit only the first three patches are really needed fo

[Xen-devel] [PATCH 4/4] x86/PoD: shorten certain operations on higher order ranges

2015-09-15 Thread Jan Beulich
Now that p2m->get_entry() always returns a valid order, utilize this to accelerate some of the operations in PoD code. (There are two uses of p2m->get_entry() left which don't easily lend themselves to this optimization.) Also adjust a few types as needed and remove stale comments from p2m_pod_cac

[Xen-devel] [qemu-mainline test] 61883: regressions - FAIL

2015-09-15 Thread osstest service owner
flight 61883 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/61883/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-multivcpu 16 guest-start/debian.repeat fail REGR. vs. 61666 test-amd64-amd64-x

[Xen-devel] [libvirt test] 61890: regressions - FAIL

2015-09-15 Thread osstest service owner
flight 61890 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/61890/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm 16 guest-start/debianhvm.repeat fail REGR. vs. 61770

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-15 Thread Chun Yan Liu
>>> On 9/11/2015 at 09:26 PM, in message <1441978018.3549.33.ca...@citrix.com>, >>> Ian Campbell wrote: > On Thu, 2015-09-10 at 23:42 -0600, Chun Yan Liu wrote: > > > > > Do these fields have any particular size requirements arising from e.g. > the > > > USB spec or from possible dom0 i

[Xen-devel] [PATCH 2/4] tools/mceinject: Fix code style

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

[Xen-devel] [PATCH 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 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. Signed-off-by: Haozhong Zhang --- tools/tests/mce-test/tools/xen-mceinj.c | 135 +--- 1 file change

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

2015-09-15 Thread Haozhong Zhang
Remove trailing spaces 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 | 28 ++-- 2 files changed, 19 insertions(+), 19 deletions(-) diff --git a/x

[Xen-devel] [PATCH 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/

Re: [Xen-devel] [PATCH v11 11/11] (lib)xl: soft reset support

2015-09-15 Thread Ian Campbell
On Mon, 2015-09-14 at 17:15 +0100, Wei Liu wrote: > FYI all other patches of this series were applied by Jan. You only need > to resend this one. Jan, I appreciate you do not want to do so routinely for all commits you make but when you are partially applying a series it would be very useful if y

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

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 00:08 +0100, Julien Grall wrote: > On 14/09/2015 16:58, Will Deacon wrote: > > > The Xen ctxt switch code[0] has DACR_EL2 in the midst of it all, and > > > certainly followed by some sysregs, which I've got my fingers crossed > > > happens to be sufficient (other than maybe ad

Re: [Xen-devel] linux-3.4 broken on chardonnay and huxelrebe (Re: [linux-3.4 test] 61301: regressions - FAIL)

2015-09-15 Thread Ian Campbell
On Thu, 2015-09-10 at 10:32 +0100, Ian Campbell wrote: > The Chardonnay case suggests that either something has been backported into > 3.4.x which has broken things (current real flights, which reliably fail, > are running on 3.4.108) or that it is simply unreliable (or both). I think > I need to r

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

2015-09-15 Thread Egger, Christoph
On 2015/09/15 10:29, Haozhong Zhang wrote: > Remove trailing spaces and fix indentations in mce.c and xen-mca.h. > > Signed-off-by: Haozhong Zhang Acked-by: Christoph Egger > --- > xen/arch/x86/cpu/mcheck/mce.c | 10 +- > xen/include/public/arch-x86/xen-mca.h | 28

[Xen-devel] [distros-debian-squeeze test] 37928: all pass

2015-09-15 Thread Platform Team regression test user
flight 37928 distros-debian-squeeze real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/37928/ Perfect :-) All tests in this flight passed baseline version: flight 37917 jobs: build-amd64 pass build-armhf

Re: [Xen-devel] [PATCH 2/4] tools/mceinject: Fix code style

2015-09-15 Thread Egger, Christoph
On 2015/09/15 10:29, Haozhong Zhang wrote: > Remove trailing spaces in xen-mceinj.c. > > Signed-off-by: Haozhong Zhang Acked-by: Christoph Egger > --- > tools/tests/mce-test/tools/xen-mceinj.c | 16 > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/tools/tes

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

2015-09-15 Thread George Dunlap
On Mon, Sep 14, 2015 at 7:33 PM, Olaf Hering wrote: > On Mon, Sep 14, George Dunlap wrote: > >> Well if you "know nothing about SELinux", and you don't use it, and >> don't have any test systems that use it, then why did you assert >> "The proper place to specify [an SELinux mount context] is /etc

[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 Campbell
When constructing an adhoc test it may be useful to copy an existing job's configuration but run it with a custom recipe. Signed-off-by: Ian Campbell --- cs-adjust-flight | 16 1 file changed, 16 insertions(+) diff --git a/cs-adjust-flight b/cs-adjust-flight index 9e011c6..662b

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

2015-09-15 Thread Ian Campbell
By reading sg-run-job-adhoc if it exists. Signed-off-by: Ian Campbell --- sg-run-job | 4 1 file changed, 4 insertions(+) diff --git a/sg-run-job b/sg-run-job index 0b0449b..c51a508 100755 --- a/sg-run-job +++ b/sg-run-job @@ -370,6 +370,10 @@ proc run-job/test-rumpuserxen {} {

[Xen-devel] [PATCH OSSTEST RFC 0/2] simplify testing with adhoc jobs

2015-09-15 Thread Ian Campbell
While I was doing some adhoc testing on chardonnay I needed to define some adhoc recipes and ended up writing the following two patches. In the end I didn't end up using the first (I just created the job out of whole cloth rather than copying an existing template and modifying). For the second I

Re: [Xen-devel] [PATCH] xl: tighten parsing of "irq" and "iomem" list elements

2015-09-15 Thread Dario Faggioli
On Mon, 2015-09-14 at 07:53 -0600, Jan Beulich wrote: > While "ioport" list element parsing already validates that the entire > input string got consumed, its two siblings so far didn't. > > Signed-off-by: Jan Beulich > Reviewed-by: Dario Faggioli Regards, Dario -- <> (Raistlin Majere) --

Re: [Xen-devel] [v2][PATCH] xen/vtd/iommu: permit group devices to passthrough in relaxed mode

2015-09-15 Thread Wei Liu
On Tue, Sep 15, 2015 at 09:17:07AM +0800, Chen, Tiejun wrote: > >>But looks its not better, so any idea? > > > >Did you at least make an attempt to find other examples of where > >we dynamically determine the log level to be used for a message? > >I would assume that if you did, you'd have come to

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

2015-09-15 Thread Egger, Christoph
On 2015/09/15 10:29, Haozhong Zhang wrote: > 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. >

[Xen-devel] [PATCH 0/2] Introspection optimization helpers

2015-09-15 Thread Razvan Cojocaru
Hello, This series adds two minor patches. The first one allows finer-grained control over the emulation behaviour of REP instructions. Previously, once vm_event-based emulation was enabled, no optimizations were allowed. However, this has a performance impact on the monitored guest, so I've added

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

2015-09-15 Thread Razvan Cojocaru
Previously, if vm_event emulation support was enabled, then REP optimizations were disabled when emulating REP-compatible instructions. This patch allows fine-tuning of this behaviour by providing a dedicated libxc helper function. Signed-off-by: Razvan Cojocaru --- tools/libxc/include/xenctrl.h

[Xen-devel] [PATCH 2/2] xen: Introduce VM_EVENT_FLAG_SET_EIP

2015-09-15 Thread Razvan Cojocaru
A previous version of this patch dealing with support for skipping the current instruction when a vm_event response requested it computed the instruction length in the hypervisor, adding non-trivial code dependencies. This patch allows a userspace vm_event client to simply request that the guest's

Re: [Xen-devel] [PATCH] libxl: slightly refine pci-assignable-{add, remove} handling

2015-09-15 Thread Ian Campbell
On Mon, 2015-09-14 at 10:50 +0100, George Dunlap wrote: > On Thu, Sep 10, 2015 at 1:36 PM, Jan Beulich wrote: > > While it appears to be intentional for "xl pci-assignable-remove" to > > not re-bind the original driver by default (requires the -r option), > > permanently losing the information whi

Re: [Xen-devel] [PATCH] xl: tighten parsing of "irq" and "iomem" list elements

2015-09-15 Thread Ian Campbell
On Mon, 2015-09-14 at 07:53 -0600, Jan Beulich wrote: > While "ioport" list element parsing already validates that the entire > input string got consumed, its two siblings so far didn't. > > Signed-off-by: Jan Beulich Acked-by: Ian Campbell ___ Xen-

Re: [Xen-devel] [PATCH] xen/arm: hvm_domain drop unused field instropection_enabled

2015-09-15 Thread Ian Campbell
On Mon, 2015-09-14 at 16:30 +0100, Julien Grall wrote: > Signed-off-by: Julien Grall Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 1/2] xen/arm: gic-v3: Clean-up the GIC*_PIDR2_* definitions

2015-09-15 Thread Ian Campbell
On Mon, 2015-09-14 at 16:32 +0100, Julien Grall wrote: > GICR_PIDR2 and GICD_PIDR2 use the same register layout. Rather than > define twice, one of which is an alias to the other, introduce > GIC_PIDR2_* > defines. > > Also: > * Use the same prefix for the mask and the value > * Integrate

[Xen-devel] [PATCH 4/4] libxl: fix places missed by spatch

2015-09-15 Thread Wei Liu
The spatch provided in previous patch didn't handle all sites that need transformation. Signed-off-by: Wei Liu --- tools/libxl/libxl.c| 17 - tools/libxl/libxl_create.c | 4 ++-- tools/libxl/libxl_pci.c| 39 ++- tools/libxl/libxl_x

[Xen-devel] [PATCH 3/4] libxl: map LIBXL__LOG_VERBOSE to XTL_VERBOSE

2015-09-15 Thread Wei Liu
There is code in libxl using XTL_VERBOSE. We should provide a libxl mapping for it. Signed-off-by: Wei Liu --- tools/libxl/libxl_internal.h | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h index 5fa55a7..070b7d3 100644 --- a/tools/li

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

2015-09-15 Thread Wei Liu
There are mixed usage of different logging macros. Ideally we only use one style to avoid confusion. Wei Liu (4): libxl: convert to use LOG() macro libxl: fix overly lines and delete extraneous quotes libxl: map LIBXL__LOG_VERBOSE to XTL_VERBOSE libxl: fix places missed by spatch tools/l

[Xen-devel] [PATCH 2/4] libxl: fix overly lines and delete extraneous quotes

2015-09-15 Thread Wei Liu
Signed-off-by: Wei Liu --- tools/libxl/libxl.c | 21 + tools/libxl/libxl_utils.c | 8 ++-- 2 files changed, 19 insertions(+), 10 deletions(-) diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c index ce8965d..9b2047f 100644 --- a/tools/libxl/libxl.c +++ b/tools/

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

2015-09-15 Thread Wei Liu
This patch converts most LIBXL__LOG* macros to LOG macro. It's done with spatch plus some hand coding. Using spatch rune: spatch --in-place --no-includes --include-headers \ --sp-file libxl.spatch \ tools/libxl/libxl*.c with some exceptions. libxl_json.c is untouched because

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 11:14:26AM +0200, Egger, Christoph wrote: > On 2015/09/15 10:29, Haozhong Zhang wrote: > > 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

Re: [Xen-devel] [PATCH 2/2] xen/arm: gic-v3: Allow Xen to run on hardware reporting GICv4

2015-09-15 Thread Ian Campbell
On Mon, 2015-09-14 at 16:32 +0100, Julien Grall wrote: > It seems that there is some hardware which report start to report GICv4 s/report start to reports/reports/ ? Also, this is an odd way to express it, what you mean is that some hardware is now shipping with GICv4. Unless you are trying to im

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

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 12:19 +0300, Razvan Cojocaru wrote: > Previously, if vm_event emulation support was enabled, then REP > optimizations were disabled when emulating REP-compatible > instructions. This patch allows fine-tuning of this behaviour by > providing a dedicated libxc helper function. >

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

2015-09-15 Thread Stefano Stabellini
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 error in case the user requested a non-cdrom read-only drive. This is XSA-142, discovered by Lin Liu (ht

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

2015-09-15 Thread Stefano Stabellini
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 { > > -rc = pread(config_fd, (uint8_t *)&val, len, pos); > > +rc = read(config_fd, (ui

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

2015-09-15 Thread Wei Liu
I don't know this piece of code so my comments might be stupid. On Tue, Sep 15, 2015 at 04:29:40PM +0800, Haozhong Zhang wrote: > 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 addr

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

2015-09-15 Thread Stefano Stabellini
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, > > (uint8_t *)&val); > > A bit ugly, and it relies on the host being little endia

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

2015-09-15 Thread Egger, Christoph
On 2015/09/15 12:02, Wei Liu wrote: > I don't know this piece of code so my comments might be stupid. > > On Tue, Sep 15, 2015 at 04:29:40PM +0800, Haozhong Zhang wrote: >> This patch removes the address translation in xen-mceinj which >> translates the guest physical address passed-in through the

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

2015-09-15 Thread Wei Liu
On Tue, Sep 15, 2015 at 12:08:32PM +0200, Egger, Christoph wrote: > On 2015/09/15 12:02, Wei Liu wrote: > > I don't know this piece of code so my comments might be stupid. > > > > On Tue, Sep 15, 2015 at 04:29:40PM +0800, Haozhong Zhang wrote: > >> This patch removes the address translation in xen

Re: [Xen-devel] [PATCH 2/4] tools/mceinject: Fix code style

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 10:48 +0200, Egger, Christoph wrote: > On 2015/09/15 10:29, Haozhong Zhang wrote: > > Remove trailing spaces in xen-mceinj.c. > > > > Signed-off-by: Haozhong Zhang > > Acked-by: Christoph Egger Acked-by: Ian Campbell ___ Xen-

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

2015-09-15 Thread Egger, Christoph
On 2015/09/15 10:29, Haozhong Zhang wrote: > 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. > > Signed-off-by: Haozhong Zhang Comments inline. > --- > tools/tests/mce-t

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

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 16:29 +0800, Haozhong Zhang wrote: > 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. I think there is a missing/implied "... and replaces it with $some

Re: [Xen-devel] [PATCH v2] arm: reduce power use by contented spin locks with WFE/SEV

2015-09-15 Thread Ian Campbell
On Mon, 2015-08-03 at 12:29 +0100, Ian Campbell wrote: > From: David Vrabel > > Instead of cpu_relax() while spinning and observing the ticket head, > introduce arch_lock_relax() which executes a WFE instruction. After > the ticket head is changed call arch_lock_signal() to execute an SEV > inst

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

2015-09-15 Thread Haozhong Zhang
On Tue, Sep 15, 2015 at 12:17:01PM +0200, Egger, Christoph wrote: > On 2015/09/15 10:29, Haozhong Zhang wrote: > > 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. > > > >

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

2015-09-15 Thread David Vrabel
On 09/09/15 20:34, David Miller wrote: > From: David Vrabel > Date: Tue, 8 Sep 2015 14:25:14 +0100 > >> Commit f48da8b14d04ca87ffcffe68829afd45f926ec6a (xen-netback: fix >> unlimited guest Rx internal queue and carrier flapping) introduced a >> regression. >> >> The PV frontend in IPXE only place

[Xen-devel] [PATCH] tools/hotplug: Use env rather than sh in xenstored.service

2015-09-15 Thread George Dunlap
Using sh to exec xenstored breaks on selinux systems (at least, on CentOS 7). The only purpose of doing that was to be able to expand the $XENSTORED variable; this can be done with /usr/bin/env instead, which still works on systemd systems (at least on CentOS 7). Suggested-by: Olaf Hering Signed

Re: [Xen-devel] [PATCH 2/2] xen/arm: gic-v3: Allow Xen to run on hardware reporting GICv4

2015-09-15 Thread Andre Przywara
Hi, On 15/09/15 10:49, Ian Campbell wrote: > On Mon, 2015-09-14 at 16:32 +0100, Julien Grall wrote: >> It seems that there is some hardware which report start to report GICv4 > > s/report start to reports/reports/ ? > > Also, this is an odd way to express it, what you mean is that some hardware

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

2015-09-15 Thread Egger, Christoph
On 2015/09/15 12:28, Haozhong Zhang wrote: > On Tue, Sep 15, 2015 at 12:17:01PM +0200, Egger, Christoph wrote: >> On 2015/09/15 10:29, Haozhong Zhang wrote: >>> This patch removes the address translation in xen-mceinj which >>> translates the guest physical address passed-in through the argument >>

Re: [Xen-devel] [PATCH] tools/hotplug: Use env rather than sh in xenstored.service

2015-09-15 Thread Wei Liu
On Tue, Sep 15, 2015 at 11:30:00AM +0100, George Dunlap wrote: > Using sh to exec xenstored breaks on selinux systems (at least, on > CentOS 7). The only purpose of doing that was to be able to expand > the $XENSTORED variable; this can be done with /usr/bin/env instead, > which still works on sys

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

2015-09-15 Thread Haozhong Zhang
On Tue, Sep 15, 2015 at 11:20:20AM +0100, Ian Campbell wrote: > On Tue, 2015-09-15 at 16:29 +0800, Haozhong Zhang wrote: > > 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 addres

Re: [Xen-devel] [PATCH v2] arm: reduce power use by contented spin locks with WFE/SEV

2015-09-15 Thread Stefano Stabellini
On Tue, 15 Sep 2015, Ian Campbell wrote: > On Mon, 2015-08-03 at 12:29 +0100, Ian Campbell wrote: > > From: David Vrabel > > > > Instead of cpu_relax() while spinning and observing the ticket head, > > introduce arch_lock_relax() which executes a WFE instruction. After > > the ticket head is cha

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

2015-09-15 Thread Paolo Bonzini
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 { >>> > > -rc = pread(co

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

2015-09-15 Thread David Vrabel
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 asked for two queues. Can you rewrite the commit message to b

Re: [Xen-devel] [PATCH 2/2] xen/arm: gic-v3: Allow Xen to run on hardware reporting GICv4

2015-09-15 Thread Julien Grall
On 15/09/15 10:49, Ian Campbell wrote: > On Mon, 2015-09-14 at 16:32 +0100, Julien Grall wrote: >> It seems that there is some hardware which report start to report GICv4 > > s/report start to reports/reports/ ? Yes, reports. > Also, this is an odd way to express it, what you mean is that some h

Re: [Xen-devel] linux-3.4 broken on chardonnay and huxelrebe (Re: [linux-3.4 test] 61301: regressions - FAIL)

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 09:47 +0100, Ian Campbell wrote: > On Thu, 2015-09-10 at 10:32 +0100, Ian Campbell wrote: > > The Chardonnay case suggests that either something has been backported > > into > > 3.4.x which has broken things (current real flights, which reliably > > fail, > > are running on 3.

Re: [Xen-devel] [PATCH] tools/hotplug: Use env rather than sh in xenstored.service

2015-09-15 Thread George Dunlap
On 09/15/2015 11:30 AM, George Dunlap wrote: > Using sh to exec xenstored breaks on selinux systems (at least, on > CentOS 7). The only purpose of doing that was to be able to expand > the $XENSTORED variable; this can be done with /usr/bin/env instead, > which still works on systemd systems (at l

Re: [Xen-devel] [PATCH 2/2] xen/arm: gic-v3: Allow Xen to run on hardware reporting GICv4

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 12:02 +0100, Julien Grall wrote: > What about: > > "The GICv4 is an extension of GICv3 (see 1.1 in ARM IHI 0069A) which > means that the GICv3 driver can run normally on a GICv4 hardware. > > Though, the GICv4-only feature won't be used." I've a few grammatical quibbles, pl

Re: [Xen-devel] [PATCH] libxl: slightly refine pci-assignable-{add, remove} handling

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 10:31 +0100, Ian Campbell wrote: > On Mon, 2015-09-14 at 10:50 +0100, George Dunlap wrote: > > On Thu, Sep 10, 2015 at 1:36 PM, Jan Beulich wrote: > > > While it appears to be intentional for "xl pci-assignable-remove" to > > > not re-bind the original driver by default (requ

Re: [Xen-devel] [PATCH] xl: tighten parsing of "irq" and "iomem" list elements

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 11:09 +0200, Dario Faggioli wrote: > On Mon, 2015-09-14 at 07:53 -0600, Jan Beulich wrote: > > While "ioport" list element parsing already validates that the entire > > input string got consumed, its two siblings so far didn't. > > > > Signed-off-by: Jan Beulich > > > Revie

Re: [Xen-devel] [PATCH] xen/arm: hvm_domain drop unused field instropection_enabled

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 10:32 +0100, Ian Campbell wrote: > On Mon, 2015-09-14 at 16:30 +0100, Julien Grall wrote: > > Signed-off-by: Julien Grall > > Acked-by: Ian Campbell Applied. I wish I'd spotted the typo in the title first though :-/ Ian. ___ Xe

Re: [Xen-devel] [PATCH v2] arm: reduce power use by contented spin locks with WFE/SEV

2015-09-15 Thread Ian Campbell
On Tue, 2015-08-11 at 09:07 -0600, Jan Beulich wrote: > > > > On 03.08.15 at 13:29, wrote: > > From: David Vrabel > > > > Instead of cpu_relax() while spinning and observing the ticket head, > > introduce arch_lock_relax() which executes a WFE instruction. After > > the ticket head is changed c

Re: [Xen-devel] [PATCH v2] arm: reduce power use by contented spin locks with WFE/SEV

2015-09-15 Thread Ian Campbell
On Tue, 2015-09-15 at 11:39 +0100, Stefano Stabellini wrote: > On Tue, 15 Sep 2015, Ian Campbell wrote: > > On Mon, 2015-08-03 at 12:29 +0100, Ian Campbell wrote: > > > From: David Vrabel > > > > > > Instead of cpu_relax() while spinning and observing the ticket head, > > > introduce arch_lock_re

Re: [Xen-devel] [PATCH 2/2] xen/arm: gic-v3: Allow Xen to run on hardware reporting GICv4

2015-09-15 Thread Julien Grall
On 15/09/15 12:15, Ian Campbell wrote: > On Tue, 2015-09-15 at 12:02 +0100, Julien Grall wrote: >> What about: >> >> "The GICv4 is an extension of GICv3 (see 1.1 in ARM IHI 0069A) which >> means that the GICv3 driver can run normally on a GICv4 hardware. >> >> Though, the GICv4-only feature won't b

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

2015-09-15 Thread Olaf Hering
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 at all reasonable. Konrad did some testing at that time and said 4.5

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

2015-09-15 Thread George Dunlap
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 systems by simply >> breaking it on SELinux systems -- that's not at all reas

Re: [Xen-devel] [PATCH v11 11/11] (lib)xl: soft reset support

2015-09-15 Thread Jan Beulich
>>> On 15.09.15 at 10:38, wrote: > On Mon, 2015-09-14 at 17:15 +0100, Wei Liu wrote: >> FYI all other patches of this series were applied by Jan. You only need >> to resend this one. > > I appreciate you do not want to do so routinely for all commits you make > but when you are partially applying

[Xen-devel] [PATCH] x86/MSI: fail if no hardware support

2015-09-15 Thread Jan Beulich
This is to guard against buggy callers (luckily Dom0 only) invoking the respective hypercall for a device not being MSI-capable. Signed-off-by: Jan Beulich --- While for a well behaved Dom0 this is a benign change, I still think this should go into 4.6. --- a/xen/arch/x86/msi.c +++ b/xen/arch/x8

Re: [Xen-devel] [PATCH 2/4] tools/mceinject: Fix code style

2015-09-15 Thread Wei Liu
On Tue, Sep 15, 2015 at 04:29:38PM +0800, Haozhong Zhang wrote: > Remove trailing spaces in xen-mceinj.c. > > Signed-off-by: Haozhong Zhang Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH] x86/MSI: fail if no hardware support

2015-09-15 Thread Andrew Cooper
On 15/09/15 14:07, Jan Beulich wrote: > This is to guard against buggy callers (luckily Dom0 only) invoking > the respective hypercall for a device not being MSI-capable. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing

Re: [Xen-devel] [PATCH] x86/MSI: fail if no hardware support

2015-09-15 Thread Wei Liu
On Tue, Sep 15, 2015 at 07:07:37AM -0600, Jan Beulich wrote: > This is to guard against buggy callers (luckily Dom0 only) invoking > the respective hypercall for a device not being MSI-capable. > > Signed-off-by: Jan Beulich > --- > While for a well behaved Dom0 this is a benign change, I still t

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

2015-09-15 Thread Jan Beulich
>>> On 15.09.15 at 10:29, wrote: > --- a/xen/arch/x86/cpu/mcheck/mce.c > +++ b/xen/arch/x86/cpu/mcheck/mce.c > @@ -105,7 +105,7 @@ void x86_mce_callback_register(x86_mce_callback_t cbfunc) > mc_callback_bank_extended = cbfunc; > } > > -/* Machine check recoverable judgement callback handle

Re: [Xen-devel] [PATCH v4 00/20] xen/arm64: Add support for 64KB page in Linux

2015-09-15 Thread David Vrabel
On 14/09/15 12:32, Arnd Bergmann wrote: > On Monday 14 September 2015 13:04:59 Roger Pau Monné wrote: >>> TBH, I'm expecting a small impact to the performance. It would be hard >>> to get the exactly the same performance as today if we keep the helpers >>> to avoid the backend dealing himself with

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

2015-09-15 Thread Haozhong Zhang
On Tue, Sep 15, 2015 at 07:13:16AM -0600, Jan Beulich wrote: > >>> On 15.09.15 at 10:29, wrote: > > --- a/xen/arch/x86/cpu/mcheck/mce.c > > +++ b/xen/arch/x86/cpu/mcheck/mce.c > > @@ -105,7 +105,7 @@ void x86_mce_callback_register(x86_mce_callback_t > > cbfunc) > > mc_callback_bank_extended

Re: [Xen-devel] [PATCH v4 00/20] xen/arm64: Add support for 64KB page in Linux

2015-09-15 Thread Arnd Bergmann
On Tuesday 15 September 2015 14:14:09 David Vrabel wrote: > On 14/09/15 12:32, Arnd Bergmann wrote: > > On Monday 14 September 2015 13:04:59 Roger Pau Monné wrote: > >>> TBH, I'm expecting a small impact to the performance. It would be hard > >>> to get the exactly the same performance as today if

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

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] [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 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] [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

[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 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); >> > +

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 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

[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] [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

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] [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

  1   2   >