Re: [Xen-devel] VPMU interrupt unreliability

2017-10-20 Thread Jan Beulich
>>> On 19.10.17 at 20:20, wrote: > Is there any document about the possible attack via the vPMU? The > document I found (such as [1] and XSA-163) just briefly say that the > vPMU should be disabled due to security concern. Besides the other responses you've already got, I also recall there being

Re: [Xen-devel] [PATCH for-4.10 v2] passthrough/vtd: Don't DMA to the stack in queue_invalidate_wait()

2017-10-20 Thread Jan Beulich
>>> On 19.10.17 at 18:22, wrote: > DMA-ing to the stack is generally considered bad practice. In this case, if a > timeout occurs because of a sluggish device which is processing the request, > the completion notification will corrupt the stack of a subsequent deeper call > tree. > > Place the p

Re: [Xen-devel] [PATCH V3 11/29] x86/hvm: Introduce a emulated VTD for HVM

2017-10-20 Thread Chao Gao
On Fri, Oct 20, 2017 at 12:56:03AM -0600, Jan Beulich wrote: On 20.10.17 at 04:46, wrote: >> On Thu, Oct 19, 2017 at 12:20:35PM +0100, Roger Pau Monné wrote: >>>On Thu, Sep 21, 2017 at 11:01:52PM -0400, Lan Tianyu wrote: From: Chao Gao This patch adds create/destroy function

Re: [Xen-devel] [PATCH for-4.10] string: fix memmove when size is 0

2017-10-20 Thread Jan Beulich
>>> On 17.10.17 at 14:03, wrote: > --- a/xen/arch/x86/string.c > +++ b/xen/arch/x86/string.c > @@ -39,6 +39,9 @@ void *(memmove)(void *dest, const void *src, size_t n) > { > long d0, d1, d2; > > +if ( !n ) > +return; Actually - I can't see how this would build successfully: Th

[Xen-devel] [xen-4.7-testing test] 114698: regressions - FAIL

2017-10-20 Thread osstest service owner
flight 114698 xen-4.7-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/114698/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-xtf-amd64-amd64-4 49 xtf/test-hvm64-lbr-tsx-vmentry fail REGR. vs. 114483 Tests which are

[Xen-devel] [libvirt test] 114713: tolerable all pass - PUSHED

2017-10-20 Thread osstest service owner
flight 114713 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/114713/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt-xsm 14 saverestore-support-checkfail like 114669 test-armhf-armhf-libvirt 14 saveresto

Re: [Xen-devel] x86: PIE support and option to extend KASLR randomization

2017-10-20 Thread Ingo Molnar
* Pavel Machek wrote: > On Mon 2017-09-25 09:33:42, Ingo Molnar wrote: > > > > * Pavel Machek wrote: > > > > > > For example, there would be collision with regular user-space mappings, > > > > right? > > > > Can local unprivileged users use mmap(MAP_FIXED) probing to figure out > > > > whe

Re: [Xen-devel] [PATCH v1 01/27] x86/crypto: Adapt assembly for PIE support

2017-10-20 Thread Ingo Molnar
* Thomas Garnier wrote: > Change the assembly code to use only relative references of symbols for the > kernel to be PIE compatible. > > Position Independent Executable (PIE) support will allow to extended the > KASLR randomization range below the -2G memory limit. > diff --git a/arch/x86/cryp

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 20 October 2017 07:25 > To: Julien Grall > Cc: Julien Grall ; Andrew Cooper > ; George Dunlap > ; Ian Jackson ; Paul > Durrant ; Roger Pau Monne > ; Wei Liu ; Stefano Stabellini > ; xen-de...@lists.xenproject.org;

[Xen-devel] [PATCH] pci-assign: Remove

2017-10-20 Thread Paolo Bonzini
Legacy PCI device assignment has been removed from Linux in 4.12, and had been deprecated 2 years ago there. We can remove it from QEMU as well. The ROM loading code was shared with Xen PCI passthrough, so move it to hw/xen. Cc: Stefano Stabellini Cc: Anthony Perard Cc: xen-de...@lists.xenproj

Re: [Xen-devel] [PATCH v1 06/27] x86/entry/64: Adapt assembly for PIE support

2017-10-20 Thread Ingo Molnar
* Thomas Garnier wrote: > Change the assembly code to use only relative references of symbols for the > kernel to be PIE compatible. > > Position Independent Executable (PIE) support will allow to extended the > KASLR randomization range below the -2G memory limit. > > Signed-off-by: Thomas Ga

Re: [Xen-devel] [PATCH v1 01/27] x86/crypto: Adapt assembly for PIE support

2017-10-20 Thread Ard Biesheuvel
On 20 October 2017 at 09:24, Ingo Molnar wrote: > > * Thomas Garnier wrote: > >> Change the assembly code to use only relative references of symbols for the >> kernel to be PIE compatible. >> >> Position Independent Executable (PIE) support will allow to extended the >> KASLR randomization range

[Xen-devel] [PATCH v3 for 4.10] x86/vpt: guarantee the return value of pt_update_irq() set in vIRR or PIR

2017-10-20 Thread Chao Gao
pt_update_irq() is expected to return the vector number of periodic timer interrupt, which should be set in vIRR of vlapic or in PIR. Otherwise it would trigger the assertion in vmx_intr_assist(), please seeing https://lists.xenproject.org/archives/html/xen-devel/2017-10/msg00915.html. But it fai

Re: [Xen-devel] [PATCH RFC 00/14] Intel EPT-Based Sub-page Write Protection Support.

2017-10-20 Thread Yi Zhang
On 2017-10-19 at 12:07:44 +0300, Razvan Cojocaru wrote: > On 19.10.2017 11:04, Zhang Yi wrote: > > From: Zhang Yi Z > > > > Hi All, > > > > Here is a patch-series which adding EPT-Based Sub-page Write Protection > > Support. You can get It's software developer manuals from: > > > > https://sof

Re: [Xen-devel] [PATCH] x86/xen: support priv-mapping in an HVM tools domain

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Boris Ostrovsky > Sent: 19 October 2017 18:45 > To: Paul Durrant ; x...@kernel.org; xen- > de...@lists.xenproject.org; linux-ker...@vger.kernel.org > Cc: Juergen Gross ; Thomas Gleixner > ; Ingo M

Re: [Xen-devel] [PATCH v2] arm: configure interrupts to be in non-secure group1

2017-10-20 Thread Andre Przywara
Hi, On 18/10/17 22:29, Stefano Stabellini wrote: > Xen uses non-secure group1 interrupts, however it doesn't configure the > GICv3 accordingly. Xen needs to set GICD_IGROUPR for SPIs and > GICR_IGROUPR0 for local interrupt to "1" to specify that interrupts > belong to group1. This is particularly

Re: [Xen-devel] [PATCH RFC 14/14] xen: tools: Added xen-subpage tool.

2017-10-20 Thread Yi Zhang
On 2017-10-19 at 11:42:17 +0300, Razvan Cojocaru wrote: > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > + > > +#include > > + > > +#define DPRINTF(a, b...) fprintf(stderr, a, ##

Re: [Xen-devel] [PATCH RFC 00/14] Intel EPT-Based Sub-page Write Protection Support.

2017-10-20 Thread Razvan Cojocaru
On 20.10.2017 11:37, Yi Zhang wrote: > On 2017-10-19 at 12:07:44 +0300, Razvan Cojocaru wrote: >> On 19.10.2017 11:04, Zhang Yi wrote: >>> From: Zhang Yi Z >>> >>> Hi All, >>> >>> Here is a patch-series which adding EPT-Based Sub-page Write Protection >>> Support. You can get It's software develo

Re: [Xen-devel] [PATCH RFC 00/14] Intel EPT-Based Sub-page Write Protection Support.

2017-10-20 Thread Razvan Cojocaru
On 20.10.2017 11:37, Yi Zhang wrote: > On 2017-10-19 at 12:07:44 +0300, Razvan Cojocaru wrote: >> On 19.10.2017 11:04, Zhang Yi wrote: >>> From: Zhang Yi Z >>> >>> Hi All, >>> >>> Here is a patch-series which adding EPT-Based Sub-page Write Protection >>> Support. You can get It's software develo

Re: [Xen-devel] [PATCH RFC 09/14] xen: vmx: Introduce a Hyper call to set subpage

2017-10-20 Thread Yi Zhang
On 2017-10-19 at 12:34:07 -0600, Tamas K Lengyel wrote: > On Thu, Oct 19, 2017 at 2:13 AM, Zhang Yi wrote: > > From: Zhang Yi Z > > > > The Hypercall is defined as HVMOP_set_subpage > > Are there any expected use-cases where a HVM guest would need access > to this hypercall? Is spp compatible wi

Re: [Xen-devel] [PATCH RFC 13/14] xen: tools: Introduce the set-subpage into xenctrl

2017-10-20 Thread Yi Zhang
On 2017-10-19 at 11:37:25 +0300, Razvan Cojocaru wrote: > Would you consider a small comment here explaining at least the access > parameter? Yes, Thanks Rezvan, will add some explanation. > > > Thanks, > Razvan ___ Xen-devel mailing list Xen-devel@l

Re: [Xen-devel] [PATCH RFC 08/14] xen: vmx: Added setup spp page structure.

2017-10-20 Thread Yi Zhang
On 2017-10-19 at 12:26:32 -0600, Tamas K Lengyel wrote: > > So nothing in this patch makes use of this function. Could you please > re-organize the patchset so this is included with the patch that > starts using it? > Yes, will follow up, Thanks for your review. Tamas. _

Re: [Xen-devel] [PATCH V3 11/29] x86/hvm: Introduce a emulated VTD for HVM

2017-10-20 Thread Lan Tianyu
On 2017年10月20日 14:56, Jan Beulich wrote: On 20.10.17 at 04:46, wrote: >> On Thu, Oct 19, 2017 at 12:20:35PM +0100, Roger Pau Monné wrote: >>> On Thu, Sep 21, 2017 at 11:01:52PM -0400, Lan Tianyu wrote: From: Chao Gao This patch adds create/destroy function for the emulated VTD

Re: [Xen-devel] [PATCH RFC 05/14] xen: vmx: Disable the 2M/1G superpage when SPP enabled

2017-10-20 Thread Yi Zhang
On 2017-10-19 at 12:17:12 -0600, Tamas K Lengyel wrote: > On Thu, Oct 19, 2017 at 2:11 AM, Zhang Yi wrote: > > From: Zhang Yi Z > > > > Current we only support Sub-page Protection on the 4k > > page table. > > > > Signed-off-by: Zhang Yi Z > > --- > > xen/arch/x86/hvm/vmx/vmx.c | 6 ++ > >

[Xen-devel] [PATCH v9 00/16] Enable Memory Bandwidth Allocation in Xen

2017-10-20 Thread Yi Sun
Hi, all, We plan to bring a new PSR (Platform Shared Resource) feature called Intel Memory Bandwidth Allocation (MBA) to Xen. Besides the MBA enabling, we change some interfaces to make them more general but not only for CAT. Any comments are welcome! You can find this series at: https://github

[Xen-devel] [PATCH v9 09/16] tools: create general interfaces to support psr allocation features

2017-10-20 Thread Yi Sun
This patch creates general interfaces in libxl to support all psr allocation features. Add 'LIBXL_HAVE_PSR_GENERIC' to indicate interface change. Please note, the functionality cannot work until later patches are applied. Signed-off-by: Yi Sun Reviewed-by: Roger Pau Monné Acked-by: Wei Liu --

[Xen-devel] [PATCH v9 08/16] x86: implement set value flow for MBA

2017-10-20 Thread Yi Sun
This patch implements set value flow for MBA including its callback function and domctl interface. Signed-off-by: Yi Sun Reviewed-by: Jan Beulich --- CC: Jan Beulich CC: Andrew Cooper CC: Wei Liu CC: Roger Pau Monné CC: Chao Peng v9: - adjust codes in 'mba_sanitize_thrtl'. (sugge

[Xen-devel] [PATCH v9 12/16] tools: implement the new xl get hw info interface

2017-10-20 Thread Yi Sun
This patch implements a new xl get HW info interface. A new argument is added for psr-hwinfo command to get and show MBA HW info. Signed-off-by: Yi Sun Reviewed-by: Roger Pau Monné Acked-by: Wei Liu --- CC: Wei Liu CC: Ian Jackson CC: Roger Pau Monné CC: Chao Peng v3: - change the form

[Xen-devel] [PATCH v9 16/16] docs: add MBA description in docs

2017-10-20 Thread Yi Sun
This patch adds MBA description in related documents. Signed-off-by: Yi Sun Acked-by: Wei Liu Reviewed-by: Roger Pau Monné --- CC: Ian Jackson CC: Wei Liu CC: Roger Pau Monné CC: Chao Peng v5: - remove 'closed-loop' in 'xl-psr.markdown' (suggested by Roger Pau Monné) v4: - mo

[Xen-devel] [PATCH v9 02/16] Rename PSR sysctl/domctl interfaces and xsm policy to make them be general

2017-10-20 Thread Yi Sun
This patch renames PSR sysctl/domctl interfaces and related xsm policy to make them be general for all resource allocation features but not only for CAT. Then, we can resuse the interfaces for all allocation features. Basically, it changes 'psr_cat_op' to 'psr_alloc', and remove 'CAT_' from some m

[Xen-devel] [PATCH v9 11/16] tools: implement the new libxl get hw info interface

2017-10-20 Thread Yi Sun
This patch implements the new libxl get hw info interface, 'libxl_psr_get_hw_info', which is suitable to all psr allocation features. It also implements corresponding list free function, 'libxl_psr_hw_info_list_free' and makes 'libxl_psr_cat_get_info' call 'libxl_psr_get_hw_info' to avoid redundant

[Xen-devel] [PATCH v9 04/16] x86: a few optimizations to psr codes

2017-10-20 Thread Yi Sun
This patch refines psr codes: 1. Change type of 'cat_init_feature' to 'bool' to remove the pointless returning of error code. 2. Move printk in 'cat_init_feature' to reduce a return path. 3. Define a local variable 'feat_mask' in 'psr_cpu_init' to reduce calling of 'cpuid_count_leaf()'. 4. Ch

[Xen-devel] [PATCH v9 01/16] docs: create Memory Bandwidth Allocation (MBA) feature document

2017-10-20 Thread Yi Sun
This patch creates MBA feature document in doc/features/. It describes key points to implement MBA which is described in details in Intel SDM "Introduction to Memory Bandwidth Allocation". Signed-off-by: Yi Sun Reviewed-by: Roger Pau Monné --- CC: Jan Beulich CC: Andrew Cooper CC: Wei Liu CC:

[Xen-devel] [PATCH v9 03/16] x86: rename 'cbm_type' to 'psr_type' to make it general

2017-10-20 Thread Yi Sun
This patch renames 'cbm_type' to 'psr_type' to generalize it. Then, we can reuse this for all psr allocation features. Signed-off-by: Yi Sun Reviewed-by: Wei Liu Reviewed-by: Roger Pau Monné Acked-by: Jan Beulich --- CC: Jan Beulich CC: Andrew Cooper CC: Wei Liu CC: Roger Pau Monné CC: Cha

[Xen-devel] [PATCH v9 15/16] tools: implement new generic set value interface and MBA set value command

2017-10-20 Thread Yi Sun
This patch implements new generic set value interfaces in libxc and libxl. These interfaces are suitable for all allocation features. It also adds a new MBA set value command in xl. Signed-off-by: Yi Sun Reviewed-by: Roger Pau Monné Acked-by: Wei Liu --- CC: Wei Liu CC: Ian Jackson CC: Roger

[Xen-devel] [PATCH v9 13/16] tools: rename 'xc_psr_cat_type' to 'xc_psr_type'

2017-10-20 Thread Yi Sun
This patch renames 'xc_psr_cat_type' to 'xc_psr_type' so that the structure name is common for all allocation features. Signed-off-by: Yi Sun Acked-by: Wei Liu Reviewed-by: Chao Peng Reviewed-by: Roger Pau Monné --- CC: Ian Jackson CC: Wei Liu CC: Roger Pau Monné CC: Chao Peng v5: - r

[Xen-devel] [PATCH v9 14/16] tools: implement new generic get value interface and MBA get value command

2017-10-20 Thread Yi Sun
This patch implements generic get value interfaces in libxc and libxl. It also refactors the get value flow in xl to make it be suitable for all allocation features. Based on that, a new MBA get value command is added in xl. Signed-off-by: Yi Sun Acked-by: Wei Liu Reviewed-by: Roger Pau Monné -

[Xen-devel] [PATCH v9 10/16] tools: implement the new libxc get hw info interface

2017-10-20 Thread Yi Sun
This patch implements a new libxc get hw info interface and corresponding data structures. It also changes libxl_psr.c to call this new interface. Signed-off-by: Yi Sun Reviewed-by: Roger Pau Monné Acked-by: Wei Liu --- CC: Wei Liu CC: Ian Jackson CC: Roger Pau Monné CC: Chao Peng v6:

[Xen-devel] [PATCH v9 05/16] x86: implement data structure and CPU init flow for MBA

2017-10-20 Thread Yi Sun
This patch implements main data structures of MBA. Like CAT features, MBA HW info has cos_max which means the max thrtl register number, and thrtl_max which means the max throttle value (delay value). It also has a flag to represent if the throttle value is linear or non-linear. One thrtl registe

[Xen-devel] [PATCH v9 07/16] x86: implement get value interface for MBA

2017-10-20 Thread Yi Sun
This patch implements get value domctl interface for MBA. Signed-off-by: Yi Sun Reviewed-by: Roger Pau Monné Acked-by: Jan Beulich --- CC: Andrew Cooper CC: Jan Beulich CC: Wei Liu CC: Roger Pau Monné CC: Chao Peng v5: - use newly defined macro to get MBA thrtl. (suggested by Ro

[Xen-devel] [PATCH v9 06/16] x86: implement get hw info flow for MBA

2017-10-20 Thread Yi Sun
This patch implements get HW info flow for MBA including its callback function and sysctl interface. Signed-off-by: Yi Sun Reviewed-by: Roger Pau Monné Acked-by: Jan Beulich --- CC: Jan Beulich CC: Andrew Cooper CC: Wei Liu CC: Roger Pau Monné CC: Chao Peng v7: - change 'PSR_INFO_IDX_

Re: [Xen-devel] [PATCH v1 1/5] xen/mm: Make xenmem_add_to_physmap global

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Ross Lagerwall > Sent: 18 October 2017 15:04 > To: Xen-devel > Cc: Stefano Stabellini ; Wei Liu > ; Konrad Rzeszutek Wilk ; > George Dunlap ; Andrew Cooper > ; Ian Jackson ; Tim > (Xen.org) ; Ros

Re: [Xen-devel] [PATCH v1 2/5] xen: Provide XEN_DMOP_add_to_physmap

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Ross Lagerwall > Sent: 18 October 2017 15:04 > To: Xen-devel > Cc: Stefano Stabellini ; Wei Liu > ; Konrad Rzeszutek Wilk ; > George Dunlap ; Andrew Cooper > ; Ian Jackson ; Tim > (Xen.org) ; Ros

Re: [Xen-devel] [PATCH v1 3/5] xen: Provide XEN_DMOP_pin_memory_cacheattr

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Ross Lagerwall > Sent: 18 October 2017 15:04 > To: Xen-devel > Cc: Stefano Stabellini ; Wei Liu > ; Konrad Rzeszutek Wilk ; > George Dunlap ; Andrew Cooper > ; Ian Jackson ; Tim > (Xen.org) ; Ros

Re: [Xen-devel] [PATCH v1 2/5] xen: Provide XEN_DMOP_add_to_physmap

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Paul Durrant > Sent: 20 October 2017 10:16 > To: Ross Lagerwall ; Xen-devel de...@lists.xen.org> > Cc: Stefano Stabellini ; Wei Liu > ; Konrad Rzeszutek Wilk ; > Andrew Cooper ; Tim (Xen.org) > ;

Re: [Xen-devel] [PATCH v1 4/5] tools: libxendevicemodel: Provide xendevicemodel_add_to_physmap

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Ross Lagerwall > Sent: 18 October 2017 15:04 > To: Xen-devel > Cc: Ross Lagerwall ; Ian Jackson > ; Wei Liu > Subject: [Xen-devel] [PATCH v1 4/5] tools: libxendevicemodel: Provide > xendevicemod

Re: [Xen-devel] [PATCH v1 5/5] tools: libxendevicemodel: Provide xendevicemodel_pin_memory_cacheattr

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Ross Lagerwall > Sent: 18 October 2017 15:04 > To: Xen-devel > Cc: Ross Lagerwall ; Ian Jackson > ; Wei Liu > Subject: [Xen-devel] [PATCH v1 5/5] tools: libxendevicemodel: Provide > xendevicemod

Re: [Xen-devel] [PATCH v7 for-next 01/12] x86/pio: allow internal PIO handlers to return RETRY

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Roger Pau Monne [mailto:roger@citrix.com] > Sent: 18 October 2017 12:40 > To: xen-de...@lists.xenproject.org > Cc: konrad.w...@oracle.com; boris.ostrov...@oracle.com; Roger Pau Monne > ; Paul Durrant ; Jan > Beulich ; Andrew Cooper > > Subject: [PATCH v7 fo

Re: [Xen-devel] [PATCH v7 for-next 03/12] vpci: introduce basic handlers to trap accesses to the PCI config space

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Roger Pau Monne [mailto:roger@citrix.com] > Sent: 18 October 2017 12:40 > To: xen-de...@lists.xenproject.org > Cc: konrad.w...@oracle.com; boris.ostrov...@oracle.com; Roger Pau Monne > ; Ian Jackson ; Wei Liu > ; Jan Beulich ; Andrew Cooper > ; Paul Durrant

Re: [Xen-devel] [PATCH v1 2/5] xen: Provide XEN_DMOP_add_to_physmap

2017-10-20 Thread Ross Lagerwall
On 10/20/2017 10:15 AM, Paul Durrant wrote: -Original Message- snip>> diff --git a/xen/arch/x86/hvm/dm.c b/xen/arch/x86/hvm/dm.c index 32ade95..432a863 100644 --- a/xen/arch/x86/hvm/dm.c +++ b/xen/arch/x86/hvm/dm.c @@ -640,6 +640,22 @@ static int dm_op(const struct dmop_args *op_args)

[Xen-devel] [GIT PULL] xen: fix for 4.14-rc6

2017-10-20 Thread Juergen Gross
Linus, Please git pull the following tag: git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git for-linus-4.14c-rc6-tag xen: fix for 4.14-rc6 It contains a fix for the Xen pv network drivers (frontend and backend) avoiding the network connection to become unusable due to an illegal MTU.

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

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

Re: [Xen-devel] [PATCH v7 for-next 04/12] x86/mmcfg: add handlers for the PVH Dom0 MMCFG areas

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Roger Pau Monne [mailto:roger@citrix.com] > Sent: 18 October 2017 12:40 > To: xen-de...@lists.xenproject.org > Cc: konrad.w...@oracle.com; boris.ostrov...@oracle.com; Roger Pau Monne > ; Jan Beulich ; Andrew Cooper > ; Paul Durrant > Subject: [PATCH v7 for-

Re: [Xen-devel] [PATCH V3 12/29] x86/vvtd: Add MMIO handler for VVTD

2017-10-20 Thread Roger Pau Monné
On Fri, Oct 20, 2017 at 10:58:32AM +0800, Chao Gao wrote: > On Thu, Oct 19, 2017 at 12:34:54PM +0100, Roger Pau Monné wrote: > >On Thu, Sep 21, 2017 at 11:01:53PM -0400, Lan Tianyu wrote: > >> +return 0; > >> +} > >> + > >> +static int vvtd_read(struct vcpu *v, unsigned long addr, > >> +

Re: [Xen-devel] [PATCH] tools/Makefile: unset MAKELEVEL before building QEMU

2017-10-20 Thread Julien Grall
Hi, On 19/10/17 16:19, Ian Jackson wrote: Anthony PERARD writes ("[PATCH] tools/Makefile: unset MAKELEVEL before building QEMU"): Since QEMU commits aef45d51d1204f3335fb99de6658e0c5612c2b67 "build: automatically handle GIT submodule checkout for dtc" the QEMU makefiles rely on the variable MAK

Re: [Xen-devel] [PATCH v7 for-next 10/12] vpci/msi: add MSI handlers

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Roger Pau Monne [mailto:roger@citrix.com] > Sent: 18 October 2017 12:41 > To: xen-de...@lists.xenproject.org > Cc: konrad.w...@oracle.com; boris.ostrov...@oracle.com; Roger Pau Monne > ; Jan Beulich ; Andrew Cooper > ; Paul Durrant > Subject: [PATCH v7 for-

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-20 Thread Julien Grall
Hi Paul, On 20/10/17 09:26, Paul Durrant wrote: -Original Message- From: Jan Beulich [mailto:jbeul...@suse.com] Sent: 20 October 2017 07:25 To: Julien Grall Cc: Julien Grall ; Andrew Cooper ; George Dunlap ; Ian Jackson ; Paul Durrant ; Roger Pau Monne ; Wei Liu ; Stefano Stabellini ; x

Re: [Xen-devel] [PATCH V3 15/29] x86/vvtd: Process interrupt remapping request

2017-10-20 Thread Roger Pau Monné
On Fri, Oct 20, 2017 at 01:16:37PM +0800, Chao Gao wrote: > On Thu, Oct 19, 2017 at 03:26:30PM +0100, Roger Pau Monné wrote: > >On Thu, Sep 21, 2017 at 11:01:56PM -0400, Lan Tianyu wrote: > >> +static void unmap_guest_page(void *virt) > >> +{ > >> +struct page_info *page; > >> + > >> +ASSER

Re: [Xen-devel] [PATCH v1 2/5] xen: Provide XEN_DMOP_add_to_physmap

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Ross Lagerwall [mailto:ross.lagerw...@citrix.com] > Sent: 20 October 2017 10:37 > To: Paul Durrant ; Xen-devel de...@lists.xen.org> > Cc: Stefano Stabellini ; Wei Liu > ; Konrad Rzeszutek Wilk ; > George Dunlap ; Andrew Cooper > ; Ian Jackson ; Tim > (Xen.org

Re: [Xen-devel] [PATCH for-4.10] libxl: annotate s to be nonnull in libxl__enum_from_string

2017-10-20 Thread Julien Grall
Hi, On 16/10/17 15:04, Wei Liu wrote: Hope this can placate coverity. Signed-off-by: Wei Liu --- Cc: Ian Jackson Cc: Julien Grall Release-acked-by: Julien Grall Cheers, --- tools/libxl/libxl_internal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/libxl/l

Re: [Xen-devel] [PATCH V3 26/29] x86/vvtd: Handle interrupt translation faults

2017-10-20 Thread Roger Pau Monné
On Fri, Oct 20, 2017 at 01:54:15PM +0800, Chao Gao wrote: > On Thu, Oct 19, 2017 at 05:31:37PM +0100, Roger Pau Monné wrote: > >On Thu, Sep 21, 2017 at 11:02:07PM -0400, Lan Tianyu wrote: > >> +static int vvtd_alloc_frcd(struct vvtd *vvtd) > >> +{ > >> +int prev; > >> +uint64_t cap = vvtd_g

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-20 Thread Paul Durrant
> -Original Message- > From: Julien Grall [mailto:julien.gr...@linaro.org] > Sent: 20 October 2017 11:00 > To: Paul Durrant ; 'Jan Beulich' > > Cc: Julien Grall ; Andrew Cooper > ; George Dunlap > ; Ian Jackson ; Roger > Pau Monne ; Wei Liu ; Stefano > Stabellini ; xen-de...@lists.xenproje

Re: [Xen-devel] [PATCH for-4.10] string: fix memmove when size is 0

2017-10-20 Thread Roger Pau Monné
On Fri, Oct 20, 2017 at 01:17:40AM -0600, Jan Beulich wrote: > >>> On 17.10.17 at 14:03, wrote: > > --- a/xen/arch/x86/string.c > > +++ b/xen/arch/x86/string.c > > @@ -39,6 +39,9 @@ void *(memmove)(void *dest, const void *src, size_t n) > > { > > long d0, d1, d2; > > > > +if ( !n ) > >

[Xen-devel] [xen-4.9-testing test] 114733: regressions - FAIL

2017-10-20 Thread osstest service owner
flight 114733 xen-4.9-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/114733/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt-raw broken in 114673 build-armhf-pvops

Re: [Xen-devel] [PATCH 1/8] xen: link against xentoolcore

2017-10-20 Thread Ian Jackson
Anthony PERARD writes ("Re: [PATCH 1/8] xen: link against xentoolcore"): > On Thu, Oct 19, 2017 at 05:38:10PM +0100, Ian Jackson wrote: > > We still need this patch for the rest of the series, though. > > Of course, I was only arguing that this patch on its own is not usefull. Right. I wanted to

Re: [Xen-devel] runstatedir matters when using debhelper tools in Artful

2017-10-20 Thread Ian Jackson
Wei Liu writes ("Re: runstatedir matters when using debhelper tools in Artful"): > On Thu, Oct 19, 2017 at 02:49:06PM +, Mark Pryor wrote: > > Hello, > > When finishing a debianised build of xen-4.10~rc1 in Artful I had no extra > > patches, so I left outthis patch below that I used to build x

Re: [Xen-devel] [PATCH V3 27/29] x86/vvtd: Enable Queued Invalidation through GCMD

2017-10-20 Thread Roger Pau Monné
On Thu, Sep 21, 2017 at 11:02:08PM -0400, Lan Tianyu wrote: > From: Chao Gao > > Software writes to QIE field of GCMD to enable or disable queued > invalidations. This patch emulates QIE field of GCMD. > > Signed-off-by: Chao Gao > Signed-off-by: Lan Tianyu > --- > xen/drivers/passthrough/vtd

Re: [Xen-devel] [Qemu-devel] [PATCH v5 0/8] xen: xen-domid-restrict improvements

2017-10-20 Thread Ian Jackson
no-re...@patchew.org writes ("Re: [Qemu-devel] [PATCH v5 0/8] xen: xen-domid-restrict improvements"): > This series seems to have some coding style problems. See output below for > more information: No, it doesn't have coding style problems. At least, this mail contains only one complaint which

[Xen-devel] [OSSTEST PATCH 07/16] Debian.pm: use sysvinit-core on stretch

2017-10-20 Thread Wei Liu
Signed-off-by: Wei Liu --- Osstest/Debian.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 845027a..24bc260 100644 --- a/Osstest/Debian.pm +++ b/Osstest/Debian.pm @@ -827,7 +827,7 @@ sub preseed_base ($$$;@) { # Systemd does

[Xen-devel] [OSSTEST PATCH 02/16] gitignore: ignore vim swap file

2017-10-20 Thread Wei Liu
Signed-off-by: Wei Liu --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 425506b..f7e5b77 100644 --- a/.gitignore +++ b/.gitignore @@ -1,5 +1,6 @@ *~ *.bak +*.swp tmp *.tmp bisection.ps -- 2.11.0 __

[Xen-devel] [OSSTEST PATCH 09/16] ts-host-install: don't use the new nic naming scheme

2017-10-20 Thread Wei Liu
Signed-off-by: Wei Liu --- ts-host-install | 4 1 file changed, 4 insertions(+) diff --git a/ts-host-install b/ts-host-install index 20b8a60..530bc5c 100755 --- a/ts-host-install +++ b/ts-host-install @@ -271,6 +271,10 @@ END # why this is repeated. push @hocmdline, "console=$cons

[Xen-devel] [OSSTEST PATCH 08/16] ts-leak-check: suppress systemd-shim, which leaks in stretch

2017-10-20 Thread Wei Liu
Signed-off-by: Wei Liu --- ts-leak-check | 1 + 1 file changed, 1 insertion(+) diff --git a/ts-leak-check b/ts-leak-check index 678d069..41e6245 100755 --- a/ts-leak-check +++ b/ts-leak-check @@ -202,6 +202,7 @@ xenstore /vm xenstore /libxl process .* udevd +process .* /.+/systemd-shim fi

[Xen-devel] [OSSTEST PATCH 03/16] ts-xen-build-prep: don't install w3c-dtd-xhtml for Stretch onward

2017-10-20 Thread Wei Liu
It is not included in Stretch. That package was installed because libvirt build needed it. However libvirt builds fine without it in Stretch. It should be fine to just not install it on Stretch. Signed-off-by: Wei Liu --- ts-xen-build-prep | 5 - 1 file changed, 4 insertions(+), 1 deletion(

[Xen-devel] [OSSTEST PATCH 04/16] ts-xen-build-prep: install packages for stretch

2017-10-20 Thread Wei Liu
Stubdom build needs texinfo. Libvirt build needs autopoint. Signed-off-by: Wei Liu --- ts-xen-build-prep | 4 1 file changed, 4 insertions(+) diff --git a/ts-xen-build-prep b/ts-xen-build-prep index 6152d0a..70a53ca 100755 --- a/ts-xen-build-prep +++ b/ts-xen-build-prep @@ -221,6 +221,10

[Xen-devel] [OSSTEST PATCH 01/16] XXX add a stretch config based on production-config

2017-10-20 Thread Wei Liu
diff -ub production-config production-config-stretch --- production-config 2017-10-11 15:01:16.891770210 +0100 +++ production-config-stretch 2017-10-19 16:12:20.839743993 +0100 @@ -21,6 +21,9 @@ JobDB Executive HostDB Executive +DebianSuite stretch +GuestDebianSuite stretch + HostnameSortS

[Xen-devel] [OSSTEST PATCH 05/16] mg-debian-installer-update-all: put quotes around arguments

2017-10-20 Thread Wei Liu
Signed-off-by: Wei Liu --- mg-debian-installer-update | 2 +- mg-debian-installer-update-all | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/mg-debian-installer-update b/mg-debian-installer-update index 37cec58..dc807a2 100755 --- a/mg-debian-installer-update +++ b/mg-d

[Xen-devel] [OSSTEST PATCH 00/16] Upgrade to Stretch

2017-10-20 Thread Wei Liu
Hi all This series upgrades osstest to Debian Strech, which aims to: 1. Keep osstest up-to-date. 2. Enable us to collect aggregate gcov data (depends on newer gcc in Stretch). 3. Enable us to run (yet to be implemented) hypervisor fuzzer in the (distant) future (depends on newer gcc in Stretch

[Xen-devel] [OSSTEST PATCH 06/16] ts-xen-install: install some packages on stretch

2017-10-20 Thread Wei Liu
The "route" command is now in that package. libnl is needed when running xl. Signed-off-by: Wei Liu --- ts-xen-install | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ts-xen-install b/ts-xen-install index ec907c5..d4c25c7 100755 --- a/ts-xen-install +++ b/ts-xen-install @@ -57,6 +57,9 @@

Re: [Xen-devel] [PATCH for-4.10] libxl: annotate s to be nonnull in libxl__enum_from_string

2017-10-20 Thread Ian Jackson
Julien Grall writes ("Re: [Xen-devel] [PATCH for-4.10] libxl: annotate s to be nonnull in libxl__enum_from_string"): > Release-acked-by: Julien Grall Thanks, I have applied this. Not sure whether this followup is 4.10 material, but IMO it is if we would otherwise want to add another open-coded

Re: [Xen-devel] [OSSTEST PATCH 03/16] ts-xen-build-prep: don't install w3c-dtd-xhtml for Stretch onward

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 03/16] ts-xen-build-prep: don't install w3c-dtd-xhtml for Stretch onward"): > It is not included in Stretch. > > That package was installed because libvirt build needed it. However > libvirt builds fine without it in Stretch. It should be fine to just not > install

Re: [Xen-devel] [OSSTEST PATCH 04/16] ts-xen-build-prep: install packages for stretch

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 04/16] ts-xen-build-prep: install packages for stretch"): > Stubdom build needs texinfo. Same comment as my previous patch. You should only mention old release names in these kind of tests, unless you know that the requirement is specific to only stretch and not f

[Xen-devel] [OSSTEST PATCH 02/16] gitignore: ignore vim swap file

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 02/16] gitignore: ignore vim swap file"): > Signed-off-by: Wei Liu Acked-by: Ian Jackson Although, you may find your life improved by putting this in your ~/.config/git/ignore. Ian. ___ Xen-devel mailing list Xen-deve

Re: [Xen-devel] [OSSTEST PATCH 01/16] XXX add a stretch config based on production-config

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 01/16] XXX add a stretch config based on production-config"): > diff -ub production-config production-config-stretch The changes LGTM but obviously this ought to go straight into `production-config'. Ian. ___ Xen-devel

Re: [Xen-devel] [OSSTEST PATCH 05/16] mg-debian-installer-update-all: put quotes around arguments

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 05/16] mg-debian-installer-update-all: put quotes around arguments"): > Signed-off-by: Wei Liu ... > suite=$1 > arch=$2 > -packages="$3" > +packages=$3 Not sure why this needless style change, but if you did it deliberately I don't really mind... > site=http:/

[Xen-devel] [OSSTEST PATCH 16/16] ts-guests-nbd-mirror: make it work with stretch

2017-10-20 Thread Wei Liu
On the server side, only add oldstyle= and port= on jessie. Stretch doesn't support or need those anymore. Prune check for older versions of Debian. On the client side, generate new style configuration file. Signed-off-by: Wei Liu --- ts-guests-nbd-mirror | 47 +

Re: [Xen-devel] [OSSTEST PATCH 08/16] ts-leak-check: suppress systemd-shim, which leaks in stretch

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 08/16] ts-leak-check: suppress systemd-shim, which leaks in stretch"): > Signed-off-by: Wei Liu Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

[Xen-devel] [OSSTEST PATCH 10/16] ts-debian-fixup: remove extra= before appending our own

2017-10-20 Thread Wei Liu
The original extra= was not removed, so there were two extra= in the resulting config file. It wasn't a problem for xl because the second extra= took precedence. However libvirt tests would only pick up the first extra= -- they worked by chance. Fix this issue by removing the first extra=. Sign

Re: [Xen-devel] [OSSTEST PATCH 07/16] Debian.pm: use sysvinit-core on stretch

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 07/16] Debian.pm: use sysvinit-core on stretch"): ... > diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm > index 845027a..24bc260 100644 > --- a/Osstest/Debian.pm > +++ b/Osstest/Debian.pm > @@ -827,7 +827,7 @@ sub preseed_base ($$$;@) { > > # Systemd doesn

[Xen-devel] [OSSTEST PATCH 13/16] ts-debian-hvm-install: disable new nic naming scheme

2017-10-20 Thread Wei Liu
This is required to fix nested hvm test. The L1 host is installed by this script. We want the L1 host to not use the new nic naming scheme. Signed-off-by: Wei Liu --- ts-debian-hvm-install | 12 1 file changed, 12 insertions(+) diff --git a/ts-debian-hvm-install b/ts-debian-hvm-ins

[Xen-devel] [OSSTEST PATCH 14/16] ts-xen-build-prep: install e2fslibs-dev

2017-10-20 Thread Wei Liu
The in-tree libfsimage ext2fs implementation can't handle 64bit enabled ext4, which is the default in stretch. Signed-off-by: Wei Liu --- ts-xen-build-prep | 6 ++ 1 file changed, 6 insertions(+) diff --git a/ts-xen-build-prep b/ts-xen-build-prep index 70a53ca..4224601 100755 --- a/ts-xen-b

Re: [Xen-devel] [OSSTEST PATCH 09/16] ts-host-install: don't use the new nic naming scheme

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 09/16] ts-host-install: don't use the new nic naming scheme"): > Signed-off-by: Wei Liu Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

[Xen-devel] [OSSTEST PATCH 12/16] make-flight: don't test pvgrub for Xen XXX

2017-10-20 Thread Wei Liu
XXX Need to pin down the version of Xen when the upgrade to stretch is complete because osstest configuration is branched for each version. Signed-off-by: Wei Liu --- make-flight | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/make-flight b/make-flight index

[Xen-devel] [OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume device

2017-10-20 Thread Wei Liu
See code comment for explanation. Signed-off-by: Wei Liu --- ts-debian-fixup | 12 1 file changed, 12 insertions(+) diff --git a/ts-debian-fixup b/ts-debian-fixup index d476467..fe37c30 100755 --- a/ts-debian-fixup +++ b/ts-debian-fixup @@ -176,6 +176,18 @@ sub otherfixupcfg () {

[Xen-devel] [OSSTEST PATCH 15/16] TestSupport: add dpkg option when installing packages

2017-10-20 Thread Wei Liu
Upgrading configuration file of nbd-client is controlled by dpkg in stretch. Add dpkg option to keep old configuration file(s). Signed-off-by: Wei Liu --- Osstest/TestSupport.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm

Re: [Xen-devel] [OSSTEST PATCH 13/16] ts-debian-hvm-install: disable new nic naming scheme

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 13/16] ts-debian-hvm-install: disable new nic naming scheme"): > This is required to fix nested hvm test. The L1 host is installed by > this script. We want the L1 host to not use the new nic naming scheme. The principle is fine. > +# Do not use "Predictable N

Re: [Xen-devel] [OSSTEST PATCH 10/16] ts-debian-fixup: remove extra= before appending our own

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 10/16] ts-debian-fixup: remove extra= before appending our own"): > The original extra= was not removed, so there were two extra= in the > resulting config file. What is the original extra= ? Why should we not combine them ? > It wasn't a problem for xl because t

[Xen-devel] [distros-debian-jessie test] 72334: tolerable trouble: blocked/broken/pass

2017-10-20 Thread Platform Team regression test user
flight 72334 distros-debian-jessie real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72334/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-armhf-jessie-netboot-pygrub 1 build-check(1) blocked n/a build-arm64-pvops

Re: [Xen-devel] [OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume device

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume device"): > See code comment for explanation. ... > +# There might be stale entries in /etc/initramfs-tools/conf.d/resume > +# which get stored in the initramfs. That introduces delay in guest > booting > +# wh

[Xen-devel] [PATCH for-4.10] Config.mk: update mini-os changeset

2017-10-20 Thread Wei Liu
The new changeset contains the new console.h fix in xen.git. Signed-off-by: Wei Liu --- Cc: Julien Grall This is rather low risk because stubdom build in xen.git uses xen headers directly. I just don't want to ship a version of xen which points to a buggy mini-os changeset. --- Config.mk | 6

Re: [Xen-devel] [OSSTEST PATCH 16/16] ts-guests-nbd-mirror: make it work with stretch

2017-10-20 Thread Ian Jackson
Wei Liu writes ("[OSSTEST PATCH 16/16] ts-guests-nbd-mirror: make it work with stretch"): > On the server side, only add oldstyle= and port= on jessie. Stretch > doesn't support or need those anymore. See my earlier comments about old vs new Debian suite names. > Prune check for older versions

  1   2   3   >