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

2015-09-23 Thread Dario Faggioli
On Wed, 2015-09-23 at 06:35 +, Wu, Feng wrote: > > From: George Dunlap [mailto:george.dun...@citrix.com] > > On 09/22/2015 08:19 AM, Wu, Feng wrote: > > > In the arch_block() hook, we actually need to > > > - Put vCPU to the blocking list > > > - Set the NV to wakeup vector > > > - Set N

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

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

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

2015-09-23 Thread Dario Faggioli
On Mon, 2015-09-21 at 07:49 +0200, Juergen Gross wrote: > On 09/15/2015 06:50 PM, Dario Faggioli wrote: > > 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

Re: [Xen-devel] [PATCH v2 21/23] x86/boot: implement early command line parser in C

2015-09-23 Thread Jan Beulich
>>> On 22.09.15 at 19:03, wrote: > On Thu, Aug 27, 2015 at 06:43:39AM -0600, Jan Beulich wrote: >> >>> On 20.07.15 at 16:29, wrote: >> > +#define __packed __attribute__((__packed__)) >> > +#define __text__attribute__((__section__(".text"))) >> > +#define __used__attribute

Re: [Xen-devel] [PATCH v7 05/17] vmx: Extend struct pi_desc to support VT-d Posted-Interrupts

2015-09-23 Thread Jan Beulich
>>> On 23.09.15 at 03:02, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: Tuesday, September 22, 2015 10:20 PM >> >>> On 11.09.15 at 10:28, wrote: >> > --- a/xen/include/asm-x86/hvm/vmx/vmcs.h >> > +++ b/xen/include/asm-x86/hvm/vmx/vmcs.h >> > @@ -80,8 +80,18 @@ struct vmx_domain

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

2015-09-23 Thread Juergen Gross
On 09/23/2015 09:24 AM, Dario Faggioli wrote: On Mon, 2015-09-21 at 07:49 +0200, Juergen Gross wrote: On 09/15/2015 06:50 PM, Dario Faggioli wrote: 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

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

2015-09-23 Thread Jan Beulich
>>> 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 vcpu > *v) > { > unsigned int cpu = v->processor; > > -if ( !test_and_set_bit(VCPU_KICK_SOFTIRQ,

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

2015-09-23 Thread Dario Faggioli
On Wed, 2015-09-23 at 05:52 +, Wu, Feng wrote: > George & Dario, thanks much for sharing so many scheduler knowledge > to me, it is very useful. > Well, we're lucky enough that it's our job to do that. :-D > > > So the only downside to doing everything in block(), wake(), and > > > __context_

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

2015-09-23 Thread Jan Beulich
>>> On 23.09.15 at 02:49, wrote: > Regarding Runtime Services, the EFI spec doesn't allow a NULL pointer to > the Runtime Services table, so Mark would like to see a proper pointer > being passed there. The function table could be populated with > hypercall wrappers in assembly, keeping the same

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

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

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

2015-09-23 Thread Ard Biesheuvel
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 Runtime Services table, so Mark would like to see a proper pointer >> being passed there. The function table could be populated w

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

2015-09-23 Thread Ian Campbell
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"): > > > On Tue, 2015-09-22 at 16:41 +0100, Ian Jackson wr

[Xen-devel] [linux-mingo-tip-master test] 62245: regressions - FAIL

2015-09-23 Thread osstest service owner
flight 62245 linux-mingo-tip-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/62245/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-pvops 5 kernel-build fail REGR. vs. 60684 build-amd6

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 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 would like a warning to be > sent ahead of time t

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

2015-09-23 Thread Dario Faggioli
On Wed, 2015-09-23 at 06:36 +0200, Juergen Gross wrote: > On 09/22/2015 06:22 PM, George Dunlap wrote: > > Juergen / Dario, could one of you summarize your two approaches, > > and the > > (alleged) advantages and disadvantages of each one? > > Okay, I'll have a try: > Thanks for this! ;-) > Th

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

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/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 > @@ -37,6 +37,7 @@ > #include > #include > #include > +#include

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 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 > you say that p

Re: [Xen-devel] Page Table Entry Modifications

2015-09-23 Thread Andrew Cooper
On 22/09/15 20:35, Gohar Irfan wrote: Hi All, I want to know if, in the case of HVM (shadow page tables), we can change the machine frame number pointed to by a page table entry? Yes - Xen is able to change the mfn in a pte. An overly simplified example would be: let's say I have page number

Re: [Xen-devel] [PATCH v7 19/28] xen/arm: ITS: Store LPIs allocated per domain

2015-09-23 Thread Julien Grall
Hi Vijay, Commit title: Store the number of LPIs allocated per domain On 18/09/15 14:09, vijay.kil...@gmail.com wrote: > From: Vijaya Kumar K > > Store number of lpis allocated per domain in vgic structure NIT: It's not neccessary to repeat the commit title in the commit message. It would just

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

2015-09-23 Thread Ian Campbell
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 Runtime Services table, so Mark would like to see a pr

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

2015-09-23 Thread Andreas Sundstrom
Citerar Samuel Thibault : 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 you say that pvgrub1 loads it fine. (XEN) domain_crash_

Re: [Xen-devel] Page Table Entry Modifications

2015-09-23 Thread Gohar Irfan
It is for a project, I'm trying to experiment with a couple of things. Could you please guide me how this could be achieved? Some relevant chunks of code, perhaps? Thanks! On Wed, Sep 23, 2015 at 1:35 PM Andrew Cooper wrote: > On 22/09/15 20:35, Gohar Irfan wrote: > > Hi All, > > I want to kno

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

2015-09-23 Thread Ian Campbell
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")) > +/* Check if event_path ends with "state" or "online" and truncate it. */ > +if (strlen(event_path) < strlen("s

Re: [Xen-devel] [PATCH 1/2] x86/hvm: Refine hap_has_{2mb, 1gb} checks

2015-09-23 Thread George Dunlap
On Tue, Sep 22, 2015 at 6:33 PM, Andrew Cooper wrote: > HAP superpages are a host property and not dependent on domain configuration. > Drop the domain paramter (which was only used in one of the two callsites), > and drop the redundant hvm_ prefix to mirror the cpu_has_* style of feature > detect

Re: [Xen-devel] [PATCH 2/2] x86/hvm: Fold opt_hap_{2mb, 1gb} into hap_capabilities

2015-09-23 Thread George Dunlap
On Tue, Sep 22, 2015 at 6:33 PM, Andrew Cooper wrote: > This allows all runtime users to simply check hap_has_{2mb,1gb} rather than > having to check opt_hap_{2mb,1gb} as well. > > As a result, opt_hap_{2mb,1gb} can move into __initdata. > > Signed-off-by: Andrew Cooper Acked-by: George Dunlap

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

2015-09-23 Thread Ian Campbell
On Tue, 2015-09-22 at 18:23 +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

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

2015-09-23 Thread Julien Grall
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: - its_enabled field is added to vgic structure > --- > xen/arch

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 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 void __vmx_deliver_posted_interrupt(struct vcpu >> *v) >> { >> unsigned int cpu = v->process

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 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 > vcpu *v) >> { >> unsigned int cpu = v->processor; >> -if ( !

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

2015-09-23 Thread Zhang, Yang Z
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 void > __vmx_deliver_posted_interrupt(struct vcpu *v) >>> { >

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

2015-09-23 Thread Jan Beulich
>>> On 23.09.15 at 11:08, wrote: > 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 void __vmx_deliver_posted_interrupt(struct >>> vcpu > *v) >>>

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

2015-09-23 Thread Jan Beulich
>>> 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 >> vcpu *v) >>> { >>> unsigned

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

2015-09-23 Thread Roger Pau Monné
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")) >> +/* Check if event_path ends with "state" or "online" and tr

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

2015-09-23 Thread George Dunlap
On 09/22/2015 03:01 PM, Jan Beulich wrote: On 22.09.15 at 15:40, wrote: > >> >>> -Original Message- >>> From: Jan Beulich [mailto:jbeul...@suse.com] >>> Sent: Tuesday, September 22, 2015 5:00 PM >>> To: Wu, Feng >>> Cc: Andrew Cooper; Dario Faggioli; George Dunlap; George Dunlap; Tia

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

2015-09-23 Thread Juergen Gross
On 09/23/2015 10:30 AM, Dario Faggioli wrote: On Wed, 2015-09-23 at 06:36 +0200, Juergen Gross wrote: On 09/22/2015 06:22 PM, George Dunlap wrote: Juergen / Dario, could one of you summarize your two approaches, and the (alleged) advantages and disadvantages of each one? Okay, I'll have a tr

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

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

[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. ___

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

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

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

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

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

[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

[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

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 >

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

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

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

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

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

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

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

[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

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

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

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

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

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

[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

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

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

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;

  1   2   >