Re: [Xen-devel] [PATCH v4 07/10] IOMMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (leaf ones).

2016-05-10 Thread Xu, Quan
On May 10, 2016 5:07 PM, Jan Beulich wrote: > >>> On 06.05.16 at 10:54, wrote: > > --- a/xen/drivers/passthrough/vtd/iommu.c > > +++ b/xen/drivers/passthrough/vtd/iommu.c > > @@ -604,15 +604,15 @@ static int iommu_flush_iotlb(struct domain *d, > unsigned long gfn, > > return rc; > > } > > >

Re: [Xen-devel] [PATCH v2 for-4.7 5/5] x86/hvm: Fix invalidation for emulated invlpg instructions

2016-05-10 Thread Tian, Kevin
> From: Andrew Cooper [mailto:andrew.coop...@citrix.com] > Sent: Tuesday, May 10, 2016 2:28 AM > > hap_invlpg() is reachable from the instruction emulator, which means > introspection and tests using hvm_fep can end up here. As such, crashing the > domain is not an appropriate action to take. >

Re: [Xen-devel] crash on boot with 4.6.1 on fedora 24

2016-05-10 Thread Jan Beulich
>>> On 11.05.16 at 07:49, wrote: > On 10/05/16 18:35, Boris Ostrovsky wrote: >> On 05/10/2016 11:43 AM, Juergen Gross wrote: >>> On 10/05/16 17:35, Jan Beulich wrote: >>> On 10.05.16 at 17:19, wrote: > On 10/05/16 15:57, Jan Beulich wrote: > On 10.05.16 at 15:39, wrote: >>> I

[Xen-devel] [qemu-mainline baseline-only test] 44402: regressions - trouble: broken/fail/pass

2016-05-10 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 44402 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/44402/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt-xsm 3 host-install(3

Re: [Xen-devel] [PATCH 1/3] libxl: switch to using libxl_domain_create_restore from v4.4 API

2016-05-10 Thread Olaf Hering
On Mon, May 02, Jim Fehlig wrote: > In LIBXL_API_VERSION 0x040400, the libxl_domain_create_restore API > gained a parameter for specifying restore parameters. Switch to > using version 0x040400, which will be useful in a subsequent commit > to specify the Xen migration stream version when restorin

Re: [Xen-devel] [PATCH v4 06/10] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones).

2016-05-10 Thread Xu, Quan
On May 10, 2016 5:04 PM, Jan Beulich wrote: > >>> On 06.05.16 at 10:54, wrote: > > --- a/xen/common/memory.c > > +++ b/xen/common/memory.c > > @@ -677,9 +677,19 @@ static int xenmem_add_to_physmap(struct > domain > > *d, #ifdef CONFIG_HAS_PASSTHROUGH > > if ( need_iommu(d) ) > > { > >

Re: [Xen-devel] crash on boot with 4.6.1 on fedora 24

2016-05-10 Thread Juergen Gross
On 10/05/16 18:35, Boris Ostrovsky wrote: > On 05/10/2016 11:43 AM, Juergen Gross wrote: >> On 10/05/16 17:35, Jan Beulich wrote: >> On 10.05.16 at 17:19, wrote: On 10/05/16 15:57, Jan Beulich wrote: On 10.05.16 at 15:39, wrote: >> I didn't finish unwrapping the stack yester

[Xen-devel] [qemu-upstream-4.3-testing test] 93977: trouble: blocked/broken

2016-05-10 Thread osstest service owner
flight 93977 qemu-upstream-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/93977/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-pvops 3 host-install(3) broken REG

Re: [Xen-devel] [PATCH v4 04/10] vt-d: propagate IOMMU Device-TLB flush error up to IOMMU unmapping.

2016-05-10 Thread Xu, Quan
On May 10, 2016 4:51 PM, Jan Beulich wrote: > >>> On 06.05.16 at 10:54, wrote: > > Propagate the IOMMU Device-TLB flush error up to IOMMU unmapping. > > > > Signed-off-by: Quan Xu > > Acked-by: Kevin Tian > > Reviewed-by: Jan Beulich > > but note ... > > > @@ -1766,9 +1769,7 @@ static int i

Re: [Xen-devel] [PATCH v4 03/10] IOMMU/MMU: enhance the call trees of IOMMU unmapping and mapping

2016-05-10 Thread Xu, Quan
On May 10, 2016 4:44 PM, Jan Beulich wrote: > >>> On 06.05.16 at 10:54, wrote: > > --- a/xen/arch/x86/mm/p2m.c > > +++ b/xen/arch/x86/mm/p2m.c > > @@ -638,13 +638,20 @@ p2m_remove_page(struct p2m_domain *p2m, > unsigned long gfn, unsigned long mfn, > > mfn_t mfn_return; > > p2m_type_t t

Re: [Xen-devel] xc_altp2m_set_vcpu_enable_notify fail

2016-05-10 Thread Big Strong
Further debugging shows that the domain is changed to domain 0 during the check of whether the cmd of do_altp2m_op is HVMOP_altp2m_vcpu_enable_notify, located at here

Re: [Xen-devel] [PATCH v4 03/10] IOMMU/MMU: enhance the call trees of IOMMU unmapping and mapping

2016-05-10 Thread Xu, Quan
On May 10, 2016 10:45 PM, George Dunlap wrote: > On Tue, May 10, 2016 at 9:44 AM, Jan Beulich wrote: > On 06.05.16 at 10:54, wrote: > >> When IOMMU mapping is failed, we issue a best effort rollback, > >> stopping IOMMU mapping, unmapping the previous IOMMU maps and > then > >> reporting th

Re: [Xen-devel] [RFC 00/16] xen/arm: Introduce alternative runtime patching for ARM64

2016-05-10 Thread Wei Chen
Hi Julien, I have created a dummy CPU feature to test alternative runtime patching. It seems working well: Before enable dummy CPU feature: (XEN) I/O virtualisation disabled (XEN) Patching kernel code (XEN) Testing alternative branch (XEN) Hello, this is the original code... (XEN) *** LOADING

Re: [Xen-devel] [PATCH v4 03/10] IOMMU/MMU: enhance the call trees of IOMMU unmapping and mapping

2016-05-10 Thread Xu, Quan
On May 10, 2016 11:00 PM, George Dunlap wrote: > On Tue, May 10, 2016 at 3:45 PM, George Dunlap > wrote: > > On Tue, May 10, 2016 at 9:44 AM, Jan Beulich wrote: > > On 06.05.16 at 10:54, wrote: > >>> When IOMMU mapping is failed, we issue a best effort rollback, > >>> stopping IOMMU mapping

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

2016-05-10 Thread osstest service owner
flight 93954 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/93954/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt-xsm 14 guest-saverestore fail REGR. vs. 91479 test-amd64-i386-libvirt

[Xen-devel] [xen-4.6-testing baseline-only test] 44401: regressions - FAIL

2016-05-10 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 44401 xen-4.6-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/44401/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm

[Xen-devel] [qemu-upstream-unstable test] 93947: tolerable FAIL - PUSHED

2016-05-10 Thread osstest service owner
flight 93947 qemu-upstream-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/93947/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm 9 debian-hvm-install fail in 93919 pass in 93947 test-arm

[Xen-devel] [ovmf test] 93944: regressions - FAIL

2016-05-10 Thread osstest service owner
flight 93944 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/93944/ 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-install fail REGR. vs. 65543 test-amd64-amd64-xl-qemuu-ov

[Xen-devel] [xen-unstable-smoke test] 94004: tolerable all pass - PUSHED

2016-05-10 Thread osstest service owner
flight 94004 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/94004/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 12 migrate-support-checkfail never pass test-armhf-armhf-xl 12

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

2016-05-10 Thread osstest service owner
flight 93937 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/93937/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stop fail like 93910 test-amd64-i386-xl-qemuu-win

[Xen-devel] [PATCH v3 2/6] build: convert crash_debug to Kconfig

2016-05-10 Thread Doug Goldstein
Convert the crash_debug option to Kconfig as CONFIG_CRASH_DEBUG. This was previously togglable on the command line so this adds a message for users enabling it from the command line to tell them to enable it from make menuconfig. Signed-off-by: Doug Goldstein --- CC: Andrew Cooper CC: George Dun

[Xen-devel] [PATCH v3 5/6] build: convert perfc{, _arrays} to Kconfig

2016-05-10 Thread Doug Goldstein
Convert the 'perfc' and 'perfc_arrays' options to Kconfig as CONFIG_PERF_COUNTERS and CONFIG_PERF_ARRAYS to minimize code changes. Signed-off-by: Doug Goldstein --- CC: Andrew Cooper CC: George Dunlap CC: Ian Jackson CC: Jan Beulich CC: Konrad Rzeszutek Wilk CC: Stefano Stabellini CC: Tim D

[Xen-devel] [PATCH v3 4/6] build: convert frame_pointer to Kconfig

2016-05-10 Thread Doug Goldstein
Converts the frame_pointer option to a Kconfig option. Signed-off-by: Doug Goldstein --- CC: Andrew Cooper CC: George Dunlap CC: Ian Jackson CC: Jan Beulich CC: Konrad Rzeszutek Wilk CC: Stefano Stabellini CC: Tim Deegan CC: Wei Liu --- INSTALL | 1 - xen/Kconfig.debug | 8

[Xen-devel] [PATCH v3 6/6] build: convert lock_profile to Kconfig

2016-05-10 Thread Doug Goldstein
Convert the 'lock_profile' option to Kconfig as CONFIG_LOCK_PROFILE. Signed-off-by: Doug Goldstein --- CC: Stefano Stabellini CC: Julien Grall CC: Jan Beulich CC: Andrew Cooper --- INSTALL| 1 - xen/Kconfig.debug | 7 +++ xen/Rules.mk | 2 --

[Xen-devel] [PATCH v3 3/6] build: convert verbose to Kconfig

2016-05-10 Thread Doug Goldstein
Convert 'verbose', which was enabled by 'debug=y' to Kconfig as CONFIG_VERBOSE_DEBUG which is enabled by default when CONFIG_DEBUG is enabled. Signed-off-by: Doug Goldstein --- CC: Stefano Stabellini CC: Julien Grall CC: Jan Beulich CC: Andrew Cooper CC: Daniel De Graaf --- INSTALL

[Xen-devel] [PATCH v3 1/6] build: add debug menu to Kconfig

2016-05-10 Thread Doug Goldstein
There are a number of debugging options for Xen so the idea is to have a menu to group them all together. Enabling this menu item will also disable NDEBUG which will result in more debug prints. This was previously wired into the 'debug=y' command line option. Signed-off-by: Doug Goldstein --- CC

[Xen-devel] [PATCH v3 0/6] Kconfig debug options

2016-05-10 Thread Doug Goldstein
This converts the debug options from xen/Rules.mk to Kconfig. This is a change from the previous version in the series because the feedback I got was that the debug options should not be linked to NDEBUG if EXPERT is set. Unfortunately Kconfig does not let you descend into a menu if that menu is n

Re: [Xen-devel] [libvirt] [PATCH 0/3] libxl: support Xen migration stream V2

2016-05-10 Thread Jim Fehlig
Ján Tomko wrote: > On Mon, May 02, 2016 at 07:01:16PM -0600, Jim Fehlig wrote: >> Hi all, >> >> This patch adds support for Xen migration stream V2 to the libvirt >> libxl driver. In the process it fixes save/restore and migration >> tests in OSSTest, which have been failing since libvirt commit >>

Re: [Xen-devel] crash on boot with 4.6.1 on fedora 24

2016-05-10 Thread Boris Ostrovsky
On 05/10/2016 12:11 PM, Kevin Moraga wrote: > Can you boot your system bare-metal and post output of 'biosdecode' command? -boris ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [xen-unstable baseline-only test] 44400: tolerable FAIL

2016-05-10 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 44400 xen-unstable real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/44400/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): build-amd64-rumpuserxen 6 xen-build

[Xen-devel] [xen-unstable-smoke test] 93999: regressions - FAIL

2016-05-10 Thread osstest service owner
flight 93999 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/93999/ 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. 93921 Tests which di

[Xen-devel] Security fixes in qemus in stable trees

2016-05-10 Thread Ian Jackson
I have: * Applied XSA-179, and in some cases some other missing patches including XSA-140, to all the qemu-xen-traditional branches back to 4.3. * Updated the qemu trad Config.mk tag to point to the new tip in xen.git#staging-4.3 to #staging-4.6 and #staging. I see that the XSA-179 pa

[Xen-devel] [xen-4.6-testing test] 93932: tolerable FAIL - PUSHED

2016-05-10 Thread osstest service owner
flight 93932 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/93932/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds 11 guest-start fail REGR. vs. 92181 test-amd64-i386-xl-qemut-w

[Xen-devel] [ANNOUNCEMENT] Xen 4.7 RC2

2016-05-10 Thread Wei Liu
Hi all Xen 4.7 RC2 is tagged. You can check that out from xen.git: git://xenbits.xen.org/xen.git 4.7.0-rc2 For you convenience there is also tarball at: http://bits.xensource.com/oss-xen/release/4.7.0-rc2/xen-4.7.0-rc2.tar.gz And the signature is at: http://bits.xensource.com/oss-xen/release/

Re: [Xen-devel] [PATCH v1 0/7] Make building xSplice patches easier

2016-05-10 Thread Konrad Rzeszutek Wilk
On Fri, May 06, 2016 at 04:48:01PM +0100, Ross Lagerwall wrote: > Here is a set of changes to make building xSplice patches easier. > Tested to boot on x86. > Compile-tested on arm. > > This is probably too late to make it into 4.7, but hey, if someone wants > to put it in I've CC'd Wei. Hey, I'

Re: [Xen-devel] crash on boot with 4.6.1 on fedora 24

2016-05-10 Thread Boris Ostrovsky
On 05/10/2016 11:43 AM, Juergen Gross wrote: > On 10/05/16 17:35, Jan Beulich wrote: > On 10.05.16 at 17:19, wrote: >>> On 10/05/16 15:57, Jan Beulich wrote: >>> On 10.05.16 at 15:39, wrote: > I didn't finish unwrapping the stack yesterday. Here it is: > > setup_arch -> dmi_sc

Re: [Xen-devel] xc_altp2m_set_vcpu_enable_notify fail

2016-05-10 Thread Big Strong
> > I think you need to add more printk to find out. > > Wei. > Thanks for the suggestion, after adding printk to all the routines of xc_altp2m_set_vcpu_enable_notify, it turns out that the problem is because the check of is_hvm_domain()

Re: [Xen-devel] [PATCH v3 2/3] xen: write information about supported backends

2016-05-10 Thread Anthony PERARD
On Fri, May 06, 2016 at 11:42:45AM +0200, Juergen Gross wrote: > Add a Xenstore directory for each supported pv backend. This will allow > Xen tools to decide which backend type to use in case there are > multiple possibilities. > > The information is added under > /local/domain//device-model//bac

Re: [Xen-devel] crash on boot with 4.6.1 on fedora 24

2016-05-10 Thread Kevin Moraga
On 05/10/2016 01:23 AM, Jan Beulich wrote: On 09.05.16 at 20:40, wrote: >> On 05/09/2016 01:22 PM, Kevin Moraga wrote: >>> On 05/09/2016 11:15 AM, Boris Ostrovsky wrote: On 05/09/2016 12:40 PM, Kevin Moraga wrote: > On 05/09/2016 09:53 AM, Jan Beulich wrote: > On 09.05.16 a

Re: [Xen-devel] [PATCH] xen/events: Don't move disabled irqs

2016-05-10 Thread Boris Ostrovsky
On 05/10/2016 11:11 AM, Ross Lagerwall wrote: > Commit ff1e22e7a638 ("xen/events: Mask a moving irq") open-coded > irq_move_irq() but left out checking if the IRQ is disabled. This broke > resuming from suspend since it tries to move a (disabled) irq without > holding the IRQ's desc->lock. Fix it b

Re: [Xen-devel] crash on boot with 4.6.1 on fedora 24

2016-05-10 Thread Juergen Gross
On 10/05/16 17:35, Jan Beulich wrote: On 10.05.16 at 17:19, wrote: >> On 10/05/16 15:57, Jan Beulich wrote: >> On 10.05.16 at 15:39, wrote: I didn't finish unwrapping the stack yesterday. Here it is: setup_arch -> dmi_scan_machine -> dmi_walk_early -> early_ioremap >>> >>>

Re: [Xen-devel] Hypercall invoking

2016-05-10 Thread Meng Xu
On Tue, May 10, 2016 at 6:12 AM, tutu sky wrote: > > > > From: Dario Faggioli > Sent: Tuesday, May 10, 2016 7:32 AM > To: tutu sky; Xen-devel@lists.xen.org > Subject: Re: [Xen-devel] Hypercall invoking > > On Tue, 2016-05-10 at 06:49 +, tutu sky wrote

Re: [Xen-devel] crash on boot with 4.6.1 on fedora 24

2016-05-10 Thread Jan Beulich
>>> On 10.05.16 at 17:19, wrote: > On 10/05/16 15:57, Jan Beulich wrote: > On 10.05.16 at 15:39, wrote: >>> I didn't finish unwrapping the stack yesterday. Here it is: >>> >>> setup_arch -> dmi_scan_machine -> dmi_walk_early -> early_ioremap >> >> Ah, that makes sense. Yet why would early_io

Re: [Xen-devel] Xen Q35 & virtual VTD support

2016-05-10 Thread Lan, Tianyu
On 5/10/2016 10:52 PM, Anthony PERARD wrote: On Tue, May 10, 2016 at 10:31:38PM +0800, Lan, Tianyu wrote: On 5/10/2016 6:11 PM, Stefano Stabellini wrote: Hello Tianyu, What do you mean by reusing Q35 for Xen? If you mean using QEMU to emulate a Q35 based machine for HVM guests, I think that

Re: [Xen-devel] crash on boot with 4.6.1 on fedora 24

2016-05-10 Thread Juergen Gross
On 10/05/16 15:57, Jan Beulich wrote: On 10.05.16 at 15:39, wrote: >> I didn't finish unwrapping the stack yesterday. Here it is: >> >> setup_arch -> dmi_scan_machine -> dmi_walk_early -> early_ioremap > > Ah, that makes sense. Yet why would early_ioremap() involve an > M2P lookup? As said,

Re: [Xen-devel] [PATCH v4 03/10] IOMMU/MMU: enhance the call trees of IOMMU unmapping and mapping

2016-05-10 Thread George Dunlap
On Tue, May 10, 2016 at 3:45 PM, George Dunlap wrote: > On Tue, May 10, 2016 at 9:44 AM, Jan Beulich wrote: > On 06.05.16 at 10:54, wrote: >>> When IOMMU mapping is failed, we issue a best effort rollback, stopping >>> IOMMU mapping, unmapping the previous IOMMU maps and then reporting the >

[Xen-devel] [PATCH] xen/events: Don't move disabled irqs

2016-05-10 Thread Ross Lagerwall
Commit ff1e22e7a638 ("xen/events: Mask a moving irq") open-coded irq_move_irq() but left out checking if the IRQ is disabled. This broke resuming from suspend since it tries to move a (disabled) irq without holding the IRQ's desc->lock. Fix it by adding in a check for disabled IRQs. The resulting

Re: [Xen-devel] [PATCH v4 08/10] vt-d/ept: propagate IOMMU Device-TLB flush error up to EPT update.

2016-05-10 Thread Jan Beulich
>>> On 10.05.16 at 16:58, wrote: > On Tue, May 10, 2016 at 10:09 AM, Jan Beulich wrote: > On 06.05.16 at 10:54, wrote: >>> --- a/xen/arch/x86/mm/p2m-ept.c >>> +++ b/xen/arch/x86/mm/p2m-ept.c >>> @@ -832,7 +832,8 @@ out: >>> need_modify_vtd_table ) >>> { >>> if ( iommu

Re: [Xen-devel] [PATCH v4 03/10] IOMMU/MMU: enhance the call trees of IOMMU unmapping and mapping

2016-05-10 Thread Jan Beulich
>>> On 10.05.16 at 16:45, wrote: > On Tue, May 10, 2016 at 9:44 AM, Jan Beulich wrote: > On 06.05.16 at 10:54, wrote: >>> @@ -812,17 +813,22 @@ ept_set_entry(struct p2m_domain *p2m, unsigned long >>> gfn, mfn_t mfn, >>> rc = atomic_write_ept_entry(ept_entry, new_entry, target); >>>

Re: [Xen-devel] [PATCH v4 08/10] vt-d/ept: propagate IOMMU Device-TLB flush error up to EPT update.

2016-05-10 Thread George Dunlap
On Tue, May 10, 2016 at 10:09 AM, Jan Beulich wrote: On 06.05.16 at 10:54, wrote: >> --- a/xen/arch/x86/mm/p2m-ept.c >> +++ b/xen/arch/x86/mm/p2m-ept.c >> @@ -832,7 +832,8 @@ out: >> need_modify_vtd_table ) >> { >> if ( iommu_hap_pt_share ) >> -iommu_pte_f

Re: [Xen-devel] [PATCH v1 6/7] acpi: Remove dependency on __LINE__ for release builds

2016-05-10 Thread Jan Beulich
>>> On 06.05.16 at 17:48, wrote: > --- a/xen/drivers/acpi/utilities/utmisc.c > +++ b/xen/drivers/acpi/utilities/utmisc.c > @@ -134,6 +134,51 @@ const char *__init > acpi_ut_validate_exception(acpi_status status) > * > > **

Re: [Xen-devel] Xen Q35 & virtual VTD support

2016-05-10 Thread Anthony PERARD
On Tue, May 10, 2016 at 10:31:38PM +0800, Lan, Tianyu wrote: > On 5/10/2016 6:11 PM, Stefano Stabellini wrote: > > Hello Tianyu, > > > > What do you mean by reusing Q35 for Xen? If you mean using QEMU to > > emulate a Q35 based machine for HVM guests, I think that should be OK. > > > > From xl c

Re: [Xen-devel] [PATCH v1 5/7] iommu: Remove dependency on __LINE__ for release builds

2016-05-10 Thread Jan Beulich
>>> On 06.05.16 at 17:48, wrote: > --- a/xen/drivers/passthrough/vtd/dmar.h > +++ b/xen/drivers/passthrough/vtd/dmar.h > @@ -108,6 +108,15 @@ struct acpi_atsr_unit *acpi_find_matched_atsr_unit(const > struct pci_dev *); > > #define DMAR_OPERATION_TIMEOUT MILLISECS(1000) > > +#ifdef NDEBUG >

Re: [Xen-devel] [PATCH v1 2/7] sched: Remove dependency on __LINE__ for release builds

2016-05-10 Thread Jan Beulich
>>> On 06.05.16 at 17:48, wrote: > When using xsplice, use of __LINE__ can generate spurious changes in > functions due to embedded line numbers. For release builds, remove the > use of these line numbers in domain_crash*() and print the current text > address instead. Which makes it more cumber

Re: [Xen-devel] [PATCH v1 4/7] page-alloc: Remove dependency on __LINE__ for release builds

2016-05-10 Thread Andrew Cooper
On 06/05/16 16:48, Ross Lagerwall wrote: > When using xsplice, use of __LINE__ can generate spurious changes in > functions due to embedded line numbers. For release builds, remove the > use of these line numbers in BOOT_BUG_ON() and print the current text > address instead. > > Signed-off-by: Ros

Re: [Xen-devel] [PATCH v1 3/7] mm: Use statically defined locking order

2016-05-10 Thread Andrew Cooper
On 06/05/16 16:48, Ross Lagerwall wrote: > Instead of using a locking order based on line numbers which doesn't > play nicely with xSplice, statically define the locking order. > > Signed-off-by: Ross Lagerwall Along with this change, it would useful to modify the message in __check_lock_level()

Re: [Xen-devel] [PATCH v1 4/7] page-alloc: Remove dependency on __LINE__ for release builds

2016-05-10 Thread Jan Beulich
>>> On 06.05.16 at 17:48, wrote: > --- a/xen/common/page_alloc.c > +++ b/xen/common/page_alloc.c > @@ -206,11 +206,19 @@ struct scrub_region { > static struct scrub_region __initdata region[MAX_NUMNODES]; > static unsigned long __initdata chunk_size; > > +#ifdef NDEBUG > +static void __init bo

Re: [Xen-devel] [PATCH v4 03/10] IOMMU/MMU: enhance the call trees of IOMMU unmapping and mapping

2016-05-10 Thread George Dunlap
On Tue, May 10, 2016 at 9:44 AM, Jan Beulich wrote: On 06.05.16 at 10:54, wrote: >> When IOMMU mapping is failed, we issue a best effort rollback, stopping >> IOMMU mapping, unmapping the previous IOMMU maps and then reporting the >> error up to the call trees. When rollback is not feasible

Re: [Xen-devel] [PATCH v1 3/7] mm: Use statically defined locking order

2016-05-10 Thread Jan Beulich
>>> On 06.05.16 at 17:48, wrote: > @@ -201,11 +203,20 @@ static inline void mm_enforce_order_unlock(int > unlock_level, > > / > * * > - * To avoid dead

Re: [Xen-devel] Xen 4.7 Test Day Instructions for RC2+ : Call to action for people who added new features / functionality to Xen 4.7

2016-05-10 Thread Lars Kurth
Hi all, I cleaned the test page page up as much as I could and have time for and incorporated your input: see http://wiki.xenproject.org/wiki/Xen_4.7_RC_test_instructions Obviously there are still gaps, which could be filled. I did focus on top-level features and features which have a command

[Xen-devel] [qemu-upstream-4.5-testing baseline-only test] 44399: regressions - FAIL

2016-05-10 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 44399 qemu-upstream-4.5-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/44399/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-multivcpu 15 g

Re: [Xen-devel] [PATCH v1 1/7] lib: Add a generic implementation of current_text_addr()

2016-05-10 Thread Jan Beulich
>>> On 06.05.16 at 17:48, wrote: > --- a/xen/common/lib.c > +++ b/xen/common/lib.c > @@ -499,6 +499,18 @@ void __init init_constructors(void) > } > > /* > + * The GCC docs state that the function must be marked noinline to have the > + * expected result: > + * "When inlining the expected behav

Re: [Xen-devel] [PATCH] x86/PoD: skip eager reclaim when possible

2016-05-10 Thread Jan Beulich
>>> On 10.05.16 at 15:50, wrote: > On 10/05/16 14:44, Jan Beulich wrote: >> Reclaiming pages is pointless when the cache can already satisfy all >> outstanding PoD entries, and doing reclaims in that case can be very >> harmful to performance when that memory gets used by the guest, but >> only to

Re: [Xen-devel] Xen Q35 & virtual VTD support

2016-05-10 Thread Lan, Tianyu
On 5/10/2016 6:11 PM, Stefano Stabellini wrote: Hello Tianyu, I am CC'ing Anthony who should have a better idea about this. Also please use my kernel.org email address for future correspondence. OK. I get it. What do you mean by reusing Q35 for Xen? If you mean using QEMU to emulate a Q35

[Xen-devel] [xen-4.5-testing test] 93928: regressions - FAIL

2016-05-10 Thread osstest service owner
flight 93928 xen-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/93928/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt 14 guest-saverestore fail REGR. vs. 92345 test-armhf-armhf-

Re: [Xen-devel] [PATCH v1 7/7] Rename sections for compatibility with -ffunction-sections -fdata-sections

2016-05-10 Thread Jan Beulich
>>> On 06.05.16 at 17:48, wrote: > When building with -ffunction-sections -fdata-sections, it will generate > section names like .text.show_handlers and .data.payload_list. These > sections are in the same namespace as the special sections that Xen > uses, such as .text.kexec and .data.schedulers.

Re: [Xen-devel] [PATCH] x86/PoD: skip eager reclaim when possible

2016-05-10 Thread Wei Liu
On Tue, May 10, 2016 at 07:49:01AM -0600, Jan Beulich wrote: > >>> On 10.05.16 at 15:44, wrote: > > Reclaiming pages is pointless when the cache can already satisfy all > > outstanding PoD entries, and doing reclaims in that case can be very > > harmful to performance when that memory gets used by

Re: [Xen-devel] [PATCH for-4.7] sched/rt: Fix memory leak in rt_init()

2016-05-10 Thread Andrew Cooper
On 10/05/16 15:18, Wei Liu wrote: > On Tue, May 10, 2016 at 02:38:48PM +0100, Andrew Cooper wrote: >> c/s 2656bc7b0 "xen: adopt .deinit_pdata and improve timer handling" >> introduced a error path into rt_init() which leaked prv if the >> allocation of prv->repl_timer failed. >> >> Introduce an err

Re: [Xen-devel] [PATCH for-4.7] sched/rt: Fix memory leak in rt_init()

2016-05-10 Thread Wei Liu
On Tue, May 10, 2016 at 02:38:48PM +0100, Andrew Cooper wrote: > c/s 2656bc7b0 "xen: adopt .deinit_pdata and improve timer handling" > introduced a error path into rt_init() which leaked prv if the > allocation of prv->repl_timer failed. > > Introduce an error cleanup path. > > Spotted by Coverit

Re: [Xen-devel] [Xen-users] [QEMU #1578192] Re: xen_kbdfront: unhandled keycode 0x0

2016-05-10 Thread George Dunlap
[BCC'ing xen-users, and cc'ing Anthony] On Mon, May 9, 2016 at 3:25 PM, wrote: > Just in case anyone was following this thread, I traced the bug back to > Qemu, so it is in fact not a Xen bug. Here's my qemu-devel post: > > https://lists.nongnu.org/archive/html/qemu-devel/2016-05/msg00119.html >

Re: [Xen-devel] [PATCH for-4.7] sched/rt: Fix memory leak in rt_init()

2016-05-10 Thread Andrew Cooper
On 10/05/16 14:52, Meng Xu wrote: > On Tue, May 10, 2016 at 9:38 AM, Andrew Cooper > wrote: >> c/s 2656bc7b0 "xen: adopt .deinit_pdata and improve timer handling" >> introduced a error path into rt_init() which leaked prv if the >> allocation of prv->repl_timer failed. >> >> Introduce an error cle

[Xen-devel] [PATCH v4 0/4] arm64, xen: add xen_boot support into grup-mkconfig

2016-05-10 Thread fu . wei
From: Fu Wei This patchset add xen_boot support into grup-mkconfig for generating xen boot entrances automatically Also update the docs/grub.texi for new xen_boot commands. This patchset has been tested on Foudation model with a bug fix: http://lists.gnu.org/archive/html/grub-devel/2016-02/msg0

Re: [Xen-devel] [PATCH] x86/PoD: skip eager reclaim when possible

2016-05-10 Thread Andrew Cooper
On 10/05/16 14:44, Jan Beulich wrote: > Reclaiming pages is pointless when the cache can already satisfy all > outstanding PoD entries, and doing reclaims in that case can be very > harmful to performance when that memory gets used by the guest, but > only to store zeroes there. > > Signed-off-by:

[Xen-devel] [PATCH v4 3/4] * util/grub.d/20_linux_xen.in: Add xen_boot command support

2016-05-10 Thread fu . wei
From: Fu Wei This patch adds the support of xen_boot command: xen_hypervisor xen_module Also add a new "feature_xen_boot" to indicate this grub support xen_boot command. Signed-off-by: Fu Wei --- grub-core/normal/main.c | 2 +- util/grub.d/20_linux_xen.in | 13 ++--- 2 fi

[Xen-devel] [PATCH v4 2/4] arm64: add "--nounzip" option support in xen_module command

2016-05-10 Thread fu . wei
From: Fu Wei This patch adds "--nounzip" option support in order to be compatible with the module command of i386, then we can simplify grub-mkconfig support code. Signed-off-by: Fu Wei --- grub-core/loader/arm64/xen_boot.c | 17 + 1 file changed, 17 insertions(+) diff --git a

[Xen-devel] [PATCH v4 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi

2016-05-10 Thread fu . wei
From: Fu Wei delete: xen_linux, xen_initrd, xen_xsm add: xen_module This update bases on commit 0edd750e50698854068358ea53528100a9192902 Author: Vladimir Serbinenko Date: Fri Jan 22 10:18:47 2016 +0100 xen_boot: Remove obsolete module type distinctions. Also bases on the

[Xen-devel] [PATCH v4 1/4] i386, xen: Add xen_hypervisor and xen_module aliases for i386

2016-05-10 Thread fu . wei
From: Fu Wei Signed-off-by: Fu Wei --- grub-core/loader/i386/xen.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/grub-core/loader/i386/xen.c b/grub-core/loader/i386/xen.c index c4d9689..15b0727 100644 --- a/grub-core/loader/i386/xen.c +++ b/grub-core/loader/i386/xen.c @@ -689,6 +68

Re: [Xen-devel] crash on boot with 4.6.1 on fedora 24

2016-05-10 Thread Jan Beulich
>>> On 10.05.16 at 15:39, wrote: > I didn't finish unwrapping the stack yesterday. Here it is: > > setup_arch -> dmi_scan_machine -> dmi_walk_early -> early_ioremap Ah, that makes sense. Yet why would early_ioremap() involve an M2P lookup? As said, MMIO addresses shouldn't be subject to such loo

Re: [Xen-devel] [PATCH for-4.7] sched/rt: Fix memory leak in rt_init()

2016-05-10 Thread Meng Xu
On Tue, May 10, 2016 at 9:38 AM, Andrew Cooper wrote: > c/s 2656bc7b0 "xen: adopt .deinit_pdata and improve timer handling" > introduced a error path into rt_init() which leaked prv if the > allocation of prv->repl_timer failed. > > Introduce an error cleanup path. > > Spotted by Coverity. I'm cu

Re: [Xen-devel] [PATCH] x86/PoD: skip eager reclaim when possible

2016-05-10 Thread Jan Beulich
>>> On 10.05.16 at 15:44, wrote: > Reclaiming pages is pointless when the cache can already satisfy all > outstanding PoD entries, and doing reclaims in that case can be very > harmful to performance when that memory gets used by the guest, but > only to store zeroes there. > > Signed-off-by: Jan

Re: [Xen-devel] [PATCH net-next 2/4] xen-netback: add control protocol implementation

2016-05-10 Thread Paul Durrant
> -Original Message- > From: Wei Liu [mailto:wei.l...@citrix.com] > Sent: 10 May 2016 14:29 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org; net...@vger.kernel.org; Wei Liu > Subject: Re: [PATCH net-next 2/4] xen-netback: add control protocol > implementation > > On Thu, May 05, 20

Re: [Xen-devel] [PATCH net-next 1/4] xen-netback: add control ring boilerplate

2016-05-10 Thread Paul Durrant
> -Original Message- > From: Wei Liu [mailto:wei.l...@citrix.com] > Sent: 10 May 2016 14:29 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org; net...@vger.kernel.org; Wei Liu > Subject: Re: [PATCH net-next 1/4] xen-netback: add control ring boilerplate > > On Thu, May 05, 2016 at 12:

[Xen-devel] [PATCH] x86/PoD: skip eager reclaim when possible

2016-05-10 Thread Jan Beulich
Reclaiming pages is pointless when the cache can already satisfy all outstanding PoD entries, and doing reclaims in that case can be very harmful to performance when that memory gets used by the guest, but only to store zeroes there. Signed-off-by: Jan Beulich --- a/xen/arch/x86/mm/p2m-pod.c +++

Re: [Xen-devel] crash on boot with 4.6.1 on fedora 24

2016-05-10 Thread Boris Ostrovsky
On 05/10/2016 03:23 AM, Jan Beulich wrote: On 09.05.16 at 20:40, wrote: >> On 05/09/2016 01:22 PM, Kevin Moraga wrote: >>> On 05/09/2016 11:15 AM, Boris Ostrovsky wrote: On 05/09/2016 12:40 PM, Kevin Moraga wrote: > On 05/09/2016 09:53 AM, Jan Beulich wrote: > On 09.05.16 at

[Xen-devel] [PATCH for-4.7] sched/rt: Fix memory leak in rt_init()

2016-05-10 Thread Andrew Cooper
c/s 2656bc7b0 "xen: adopt .deinit_pdata and improve timer handling" introduced a error path into rt_init() which leaked prv if the allocation of prv->repl_timer failed. Introduce an error cleanup path. Spotted by Coverity. Signed-off-by: Andrew Cooper --- CC: Wei Liu CC: George Dunlap CC: Dar

Re: [Xen-devel] [PATCH net-next 2/4] xen-netback: add control protocol implementation

2016-05-10 Thread Wei Liu
On Thu, May 05, 2016 at 12:19:28PM +0100, Paul Durrant wrote: > My recent patch to include/xen/interface/io/netif.h defines a new shared > ring (in addition to the rx and tx rings) for passing control messages > from a VM frontend driver to a backend driver. > > A previous patch added the necessar

Re: [Xen-devel] [PATCH net-next 4/4] xen-netback: use hash value from the frontend

2016-05-10 Thread Wei Liu
On Thu, May 05, 2016 at 12:19:30PM +0100, Paul Durrant wrote: > My recent patch to include/xen/interface/io/netif.h defines a new extra > info type that can be used to pass hash values between backend and guest > frontend. > > This patch adds code to xen-netback to use the value in a hash extra >

Re: [Xen-devel] [PATCH net-next 1/4] xen-netback: add control ring boilerplate

2016-05-10 Thread Wei Liu
On Thu, May 05, 2016 at 12:19:27PM +0100, Paul Durrant wrote: [...] > > +static int connect_ctrl_ring(struct backend_info *be) > +{ Please use goto style error handling in this function. Other than this the code looks good. Wei. ___ Xen-devel mailin

Re: [Xen-devel] [PATCH net-next 3/4] xen-netback: pass hash value to the frontend

2016-05-10 Thread Wei Liu
On Thu, May 05, 2016 at 12:19:29PM +0100, Paul Durrant wrote: > My recent patch to include/xen/interface/io/netif.h defines a new extra > info type that can be used to pass hash values between backend and guest > frontend. > > This patch adds code to xen-netback to pass hash values calculated for

Re: [Xen-devel] [PATCH] xtf: allow selecting objcopy binary to use

2016-05-10 Thread Andrew Cooper
On 10/05/16 13:16, Roger Pau Monne wrote: > Signed-off-by: Roger Pau Monné Reviewed-by: Andrew Cooper and committed. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH] xtf: only set CC if it's not set

2016-05-10 Thread Andrew Cooper
On 10/05/16 13:18, Roger Pau Monne wrote: > This prevents overwriting CC if it's already set on the environment. > > Signed-off-by: Roger Pau Monné Reviewed-by: Andrew Cooper and committed. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lis

Re: [Xen-devel] [PATCH] tools: configure correct trace backend for QEMU

2016-05-10 Thread Paul Durrant
> -Original Message- > From: Wei Liu [mailto:wei.l...@citrix.com] > Sent: 10 May 2016 11:58 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org; Ian Jackson; Wei Liu > Subject: Re: [PATCH] tools: configure correct trace backend for QEMU > > On Mon, May 09, 2016 at 05:43:14PM +0100, Pau

[Xen-devel] [PATCH] xtf: only set CC if it's not set

2016-05-10 Thread Roger Pau Monne
This prevents overwriting CC if it's already set on the environment. Signed-off-by: Roger Pau Monné --- build/common.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build/common.mk b/build/common.mk index 2741508..1120a16 100644 --- a/build/common.mk +++ b/build/common.mk

[Xen-devel] [PATCH] xtf: allow selecting objcopy binary to use

2016-05-10 Thread Roger Pau Monne
Signed-off-by: Roger Pau Monné --- build/common.mk | 1 + build/gen.mk| 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/build/common.mk b/build/common.mk index 1120a16..13c468e 100644 --- a/build/common.mk +++ b/build/common.mk @@ -2,6 +2,7 @@ ROOT := $(abspath $(CURDIR)/.

Re: [Xen-devel] [PATCH v1 7/7] Rename sections for compatibility with -ffunction-sections -fdata-sections

2016-05-10 Thread Julien Grall
Hi Ross, On 06/05/16 16:48, Ross Lagerwall wrote: When building with -ffunction-sections -fdata-sections, it will generate section names like .text.show_handlers and .data.payload_list. These sections are in the same namespace as the special sections that Xen uses, such as .text.kexec and .data.

[Xen-devel] Xen Security Advisory 179 (CVE-2016-3710, CVE-2016-3712) - QEMU: Banked access to VGA memory (VBE) uses inconsistent bounds checks

2016-05-10 Thread Xen . org security team
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Xen Security Advisory CVE-2016-3710,CVE-2016-3712 / XSA-179 version 5 QEMU: Banked access to VGA memory (VBE) uses inconsistent bounds checks UPDATES IN VERSION 5 Fixed credits section. Zuoz

Re: [Xen-devel] [PATCH v2 for-4.7 5/5] x86/hvm: Fix invalidation for emulated invlpg instructions

2016-05-10 Thread Andrew Cooper
On 10/05/16 12:02, Jan Beulich wrote: On 10.05.16 at 12:41, wrote: >> On 10/05/16 11:15, Andrew Cooper wrote: >>> On 10/05/16 08:51, Jan Beulich wrote: >>> On 09.05.16 at 20:27, wrote: > --- a/xen/arch/x86/hvm/svm/svm.c > +++ b/xen/arch/x86/hvm/svm/svm.c > @@ -,10 +,1

Re: [Xen-devel] [PATCH v2 for-4.7 5/5] x86/hvm: Fix invalidation for emulated invlpg instructions

2016-05-10 Thread Jan Beulich
>>> On 10.05.16 at 12:41, wrote: > On 10/05/16 11:15, Andrew Cooper wrote: >> On 10/05/16 08:51, Jan Beulich wrote: >> On 09.05.16 at 20:27, wrote: --- a/xen/arch/x86/hvm/svm/svm.c +++ b/xen/arch/x86/hvm/svm/svm.c @@ -,10 +,13 @@ static void svm_invlpga_intercept(

Re: [Xen-devel] run xl toolstack on hikey board

2016-05-10 Thread Julien Grall
On 10/05/16 11:24, Chenxiao Zhao wrote: Hi, Hello, This is question is not related to development but using Xen. So I have CCed xen-users and BCCed xen-devel. I have followed xen wiki to get xen hypervisor and dom0 run on hikey board. when I try to run xl command to list all the domain i

Re: [Xen-devel] [PATCH] tools: configure correct trace backend for QEMU

2016-05-10 Thread Wei Liu
On Mon, May 09, 2016 at 05:43:14PM +0100, Paul Durrant wrote: > Newer versions of the QEMU source have replaced the 'stderr' trace > backend with 'log'. This patch adjusts the tools Makefile to test for > the 'log' backend and specify it if it is available. > > Signed-off-by: Paul Durrant Acked-

  1   2   >