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

2015-06-18 Thread Juergen Gross
On 06/18/2015 08:20 AM, Chun Yan Liu wrote: On 6/17/2015 at 07:34 PM, in message <1434540857.13744.334.ca...@citrix.com>, Ian Campbell wrote: On Tue, 2015-06-16 at 16:20 +0200, Juergen Gross wrote: My point was to avoid the sysfs accesses in libxl in order to support BSD as well and to red

Re: [Xen-devel] [v3][PATCH 07/16] hvmloader/pci: skip reserved ranges

2015-06-18 Thread Chen, Tiejun
On 2015/6/18 14:29, Jan Beulich wrote: On 18.06.15 at 08:17, wrote: On 2015/6/17 17:24, Jan Beulich wrote: On 17.06.15 at 11:18, wrote: On 2015/6/17 17:02, Jan Beulich wrote: On 17.06.15 at 10:26, wrote: Something hits me to generate another idea, #1. Still allocate all devices as before

[Xen-devel] [rumpuserxen test] 58712: regressions - FAIL

2015-06-18 Thread osstest service user
flight 58712 rumpuserxen real [real] http://logs.test-lab.xenproject.org/osstest/logs/58712/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-rumpuserxen 5 rumpuserxen-build fail REGR. vs. 33866 build-i386-rumpuserxe

[Xen-devel] 4.5 qemu tree tagging

2015-06-18 Thread Jan Beulich
Ian, Stefano, could you please tag the respective qemu trees for 4.5.1? Considering the little (if any at all) testing feedback we've been getting on stable RCs, and considering how late we are with it, I actually see no point in doing another RC, so unless one of you heavily objects I'd like the

Re: [Xen-devel] [v3][PATCH 04/16] xen/passthrough: extend hypercall to support rdm reservation policy

2015-06-18 Thread Chen, Tiejun
On 2015/6/17 18:11, Jan Beulich wrote: On 11.06.15 at 03:15, wrote: @@ -899,7 +899,7 @@ int set_mmio_p2m_entry(struct domain *d, unsigned long gfn, mfn_t mfn, } int set_identity_p2m_entry(struct domain *d, unsigned long gfn, - p2m_access_t p2ma) +

Re: [Xen-devel] [RFC v2 14/15] Suppress posting interrupts when 'SN' is set

2015-06-18 Thread Wu, Feng
> -Original Message- > From: Wu, Feng > Sent: Monday, June 15, 2015 5:20 PM > To: Jan Beulich > Cc: andrew.coop...@citrix.com; george.dun...@eu.citrix.com; Tian, Kevin; > Zhang, Yang Z; xen-devel@lists.xen.org; k...@xen.org; Wu, Feng > Subject: RE: [RFC v2 14/15] Suppress posting interrup

[Xen-devel] preparing for 4.4.3

2015-06-18 Thread Jan Beulich
With the intended four month cycle, 4.4.3 is due in about a month. Please indicate any backports you see missing yet consider required in the current staging-4.4 tree. As I'll be on vacation the week of July 1st, the intention for an RC1 would be late next week (if things get into shape and throug

Re: [Xen-devel] [RFC v2 14/15] Suppress posting interrupts when 'SN' is set

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 09:34, wrote: > Seems it is a little tricky here. What we need to do for this is > like something below: > > .. > > else if ( !pi_test_sn(&v->arch.hvm_vmx.pi_desc) || > !pi_test_and_set_on(&v->arch.hvm_vmx.pi_desc) ) > { > __vmx_deliver_posted_int

Re: [Xen-devel] [v3][PATCH 04/16] xen/passthrough: extend hypercall to support rdm reservation policy

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 09:14, wrote: > On 2015/6/17 18:11, Jan Beulich wrote: > On 11.06.15 at 03:15, wrote: >>> @@ -1577,9 +1578,10 @@ int iommu_do_pci_domctl( >>> seg = machine_sbdf >> 16; >>> bus = PCI_BUS(machine_sbdf); >>> devfn = PCI_DEVFN2(machine_sbdf); >>> +

Re: [Xen-devel] preparing for 4.4.3

2015-06-18 Thread Olaf Hering
On Thu, Jun 18, Jan Beulich wrote: > With the intended four month cycle, 4.4.3 is due in about a month. > Please indicate any backports you see missing yet consider required > in the current staging-4.4 tree. I would like to see the gnutls changes for qemu-tradiitional be included in maintained t

Re: [Xen-devel] [RFC v2 14/15] Suppress posting interrupts when 'SN' is set

2015-06-18 Thread Wu, Feng
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Thursday, June 18, 2015 3:51 PM > To: Wu, Feng > Cc: andrew.coop...@citrix.com; george.dun...@eu.citrix.com; Tian, Kevin; > Zhang, Yang Z; xen-devel@lists.xen.org; k...@xen.org > Subject: RE: [RFC v2 14/15] Suppre

Re: [Xen-devel] performace issue when turn on apicv

2015-06-18 Thread Liuqiming (John)
Hi, We have test on upstream Xen, it seem the upstream version also has the same problem. When using FIO to test the performance of SSD passthroughed in vm the result show that: When the apicv is on, each EXIT_REASON_MSR_WRITE event spent more time than apicv is off. Following is the xentra

Re: [Xen-devel] [v3][PATCH 07/16] hvmloader/pci: skip reserved ranges

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 09:01, wrote: > On 2015/6/18 14:29, Jan Beulich wrote: > On 18.06.15 at 08:17, wrote: >>> On 2015/6/17 17:24, Jan Beulich wrote: >>> On 17.06.15 at 11:18, wrote: > On 2015/6/17 17:02, Jan Beulich wrote: > On 17.06.15 at 10:26, wrote: >>> Something hits

Re: [Xen-devel] preparing for 4.4.3

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 09:53, wrote: > On Thu, Jun 18, Jan Beulich wrote: > >> With the intended four month cycle, 4.4.3 is due in about a month. >> Please indicate any backports you see missing yet consider required >> in the current staging-4.4 tree. > > I would like to see the gnutls changes for

Re: [Xen-devel] [RFC v2 14/15] Suppress posting interrupts when 'SN' is set

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 10:01, wrote: > >> -Original Message- >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: Thursday, June 18, 2015 3:51 PM >> To: Wu, Feng >> Cc: andrew.coop...@citrix.com; george.dun...@eu.citrix.com; Tian, Kevin; >> Zhang, Yang Z; xen-devel@lists.xen.org; k...@xe

Re: [Xen-devel] FW: VT-d async invalidation for Device-TLB.

2015-06-18 Thread Xu, Quan
> On 16.06.15 09:59, wrote: > >>> On 16.06.15 at 09:59, wrote: > > Jan Beulich wrote on 2015-06-16: > > On 16.06.15 at 05:07, wrote: > >>> Jan Beulich wrote on 2015-06-15: > >>> On 13.06.15 at 16:44, wrote: > >> On 12.06.15 at 14:47, wrote: > > On

Re: [Xen-devel] performace issue when turn on apicv

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 10:02, wrote: > When using FIO to test the performance of SSD passthroughed in vm the > result show that: When the apicv is on, each EXIT_REASON_MSR_WRITE event > spent more time than apicv is off. > > Following is the xentrace result: > > apicv on: > >VMExitCode

Re: [Xen-devel] [PATCH 3.4 000/172] 3.4.108-rc1 review

2015-06-18 Thread Ian Campbell
On Thu, 2015-06-18 at 11:19 +0800, Zefan Li wrote: > On 2015/6/16 23:13, Ian Campbell wrote: > > On Tue, 2015-06-16 at 16:33 +0800, l...@kernel.org wrote: > >> From: Zefan Li > >> > >> This is the start of the stable review cycle for the 3.4.108 release. > >> There are 172 patches in this series,

[Xen-devel] [qemu-mainline test] 58708: tolerable FAIL - PUSHED

2015-06-18 Thread osstest service user
flight 58708 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/58708/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-libvirt 11 guest-start fail like 58551 test-amd64-i386-libvirt-xsm

Re: [Xen-devel] [OSSTEST PATCH 12/25] reporting: Move job histories into history/

2015-06-18 Thread Ian Campbell
On Wed, 2015-06-17 at 17:45 +0100, Ian Jackson wrote: > Replace some `.'s in HTML filenames with `/'s, making the directory > listings easier to deal with. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@l

Re: [Xen-devel] [OSSTEST PATCH 11/25] reporting: Show slightly better info for broken jobs

2015-06-18 Thread Ian Campbell
On Wed, 2015-06-17 at 17:45 +0100, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 13/25] reporting: Move bisection report outputs

2015-06-18 Thread Ian Campbell
On Wed, 2015-06-17 at 17:45 +0100, Ian Jackson wrote: > Replace some `.'s in filenames with `/'s, making the directory > listings easier to deal with. > > (I haven't been able to conveniently do a proper test of this change, > but I have dry-run the critical parts.) > > Signed-off-by: Ian Jackson

Re: [Xen-devel] [OSSTEST PATCH 14/25] reporting: sg-report-flight should ignore missing jobs

2015-06-18 Thread Ian Campbell
On Wed, 2015-06-17 at 17:45 +0100, Ian Jackson wrote: > The function findaflight should not, when the caller specifies a job, > find a flight which does not contain that job at all. > > One effect of allowing it to find such flights is that it might find a > bisection flight and try to use it as a

Re: [Xen-devel] [OSSTEST PATCH 25/25] cri-args-hostlists: Run sg-report-host-history

2015-06-18 Thread Ian Campbell
On Wed, 2015-06-17 at 17:45 +0100, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [v3][PATCH 04/16] xen/passthrough: extend hypercall to support rdm reservation policy

2015-06-18 Thread Chen, Tiejun
On 2015/6/18 15:53, Jan Beulich wrote: On 18.06.15 at 09:14, wrote: On 2015/6/17 18:11, Jan Beulich wrote: On 11.06.15 at 03:15, wrote: @@ -1577,9 +1578,10 @@ int iommu_do_pci_domctl( seg = machine_sbdf >> 16; bus = PCI_BUS(machine_sbdf); devfn = PCI_DEVFN2(m

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

2015-06-18 Thread Ian Campbell
On Thu, 2015-06-18 at 09:02 +0200, Juergen Gross wrote: > On 06/18/2015 08:20 AM, Chun Yan Liu wrote: > > > > > On 6/17/2015 at 07:34 PM, in message > <1434540857.13744.334.ca...@citrix.com>, > > Ian Campbell wrote: > >> On Tue, 2015-06-16 at 16:20 +0200, Juergen Gross wrote: > >>> My p

Re: [Xen-devel] performace issue when turn on apicv

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 10:20, wrote: > Apart from that I notice that the EXIT_REASON_EOI_INDUCED handling > also adds about the same number of ticks... Are there any other devices in the guest causing meaningful amounts of interrupts (I suppose the SSD itself is using just one)? I ask since I wonder

Re: [Xen-devel] [v3][PATCH 05/16] xen: enable XENMEM_memory_map in hvm

2015-06-18 Thread Chen, Tiejun
On 2015/6/17 18:14, Jan Beulich wrote: On 11.06.15 at 03:15, wrote: This patch enables XENMEM_memory_map in hvm. So we can use it to setup the e820 mappings. I think saying "hvmloader" instead of "we" would make things more explicit. In the context here, "we" would be the hypervisor, and Fi

Re: [Xen-devel] [PATCH] oxenstored: fix del_watches and del_transactions

2015-06-18 Thread Ian Campbell
On Wed, 2015-06-17 at 20:44 +0100, Dave Scott wrote: > > On 17 Jun 2015, at 20:39, Wei Liu wrote: > > > > The statement to reset nb_watches should be in del_watches, not > > del_transactions. > > > > Signed-off-by: Wei Liu > > Cc: David Scott > > --- > > It was only until Ian applied previous

Re: [Xen-devel] performace issue when turn on apicv

2015-06-18 Thread Zhang, Yang Z
Jan Beulich wrote on 2015-06-18: On 18.06.15 at 10:20, wrote: >> Apart from that I notice that the EXIT_REASON_EOI_INDUCED handling >> also adds about the same number of ticks... > > Are there any other devices in the guest causing meaningful amounts of > interrupts (I suppose the SSD itself

Re: [Xen-devel] [PATCH] oxenstored: fix del_watches and del_transactions

2015-06-18 Thread Ian Campbell
On Thu, 2015-06-18 at 09:53 +0100, Ian Campbell wrote: > On Wed, 2015-06-17 at 20:44 +0100, Dave Scott wrote: > > > On 17 Jun 2015, at 20:39, Wei Liu wrote: > > > > > > The statement to reset nb_watches should be in del_watches, not > > > del_transactions. > > > > > > Signed-off-by: Wei Liu > >

Re: [Xen-devel] [OSSTEST PATCH 00/25] Reporting improvements

2015-06-18 Thread Ian Campbell
On Wed, 2015-06-17 at 17:44 +0100, Ian Jackson wrote: > Most of the early part of this series is straightforward (and acked) > and could go in whenever. Agreed. Do you happen to have a convenient directory with the new output? (I'm happy to just wait for it to happen too). > Patches 15-24 introdu

Re: [Xen-devel] [v3][PATCH 04/16] xen/passthrough: extend hypercall to support rdm reservation policy

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 10:48, wrote: > On 2015/6/18 15:53, Jan Beulich wrote: > On 18.06.15 at 09:14, wrote: >>> On 2015/6/17 18:11, Jan Beulich wrote: >>> On 11.06.15 at 03:15, wrote: > @@ -1577,9 +1578,10 @@ int iommu_do_pci_domctl( >seg = machine_sbdf >> 16; >

Re: [Xen-devel] [v3][PATCH 06/16] hvmloader: get guest memory map into memory_map[]

2015-06-18 Thread Chen, Tiejun
On 2015/6/17 18:22, Jan Beulich wrote: On 11.06.15 at 03:15, wrote: --- a/tools/firmware/hvmloader/hvmloader.c +++ b/tools/firmware/hvmloader/hvmloader.c @@ -107,6 +107,8 @@ asm ( ".text \n" ); +struct e820map memory_map; Imo this should live in e820.c.

Re: [Xen-devel] [PATCH] oxenstored: fix del_watches and del_transactions

2015-06-18 Thread Ian Campbell
On Thu, 2015-06-18 at 09:56 +0100, Ian Campbell wrote: > On Thu, 2015-06-18 at 09:53 +0100, Ian Campbell wrote: > > On Wed, 2015-06-17 at 20:44 +0100, Dave Scott wrote: > > > > On 17 Jun 2015, at 20:39, Wei Liu wrote: > > > > > > > > The statement to reset nb_watches should be in del_watches, not

Re: [Xen-devel] performace issue when turn on apicv

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 10:53, wrote: > Jan Beulich wrote on 2015-06-18: > On 18.06.15 at 10:20, wrote: >>> Apart from that I notice that the EXIT_REASON_EOI_INDUCED handling >>> also adds about the same number of ticks... >> >> Are there any other devices in the guest causing meaningful amounts

Re: [Xen-devel] [OSSTEST Nested PATCH v11 6/7] Compose the main recipe of nested test job

2015-06-18 Thread Pang, LongtaoX
> -Original Message- > From: Ian Campbell [mailto:ian.campb...@citrix.com] > Sent: Wednesday, June 17, 2015 7:49 PM > To: Pang, LongtaoX > Cc: Hu, Robert; Ian Jackson; xen-devel@lists.xen.org; wei.l...@citrix.com > Subject: Re: [OSSTEST Nested PATCH v11 6/7] Compose the main recipe of nes

Re: [Xen-devel] FW: VT-d async invalidation for Device-TLB.

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 10:09, wrote: >> On 16.06.15 09:59, wrote: >> >>> On 16.06.15 at 09:59, wrote: >> > Jan Beulich wrote on 2015-06-16: >> > On 16.06.15 at 05:07, wrote: >> >>> Jan Beulich wrote on 2015-06-15: >> >>> On 13.06.15 at 16:44, wrote: >> >> On

Re: [Xen-devel] performace issue when turn on apicv

2015-06-18 Thread Zhang, Yang Z
Jan Beulich wrote on 2015-06-18: On 18.06.15 at 10:53, wrote: >> Jan Beulich wrote on 2015-06-18: >> On 18.06.15 at 10:20, wrote: Apart from that I notice that the EXIT_REASON_EOI_INDUCED handling also adds about the same number of ticks... >>> >>> Are there any other devices

Re: [Xen-devel] 4.5 qemu tree tagging

2015-06-18 Thread Stefano Stabellini
On Thu, 18 Jun 2015, Jan Beulich wrote: > Ian, Stefano, > > could you please tag the respective qemu trees for 4.5.1? Considering > the little (if any at all) testing feedback we've been getting on stable > RCs, and considering how late we are with it, I actually see no point in > doing another RC

Re: [Xen-devel] [v3][PATCH 16/16] xen/vtd: prevent from assign the device with shared rmrr

2015-06-18 Thread Chen, Tiejun
On 2015/6/17 18:28, Jan Beulich wrote: On 11.06.15 at 03:15, wrote: --- a/xen/drivers/passthrough/vtd/iommu.c +++ b/xen/drivers/passthrough/vtd/iommu.c @@ -2277,13 +2277,37 @@ static int intel_iommu_assign_device( if ( list_empty(&acpi_drhd_units) ) return -ENODEV; +seg = p

Re: [Xen-devel] [OSSTEST Nested PATCH v11 6/7] Compose the main recipe of nested test job

2015-06-18 Thread Ian Campbell
On Thu, 2015-06-18 at 09:16 +, Pang, LongtaoX wrote: > > > -Original Message- > > From: Ian Campbell [mailto:ian.campb...@citrix.com] > > Sent: Wednesday, June 17, 2015 7:49 PM > > To: Pang, LongtaoX > > Cc: Hu, Robert; Ian Jackson; xen-devel@lists.xen.org; wei.l...@citrix.com > > Subj

Re: [Xen-devel] [PATCH] oxenstored: fix del_watches and del_transactions

2015-06-18 Thread Wei Liu
On Thu, Jun 18, 2015 at 09:56:37AM +0100, Ian Campbell wrote: > On Thu, 2015-06-18 at 09:53 +0100, Ian Campbell wrote: > > On Wed, 2015-06-17 at 20:44 +0100, Dave Scott wrote: > > > > On 17 Jun 2015, at 20:39, Wei Liu wrote: > > > > > > > > The statement to reset nb_watches should be in del_watch

Re: [Xen-devel] [OSSTEST Nested PATCH v11 6/7] Compose the main recipe of nested test job

2015-06-18 Thread Pang, LongtaoX
> -Original Message- > From: Ian Campbell [mailto:ian.campb...@citrix.com] > Sent: Thursday, June 18, 2015 5:23 PM > To: Pang, LongtaoX > Cc: Hu, Robert; Ian Jackson; wei.l...@citrix.com; xen-devel@lists.xen.org > Subject: Re: [Xen-devel] [OSSTEST Nested PATCH v11 6/7] Compose the main >

Re: [Xen-devel] [PATCH] oxenstored: fix del_watches and del_transactions

2015-06-18 Thread Ian Campbell
On Thu, 2015-06-18 at 10:23 +0100, Wei Liu wrote: > On Thu, Jun 18, 2015 at 09:56:37AM +0100, Ian Campbell wrote: > > On Thu, 2015-06-18 at 09:53 +0100, Ian Campbell wrote: > > > On Wed, 2015-06-17 at 20:44 +0100, Dave Scott wrote: > > > > > On 17 Jun 2015, at 20:39, Wei Liu wrote: > > > > > > >

Re: [Xen-devel] [v3][PATCH 04/16] xen/passthrough: extend hypercall to support rdm reservation policy

2015-06-18 Thread Chen, Tiejun
On 2015/6/18 17:13, Jan Beulich wrote: On 18.06.15 at 10:48, wrote: On 2015/6/18 15:53, Jan Beulich wrote: On 18.06.15 at 09:14, wrote: On 2015/6/17 18:11, Jan Beulich wrote: On 11.06.15 at 03:15, wrote: @@ -1577,9 +1578,10 @@ int iommu_do_pci_domctl( seg = machine_sbdf >> 16;

Re: [Xen-devel] performace issue when turn on apicv

2015-06-18 Thread Zhang, Yang Z
Jan Beulich wrote on 2015-06-18: On 18.06.15 at 10:02, wrote: >> When using FIO to test the performance of SSD passthroughed in vm >> the result show that: When the apicv is on, each >> EXIT_REASON_MSR_WRITE event spent more time than apicv is off. >> >> Following is the xentrace result: >>

Re: [Xen-devel] performace issue when turn on apicv

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 11:18, wrote: > Jan Beulich wrote on 2015-06-18: > On 18.06.15 at 10:53, wrote: >>> Jan Beulich wrote on 2015-06-18: >>> On 18.06.15 at 10:20, wrote: > Apart from that I notice that the EXIT_REASON_EOI_INDUCED handling > also adds about the same number of ticks

[Xen-devel] [PATCH v6] OSSTEST: introduce a raisin build test

2015-06-18 Thread Stefano Stabellini
Signed-off-by: Stefano Stabellini --- Changes in v6: - move divide to Osstest/BuildSupport.pm and rename it to divide_xen_build; use divide_xen_build in ts-raisin-build and ts-xen-build. Changes in v5: - remove setting DEFAULT_REVISION_RAISIN to master - disable raisin when branch is not xen* -

Re: [Xen-devel] [PATCH 23/27] tools/libxl: [RFC] Write checkpoint records into the stream

2015-06-18 Thread Andrew Cooper
On 18/06/15 04:13, Wen Congyang wrote: >> @@ -139,6 +169,15 @@ static void stream_failed(libxl__egc *egc, >> > assert(rc); >> > stream->rc = rc; >> > >> > +/* >> > + *If we are in a checkpoint, pass the failure to libxc, which will >> > come >> > + * back around to us via l

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

2015-06-18 Thread osstest service user
flight 58710 linux-3.4 real [real] http://logs.test-lab.xenproject.org/osstest/logs/58710/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl 9 debian-install fail REGR. vs. 52209-bisect test-amd64-amd64-pair

Re: [Xen-devel] [PATCHv14 1/2] gnttab: use per-VCPU maptrack free lists

2015-06-18 Thread Jan Beulich
>>> On 17.06.15 at 15:53, wrote: > @@ -1474,6 +1519,17 @@ gnttab_setup_table( > gt = d->grant_table; > write_lock(>->lock); > > +/* Tracking of mapped foreign frames table */ > +gt->maptrack = vzalloc(max_maptrack_frames * sizeof(*gt->maptrack)); > +if ( gt->maptrack == NUL

Re: [Xen-devel] [PATCHv14 2/2] gnttab: steal maptrack entries from other VCPUs

2015-06-18 Thread Jan Beulich
>>> On 17.06.15 at 15:53, wrote: > +static int steal_maptrack_handle(struct grant_table *t, > + const struct vcpu *curr) > +{ > +const struct domain *currd = curr->domain; > +unsigned int first, i; > + > +/* Find an initial victim. */ > +first = i =

Re: [Xen-devel] [v3][PATCH 03/16] xen/vtd: create RMRR mapping

2015-06-18 Thread Tim Deegan
At 14:13 +0800 on 12 Jun (1434118407), Chen, Tiejun wrote: > > could you explain why existing guest_physmap_remove_page can't > > serve the purpose so you need invent a new identity mapping > > specific one? For unmapping suppose it should be common regardless > > of whether it's identity-mapped or

Re: [Xen-devel] [PATCHv14 1/2] gnttab: use per-VCPU maptrack free lists

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 12:02, wrote: On 17.06.15 at 15:53, wrote: >> @@ -1474,6 +1519,17 @@ gnttab_setup_table( >> gt = d->grant_table; >> write_lock(>->lock); >> >> +/* Tracking of mapped foreign frames table */ >> +gt->maptrack = vzalloc(max_maptrack_frames * sizeof(*gt->ma

Re: [Xen-devel] [PATCHv3 1/6] evtchn: clear xen_consumer when clearing state

2015-06-18 Thread Jan Beulich
>>> On 17.06.15 at 14:02, wrote: > @@ -1187,7 +1188,6 @@ void free_xen_event_channel(struct domain *d, int port) > BUG_ON(!port_is_valid(d, port)); > chn = evtchn_from_port(d, port); > BUG_ON(!consumer_is_xen(chn)); > -chn->xen_consumer = 0; > > spin_unlock(&d->event_lock

Re: [Xen-devel] [PATCHv3 2/6] evtchn: defer freeing struct evtchn's until evtchn_destroy_final()

2015-06-18 Thread Jan Beulich
>>> On 17.06.15 at 14:02, wrote: > --- a/xen/common/event_channel.c > +++ b/xen/common/event_channel.c > @@ -1175,22 +1175,6 @@ int alloc_unbound_xen_event_channel( > > void free_xen_event_channel(struct domain *d, int port) > { > -struct evtchn *chn; > - > -spin_lock(&d->event_lock);

Re: [Xen-devel] [PATCHv3 2/6] evtchn: defer freeing struct evtchn's until evtchn_destroy_final()

2015-06-18 Thread David Vrabel
On 18/06/15 11:36, Jan Beulich wrote: On 17.06.15 at 14:02, wrote: >> --- a/xen/common/event_channel.c >> +++ b/xen/common/event_channel.c >> @@ -1175,22 +1175,6 @@ int alloc_unbound_xen_event_channel( >> >> void free_xen_event_channel(struct domain *d, int port) >> { >> -struct evtch

Re: [Xen-devel] [PATCHv3 2/6] evtchn: defer freeing struct evtchn's until evtchn_destroy_final()

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 12:40, wrote: > On 18/06/15 11:36, Jan Beulich wrote: > On 17.06.15 at 14:02, wrote: >>> --- a/xen/common/event_channel.c >>> +++ b/xen/common/event_channel.c >>> @@ -1175,22 +1175,6 @@ int alloc_unbound_xen_event_channel( >>> >>> void free_xen_event_channel(struct domai

Re: [Xen-devel] [PATCHv3 4/6] evtchn: use a per-event channel lock for sending events

2015-06-18 Thread Jan Beulich
>>> On 17.06.15 at 14:03, wrote: > When sending an event, use a new per-event channel lock to safely > validate the event channel state. > > This new lock must be held when changing event channel state. Note > that the event channel lock must also be held when changing state from > ECS_FREE or i

Re: [Xen-devel] [OSSTEST PATCH 00/25] Reporting improvements

2015-06-18 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH 00/25] Reporting improvements"): > On Wed, 2015-06-17 at 17:44 +0100, Ian Jackson wrote: > > Most of the early part of this series is straightforward (and acked) > > and could go in whenever. > > Agreed. Do you happen to have a convenient directory with the

Re: [Xen-devel] [PATCHv3 5/6] evtchn: remove the locking when unmasking an event channel

2015-06-18 Thread Jan Beulich
>>> On 17.06.15 at 14:03, wrote: > --- a/xen/common/event_channel.c > +++ b/xen/common/event_channel.c > @@ -978,8 +978,6 @@ int evtchn_unmask(unsigned int port) > struct domain *d = current->domain; > struct evtchn *evtchn; > > -ASSERT(spin_is_locked(&d->event_lock)); > - > i

Re: [Xen-devel] FW: VT-d async invalidation for Device-TLB.

2015-06-18 Thread Xu, Quan
>On June 18, 2015 5:19 PM, wrote: > >>> On 18.06.15 at 10:09, wrote: > > >> On 16.06.15 09:59, wrote: > >> >>> On 16.06.15 at 09:59, wrote: > >> > Jan Beulich wrote on 2015-06-16: > >> > On 16.06.15 at 05:07, wrote: > >> >>> Jan Beulich wrote on 2015-06-15: > >>

Re: [Xen-devel] [PATCHv3 6/6] evtchn: pad struct evtchn to 64 bytes

2015-06-18 Thread Jan Beulich
>>> On 17.06.15 at 14:03, wrote: > The number of struct evtchn in a page must be a power of two. Under > some workloads performance is improved slightly by padding struct > evtchn to 64 bytes (a typical cache line size), thus putting the fewer > per-channel locks into each cache line. > > This d

Re: [Xen-devel] stable trees (was: [xen-4.2-testing test] 58584: regressions)

2015-06-18 Thread Jan Beulich
>>> On 17.06.15 at 12:26, wrote: > Jan Beulich writes ("stable trees (was: [xen-4.2-testing test] 58584: > regressions)"): >> Which leaves several options: >> - the problem was always there, but hidden by some factor in the >> old osstest instance, > > I think this is most likely. The old sys

Re: [Xen-devel] [PATCHv3 5/6] evtchn: remove the locking when unmasking an event channel

2015-06-18 Thread David Vrabel
On 18/06/15 12:30, Jan Beulich wrote: On 17.06.15 at 14:03, wrote: >> --- a/xen/common/event_channel.c >> +++ b/xen/common/event_channel.c >> @@ -978,8 +978,6 @@ int evtchn_unmask(unsigned int port) >> struct domain *d = current->domain; >> struct evtchn *evtchn; >> >> -ASSERT

Re: [Xen-devel] FW: VT-d async invalidation for Device-TLB.

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 13:31, wrote: >> On June 18, 2015 5:19 PM, wrote: >> >>> On 18.06.15 at 10:09, wrote: >> >> >> On 16.06.15 09:59, wrote: >> >> >>> On 16.06.15 at 09:59, wrote: >> >> > Jan Beulich wrote on 2015-06-16: >> >> > On 16.06.15 at 05:07, wrote: >> >>

Re: [Xen-devel] [PATCHv3 4/6] evtchn: use a per-event channel lock for sending events

2015-06-18 Thread David Vrabel
On 18/06/15 12:20, Jan Beulich wrote: On 17.06.15 at 14:03, wrote: >> When sending an event, use a new per-event channel lock to safely >> validate the event channel state. >> >> This new lock must be held when changing event channel state. Note >> that the event channel lock must also be he

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

2015-06-18 Thread George Dunlap
On 06/17/2015 12:24 PM, Ian Campbell wrote: > On Tue, 2015-06-16 at 12:44 +0100, Ian Jackson wrote: >> George Dunlap writes ("Re: [Xen-devel] [PATCH V4 3/7] libxl: add pvusb API"): >>> On Tue, Jun 16, 2015 at 12:19 PM, Juergen Gross wrote: I'm pretty sure this is just a matter of timing durin

Re: [Xen-devel] [PATCH 01/44] kernel: Add support for poweroff handler call chain

2015-06-18 Thread Guenter Roeck
On 06/17/2015 11:53 PM, Frans Klaver wrote: On Thu, Jun 18, 2015 at 3:04 AM, Stephen Boyd wrote: On 10/06/2014 10:28 PM, Guenter Roeck wrote: Various drivers implement architecture and/or device specific means to remove power from the system. For the most part, those drivers set the global va

Re: [Xen-devel] FW: VT-d async invalidation for Device-TLB.

2015-06-18 Thread Xu, Quan
>June 18, 2015 7:39 PM, wrote: > >>> On 18.06.15 at 13:31, wrote: > >> On June 18, 2015 5:19 PM, wrote: > >> >>> On 18.06.15 at 10:09, wrote: > >> > >> >> On 16.06.15 09:59, wrote: > >> >> >>> On 16.06.15 at 09:59, wrote: > >> >> > Jan Beulich wrote on 2015-06-16: >

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

2015-06-18 Thread George Dunlap
On 06/17/2015 12:24 PM, Ian Campbell wrote: > On Tue, 2015-06-16 at 12:44 +0100, Ian Jackson wrote: >> George Dunlap writes ("Re: [Xen-devel] [PATCH V4 3/7] libxl: add pvusb API"): >>> On Tue, Jun 16, 2015 at 12:19 PM, Juergen Gross wrote: I'm pretty sure this is just a matter of timing durin

Re: [Xen-devel] [PATCHv3 5/6] evtchn: remove the locking when unmasking an event channel

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 13:36, wrote: > On 18/06/15 12:30, Jan Beulich wrote: > On 17.06.15 at 14:03, wrote: >>> --- a/xen/common/event_channel.c >>> +++ b/xen/common/event_channel.c >>> @@ -978,8 +978,6 @@ int evtchn_unmask(unsigned int port) >>> struct domain *d = current->domain; >>>

Re: [Xen-devel] [PATCHv3 5/6] evtchn: remove the locking when unmasking an event channel

2015-06-18 Thread David Vrabel
On 18/06/15 13:08, Jan Beulich wrote: On 18.06.15 at 13:36, wrote: >> On 18/06/15 12:30, Jan Beulich wrote: >> On 17.06.15 at 14:03, wrote: --- a/xen/common/event_channel.c +++ b/xen/common/event_channel.c @@ -978,8 +978,6 @@ int evtchn_unmask(unsigned int port)

[Xen-devel] [PATCH] x86/HVM: EOI handling function adjustments

2015-06-18 Thread Jan Beulich
The vector parameters are more usefully u8 right away. This is particularly important for the vioapic_update_EOI() invocation from vioapic_write() (which luckily is only a latent issue, as VIOAPIC_VERSION_ID is still hard coded to 0x11 right now). But it at once allows simplifying VMX's EXIT_REASON

Re: [Xen-devel] [qemu-mainline bisection] complete build-amd64

2015-06-18 Thread Ian Jackson
osstest service user writes ("[qemu-mainline bisection] complete build-amd64"): > branch xen-unstable > xen branch xen-unstable > job build-amd64 > test xen-build > > Tree: qemu git://xenbits.xen.org/staging/qemu-xen-unstable.git > Tree: qemuu git://git.qemu.org/qemu.git > Tree: xen git://xenbits.

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

2015-06-18 Thread Juergen Gross
On 06/18/2015 02:08 PM, George Dunlap wrote: On 06/17/2015 12:24 PM, Ian Campbell wrote: On Tue, 2015-06-16 at 12:44 +0100, Ian Jackson wrote: George Dunlap writes ("Re: [Xen-devel] [PATCH V4 3/7] libxl: add pvusb API"): On Tue, Jun 16, 2015 at 12:19 PM, Juergen Gross wrote: I'm pretty sure

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

2015-06-18 Thread Juergen Gross
On 06/18/2015 10:50 AM, Ian Campbell wrote: On Thu, 2015-06-18 at 09:02 +0200, Juergen Gross wrote: On 06/18/2015 08:20 AM, Chun Yan Liu wrote: On 6/17/2015 at 07:34 PM, in message <1434540857.13744.334.ca...@citrix.com>, Ian Campbell wrote: On Tue, 2015-06-16 at 16:20 +0200, Juergen Gros

[Xen-devel] [xen-4.2-testing test] 58711: FAIL

2015-06-18 Thread osstest service user
flight 58711 xen-4.2-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/58711/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-libvirt 3 host-install(3) broken in 58584 REGR. vs. 584

[Xen-devel] [PATCH v2] x86/HVM: avoid pointer wraparound in bufioreq handling

2015-06-18 Thread Jan Beulich
The number of slots per page being 511 (i.e. not a power of two) means that the (32-bit) read and write indexes going beyond 2^32 will likely disturb operation. Extend I/O req server creation so the caller can indicate that it is using suitable atomic accesses where needed (not all accesses to the

[Xen-devel] [PATCH] xen/HVM: atomically access pointers in bufioreq handling

2015-06-18 Thread Jan Beulich
The number of slots per page being 511 (i.e. not a power of two) means that the (32-bit) read and write indexes going beyond 2^32 will likely disturb operation. The hypervisor side gets I/O req server creation extended so we can indicate that we're using suitable atomic accesses where needed (not a

[Xen-devel] [ovmf test] 58714: tolerable FAIL - PUSHED

2015-06-18 Thread osstest service user
flight 58714 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/58714/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stop fail like 58681 test-amd64-i386-xl-qemuu-win7-amd64 1

Re: [Xen-devel] [PATCH] x86/HVM: EOI handling function adjustments

2015-06-18 Thread Andrew Cooper
On 18/06/15 13:51, Jan Beulich wrote: > --- a/xen/arch/x86/hvm/vlapic.c > +++ b/xen/arch/x86/hvm/vlapic.c > @@ -421,18 +421,17 @@ void vlapic_EOI_set(struct vlapic *vlapi > if ( hvm_funcs.handle_eoi ) > hvm_funcs.handle_eoi(vector); > > -if ( vlapic_test_and_clear_vector(vector,

Re: [Xen-devel] [PATCH v2] x86/HVM: avoid pointer wraparound in bufioreq handling

2015-06-18 Thread Andrew Cooper
On 18/06/15 14:14, Jan Beulich wrote: > The number of slots per page being 511 (i.e. not a power of two) means > that the (32-bit) read and write indexes going beyond 2^32 will likely > disturb operation. Extend I/O req server creation so the caller can > indicate that it is using suitable atomic a

[Xen-devel] [libvirt test] 58713: tolerable FAIL - PUSHED

2015-06-18 Thread osstest service user
flight 58713 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/58713/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-libvirt-xsm 11 guest-start fail like 58643 test-amd64-i386-libvirt 11 gu

Re: [Xen-devel] [PATCH] x86/HVM: EOI handling function adjustments

2015-06-18 Thread Jan Beulich
>>> On 18.06.15 at 15:29, wrote: > On 18/06/15 13:51, Jan Beulich wrote: >> -void vlapic_handle_EOI_induced_exit(struct vlapic *vlapic, int vector) >> +void vlapic_handle_EOI(struct vlapic *vlapic, u8 vector) >> { >> +struct domain *d = vlapic_domain(vlapic); >> + >> if ( vlapic_test_and

Re: [Xen-devel] [PATCH v3 01/11] x86/acpi: add a common interface for x86 cpu matching

2015-06-18 Thread Jan Beulich
>>> On 11.06.15 at 10:25, wrote: > Add a common interface for matching the current cpu against an > array of x86_cpu_ids. Also change mwait-idle.c to use it. > > Signed-off-by: Wei Wang > --- There's a summary of the changes in v3 missing here (and it looks like everywhere in the series). Putti

Re: [Xen-devel] [PATCH v3 02/11] x86/intel_pstate: add some calculation related support

2015-06-18 Thread Jan Beulich
>>> On 11.06.15 at 10:26, wrote: > The added calculation related functions will be used in the intel_pstate.c. > They are copied from the kernel(commit 2418f4f2, f3002134, eb18cba7). The _Linux_ kernel I assume? > --- a/xen/include/asm-x86/div64.h > +++ b/xen/include/asm-x86/div64.h > @@ -11,4 +

[Xen-devel] Dom0 kernel panic when porting xen to new arm soc

2015-06-18 Thread Peng Fan
Hi, I am porting xen to an Cortex-A7 soc and met Dom0 kernel panic. I have no clear idea about why Dom0 kernel panic. Detail log see below: U-Boot 2015.04-rc4-00145-gf12a16e (Jun 18 2015 - 10:38:06) CPU: Freescale i.MX7D rev1.0 at 792 MHz CPU: Thermal invalid data, fuse: 0x1b800 CPU: Tempera

Re: [Xen-devel] [PATCH v3 03/11] x86/intel_pstate: add new policy fields and a new driver interface

2015-06-18 Thread Jan Beulich
>>> On 11.06.15 at 10:26, wrote: > --- a/xen/drivers/cpufreq/utility.c > +++ b/xen/drivers/cpufreq/utility.c > @@ -457,6 +457,12 @@ int __cpufreq_set_policy(struct cpufreq_policy *data, > data->min = policy->min; > data->max = policy->max; > > +if (cpufreq_driver->setpolicy) { > +

Re: [Xen-devel] [PATCH v3 04/11] x86/intel_pstate: relocate the driver register function

2015-06-18 Thread Jan Beulich
>>> On 11.06.15 at 10:27, wrote: > Register the CPU hotplug notifier when the driver is > registered, and move the driver register function to > the cpufreq.c. The first half of the sentence fails to say why. And I suppose if you explained that (to yourself) you'd figure that the change is wrong

Re: [Xen-devel] performace issue when turn on apicv

2015-06-18 Thread Jiangyifei
Hi, I'm John's colleague. We looked into the details of the tracing data, and found that the number of MSR_IA32_APICTMICT_MSR event is quite high when apic-v is enabled(about 9x more compared with apic-v disabled). Below is the details: EXIT_REASON_MSR_WRITE apicv on: MSR= 0x0838(MSR_IA32

Re: [Xen-devel] [PATCH v3 06/11] x86/intel_pstate: the main boby of the intel_pstate driver

2015-06-18 Thread Jan Beulich
>>> On 11.06.15 at 10:27, wrote: > The intel_pstate driver is ported following its kernel code logic > (commit: 93f0822d).In order to port the Linux source file with > minimal modifications, some of the variable types are kept intact > (e.g. "int current_pstae", would otherwise be changed to > "un

Re: [Xen-devel] [PATCH v3 08/11] x86/intel_pstate: add a booting param to select the driver to load

2015-06-18 Thread Jan Beulich
>>> On 11.06.15 at 10:28, wrote: > --- a/xen/arch/x86/acpi/cpufreq/intel_pstate.c > +++ b/xen/arch/x86/acpi/cpufreq/intel_pstate.c > @@ -749,6 +749,9 @@ static struct cpufreq_driver intel_pstate_driver = { > .name = "intel_pstate", > }; > > +static bool_t __initdata load_intel_psta

Re: [Xen-devel] [OSSTEST PATCH 00/25] Reporting improvements

2015-06-18 Thread Ian Campbell
On Thu, 2015-06-18 at 12:21 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [OSSTEST PATCH 00/25] Reporting improvements"): > > On Wed, 2015-06-17 at 17:44 +0100, Ian Jackson wrote: > > > Most of the early part of this series is straightforward (and acked) > > > and could go in whenever. > >

Re: [Xen-devel] stable trees (was: [xen-4.2-testing test] 58584: regressions)

2015-06-18 Thread Ian Campbell
On Thu, 2015-06-18 at 12:37 +0100, Jan Beulich wrote: > >>> On 17.06.15 at 12:26, wrote: > > Jan Beulich writes ("stable trees (was: [xen-4.2-testing test] 58584: > > regressions)"): > >> Which leaves several options: > >> - the problem was always there, but hidden by some factor in the > >> ol

Re: [Xen-devel] Dom0 kernel panic when porting xen to new arm soc

2015-06-18 Thread Julien Grall
On 18/06/15 15:09, Peng Fan wrote: > Hi, Hi, > /bin/sh: can't access tty; job control turned off > / # mmcblk2: p1 p2 > mmcblk2boot1: unknown partition table > mmcblk2boot0: unknown partition table > (XEN) imx-uart.c:72: uart: rxfifo overrun It seems that you ported a new UART driver. My feelin

Re: [Xen-devel] [PATCH 01/44] kernel: Add support for poweroff handler call chain

2015-06-18 Thread Guenter Roeck
On Wed, Jun 17, 2015 at 06:04:54PM -0700, Stephen Boyd wrote: [ ... ] > > What happened to this series? I want to add shutdown support to my > platform and I need to write a register on the PMIC in one driver to > configure it for shutdown instead of restart and then write an MMIO > register to te

Re: [Xen-devel] [OSSTEST PATCH 00/25] Reporting improvements

2015-06-18 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH 00/25] Reporting improvements"): > On Thu, 2015-06-18 at 12:21 +0100, Ian Jackson wrote: > > I'll give you a chance to look at the output URL above. > > Looks good, thanks. > > OOI what is the expected usefulness of the "alloc testid" column? With the sys

Re: [Xen-devel] Dom0 kernel panic when porting xen to new arm soc

2015-06-18 Thread Ian Campbell
On Thu, 2015-06-18 at 22:09 +0800, Peng Fan wrote: > Hi, > > I am porting xen to an Cortex-A7 soc and met Dom0 kernel panic. I have > no clear idea about why Dom0 kernel panic. Have you confirmed that this same kernel runs reliably natively on this platform? [...] > / # Unable to handle kernel

  1   2   >