Re: [Xen-devel] [PATCH] xen: credit1: fix tickling when it happens from a remote pCPU

2015-09-23 Thread Juergen Gross
On 09/24/2015 06:31 AM, Dario Faggioli wrote: especially if that is also from a different cpupool than the processor of the vCPU that triggered the tickling. In fact, it is possible that we get as far as calling vcpu_unblock()--> vcpu_wake()-->csched_vcpu_wake()-->__runq_tickle() for the vCPU 'v

Re: [Xen-devel] [PATCH v7 20/28] xen/arm: ITS: Add virtual ITS availability check helper

2015-09-23 Thread Vijay Kilari
On Wed, Sep 23, 2015 at 2:22 PM, Julien Grall wrote: > Hi Vijay, > > On 18/09/15 14:09, vijay.kil...@gmail.com wrote: >> From: Vijaya Kumar K >> >> Introduce vgic_is_lpi_supported() helper function >> to know virtual ITS availability for a domain >> >> Signed-off-by: Vijaya Kumar K >> --- >> v7:

[Xen-devel] [xen-unstable test] 62265: regressions - FAIL

2015-09-23 Thread osstest service owner
flight 62265 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/62265/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-stubdom-debianhvm-amd64-xsm 13 guest-localmigrate fail REGR. vs. 61627 R

Re: [Xen-devel] [PATCH v7 18/28] xen/arm: ITS: Export ITS info to Virtual ITS

2015-09-23 Thread Vijay Kilari
On Wed, Sep 23, 2015 at 2:01 PM, Julien Grall wrote: > Hi Vijay, > > On 18/09/15 14:09, vijay.kil...@gmail.com wrote: >> diff --git a/xen/arch/arm/gic-v3-its.c b/xen/arch/arm/gic-v3-its.c >> index 0d5c61c..f3346d3 100644 >> --- a/xen/arch/arm/gic-v3-its.c >> +++ b/xen/arch/arm/gic-v3-its.c >> @@ -

Re: [Xen-devel] [PATCH v7 17/28] xen/arm: ITS: Add GITS registers emulation

2015-09-23 Thread Vijay Kilari
On Tue, Sep 22, 2015 at 8:00 PM, Julien Grall wrote: > Hi Vijay, > > On 18/09/15 14:09, vijay.kil...@gmail.com wrote: [...] >> +case 0x0010 ... 0x007c: >> +case 0xc000 ... 0xffcc: >> +/* Implementation defined -- write ignored */ >> +goto write_ignore; >> +case GITS_CBA

[Xen-devel] [PATCH] xen: credit1: fix tickling when it happens from a remote pCPU

2015-09-23 Thread Dario Faggioli
especially if that is also from a different cpupool than the processor of the vCPU that triggered the tickling. In fact, it is possible that we get as far as calling vcpu_unblock()--> vcpu_wake()-->csched_vcpu_wake()-->__runq_tickle() for the vCPU 'vc', but all while running on a pCPU that is diff

[Xen-devel] [PATCH] xen: sched: get rid of cpupool_scheduler_cpumask()

2015-09-23 Thread Dario Faggioli
Hi everyone, This was patch 4 of a series, v2 of which is in this thread: http://lists.xen.org/archives/html/xen-devel/2015-07/msg00573.html The first 3 patches have been applied long ago. However, I just noticed that this one has not (unless I'm missing something). I guess George never got rou

[Xen-devel] [PATCH] xen: sched: get rid of cpupool_scheduler_cpumask()

2015-09-23 Thread Dario Faggioli
and of (almost every) direct use of cpupool_online_cpumask(). In fact, what we really want for the most of the times, is the set of valid pCPUs of the cpupool a certain domain is part of. Furthermore, in case it's called with a NULL pool as argument, cpupool_scheduler_cpumask() does more harm than

Re: [Xen-devel] [PATCH v7 15/17] vmx: VT-d posted-interrupt core logic handling

2015-09-23 Thread Dario Faggioli
On Thu, 2015-09-24 at 01:50 +, Wu, Feng wrote: > > -Original Message- > > From: George Dunlap [mailto:george.dun...@citrix.com] > > This seems to me to be a cleaner design. It removes the need to > > modify > > any code on context-switch path. It moves modification of NDST and > > m

[Xen-devel] [qemu-upstream-4.5-testing test] 62260: regressions - FAIL

2015-09-23 Thread osstest service owner
flight 62260 qemu-upstream-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/62260/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-vhd9 debian-di-install fail REGR. vs. 60577 test-am

Re: [Xen-devel] [PATCH] Remove a set operation for VCPU_KICK_SOFTIRQ when post interrupt to vm.

2015-09-23 Thread Zhang, Yang Z
Liuqiming (John) wrote on 2015-09-24: > On 2015/9/23 21:41, Zhang, Yang Z wrote: >> Hanweidong (Randy) wrote on 2015-09-23: >>> >>> Zhang, Yang Z wrote on ent: 2015年9月23日 11:51: VCPU_KICK_SOFTIRQ when post interrupt to vm. Zhang, Yang Z wrote on 2015-09-08: > Liuqiming (John) w

[Xen-devel] [linux-next test] 62238: regressions - FAIL

2015-09-23 Thread osstest service owner
flight 62238 linux-next real [real] http://logs.test-lab.xenproject.org/osstest/logs/62238/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl 6 xen-boot fail REGR. vs. 62110 test-armhf-armhf-xl-xs

Re: [Xen-devel] [PATCH] Remove a set operation for VCPU_KICK_SOFTIRQ when post interrupt to vm.

2015-09-23 Thread Liuqiming (John)
On 2015/9/23 21:41, Zhang, Yang Z wrote: Hanweidong (Randy) wrote on 2015-09-23: Zhang, Yang Z wrote on ent: 2015年9月23日 11:51: VCPU_KICK_SOFTIRQ when post interrupt to vm. Zhang, Yang Z wrote on 2015-09-08: Liuqiming (John) wrote on 2015-09-08: Ok, I will try to explain, correct me if I got

Re: [Xen-devel] [PATCH v2] xen/x86: Record xsave features in c->x86_capabilities

2015-09-23 Thread Shuai Ruan
On Tue, Sep 22, 2015 at 06:22:27PM +0100, Andrew Cooper wrote: > Convert existing cpu_has_x??? to being functions of boot_cpu_data (matching > the prevailing style), and mask out unsupported features. > > Signed-off-by: Andrew Cooper > --- > CC: Jan Beulich > CC: Shuai Ruan > > v2: > * Reuse

Re: [Xen-devel] [PATCH v7 15/17] vmx: VT-d posted-interrupt core logic handling

2015-09-23 Thread Wu, Feng
> -Original Message- > From: George Dunlap [mailto:george.dun...@citrix.com] > Sent: Wednesday, September 23, 2015 11:26 PM > To: Wu, Feng; Jan Beulich > Cc: Andrew Cooper; Dario Faggioli; George Dunlap; Tian, Kevin; > xen-devel@lists.xen.org; Keir Fraser > Subject: Re: [Xen-devel] [PATCH

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

2015-09-23 Thread Xu, Quan
Hi, Kornrad > -Original Message- > From: Konrad Rzeszutek Wilk [mailto:konrad.w...@oracle.com] > Sent: Thursday, September 24, 2015 1:39 AM > To: Xu, Quan > Cc: andrew.coop...@citrix.com; Dong, Eddie; ian.campb...@citrix.com; > ian.jack...@eu.citrix.com; jbeul...@suse.com; Nakajima, Jun; k

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

2015-09-23 Thread osstest service owner
flight 62256 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/62256/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-stubdom-debianhvm-amd64-xsm 9 debian-hvm-install fail REGR. vs. 62053 test-a

[Xen-devel] [qemu-upstream-4.4-testing test] 62229: regressions - FAIL

2015-09-23 Thread osstest service owner
flight 62229 qemu-upstream-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/62229/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-vhd9 debian-di-install fail REGR. vs. 60565 test-am

Re: [Xen-devel] [PATCH v3 1/9] xen-blkfront: convert to blk-mq APIs

2015-09-23 Thread Konrad Rzeszutek Wilk
On Wed, Sep 23, 2015 at 04:31:21PM -0400, Konrad Rzeszutek Wilk wrote: > On Sat, Sep 05, 2015 at 08:39:34PM +0800, Bob Liu wrote: > > Note: This patch is based on original work of Arianna's internship for > > GNOME's Outreach Program for Women. > > > > Only one hardware queue is used now, so there

[Xen-devel] [ovmf baseline-only test] 38024: regressions - FAIL

2015-09-23 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 38024 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38024/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ovmf-amd64 9 debian-hvm-insta

Re: [Xen-devel] [PATCH v3 2/9] xen-block: add document for mutli hardware queues/rings

2015-09-23 Thread Konrad Rzeszutek Wilk
On Sat, Sep 05, 2015 at 08:39:35PM +0800, Bob Liu wrote: > Document multi queues/rings of xen-block. This needs to be posted on Xen devel as well so that the blkif.h header in Xen has this. > > Signed-off-by: Bob Liu > --- > include/xen/interface/io/blkif.h | 32 +

Re: [Xen-devel] [PATCH v3 1/9] xen-blkfront: convert to blk-mq APIs

2015-09-23 Thread Konrad Rzeszutek Wilk
On Sat, Sep 05, 2015 at 08:39:34PM +0800, Bob Liu wrote: > Note: This patch is based on original work of Arianna's internship for > GNOME's Outreach Program for Women. > > Only one hardware queue is used now, so there is no significant > performance change > > The legacy non-mq code is deleted co

Re: [Xen-devel] [PATCH RFC v3 0/6] HVM x86 deprivileged mode summary

2015-09-23 Thread Konrad Rzeszutek Wilk
On Fri, Sep 11, 2015 at 05:08:31PM +0100, Ben Catterall wrote: > Hi all, > > I have now finished my internship at Citrix and am posting this final version > of > my RFC series. I would like to express my thanks to all of those who have > taken > the time to review, comment and discuss this serie

Re: [Xen-devel] [PATCH] x86: detect CMOS aliasing on ports other then 0x70/0x71

2015-09-23 Thread Andrew Cooper
On 22/09/15 14:10, Jan Beulich wrote: > +for ( offs = 2; offs < 8; offs <<= 1 ) > +{ > +bool_t read = 1; > + > +for ( i = RTC_REG_D + 1; i < 0x80; ++i ) > +{ > +uint8_t normal, alt; > +unsigned long flags; > + > +if ( i == acpi_gbl

[Xen-devel] [PATCH v7 2/2] xen/arm: support gzip compressed kernels

2015-09-23 Thread Stefano Stabellini
Free the memory used for the compressed kernel and update the relative mod->start and mod->size parameters with the uncompressed ones. To decompress the kernel, allocate memory from dommheap, because freeing the modules is done by calling init_heap_pages, which frees to domheap. Map these pages us

[Xen-devel] [PATCH v7 0/2] support gzipped kernels on arm

2015-09-23 Thread Stefano Stabellini
Hi all, this patch series introduces support for gzipped kernels, such as the standard Image.gz format used by Linux on arm64 by default, in Xen on arm. Without it, Xen cannot load the default kernel shipped by distros, such as CentOS 7. Stefano Stabellini (2): xen: move perform_gunzip to

[Xen-devel] [PATCH v7 1/2] xen: move perform_gunzip to common

2015-09-23 Thread Stefano Stabellini
The current gunzip code to decompress the Dom0 kernel is implemented in inflate.c which is included by bzimage.c. I am looking to doing the same on ARM64 but there is quite a bit of boilerplate definitions that I would need to import in order for inflate.c to work correctly. Instead of copying/pa

Re: [Xen-devel] [PATCH OSSTEST] make_qcow2: Look for qemu-img under /usr as well as /usr/local

2015-09-23 Thread Stefano Stabellini
On Wed, 23 Sep 2015, Ian Campbell wrote: > On Tue, 2015-09-22 at 22:36 +0100, Stefano Stabellini wrote: > > On Tue, 22 Sep 2015, Ian Jackson wrote: > > > (CCing Stefano) > > > > > > Ian Campbell writes ("Re: [PATCH OSSTEST] make_qcow2: Look for qemu-img > > > under /usr as well as /usr/local"): >

Re: [Xen-devel] [PATCH 0/2] x86/hvm: Improvements to HAP superpage feature detection

2015-09-23 Thread Konrad Rzeszutek Wilk
On Tue, Sep 22, 2015 at 06:33:01PM +0100, Andrew Cooper wrote: > Two small patches which reduce the runtime overhead of detecting HAP > superpages by moving more initialisation into __init. > > Noticed while reviewing "x86/p2m: use large pages for MMIO mappings". Reviewed-by: Konrad Rzeszutek Wi

[Xen-devel] [xen-4.5-testing baseline-only test] 38021: regressions - FAIL

2015-09-23 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 38021 xen-4.5-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38021/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-pair10 xen-boot/dst

Re: [Xen-devel] Xen, ACPI and Linux

2015-09-23 Thread Stefano Stabellini
On Wed, 23 Sep 2015, Ian Campbell wrote: > On Wed, 2015-09-23 at 01:18 -0700, Ard Biesheuvel wrote: > > On 23 September 2015 at 01:12, Jan Beulich wrote: > > > > > > On 23.09.15 at 02:49, wrote: > > > > Regarding Runtime Services, the EFI spec doesn't allow a NULL pointer > > > > to > > > > the R

Re: [Xen-devel] Oldest supported Xen version in upstream QEMU (Was: Re: [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries)

2015-09-23 Thread Stefano Stabellini
On Wed, 23 Sep 2015, Ian Campbell wrote: > On Tue, 2015-09-22 at 22:31 +0100, Stefano Stabellini wrote: > > The oldest Xen version I build-test for every pull request is Xen 4.0.0, > > I think it is very reasonable to remove anything older than that. > > I am OK with removing Xen 4.0.0 too, but I

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

2015-09-23 Thread Konrad Rzeszutek Wilk
> index 5dc0033..f661c8c 100644 > --- a/xen/include/xen/hvm/iommu.h > +++ b/xen/include/xen/hvm/iommu.h > @@ -20,6 +20,7 @@ > #define __XEN_HVM_IOMMU_H__ > > #include > +#include > #include > #include > > @@ -56,12 +57,15 @@ struct hvm_iommu { > struct page_list_head qi_hold_page_l

Re: [Xen-devel] [PATCH] x86emul: support clzero

2015-09-23 Thread Andrew Cooper
On 22/09/15 14:06, Jan Beulich wrote: > ... in anticipation of this possibly going to get used by guests for > basic thinks like memset() or clearing or pages. > > Since the emulation doesn't use clzero itself, checking the guest's > CPUID for the feature to be exposed is (intentionally) being avoi

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

2015-09-23 Thread George Dunlap
On Wed, Sep 23, 2015 at 6:10 PM, George Dunlap wrote: > It occurs to me that the steal_for_cache calculations are also wrong > here --it should be (p2m->pod.entry_count - pod > p2m->pod.count) -- > i.e., we should steal if liabilities would be greater than assets > *after* this function completed,

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

2015-09-23 Thread George Dunlap
On 09/15/2015 08:37 AM, Jan Beulich wrote: > @@ -574,41 +576,46 @@ recount: > * + There are PoD entries to handle, or > * + There is ram left, and we want to steal it > */ > -for ( i=0; > - i<(1<0 || (steal_for_cache && ram > 0)); > - i++) > +for ( i = 0;

Re: [Xen-devel] [GIT PULL] (xen) stable/for-jens-4.3

2015-09-23 Thread Jens Axboe
On 09/23/2015 10:15 AM, Konrad Rzeszutek Wilk wrote: Hey Jens, Please git pull git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git stable/for-jens-4.3 which is based on your 'for-4.3/drivers' branch. It has one fix that should go in and also be put in stable tree (I've added the CC al

[Xen-devel] [seabios test] 62247: regressions - FAIL

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

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

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

[Xen-devel] [linux-3.10 test] 62226: regressions - FAIL

2015-09-23 Thread osstest service owner
flight 62226 linux-3.10 real [real] http://logs.test-lab.xenproject.org/osstest/logs/62226/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qcow2 9 debian-di-install fail REGR. vs. 60670 test-amd64-amd64-xl-vh

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

2015-09-23 Thread Tim Deegan
Hi, At 14:09 + on 21 Sep (1442844587), Xu, Quan wrote: > George / Tim, > Could you help me review these memory patches? Thanks! The interrupt-mapping and chipset control parts of this are outside my understanding. :) And I'm not an x86/mm maintainer any more, but I'll have a look: 7/13: I'm

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

2015-09-23 Thread George Dunlap
On 09/22/2015 01:56 PM, Jan Beulich wrote: > When mapping large BARs (e.g. the frame buffer of a graphics card) the > overhead of establishing such mappings using only 4k pages has, > particularly after the XSA-125 fix, become unacceptable. Alter the > XEN_DOMCTL_memory_mapping semantics once again

[Xen-devel] [GIT PULL] (xen) stable/for-jens-4.3

2015-09-23 Thread Konrad Rzeszutek Wilk
Hey Jens, Please git pull git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git stable/for-jens-4.3 which is based on your 'for-4.3/drivers' branch. It has one fix that should go in and also be put in stable tree (I've added the CC already). It is a fix for a memory leak that can exposed

Re: [Xen-devel] [PATCH OSSTEST] Debian: Wait for udev devices to settle in erase-other-disks

2015-09-23 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH OSSTEST] Debian: Wait for udev devices to settle in erase-other-disks"): > On Wed, 2015-09-23 at 16:59 +0100, Ian Jackson wrote: > > logmsg () { logger -t osstest-erase-other-disks-\$\$ "\$@" } > > I wasn't confident in getting the shell-in-perl backticks correc

Re: [Xen-devel] [PATCH OSSTEST] Debian: Wait for udev devices to settle in erase-other-disks

2015-09-23 Thread Ian Campbell
On Wed, 2015-09-23 at 16:59 +0100, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST] Debian: Wait for udev devices to > settle in erase-other-disks"): > > Otherwise we, apparently, have a race between the "test -b" + dd > > against the sda1 device node being removed and end up creating a f

Re: [Xen-devel] [PATCH OSSTEST] Debian: Wait for udev devices to settle in erase-other-disks

2015-09-23 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] Debian: Wait for udev devices to settle in erase-other-disks"): > Otherwise we, apparently, have a race between the "test -b" + dd > against the sda1 device node being removed and end up creating a file > called /dev/sda1 containing 32K of zeroes, which will l

[Xen-devel] [PATCH v2] xen/keyhandler: Rework keyhandler infrastructure

2015-09-23 Thread Andrew Cooper
struct keyhandler does not contain much information, and requires a lot of boilerplate to use. It is far more convenient to have register_keyhandler() take each piece of information a parameter, especially when introducing temporary debugging keyhandlers. This in turn allows struct keyhandler its

[Xen-devel] Regression in RMRRs identity mapping for PVH Dom0

2015-09-23 Thread Elena Ufimtseva
Hi There is a regression in RMRR patch 5ae03990c120a7b3067a52d9784c9aa72c0705a6 in new set_identity_p2m_entry. RMRRs are not being mapped in

[Xen-devel] [PATCH OSSTEST] Debian: Wait for udev devices to settle in erase-other-disks

2015-09-23 Thread Ian Campbell
Otherwise we, apparently, have a race between the "test -b" + dd against the sda1 device node being removed and end up creating a file called /dev/sda1 containing 32K of zeroes, which will later render mke2fs very confused. For some reason we seem to reliably loose the race on Jessie i386 host ins

Re: [Xen-devel] [PATCH] x86/PV: properly populate descriptor tables

2015-09-23 Thread Andrew Cooper
On 23/09/15 16:34, Jan Beulich wrote: Us extending the GDT limit past the Xen descriptors so far meant that guests (including user mode programs) accessing any descriptor table slot above the original OS'es limit but below the first Xen descriptor caused a #PF, converted to a #GP in our #PF handl

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-23 Thread Tim Deegan
At 16:18 +0100 on 23 Sep (1443025126), Wei Liu wrote: > With the discussion still not finalised I'm a bit worried that this > issue will block the release. > > I think we have a few options here. I will list them in order of my > preference. Please correct me if I'm talking non-sense, and feel fre

Re: [Xen-devel] [PATCH v6 12/29] xen/x86: allow disabling the emulated local apic

2015-09-23 Thread Roger Pau Monné
El 16/09/15 a les 12.05, Jan Beulich ha escrit: On 04.09.15 at 14:08, wrote: >> --- a/xen/arch/x86/hvm/svm/svm.c >> +++ b/xen/arch/x86/hvm/svm/svm.c >> @@ -1035,6 +1035,7 @@ static void noreturn svm_do_resume(struct vcpu *v) >> struct vmcb_struct *vmcb = v->arch.hvm_svm.vmcb; >> boo

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-23 Thread George Dunlap
On 09/23/2015 04:18 PM, Wei Liu wrote: > With the discussion still not finalised I'm a bit worried that this > issue will block the release. > > I think we have a few options here. I will list them in order of my > preference. Please correct me if I'm talking non-sense, and feel free to > add more

Re: [Xen-devel] [PATCH v7 15/17] vmx: VT-d posted-interrupt core logic handling

2015-09-23 Thread Jan Beulich
>>> On 23.09.15 at 17:25, wrote: > So, at the moment my *advice* is to look into setting SN / NDST on > vmexit/vmentry, and only having hooks at block/unblock. > > However, the __context_switch() path is also OK with me, if Jan and > Dario are happy. > > Jan / Dario, do you guys have any opinion

[Xen-devel] [PATCH] x86/PV: properly populate descriptor tables

2015-09-23 Thread Jan Beulich
Us extending the GDT limit past the Xen descriptors so far meant that guests (including user mode programs) accessing any descriptor table slot above the original OS'es limit but below the first Xen descriptor caused a #PF, converted to a #GP in our #PF handler. Which is quite different from the na

Re: [Xen-devel] [PATCH v7 15/17] vmx: VT-d posted-interrupt core logic handling

2015-09-23 Thread George Dunlap
On 09/23/2015 01:35 PM, Wu, Feng wrote: > > >> -Original Message- >> From: George Dunlap [mailto:george.dun...@citrix.com] >> Sent: Wednesday, September 23, 2015 5:44 PM >> To: Jan Beulich; Wu, Feng >> Cc: Andrew Cooper; Dario Faggioli; George Dunlap; Tian, Kevin; >> xen-devel@lists.xen.o

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-23 Thread Konrad Rzeszutek Wilk
On Wed, Sep 23, 2015 at 04:18:46PM +0100, Wei Liu wrote: > With the discussion still not finalised I'm a bit worried that this > issue will block the release. > > I think we have a few options here. I will list them in order of my > preference. Please correct me if I'm talking non-sense, and feel

Re: [Xen-devel] [PATCH v2 for-4.6 0/2] libxl: devd fixes

2015-09-23 Thread Wei Liu
On Wed, Sep 23, 2015 at 04:10:20PM +0200, Roger Pau Monné wrote: > El 23/09/15 a les 15.20, Wei Liu ha escrit: > > On Wed, Sep 23, 2015 at 12:06:54PM +0200, Roger Pau Monne wrote: > >> The following patches fix an error when reconnecting a device that's > >> handled > >> by a driver domain and a

Re: [Xen-devel] [PATCH for-4.6] p2m/ept: Set the A bit only if PML is enabled

2015-09-23 Thread Wei Liu
With the discussion still not finalised I'm a bit worried that this issue will block the release. I think we have a few options here. I will list them in order of my preference. Please correct me if I'm talking non-sense, and feel free to add more options if I miss anything. 1. Disable PML on bro

Re: [Xen-devel] [BUG] XEN domU crash when PV grub chainloads 32-bit domU grub

2015-09-23 Thread Andreas Sundstrom
Citerar Ian Campbell : On Wed, 2015-09-23 at 00:37 +0200, Samuel Thibault wrote: Andreas Sundstrom, le Mon 21 Sep 2015 22:03:22 +0200, a écrit : > Note that my original thought was that this bug probably is within > GRUB. It's probably in the GRUB implementation of loading the domU GRUB, since

Re: [Xen-devel] [PATCH v6 11/29] xen/x86: add bitmap of enabled emulated devices

2015-09-23 Thread Roger Pau Monné
El 23/09/15 a les 15.24, Jan Beulich ha escrit: On 23.09.15 at 14:35, wrote: >> El 16/09/15 a les 11.50, Jan Beulich ha escrit: >> On 04.09.15 at 14:08, wrote: + d->domain_id, config->emulation_flags); +return -EINVAL; +} +

Re: [Xen-devel] Oldest supported Xen version in upstream QEMU (Was: Re: [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries)

2015-09-23 Thread Ian Campbell
On Wed, 2015-09-23 at 10:21 -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Sep 23, 2015 at 04:17:05PM +0200, Juergen Gross wrote: > > On 09/23/2015 04:09 PM, Konrad Rzeszutek Wilk wrote: > > > On Wed, Sep 23, 2015 at 09:29:10AM +0100, Ian Campbell wrote: > > > > On Tue, 2015-09-22 at 22:31 +0100, St

Re: [Xen-devel] Oldest supported Xen version in upstream QEMU (Was: Re: [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries)

2015-09-23 Thread Konrad Rzeszutek Wilk
On Wed, Sep 23, 2015 at 04:17:05PM +0200, Juergen Gross wrote: > On 09/23/2015 04:09 PM, Konrad Rzeszutek Wilk wrote: > >On Wed, Sep 23, 2015 at 09:29:10AM +0100, Ian Campbell wrote: > >>On Tue, 2015-09-22 at 22:31 +0100, Stefano Stabellini wrote: > >>>The oldest Xen version I build-test for every

Re: [Xen-devel] [BUG] XEN domU crash when PV grub chainloads 32-bit domU grub

2015-09-23 Thread Ian Campbell
On Wed, 2015-09-23 at 12:47 +, Andreas Sundstrom wrote: > Citerar Ian Campbell : > > > Along those lines, if the _host_ has buckets of RAM then might it be > > worth > > restricting it in case the issue is with getting MFNs which are not > > representably by the 32-bit PV interfaces? (IIRC the

Re: [Xen-devel] Oldest supported Xen version in upstream QEMU (Was: Re: [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries)

2015-09-23 Thread Juergen Gross
On 09/23/2015 04:09 PM, Konrad Rzeszutek Wilk wrote: On Wed, Sep 23, 2015 at 09:29:10AM +0100, Ian Campbell wrote: On Tue, 2015-09-22 at 22:31 +0100, Stefano Stabellini wrote: The oldest Xen version I build-test for every pull request is Xen 4.0.0, I think it is very reasonable to remove anythi

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

2015-09-23 Thread osstest service owner
flight 62208 xen-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/62208/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-raw 10 guest-start fail REGR. vs. 60742 Regressions which

Re: [Xen-devel] [PATCH v2 for-4.6 0/2] libxl: devd fixes

2015-09-23 Thread Roger Pau Monné
El 23/09/15 a les 15.20, Wei Liu ha escrit: > On Wed, Sep 23, 2015 at 12:06:54PM +0200, Roger Pau Monne wrote: >> The following patches fix an error when reconnecting a device that's handled >> by a driver domain and a possible race when doing the cleanup of the backend >> path. >> >> I think bot

Re: [Xen-devel] Oldest supported Xen version in upstream QEMU (Was: Re: [Minios-devel] [PATCH v2 0/15+5+5] Begin to disentangle libxenctrl and provide some stable libraries)

2015-09-23 Thread Konrad Rzeszutek Wilk
On Wed, Sep 23, 2015 at 09:29:10AM +0100, Ian Campbell wrote: > On Tue, 2015-09-22 at 22:31 +0100, Stefano Stabellini wrote: > > The oldest Xen version I build-test for every pull request is Xen 4.0.0, > > I think it is very reasonable to remove anything older than that. > > I am OK with removing

Re: [Xen-devel] [PATCH 1/3] x86/p2m: tighten conditions of IOMMU mapping updates

2015-09-23 Thread Wei Liu
On Tue, Sep 22, 2015 at 08:15:39AM -0600, Jan Beulich wrote: > >>> On 21.09.15 at 16:02, wrote: > > In the EPT case permission changes should also result in updates or > > TLB flushes. > > > > In the NPT case the old MFN does not depend on the new entry being > > valid (but solely on the old one)

[Xen-devel] [ovmf test] 62244: all pass - PUSHED

2015-09-23 Thread osstest service owner
flight 62244 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/62244/ Perfect :-) All tests in this flight passed version targeted for testing: ovmf 8cfd008ef8da26d97314816e0635691955d475d5 baseline version: ovmf 9a470dae60ed0c57afdf61342616dd1768b

Re: [Xen-devel] [PATCH] Remove a set operation for VCPU_KICK_SOFTIRQ when post interrupt to vm.

2015-09-23 Thread Zhang, Yang Z
Hanweidong (Randy) wrote on 2015-09-23: > > > Zhang, Yang Z wrote on ent: 2015年9月23日 11:51: >> VCPU_KICK_SOFTIRQ when post interrupt to vm. >> >> Zhang, Yang Z wrote on 2015-09-08: >>> Liuqiming (John) wrote on 2015-09-08: Ok, I will try to explain, correct me if I got anything wrong:

Re: [Xen-devel] [PATCH] Remove a set operation for VCPU_KICK_SOFTIRQ when post interrupt to vm.

2015-09-23 Thread Hanweidong (Randy)
> -Original Message- > From: Zhang, Yang Z [mailto:yang.z.zh...@intel.com] > Sent: 2015年9月23日 11:51 > To: Liuqiming (John); Hanweidong (Randy); Jan Beulich > Cc: Zhangwei (FF); xen-de...@lists.xenproject.org > Subject: RE: [Xen-devel] [PATCH] Remove a set operation for > VCPU_KICK_SOFTIRQ

Re: [Xen-devel] [PATCH v6 11/29] xen/x86: add bitmap of enabled emulated devices

2015-09-23 Thread Jan Beulich
>>> On 23.09.15 at 14:35, wrote: > El 16/09/15 a les 11.50, Jan Beulich ha escrit: > On 04.09.15 at 14:08, wrote: >>> + d->domain_id, config->emulation_flags); >>> +return -EINVAL; >>> +} >>> +if ( config->emulation_flags != emulation_mask ) >>> +

Re: [Xen-devel] [PATCH for 4.6] xen/arm: vgic: Correctly emulate write when byte is used

2015-09-23 Thread Wei Liu
On Tue, Sep 22, 2015 at 09:18:48PM +0100, Julien Grall wrote: > When a guest is writing a byte, the value will be located in bits[7:0] > of the register. > > Although the current implementation is expecting the byte at the Nth > byte of the register where N = address & 4; > > When the address is

Re: [Xen-devel] [Patch V2] libxc: remove useless stuff from domain builder

2015-09-23 Thread Wei Liu
On Tue, Sep 22, 2015 at 02:20:52PM +0200, Juergen Gross wrote: > Remove unused fields from the domain builder and associated functions. > > Signed-off-by: Juergen Gross Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists

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

2015-09-23 Thread Wei Liu
On Mon, Sep 21, 2015 at 11:57:34AM +0200, Vitaly Kuznetsov wrote: > Use existing create/restore path to perform 'soft reset' for HVM > domains. Tear everything down, e.g. destroy domain's device model, > remove the domain from xenstore, save toolstack record and start > over. > > Signed-off-by: Vi

Re: [Xen-devel] [PATCH v2 for-4.6 0/2] libxl: devd fixes

2015-09-23 Thread Wei Liu
On Wed, Sep 23, 2015 at 12:06:54PM +0200, Roger Pau Monne wrote: > The following patches fix an error when reconnecting a device that's handled > by a driver domain and a possible race when doing the cleanup of the backend > path. > > I think both should be included in 4.6, since this a regressi

Re: [Xen-devel] [PATCH] Remove a set operation for VCPU_KICK_SOFTIRQ when post interrupt to vm.

2015-09-23 Thread Zhang, Yang Z
Jan Beulich wrote on 2015-09-23: On 23.09.15 at 11:15, wrote: >> Jan Beulich wrote on 2015-09-23: >> On 23.09.15 at 05:50, wrote: --- a/xen/arch/x86/hvm/vmx/vmx.c +++ b/xen/arch/x86/hvm/vmx/vmx.c @@ -1678,8 +1678,9 @@ static void __vmx_deliver_posted_interrupt(struct

Re: [Xen-devel] [PATCH OSSTEST] ts-xen-build-prep: don't attempt to extend volume group by 0.

2015-09-23 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] ts-xen-build-prep: don't attempt to extend volume group by 0."): > overall_limit_pe can set its argument to 0 if the VG is already as > large as it is allowed to be. > > In which case we would end up passing "-l +0" to lvextend, which it > complains about and

Re: [Xen-devel] [PATCH v6 11/29] xen/x86: add bitmap of enabled emulated devices

2015-09-23 Thread Andrew Cooper
On 23/09/15 13:42, Roger Pau Monné wrote: El 16/09/15 a les 12.10, Jan Beulich ha escrit: On 04.09.15 at 14:08, wrote: --- a/xen/include/asm-x86/domain.h +++ b/xen/include/asm-x86/domain.h @@ -387,8 +387,21 @@ struct arch_domain bool_t mem_access_emulate_enabled; struct monito

Re: [Xen-devel] [PATCH v6 11/29] xen/x86: add bitmap of enabled emulated devices

2015-09-23 Thread Roger Pau Monné
El 16/09/15 a les 12.10, Jan Beulich ha escrit: On 04.09.15 at 14:08, wrote: >> --- a/xen/include/asm-x86/domain.h >> +++ b/xen/include/asm-x86/domain.h >> @@ -387,8 +387,21 @@ struct arch_domain >> bool_t mem_access_emulate_enabled; >> >> struct monitor_write_data *event_write_da

Re: [Xen-devel] [PATCH v7 15/17] vmx: VT-d posted-interrupt core logic handling

2015-09-23 Thread Wu, Feng
> -Original Message- > From: George Dunlap [mailto:george.dun...@citrix.com] > Sent: Wednesday, September 23, 2015 5:44 PM > To: Jan Beulich; Wu, Feng > Cc: Andrew Cooper; Dario Faggioli; George Dunlap; Tian, Kevin; > xen-devel@lists.xen.org; Keir Fraser > Subject: Re: [Xen-devel] [PATCH

Re: [Xen-devel] [PATCH v6 11/29] xen/x86: add bitmap of enabled emulated devices

2015-09-23 Thread Roger Pau Monné
El 16/09/15 a les 11.50, Jan Beulich ha escrit: On 04.09.15 at 14:08, wrote: >> --- a/tools/libxl/libxl_x86.c >> +++ b/tools/libxl/libxl_x86.c >> @@ -7,8 +7,12 @@ int libxl__arch_domain_prepare_config(libxl__gc *gc, >>libxl_domain_config *d_config, >>

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

2015-09-23 Thread Boris Ostrovsky
On 09/23/2015 03:35 AM, Juergen Gross wrote: Depends on the hardware. On some AMD processors one socket covers multiple NUMA nodes. This is the critical case. set_sched_topology() will be called on those machines possibly multiple times when bringing up additional cpus. I'm asking because tryi

Re: [Xen-devel] [PATCH v6 11/29] xen/x86: add bitmap of enabled emulated devices

2015-09-23 Thread Roger Pau Monné
El 04/09/15 a les 15.55, Jan Beulich ha escrit: On 04.09.15 at 15:51, wrote: >> El 04/09/15 a les 14.25, Wei Liu ha escrit: >>> On Fri, Sep 04, 2015 at 02:08:50PM +0200, Roger Pau Monne wrote: diff --git a/xen/arch/x86/domain.c b/xen/arch/x86/domain.c index 045f6ff..fe9504f 100644 >

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

2015-09-23 Thread osstest service owner
flight 62204 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/62204/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-multivcpu 11 guest-start fail REGR. vs. 60727 Regressions which

[Xen-devel] [qemu-mainline baseline-only test] 38018: regressions - FAIL

2015-09-23 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 38018 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38018/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-pvh-amd 6 xen-boot

Re: [Xen-devel] [PATCH v2 for-4.6 1/2] libxl: fix devd removal path

2015-09-23 Thread Ian Campbell
On Wed, 2015-09-23 at 12:06 +0200, Roger Pau Monne wrote: > The current flow of the devd helper (in charge of launching hotplug > scripts > inside of driver domains) is to wait for the device backend to switch to > state 6 (XenbusStateClosed) and then remove it. This is not correct, > since > a do

Re: [Xen-devel] [PATCH v2 for-4.6 2/2] libxl: fix the cleanup of the backend path when using driver domains

2015-09-23 Thread Ian Campbell
On Wed, 2015-09-23 at 12:06 +0200, Roger Pau Monne wrote: > With the current libxl implementation the control domain will remove both > the frontend and the backend xenstore paths of a device that's handled by > a > driver domain. This is incorrect, since the driver domain possibly needs > to > acc

[Xen-devel] [PATCH OSSTEST] ts-xen-build-prep: don't attempt to extend volume group by 0.

2015-09-23 Thread Ian Campbell
overall_limit_pe can set its argument to 0 if the VG is already as large as it is allowed to be. In which case we would end up passing "-l +0" to lvextend, which it complains about and then fails. Move the clamping of $vg_more_free_pe to outside the check for it being 0. Based on a patch by Robe

Re: [Xen-devel] [PATCH v6 09/29] libxl: switch HVM domain building to use xc_dom_* helpers

2015-09-23 Thread Roger Pau Monné
El 18/09/15 a les 17.53, Anthony PERARD ha escrit: > On Fri, Sep 04, 2015 at 02:08:48PM +0200, Roger Pau Monne wrote: >> Now that we have all the code in place HVM domain building in libxl can be >> switched to use the xc_dom_* family of functions, just like they are used in >> order to build PV gu

Re: [Xen-devel] [PATCH v6 08/29] libxc: introduce a xc_dom_arch for hvm-3.0-x86_32 guests

2015-09-23 Thread Roger Pau Monné
El 18/09/15 a les 17.53, Anthony PERARD ha escrit: > On Fri, Sep 04, 2015 at 02:08:47PM +0200, Roger Pau Monne wrote: >> +total_pages = 0; >> +p2m_size = 0; >> +for ( i = 0; i < nr_vmemranges; i++ ) >> +{ >> +total_pages += ((vmemranges[i].end - vmemranges[i].start) >> +

Re: [Xen-devel] [PATCH v7 22/28] xen/arm: ITS: Allocate irq descriptors for LPIs

2015-09-23 Thread Julien Grall
Hi Vijay, On 18/09/15 14:09, vijay.kil...@gmail.com wrote: > From: Vijaya Kumar K > > Allocate dynamically irq descriptors for LPIs This patch is the best example why we should only use nr_lpis to setup the number of ID bits in the GICv3. Here you are using gic_is_lpi to know if the IRQ is an

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

2015-09-23 Thread George Dunlap
On 09/23/2015 05:36 AM, Juergen Gross wrote: > On 09/22/2015 06:22 PM, George Dunlap wrote: >> On 09/22/2015 05:42 AM, Juergen Gross wrote: >>> One other thing I just discovered: there are other consumers of the >>> topology sibling masks (e.g. topology_sibling_cpumask()) as well. >>> >>> I think w

Re: [Xen-devel] [PATCH v7 21/28] xen/arm: ITS: Add GICR register emulation

2015-09-23 Thread Julien Grall
Hi Vijay, On 18/09/15 14:09, vijay.kil...@gmail.com wrote: > diff --git a/xen/arch/arm/vgic-v3-its.c b/xen/arch/arm/vgic-v3-its.c > index 4285af3..37c2105 100644 > --- a/xen/arch/arm/vgic-v3-its.c > +++ b/xen/arch/arm/vgic-v3-its.c > @@ -29,6 +29,7 @@ > #include > #include > #include > +#inc

[Xen-devel] [PATCH v2 for-4.6 2/2] libxl: fix the cleanup of the backend path when using driver domains

2015-09-23 Thread Roger Pau Monne
With the current libxl implementation the control domain will remove both the frontend and the backend xenstore paths of a device that's handled by a driver domain. This is incorrect, since the driver domain possibly needs to access the backend path in order to perform the disconnection and cleanup

[Xen-devel] [PATCH v2 for-4.6 1/2] libxl: fix devd removal path

2015-09-23 Thread Roger Pau Monne
The current flow of the devd helper (in charge of launching hotplug scripts inside of driver domains) is to wait for the device backend to switch to state 6 (XenbusStateClosed) and then remove it. This is not correct, since a domain can reconnect it's PV devices as many times as it wants. In orde

[Xen-devel] [PATCH v2 for-4.6 0/2] libxl: devd fixes

2015-09-23 Thread Roger Pau Monne
The following patches fix an error when reconnecting a device that's handled by a driver domain and a possible race when doing the cleanup of the backend path. I think both should be included in 4.6, since this a regression as compared to using udev inside of the driver domain. Roger. ___

Re: [Xen-devel] [PATCH] Remove a set operation for VCPU_KICK_SOFTIRQ when post interrupt to vm.

2015-09-23 Thread George Dunlap
On Wed, Sep 23, 2015 at 10:18 AM, Zhang, Yang Z wrote: > George Dunlap wrote on 2015-09-23: >> On Wed, Sep 23, 2015 at 8:42 AM, Jan Beulich wrote: >> On 23.09.15 at 05:50, wrote: --- a/xen/arch/x86/hvm/vmx/vmx.c +++ b/xen/arch/x86/hvm/vmx/vmx.c @@ -1678,8 +1678,9 @@ static voi

Re: [Xen-devel] [PATCH for-4.6 1/2] libxl: fix devd removal path

2015-09-23 Thread Ian Campbell
On Wed, 2015-09-23 at 11:36 +0200, Roger Pau Monné wrote: > El 23/09/15 a les 10.46, Ian Campbell ha escrit: > > On Tue, 2015-09-22 at 18:23 +0200, Roger Pau Monne wrote: > > > > > > -/* Check if event_path ends with "state" and truncate it */ > > > -if (strlen(event_path) < strlen("state

  1   2   >