Re: [Xen-devel] [PATCH v2 4/4] dt-uart: support /chosen/stdout-path property.

2015-01-12 Thread Ian Campbell
On Thu, 2015-01-08 at 13:30 +, Julien Grall wrote: > On 08/01/15 13:22, Ian Campbell wrote: > > On Thu, 2015-01-08 at 13:15 +, Julien Grall wrote: > >> Hi Ian, > >> > >> On 08/01/15 11:53, Ian Campbell wrote: > >>> +ret = dt_property_read_string(chosen, "stdout-path", > >>> &st

Re: [Xen-devel] [PATCH v2] xenstored: log tdb message via xenstored's logging mechanisms

2015-01-12 Thread Wei Liu
On Mon, Jan 12, 2015 at 03:18:02PM +, Ian Campbell wrote: > TDB provides us with a callback for this purpose. Use it in both > xenstored and xs_tdb_dump. > > While at it make the existing log() macro tollerate memory failures. > > Signed-off-by: Ian Campbell Acked-by: Wei Liu

Re: [Xen-devel] [v3 5/5] Qemu-Xen-vTPM: QEMU machine class is initialized before tpm_init()

2015-01-12 Thread Paolo Bonzini
On 31/12/2014 00:03, Quan Xu wrote: > make sure QEMU machine class is initialized and QEMU has registered > Xen stubdom vTPM driver when call tpm_init() > > Signed-off-by: Quan Xu > --- > vl.c | 16 ++-- > 1 file changed, 10 insertions(+), 6 deletions(-) > > diff --git a/vl.c b/vl

Re: [Xen-devel] [PATCH v2 4/4] dt-uart: support /chosen/stdout-path property.

2015-01-12 Thread Julien Grall
Hi Ian, On 12/01/15 15:22, Ian Campbell wrote: > On Thu, 2015-01-08 at 13:30 +, Julien Grall wrote: >> On 08/01/15 13:22, Ian Campbell wrote: >>> On Thu, 2015-01-08 at 13:15 +, Julien Grall wrote: Hi Ian, On 08/01/15 11:53, Ian Campbell wrote: > +ret = dt_pro

Re: [Xen-devel] [PATCH v4 2/2] gnttab: refactor locking for scalability

2015-01-12 Thread Jan Beulich
>>> On 09.01.15 at 16:12, wrote: > @@ -188,6 +191,26 @@ nr_active_grant_frames(struct grant_table *gt) > return num_act_frames_from_sha_frames(nr_grant_frames(gt)); > } > > +static inline struct active_grant_entry * > +active_entry_acquire(struct grant_table *t, grant_ref_t e) > +{ > +

Re: [Xen-devel] [PATCH v4 1/2] gnttab: Introduce rwlock to protect updates to grant table state

2015-01-12 Thread Jan Beulich
>>> On 09.01.15 at 16:12, wrote: > @@ -899,26 +899,27 @@ __gnttab_unmap_common( > > op->frame = (unsigned long)(op->dev_bus_addr >> PAGE_SHIFT); > > +read_lock(&lgt->lock); > if ( unlikely(op->handle >= lgt->maptrack_limit) ) > { > +read_unlock(&lgt->lock); >

Re: [Xen-devel] [libvirt] [PATCH 10/12] Introduce support for parsing/formatting Xen xl config format

2015-01-12 Thread John Ferlan
On 01/10/2015 12:03 AM, Jim Fehlig wrote: > Introduce a parser/formatter for the xl config format. Since the > deprecation of xm/xend, the VM config file format has diverged as > new features are added to libxl. This patch adds support for parsing > and formating the xl config format. It suppo

Re: [Xen-devel] [PATCH v2 1/3] x86: also allow REP STOS emulation acceleration

2015-01-12 Thread Jan Beulich
>>> On 12.01.15 at 15:54, wrote: > On Fri, Jan 9, 2015 at 12:10 PM, Jan Beulich wrote: > On 09.01.15 at 12:45, wrote: >>> At 11:24 + on 09 Jan (1420799087), Jan Beulich wrote: >>> On 09.01.15 at 12:18, wrote: >> > +default: >> > +xfree(buf); >>

Re: [Xen-devel] [v3 5/5] Qemu-Xen-vTPM: QEMU machine class is initialized before tpm_init()

2015-01-12 Thread Xu, Quan
> -Original Message- > From: Paolo Bonzini [mailto:pbonz...@redhat.com] > Sent: Monday, January 12, 2015 11:23 PM > To: Xu, Quan; qemu-de...@nongnu.org > Cc: xen-devel@lists.xen.org; aligu...@amazon.com > Subject: Re: [v3 5/5] Qemu-Xen-vTPM: QEMU machine class is initialized before > tpm_

Re: [Xen-devel] preparing for 4.4.2 / 4.3.4

2015-01-12 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] preparing for 4.4.2 / 4.3.4"): > On 12.01.15 at 09:47, wrote: > > On Thu, Jan 08, Jan Beulich wrote: > >> now that 4.5 is almost out the door, I'd like to get stable releases > >> prepared on the other two active branches. 4.3.4 is expected to be > >> the last

Re: [Xen-devel] [PATCH] tools: libxl: do not overrun input buffer in libxl__parse_mac

2015-01-12 Thread Ian Jackson
Ian Jackson writes ("Re: [PATCH] tools: libxl: do not overrun input buffer in libxl__parse_mac"): > Wei Liu writes ("Re: [PATCH] tools: libxl: do not overrun input buffer in > libx\ > > This is a candidate for backporting. > > Queued. Backported to 4.3 and 4.4. Thanks, Ian. __

Re: [Xen-devel] [PATCH] tools: libxl: do not leak diskpath during local disk attach

2015-01-12 Thread Ian Jackson
Ian Jackson writes ("Re: [PATCH] tools: libxl: do not leak diskpath during local disk attach"): > I will apply this patch to staging and queue it for backport. Backported to 4.3 and 4.4. Thanks, Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org

[Xen-devel] [PATCH 01/14] mm: provide a find_page vma operation

2015-01-12 Thread David Vrabel
The optional find_page VMA operation is used to lookup the pages backing a VMA. This is useful in cases where the normal mechanisms for finding the page don't work. This is only called if the PTE is special. One use case is a Xen PV guest mapping foreign pages into userspace. In a Xen PV guest,

[Xen-devel] [PATCH 03/14] xen/grant-table: pre-populate kernel unmap ops for xen_gnttab_unmap_refs()

2015-01-12 Thread David Vrabel
When unmapping grants, instead of converting the kernel map ops to unmap ops on the fly, pre-populate the set of unmap ops. This allows the grant unmap for the kernel mappings to be trivially batched in the future. Signed-off-by: David Vrabel --- arch/arm/include/asm/xen/page.h |2 +- arch/

[Xen-devel] [PATCHv2 00/14] xen: fix many long-standing grant mapping bugs

2015-01-12 Thread David Vrabel
This series fixes a number of long-standing bugs in the handling of grant maps. Refer to the following for all the details. http://xenbits.xen.org/people/dvrabel/grant-improvements-C.pdf In summary, the important uses that this enables are: 1. Block backends can use networked storage safely.

[Xen-devel] [PATCH 05/14] x86/xen: require ballooned pages for grant maps

2015-01-12 Thread David Vrabel
From: Jenny Herbert Ballooned pages are always used for grant maps which means the original frame does not need to be saved in page->index nor restored after the grant unmap. This allows the workaround in netback for the conflicting use of the (unionized) page->index and page->pfmemalloc to be r

[Xen-devel] [PATCH 06/14] xen/grant-table: add helpers for allocating pages

2015-01-12 Thread David Vrabel
Add gnttab_alloc_pages() and gnttab_free_pages() to allocate/free pages suitable to for granted maps. Signed-off-by: David Vrabel --- drivers/block/xen-blkback/blkback.c |8 drivers/net/xen-netback/interface.c |7 +++ drivers/xen/gntdev.c|4 ++-- drivers/

[Xen-devel] [PATCH 02/14] mm: add 'foreign' alias for the 'pinned' page flag

2015-01-12 Thread David Vrabel
From: Jenny Herbert The foreign page flag will be used by Xen guests to mark pages that have grant mappings of frames from other (foreign) guests. The foreign flag is an alias for the existing (Xen-specific) pinned flag. This is safe because pinned is only used on pages used for page tables and

[Xen-devel] [PATCH 07/14] xen: mark grant mapped pages as foreign

2015-01-12 Thread David Vrabel
From: Jenny Herbert Use the "foreign" page flag to mark pages that have a grant map. Use page->private to store information of the grant (the granting domain and the grant reference). Signed-off-by: Jenny Herbert Signed-off-by: David Vrabel --- arch/x86/xen/p2m.c|7 --- drive

[Xen-devel] [PATCH 09/14] xen/grant-table: add a mechanism to safely unmap pages that are in use

2015-01-12 Thread David Vrabel
From: Jenny Herbert Introduce gnttab_unmap_refs_async() that can be used to safely unmap pages that may be in use (ref count > 1). If the pages are in use the unmap is deferred and retried later. This polling is not very clever but it should be good enough if the cases where the delay is necess

[Xen-devel] [PATCH 04/14] xen: remove scratch frames for ballooned pages and m2p override

2015-01-12 Thread David Vrabel
The scratch frame mappings for ballooned pages and the m2p override are broken. Remove them in preparation for replacing them with simpler mechanisms that works. The scratch pages did not ensure that the page was not in use. In particular, the foreign page could still be in use by hardware. If

Re: [Xen-devel] Backport request for "tools/hotplug: set mtu from bridge for tap interface"

2015-01-12 Thread Ian Jackson
Daniel Kiper writes ("Re: Backport request for "tools/hotplug: set mtu from bridge for tap interface""): > On Tue, Nov 18, 2014 at 06:24:31PM +, Ian Jackson wrote: > > Does it apply cleanly to 4.3 and 4.4? I haven't checked. Daniel, if > > you could check that, that would be helpful. If it

[Xen-devel] [PATCH 08/14] xen-netback: use foreign page information from the pages themselves

2015-01-12 Thread David Vrabel
From: Jenny Herbert Use the foreign page flag in netback to get the domid and grant ref needed for the grant copy. This signficiantly simplifies the netback code and makes netback work with foreign pages from other backends (e.g., blkback). This allows blkback to use iSCSI disks provided by dom

Re: [Xen-devel] [PATCH] xen-pt: Fix PCI devices re-attach failed

2015-01-12 Thread Stefano Stabellini
On Mon, 12 Jan 2015, Paolo Bonzini wrote: > On 12/01/2015 14:35, Li, Liang Z wrote: > > > > diff --git a/hw/xen/xen_pt.c b/hw/xen/xen_pt.c index c1bf357..f2893b2 100644 > > --- a/hw/xen/xen_pt.c > > +++ b/hw/xen/xen_pt.c > > @@ -736,7 +736,7 @@ static int xen_pt_initfn(PCIDevice *d) > > } > >

Re: [Xen-devel] Architecture for dom0 integrity measurements.

2015-01-12 Thread Xu, Quan
Hi, Dr. G.W. Wettstein cc Graaf who is vTPM / XSM maintainer. Also cc Stefano. > -Original Message- > From: xen-devel-boun...@lists.xen.org > [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of Dr. Greg Wettstein > Sent: Saturday, January 10, 2015 10:59 PM > To: xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH] evtchn: simplify sending of notifications

2015-01-12 Thread Andrew Cooper
On 12/01/15 11:42, Jan Beulich wrote: On 12.01.15 at 12:33, wrote: >> On 12/01/15 08:57, Jan Beulich wrote: >>> --- a/xen/include/xen/event.h >>> +++ b/xen/include/xen/event.h >>> @@ -152,10 +152,11 @@ static inline void evtchn_port_init(stru >>> d->evtchn_port_ops->init(d, evtchn);

Re: [Xen-devel] [libvirt] [PATCH 00/12] Replace Xen xl parsing/formatting impl

2015-01-12 Thread Eric Blake
On 01/12/2015 08:06 AM, Ian Campbell wrote: > On Fri, 2015-01-09 at 22:03 -0700, Jim Fehlig wrote: >> The first attempt to implement support for parsing/formatting Xen's >> xl disk config format copied Xen's flex-based parser into libvirt, which >> has proved to be challenging in the context of aut

Re: [Xen-devel] [PATCH v3 00/14] Enable vTPM subsystem on TPM 2.0

2015-01-12 Thread Xu, Quan
Graaf, Now there are no more comments for this series of patch. Can this series of patch be merged in staging branch? or any other AR, let me know. If the series of patch are in staging branch, the Community and I can continue to develop and enhance it. Thanks Quan > -Original Message--

Re: [Xen-devel] [libvirt] [PATCH 09/12] Revert "src/xenconfig: Xen-xl parser"

2015-01-12 Thread Eric Blake
On 01/12/2015 07:51 AM, John Ferlan wrote: > > > On 01/10/2015 12:03 AM, Jim Fehlig wrote: >> This reverts commit 2c78051a14acfb7aba078d569b1632dfe0ca0853. >> >> Conflicts: >> src/Makefile.am >> >> Signed-off-by: Jim Fehlig >> --- >> .gitignore| 1 - >> cfg.mk

Re: [Xen-devel] [PATCH] Add a flight to test OVMF master branch

2015-01-12 Thread Ian Jackson
Wei Liu writes ("[PATCH] Add a flight to test OVMF master branch"): > Do the usual stuffs for adding a new branch (ap-* cr-* etc). > > Modify ts-xen-build so that it builds Xen with the specified ovmf tree > and revision. > > Only build and test on x86 by modifying make-flight and mfi-common. >

Re: [Xen-devel] [PATCH v4 1/2] gnttab: Introduce rwlock to protect updates to grant table state

2015-01-12 Thread Egger, Christoph
On 2015/01/12 16:09, Jan Beulich wrote: On 09.01.15 at 16:12, wrote: >> @@ -899,26 +899,27 @@ __gnttab_unmap_common( >> >> op->frame = (unsigned long)(op->dev_bus_addr >> PAGE_SHIFT); >> >> +read_lock(&lgt->lock); >> if ( unlikely(op->handle >= lgt->maptrack_limit) ) >>

Re: [Xen-devel] [Xen-users] [TestDay] minor bug + possible configuration bug 4.5rc4 archlinux

2015-01-12 Thread Wei Liu
On Mon, Jan 12, 2015 at 02:58:53PM +, Ian Campbell wrote: > Adding some CC's, including the devel list. > > On Fri, 2015-01-09 at 19:49 -0600, Doug McMillan wrote: > > > > > > configuration(?) > > I compiled booted straight from bios with xen.efi during boot I received > > several errors. >

[Xen-devel] [PATCH 11/14] xen/gntdev: safely unmap grants in case they are still in use

2015-01-12 Thread David Vrabel
From: Jenny Herbert Use gnttab_unmap_refs_async() to wait until the mapped pages are no longer in use before unmapping them. This allows userspace programs to safely use Direct I/O and AIO to a network filesystem which may retain refs to pages in queued skbs after the filesystem I/O has complete

[Xen-devel] [PATCH 14/14] xen/gntdev: provide find_page VMA operation

2015-01-12 Thread David Vrabel
For a PV guest, use the find_page op to find the right page. To handle VMAs being split, remember the start of the original VMA so the correct index in the pages array can be calculated. Signed-off-by: David Vrabel --- drivers/xen/gntdev.c | 12 1 file changed, 12 insertions(+)

[Xen-devel] [PATCH 10/14] xen/gntdev: convert priv->lock to a mutex

2015-01-12 Thread David Vrabel
Unmapping may require sleeping and we unmap while holding priv->lock, so convert it to a mutex. Signed-off-by: David Vrabel --- drivers/xen/gntdev.c | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/drivers/xen/gntdev.c b/drivers/xen

[Xen-devel] [PATCH 12/14] xen-blkback: safely unmap grants in case they are still in use

2015-01-12 Thread David Vrabel
From: Jenny Herbert Use gnttab_unmap_refs_async() to wait until the mapped pages are no longer in use before unmapping them. This allows blkback to use network storage which may retain refs to pages in queued skbs after the block I/O has completed. Signed-off-by: Jenny Herbert Signed-off-by: D

[Xen-devel] [PATCH 13/14] xen/gntdev: mark userspace PTEs as special on x86 PV guests

2015-01-12 Thread David Vrabel
In an x86 PV guest, get_user_pages_fast() on a userspace address range containing foreign mappings does not work correctly because the M2P lookup of the MFN from a userspace PTE may return the wrong page. Force get_user_pages_fast() to fail on such addresses by marking the PTEs as special. If Xen

Re: [Xen-devel] [PATCH v2 1/5] vTPM: event channel bind interdomain with para/hvm virtual machine

2015-01-12 Thread Xu, Quan
> -Original Message- > From: xen-devel-boun...@lists.xen.org > [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of Xu, Quan > Sent: Friday, January 09, 2015 9:25 AM > To: Daniel De Graaf; xen-devel@lists.xen.org > Cc: samuel.thiba...@ens-lyon.org; stefano.stabell...@eu.citrix.com > Subj

Re: [Xen-devel] [Xen-users] [TestDay] minor bug + possible configuration bug 4.5rc4 archlinux

2015-01-12 Thread Olaf Hering
On Mon, Jan 12, Ian Campbell wrote: > Adding some CC's, including the devel list. > > On Fri, 2015-01-09 at 19:49 -0600, Doug McMillan wrote: > > configuration(?) > > I compiled booted straight from bios with xen.efi during boot I received > > several errors. > > xl info works (see attachment).

[Xen-devel] 答复: [PATCH] x86/MCE: allow overriding the CMCI threshold

2015-01-12 Thread 刘劲松(凯耳)
Same comments with Egger. Acked-by: Liu Jinsong -邮件原件- 发件人: Egger, Christoph [mailto:cheg...@amazon.de] 发送时间: 2015年1月12日 18:18 收件人: Jan Beulich; xen-devel 抄送: 刘劲松(凯耳) 主题: Re: [PATCH] x86/MCE: allow overriding the CMCI threshold On 2015/01/12 9:44, Jan Beulich wrote: > We've had reports

Re: [Xen-devel] [PATCH 00/12] Replace Xen xl parsing/formatting impl

2015-01-12 Thread Jim Fehlig
Ian Campbell wrote: > On Fri, 2015-01-09 at 22:03 -0700, Jim Fehlig wrote: > >> The first attempt to implement support for parsing/formatting Xen's >> xl disk config format copied Xen's flex-based parser into libvirt, which >> has proved to be challenging in the context of autotools. But as it

Re: [Xen-devel] [PATCH v4 1/2] gnttab: Introduce rwlock to protect updates to grant table state

2015-01-12 Thread Jan Beulich
>>> On 12.01.15 at 17:03, wrote: > On 2015/01/12 16:09, Jan Beulich wrote: > On 09.01.15 at 16:12, wrote: >>> @@ -899,26 +899,27 @@ __gnttab_unmap_common( >>> >>> op->frame = (unsigned long)(op->dev_bus_addr >> PAGE_SHIFT); >>> >>> +read_lock(&lgt->lock); >>> if ( unlikely(o

Re: [Xen-devel] [PATCH 00/12] Replace Xen xl parsing/formatting impl

2015-01-12 Thread Ian Campbell
On Mon, 2015-01-12 at 09:23 -0700, Jim Fehlig wrote: > Ian Campbell wrote: > > On Fri, 2015-01-09 at 22:03 -0700, Jim Fehlig wrote: > > > >> The first attempt to implement support for parsing/formatting Xen's > >> xl disk config format copied Xen's flex-based parser into libvirt, which > >> has

Re: [Xen-devel] [PATCH v2] xenstored: log tdb message via xenstored's logging mechanisms

2015-01-12 Thread Ian Campbell
On Mon, 2015-01-12 at 15:22 +, Wei Liu wrote: > On Mon, Jan 12, 2015 at 03:18:02PM +, Ian Campbell wrote: > > TDB provides us with a callback for this purpose. Use it in both > > xenstored and xs_tdb_dump. > > > > While at it make the existing log() macro tollerate memory failures. > > >

Re: [Xen-devel] [PATCH] tools: libxl: directly initialise saved_* in _libxl_types.c

2015-01-12 Thread Ian Campbell
On Thu, 2015-01-08 at 11:18 +, Wei Liu wrote: > Acked-by: Wei Liu > Applied, thanks. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 4/4] dt-uart: support /chosen/stdout-path property.

2015-01-12 Thread Ian Campbell
On Mon, 2015-01-12 at 15:24 +, Julien Grall wrote: > Ok. I'm fine with it. Thanks, I've applied the whole series, fixing the comment you noted in #3 as I went. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 16/21] xen/passthrough: iommu_deassign_device_dt: By default reassign device to nobody

2015-01-12 Thread Julien Grall
Hi Stefano, Sorry for the very late answer on this patch. On 06/08/14 17:23, Stefano Stabellini wrote: >> diff --git a/xen/drivers/passthrough/device_tree.c >> b/xen/drivers/passthrough/device_tree.c >> index 45d4a59..f8affa0 100644 >> --- a/xen/drivers/passthrough/device_tree.c >> +++ b/xen/dri

Re: [Xen-devel] [PATCH] libxl/arm: Correctly spelled FDT_ERR_* in a comment

2015-01-12 Thread Ian Campbell
On Fri, 2015-01-09 at 16:13 +, Julien Grall wrote: > Signed-off-by: Julien Grall > Cc: Ian Jackson > Cc: Stefano Stabellini > Cc: Ian Campbell > Cc: Wei Liu Applied, thanks. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.

Re: [Xen-devel] [PATCH v2] libxl: Don't ignore error when we fail to give access to ioport/irq/iomem

2015-01-12 Thread Ian Campbell
On Fri, 2015-01-09 at 16:06 +, Wei Liu wrote: > On Fri, Jan 09, 2015 at 03:56:45PM +, Julien Grall wrote: > > If we fail to give the access, the domain will unlikely work correctly. > > So we should bail out at the first error. > > > > Signed-off-by: Julien Grall > > Cc: Ian Jackson > >

Re: [Xen-devel] [PATCH v2] libxl: Don't ignore error when we fail to give access to ioport/irq/iomem

2015-01-12 Thread Julien Grall
Hi Ian, On 12/01/15 16:35, Ian Campbell wrote: > On Fri, 2015-01-09 at 16:06 +, Wei Liu wrote: >> On Fri, Jan 09, 2015 at 03:56:45PM +, Julien Grall wrote: >>> If we fail to give the access, the domain will unlikely work correctly. >>> So we should bail out at the first error. >>> >>> Sign

Re: [Xen-devel] Backport request for "tools/hotplug: set mtu from bridge for tap interface" - for Xen 4.3

2015-01-12 Thread Charles Arnold
With changeset 22885 support was added for setting the MTU in the vif-bridge script for when a vif interface was set to 'online'. The was not done for the 'add' operation. The 'add' operation was added to the script for when tap devices were specified (c/s 21944). With the setting of the MTU for

Re: [Xen-devel] Backport request for "tools/hotplug: set mtu from bridge for tap interface" - for Xen 4.4, 4.5, and unstable

2015-01-12 Thread Charles Arnold
Add quotes around $bridge and $dev to handle spaces in names. This should go into 4.4, 4.5 and unstable. Signed-off-by: Charles Arnold diff --git a/tools/hotplug/Linux/vif-bridge b/tools/hotplug/Linux/vif-bridge index 3d72ca4..8b67b0a 100644 --- a/tools/hotplug/Linux/vif-bridge +++ b/tools/hotpl

Re: [Xen-devel] [PATCH v2] libxl: Don't ignore error when we fail to give access to ioport/irq/iomem

2015-01-12 Thread Wei Liu
On Mon, Jan 12, 2015 at 04:35:19PM +, Ian Campbell wrote: > On Fri, 2015-01-09 at 16:06 +, Wei Liu wrote: > > On Fri, Jan 09, 2015 at 03:56:45PM +, Julien Grall wrote: > > > If we fail to give the access, the domain will unlikely work correctly. > > > So we should bail out at the first

Re: [Xen-devel] [libvirt] [PATCH 10/12] Introduce support for parsing/formatting Xen xl config format

2015-01-12 Thread Jim Fehlig
John Ferlan wrote: > On 01/10/2015 12:03 AM, Jim Fehlig wrote: > >> Introduce a parser/formatter for the xl config format. Since the >> deprecation of xm/xend, the VM config file format has diverged as >> new features are added to libxl. This patch adds support for parsing >> and formating the

Re: [Xen-devel] [PATCH for-4.6] libxl, hotplug/Linux: default to phy backend for raw format file

2015-01-12 Thread Ian Campbell
On Thu, 2015-01-08 at 16:47 +, Stefano Stabellini wrote: > On Thu, 8 Jan 2015, Ian Campbell wrote: > > On Thu, 2015-01-08 at 16:07 +, Wei Liu wrote: > > > On Thu, Jan 08, 2015 at 02:07:42PM +, Ian Campbell wrote: > > > > On Wed, 2014-11-26 at 16:55 +, Wei Liu wrote: > > > > > Modify

Re: [Xen-devel] [PATCH v2] tools/Rules.mk: Don't optimize debug builds; add macro debugging information

2015-01-12 Thread Ian Campbell
On Thu, 2015-01-08 at 17:16 +, Ian Campbell wrote: > > +ifeq ($(debug),y) > > +# Disable optimizations and debugging information for macros >^enable > > ... or else it looks like you are saying it should be turning of macro > debug info too. This is something w

Re: [Xen-devel] [PATCH] tools/misc: Cleanup makefile

2015-01-12 Thread Ian Campbell
On Fri, 2015-01-09 at 14:53 +, Wei Liu wrote: > On Thu, Jan 08, 2015 at 01:34:22PM +, Andrew Cooper wrote: > > The existing makefile was awkward with needing to express conditional > > inclusion for both the build and install rules, and contained both split and > > unsplit long lines. > >

Re: [Xen-devel] [PATCH 02/11] VMX: implement suppress #VE.

2015-01-12 Thread Andrew Cooper
On 09/01/15 21:26, Ed White wrote: > In preparation for selectively enabling hardware #VE in a later patch, > set suppress #VE on all EPTE's on #VE-capable hardware. > > Suppress #VE should always be the default condition for two reasons: > it is generally not safe to deliver #VE into a guest unles

Re: [Xen-devel] Backport request for "tools/hotplug: set mtu from bridge for tap interface"

2015-01-12 Thread Charles Arnold
Forgot to put [Patch] in the subject line for the last two replies :( - Charles ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [RFC PATCH] dpci: Put the dpci back on the list if running on another CPU.

2015-01-12 Thread Konrad Rzeszutek Wilk
There is race when we clear the STATE_SCHED in the softirq - which allows the 'raise_softirq_for' to progress and schedule another dpci. During that time the other CPU could receive an interrupt and calls 'raise_softirq_for' and put the dpci on its per-cpu list. There would be two 'dpci_softirq' ru

[Xen-devel] Xen 4.5: revert dpci patches

2015-01-12 Thread Konrad Rzeszutek Wilk
Hey, Two folks (Malcom, Sander) have reported issues with the dpci softirq code and while I've an fix that fixes it, and this stage I am uncomfortable putting it in Xen 4.5. As such I am going to revert from Xen 4.5 tree (only) these patches: 104072f dpci: add 'masked' as a gate for hvm_dirq_assi

Re: [Xen-devel] [PATCH 07/14] xen: mark grant mapped pages as foreign

2015-01-12 Thread Ian Campbell
On Mon, 2015-01-12 at 15:43 +, David Vrabel wrote: > + set_page_set_private(pages[i], (unsigned long)foreign); Is set twice here correct or a typo? ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 08/14] xen-netback: use foreign page information from the pages themselves

2015-01-12 Thread Ian Campbell
On Mon, 2015-01-12 at 15:43 +, David Vrabel wrote: > From: Jenny Herbert > > Use the foreign page flag in netback to get the domid and grant ref > needed for the grant copy. This signficiantly simplifies the netback > code and makes netback work with foreign pages from other backends > (e.g.

Re: [Xen-devel] [PATCH 07/14] xen: mark grant mapped pages as foreign

2015-01-12 Thread David Vrabel
On 12/01/15 16:54, Ian Campbell wrote: > On Mon, 2015-01-12 at 15:43 +, David Vrabel wrote: >> +set_page_set_private(pages[i], (unsigned long)foreign); > > Is set twice here correct or a typo? Oops. I forgot to try a 32-bit build. David _

Re: [Xen-devel] [OSSTEST PATCH] make-flight: reorganize scheduling related test jobs

2015-01-12 Thread Ian Jackson
Dario Faggioli writes ("[OSSTEST PATCH] make-flight: reorganize scheduling related test jobs"): > Scheduling related tests are ok to run on ARM, so do > not cut them off. They also do not depend on a > particular Dom0 architecture. > > The net effect is that the following tests are removed: > te

Re: [Xen-devel] [PATCH v2] libxl: Don't ignore error when we fail to give access to ioport/irq/iomem

2015-01-12 Thread Ian Jackson
Julien Grall writes ("Re: [PATCH v2] libxl: Don't ignore error when we fail to give access to ioport/irq/iomem"): > On 12/01/15 16:35, Ian Campbell wrote: > > Should this be backported? > > Yes. But it may not apply directly on Xen 4.4. I have made a note of this. I will apply it to 4.5 after t

Re: [Xen-devel] [PATCH v2] libxl: Don't ignore error when we fail to give access to ioport/irq/iomem

2015-01-12 Thread Julien Grall
Hi Ian, On 12/01/15 16:59, Ian Jackson wrote: > Julien Grall writes ("Re: [PATCH v2] libxl: Don't ignore error when we fail > to give access to ioport/irq/iomem"): >> On 12/01/15 16:35, Ian Campbell wrote: >>> Should this be backported? >> >> Yes. But it may not apply directly on Xen 4.4. > > I

Re: [Xen-devel] xl only waits 33 seconds for ballooning to complete

2015-01-12 Thread Mike Latimer
On Monday, January 12, 2015 12:36:01 PM George Dunlap wrote: > I would: > 1. Reset the retries after a successful increase > 2. Not allow free_memkb_prev to go down. Thanks, George. Good points, which definitely improve the situation. > So maybe something like the following? > > if (free_memkb

Re: [Xen-devel] Backport request for "tools/hotplug: set mtu from bridge for tap interface" - for Xen 4.4, 4.5, and unstable

2015-01-12 Thread Ian Jackson
Charles Arnold writes ("Re: Backport request for "tools/hotplug: set mtu from bridge for tap interface" - for Xen 4.4, 4.5, and unstable"): > Add quotes around $bridge and $dev to handle spaces in names. > This should go into 4.4, 4.5 and unstable. Is this really necessary for backporting ? Fran

Re: [Xen-devel] Backport request for "tools/hotplug: set mtu from bridge for tap interface" - for Xen 4.3

2015-01-12 Thread Ian Jackson
Charles Arnold writes ("Re: Backport request for "tools/hotplug: set mtu from bridge for tap interface" - for Xen 4.3"): > With changeset 22885 support was added for setting the MTU in the vif-bridge > script for when a vif interface was set to 'online'. The was not done for the > 'add' operation

Re: [Xen-devel] [PATCH 03/11] x86/HVM: Hardware alternate p2m support detection.

2015-01-12 Thread Andrew Cooper
On 09/01/15 21:26, Ed White wrote: > As implemented here, only supported on platforms with VMX HAP. > > Signed-off-by: Ed White > --- > xen/arch/x86/hvm/hvm.c| 8 > xen/arch/x86/hvm/vmx/vmx.c| 1 + > xen/include/asm-x86/hvm/hvm.h | 6 ++ > 3 files changed, 15 insertions(+

Re: [Xen-devel] Backport request for "tools/hotplug: set mtu from bridge for tap interface" - for Xen 4.4, 4.5, and unstable

2015-01-12 Thread Charles Arnold
>>> On 1/12/2015 at 10:03 AM, Ian Jackson wrote: > Charles Arnold writes ("Re: Backport request for "tools/hotplug: set mtu from > bridge for tap interface" - for Xen 4.4, 4.5, and unstable"): >> Add quotes around $bridge and $dev to handle spaces in names. >> This should go into 4.4, 4.5 and un

Re: [Xen-devel] [PATCH v2 04/19] libxl: add emacs local variables in libxl_{x86, arm}.c

2015-01-12 Thread Ian Campbell
On Mon, 2014-12-01 at 15:33 +, Wei Liu wrote: > Signed-off-by: Wei Liu > Cc: Ian Campbell > Cc: Ian Jackson > Cc: Dario Faggioli > Cc: Elena Ufimtseva Acked + applied. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xe

Re: [Xen-devel] [PATCH v2 03/19] libxc: allocate memory with vNUMA information for PV guest

2015-01-12 Thread Ian Campbell
On Mon, 2014-12-01 at 15:33 +, Wei Liu wrote: No longer description of what is going on? Or at least comments on what the new fields mean. (I figure two of them form an array mapping v to p, I'm not sure what the other one is...) > Signed-off-by: Wei Liu > Cc: Ian Campbell > Cc: Ian Jackson

Re: [Xen-devel] [PATCH v2 05/19] libxl: introduce vNUMA types

2015-01-12 Thread Ian Campbell
On Mon, 2014-12-01 at 15:33 +, Wei Liu wrote: Looking ahead a few patches I see a disturbing trend to include little or no commit message in this series. I also see no overview of what the design for vnuma at the libxl level is going to look like, just a bunch of moving parts in isolation, whi

Re: [Xen-devel] [PATCH 08/14] xen-netback: use foreign page information from the pages themselves

2015-01-12 Thread David Vrabel
On 12/01/15 16:56, Ian Campbell wrote: > On Mon, 2015-01-12 at 15:43 +, David Vrabel wrote: >> From: Jenny Herbert >> >> Use the foreign page flag in netback to get the domid and grant ref >> needed for the grant copy. This signficiantly simplifies the netback >> code and makes netback work w

Re: [Xen-devel] [PATCH 08/14] xen-netback: use foreign page information from the pages themselves

2015-01-12 Thread Ian Campbell
On Mon, 2015-01-12 at 17:16 +, David Vrabel wrote: > On 12/01/15 16:56, Ian Campbell wrote: > > On Mon, 2015-01-12 at 15:43 +, David Vrabel wrote: > >> From: Jenny Herbert > >> > >> Use the foreign page flag in netback to get the domid and grant ref > >> needed for the grant copy. This si

Re: [Xen-devel] xl only waits 33 seconds for ballooning to complete

2015-01-12 Thread Ian Campbell
On Mon, 2015-01-12 at 10:04 -0700, Mike Latimer wrote: > On Monday, January 12, 2015 12:36:01 PM George Dunlap wrote: > > I would: > > 1. Reset the retries after a successful increase > > 2. Not allow free_memkb_prev to go down. > > Thanks, George. Good points, which definitely improve the situati

Re: [Xen-devel] [PATCH] Add a flight to test OVMF master branch

2015-01-12 Thread Wei Liu
On Mon, Jan 12, 2015 at 04:07:46PM +, Ian Jackson wrote: > Wei Liu writes ("[PATCH] Add a flight to test OVMF master branch"): > > Do the usual stuffs for adding a new branch (ap-* cr-* etc). > > > > Modify ts-xen-build so that it builds Xen with the specified ovmf tree > > and revision. > >

Re: [Xen-devel] [PATCH v2 05/19] libxl: introduce vNUMA types

2015-01-12 Thread Wei Liu
On Mon, Jan 12, 2015 at 05:17:24PM +, Ian Campbell wrote: > On Mon, 2014-12-01 at 15:33 +, Wei Liu wrote: > > Looking ahead a few patches I see a disturbing trend to include little > or no commit message in this series. I also see no overview of what the > design for vnuma at the libxl lev

Re: [Xen-devel] [PATCH v2] libxl: Don't ignore error when we fail to give access to ioport/irq/iomem

2015-01-12 Thread Ian Jackson
Julien Grall writes ("Re: [PATCH v2] libxl: Don't ignore error when we fail to give access to ioport/irq/iomem"): > On 12/01/15 16:59, Ian Jackson wrote: > > As you suspected, it doesn't apply cleanly to 4.4. I would welcome a > > backport. If no-one else backports it I will do it myself. > > I

Re: [Xen-devel] [PATCH v2 05/19] libxl: introduce vNUMA types

2015-01-12 Thread Ian Campbell
On Mon, 2015-01-12 at 17:21 +, Wei Liu wrote: > On Mon, Jan 12, 2015 at 05:17:24PM +, Ian Campbell wrote: > > On Mon, 2014-12-01 at 15:33 +, Wei Liu wrote: > > > > Looking ahead a few patches I see a disturbing trend to include little > > or no commit message in this series. I also see

Re: [Xen-devel] [RFC PATCH] dpci: Put the dpci back on the list if running on another CPU.

2015-01-12 Thread Sander Eikelenboom
Monday, January 12, 2015, 5:45:30 PM, you wrote: > There is race when we clear the STATE_SCHED in the softirq > - which allows the 'raise_softirq_for' to progress and > schedule another dpci. During that time the other CPU could > receive an interrupt and calls 'raise_softirq_for' and put > the d

Re: [Xen-devel] Xen 4.5: revert dpci patches

2015-01-12 Thread Sander Eikelenboom
Monday, January 12, 2015, 5:49:44 PM, you wrote: > Hey, > Two folks (Malcom, Sander) have reported issues with the dpci softirq code and > while I've an fix that fixes it, and this stage I am uncomfortable putting > it in Xen 4.5. As such I am going to revert from Xen 4.5 tree > (only) these pat

Re: [Xen-devel] xl only waits 33 seconds for ballooning to complete

2015-01-12 Thread George Dunlap
On 01/12/2015 05:04 PM, Mike Latimer wrote: >> I'm inclined to say we could add an option to say "wait forever", or >> to increase the period of the checks; but ultimately at some point >> someone (either xl or the human) needs to timeout and say, "This is >> never going to finish". 10s seems like

Re: [Xen-devel] [RFC PATCH] dpci: Put the dpci back on the list if running on another CPU.

2015-01-12 Thread Konrad Rzeszutek Wilk
On Mon, Jan 12, 2015 at 11:45:30AM -0500, Konrad Rzeszutek Wilk wrote: > There is race when we clear the STATE_SCHED in the softirq > - which allows the 'raise_softirq_for' to progress and > schedule another dpci. During that time the other CPU could > receive an interrupt and calls 'raise_softirq_

Re: [Xen-devel] [PATCH 00/11] Alternate p2m: support multiple copies of host p2m

2015-01-12 Thread Ed White
On 01/12/2015 02:00 AM, Jan Beulich wrote: On 10.01.15 at 00:04, wrote: >> On 01/09/2015 02:41 PM, Andrew Cooper wrote: >>> Having some non-OS part of the guest swap the EPT tables and >>> accidentally turn a DMA buffer read-only is not going to end well. >>> >> >> The agent can certainly do

Re: [Xen-devel] [PATCH 00/11] Alternate p2m: support multiple copies of host p2m

2015-01-12 Thread Ed White
On 01/12/2015 04:17 AM, Ian Jackson wrote: > Ed White writes ("[PATCH 00/11] Alternate p2m: support multiple copies of > host p2m"): >> This set of patches adds support to hvm domains for EPTP switching >> by creating multiple copies of the host p2m (currently limited to 10 >> copies). > > Thanks

Re: [Xen-devel] [PATCH 00/11] Alternate p2m: support multiple copies of host p2m

2015-01-12 Thread Ian Jackson
Ed White writes ("Re: [PATCH 00/11] Alternate p2m: support multiple copies of host p2m"): > On 01/12/2015 04:17 AM, Ian Jackson wrote: > > Are there tools parts to come later ? > > I copied you because get_maintainer picked you. There is scope for tools > parts, but I don't have the relevant subj

Re: [Xen-devel] [PATCH 02/11] VMX: implement suppress #VE.

2015-01-12 Thread Ed White
On 01/12/2015 08:43 AM, Andrew Cooper wrote: > On 09/01/15 21:26, Ed White wrote: >> In preparation for selectively enabling hardware #VE in a later patch, >> set suppress #VE on all EPTE's on #VE-capable hardware. >> >> Suppress #VE should always be the default condition for two reasons: >> it is

Re: [Xen-devel] [PATCH 03/11] x86/HVM: Hardware alternate p2m support detection.

2015-01-12 Thread Ed White
On 01/12/2015 09:08 AM, Andrew Cooper wrote: > On 09/01/15 21:26, Ed White wrote: >> As implemented here, only supported on platforms with VMX HAP. >> >> Signed-off-by: Ed White >> --- >> xen/arch/x86/hvm/hvm.c| 8 >> xen/arch/x86/hvm/vmx/vmx.c| 1 + >> xen/include/asm-x86/hv

Re: [Xen-devel] [libvirt] [PATCH 09/12] Revert "src/xenconfig: Xen-xl parser"

2015-01-12 Thread Jim Fehlig
Eric Blake wrote: > On 01/12/2015 07:51 AM, John Ferlan wrote: > >> On 01/10/2015 12:03 AM, Jim Fehlig wrote: >> >>> This reverts commit 2c78051a14acfb7aba078d569b1632dfe0ca0853. >>> >>> Conflicts: >>> src/Makefile.am >>> >>> Signed-off-by: Jim Fehlig >>> --- >>> .gitignore

Re: [Xen-devel] [PATCH 04/11] x86/MM: Improve p2m type checks.

2015-01-12 Thread Andrew Cooper
On 09/01/15 21:26, Ed White wrote: > diff --git a/xen/include/asm-x86/p2m.h b/xen/include/asm-x86/p2m.h > index 5f7fe71..8193901 100644 > --- a/xen/include/asm-x86/p2m.h > +++ b/xen/include/asm-x86/p2m.h > @@ -193,6 +193,9 @@ struct p2m_domain { > * threaded on in LRU order. */ > struct

Re: [Xen-devel] [libvirt] [PATCH 10/12] Introduce support for parsing/formatting Xen xl config format

2015-01-12 Thread Eric Blake
On 01/09/2015 10:03 PM, Jim Fehlig wrote: > Introduce a parser/formatter for the xl config format. Since the > deprecation of xm/xend, the VM config file format has diverged as > new features are added to libxl. This patch adds support for parsing > and formating the xl config format. It support

Re: [Xen-devel] [PATCH 00/11] Alternate p2m: support multiple copies of host p2m

2015-01-12 Thread Ed White
On 01/12/2015 09:43 AM, Ian Jackson wrote: > Ed White writes ("Re: [PATCH 00/11] Alternate p2m: support multiple copies of > host p2m"): >> On 01/12/2015 04:17 AM, Ian Jackson wrote: >>> Are there tools parts to come later ? >> >> I copied you because get_maintainer picked you. There is scope for

Re: [Xen-devel] [PATCH 00/11] Alternate p2m: support multiple copies of host p2m

2015-01-12 Thread Andrew Cooper
On 12/01/15 17:43, Ian Jackson wrote: > Ed White writes ("Re: [PATCH 00/11] Alternate p2m: support multiple copies of > host p2m"): >> On 01/12/2015 04:17 AM, Ian Jackson wrote: >>> Are there tools parts to come later ? >> I copied you because get_maintainer picked you. There is scope for tools >>

Re: [Xen-devel] xl only waits 33 seconds for ballooning to complete

2015-01-12 Thread Mike Latimer
On Monday, January 12, 2015 05:29:25 PM George Dunlap wrote: > When I said "10s seems very conservative", I meant, "10s should be by > far long enough for something to happen". If you can't free up at least > 1k in 30s, then there is certainly something very unusual with your > system. So I was r

Re: [Xen-devel] [OPW PATCH V4] tools: xl: refactor code to parse network device options

2015-01-12 Thread Ian Campbell
On Wed, 2014-10-22 at 12:35 +0100, Ian Campbell wrote: > On Wed, 2014-10-22 at 00:36 +0300, Alexandra Sandulescu wrote: > > This patch removes duplicate code in /tools/libxl/xl_cmdimpl.c by > > adding parse_nic_config function. This function parses configuration > > data and adds the information in

Re: [Xen-devel] Fwd: [OPW PATCH 1/4] tools/xl: Calling _init and _dispose function for libxl types

2015-01-12 Thread Ian Campbell
On Tue, 2014-10-21 at 18:04 +0100, George Dunlap wrote: Just getting back to these after the freeze. > On Tue, Oct 21, 2014 at 5:34 PM, Uma Sharma wrote: > > Should I resend the patches then? > > On the xen-devel list, always reply at the bottom, like this. :-) > > I think normally it wouldn'

Re: [Xen-devel] [PATCH 1/2] sched: credit2: respect per-vcpu hard affinity

2015-01-12 Thread Dario Faggioli
On Sat, 2014-11-29 at 14:33 -1000, Justin T. Weaver wrote: > by making sure that vcpus only run on the pcpu(s) they are allowed to > run on based on their hard affinity cpu masks. > > Signed-off-by: Justin T. Weaver > Still about referencing "history", one usually puts here a quick summary of wha

<    1   2   3   >