Re: [Xen-devel] [Qemu-devel] Question about scsi emulation

2015-03-24 Thread Yaoli Zheng
Thank you for the advice! It will be grateful if someone can provide any guide how to config MegaSAS HBA or how to use xen pvscsi in XEN. There seems no option in XEN for these two driver emulation and no document found online. Thanks! -Original Message- From: Stefano Stabellini [mailt

Re: [Xen-devel] [PATCH] x86: don't change affinity with interrupt unmasked

2015-03-24 Thread Jan Beulich
>>> On 23.03.15 at 19:46, wrote: > On Fri, Mar 20, 2015 at 03:40:02PM +, Jan Beulich wrote: >> With ->startup unmasking the IRQ, setting the affinity afterwards >> without masking the IRQ again is invalid namely for MSI (which can't >> have their affinity updated atomically). > > That took a

Re: [Xen-devel] [PATCH 2/2] x86/vMSI-X: add valid bits for read acceleration

2015-03-24 Thread Jan Beulich
>>> On 23.03.15 at 20:11, wrote: > On Fri, Mar 20, 2015 at 04:27:57PM +, Jan Beulich wrote: >> Again because Xen doesn't get to see all guest writes, it shouldn't >> serve reads from its cache before having seen a write to the respective >> address. >> >> Signed-off-by: Jan Beulich > > Revi

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-03-24 Thread Jan Beulich
>>> On 24.03.15 at 07:42, wrote: > > On 02/17/2015 06:19 PM, Jan Beulich wrote: > On 12.02.15 at 03:39, wrote: >>> On 02/11/2015 07:52 PM, Andrew Cooper wrote: On 11/02/15 08:28, Kai Huang wrote: > Design > == > > - PML feature is used globally > > A new Xen

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-03-24 Thread Kai Huang
On 03/24/2015 03:53 PM, Jan Beulich wrote: On 24.03.15 at 07:42, wrote: On 02/17/2015 06:19 PM, Jan Beulich wrote: On 12.02.15 at 03:39, wrote: On 02/11/2015 07:52 PM, Andrew Cooper wrote: On 11/02/15 08:28, Kai Huang wrote: Design == - PML feature is used globally A new Xen boot p

Re: [Xen-devel] [PATCH 2/2] x86: Don't use BAD_APICID for non-APICID fields

2015-03-24 Thread Jan Beulich
>>> On 23.03.15 at 20:54, wrote: > BAD_APICID is used by cpuinfo_x86's phys_proc_id, cpu_core_id > and compute_unit_id even though these fields don't hold an APIC ID > itself but rather its derivative. > > Provide appropriate macros for each of those three (and make them > unsigned). > > This al

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-03-24 Thread Jan Beulich
>>> On 24.03.15 at 09:06, wrote: > On 03/24/2015 03:53 PM, Jan Beulich wrote: >> The command line >> option parsing is (and should be) independent of the chosen >> default anyway, i.e. overrides in either direction should be >> possible. > While the parse_ept_param function does support "ept=pml

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-03-24 Thread Kai Huang
On 03/24/2015 04:14 PM, Jan Beulich wrote: On 24.03.15 at 09:06, wrote: On 03/24/2015 03:53 PM, Jan Beulich wrote: The command line option parsing is (and should be) independent of the chosen default anyway, i.e. overrides in either direction should be possible. While the parse_ept_param

Re: [Xen-devel] [libvirt] libvirt/libxl implemetation of get_online_cpu / virNodeGetCPUMap?

2015-03-24 Thread Manish Jaggi
On Tuesday 24 February 2015 08:30 PM, Anthony PERARD wrote: On Tue, Feb 24, 2015 at 01:22:19PM +, Daniel P. Berrange wrote: On Tue, Feb 24, 2015 at 01:15:57PM +, Anthony PERARD wrote: On Tue, Feb 24, 2015 at 12:46:44PM +, Daniel P. Berrange wrote: On Tue, Feb 24, 2015 at 12:41:01P

[Xen-devel] One question to lowlevel/xl/xl.c and lowlevel/xc/xc.c

2015-03-24 Thread Chen, Tiejun
All guys, Sorry to bother you. I have a question to two files, tools/python/xen/lowlevel/xc/xc.c and tools/python/xen/lowlevel/xl/xl.c. Who is a caller to those methods like pyxc_methods[] and pyxl_methods[]? And how should we call these approaches? In my specific case, I'm trying to introdu

Re: [Xen-devel] Xen crash with mem-sharing and cloning

2015-03-24 Thread Tamas K Lengyel
On Tue, Mar 24, 2015 at 4:54 AM, Andres Lagar Cavilla < and...@lagarcavilla.org> wrote: > > > On Mon, Mar 23, 2015 at 11:25 AM, Tamas K Lengyel > wrote: > >> On Mon, Mar 23, 2015 at 6:59 PM, Andres Lagar Cavilla < >> and...@lagarcavilla.org> wrote: >> >>> On Mon, Mar 23, 2015 at 9:10 AM, Tamas K

Re: [Xen-devel] [OSSTEST Nested PATCH 2/6] Add and expose some testsupport APIs

2015-03-24 Thread Wei Liu
On Tue, Mar 24, 2015 at 05:13:31AM +, Pang, LongtaoX wrote: > > > > -Original Message- > > From: Ian Campbell [mailto:ian.campb...@citrix.com] > > Sent: Tuesday, March 24, 2015 1:37 AM > > To: Wei Liu > > Cc: Pang, LongtaoX; Hu, Robert; ian.jack...@eu.citrix.com; > > xen-devel@lists.x

Re: [Xen-devel] [OSSTEST Nested PATCH 2/6] Add and expose some testsupport APIs

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 05:13 +, Pang, LongtaoX wrote: > So, maybe I should write some code in 'ts-nested-setup' script to > implement ssh into L1, edit sources.list and comment out the CDROM > repo entry. Please do it in ts-debian-hvm-install, either as a postinst step or, better (IMHO), by arr

Re: [Xen-devel] [OSSTEST Nested PATCH 2/6] Add and expose some testsupport APIs

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 03:25 +, Hu, Robert wrote: > Though no harm as far as I see now, I cannot forcast any harm in the > future. Anyway, this 'nestedhvm' is for a nested usage. Better to enable > it when we are actually going enable a nested environment. > Practically, we can do it in either

Re: [Xen-devel] [OSSTEST Nested PATCH 2/6] Add and expose some testsupport APIs

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 03:34 +, Hu, Robert wrote: > > Make it an option to the function which creates the configuration in the > > first place. Quoting myself from earlier in this very thread: > Then we still need to designate to use e1000 device somewhere; say, in > ts-nested-setup, we store

Re: [Xen-devel] One question to lowlevel/xl/xl.c and lowlevel/xc/xc.c

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 16:47 +0800, Chen, Tiejun wrote: > All guys, > > Sorry to bother you. > > I have a question to two files, tools/python/xen/lowlevel/xc/xc.c and > tools/python/xen/lowlevel/xl/xl.c. Who is a caller to those methods like > pyxc_methods[] and pyxl_methods[]? They are registe

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

2015-03-24 Thread xen . org
flight 36590 ovmf real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/36590/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-pair17 guest-migrate/src_host/dst_host fail like 36525 Tests which did not succeed, but are

Re: [Xen-devel] [Qemu-devel] Question about scsi emulation

2015-03-24 Thread Juergen Gross
On 03/24/2015 01:20 AM, Yaoli Zheng wrote: Thank you for the advice! It will be grateful if someone can provide any guide how to config MegaSAS HBA or how to use xen pvscsi in XEN. There seems no option in XEN for these two driver emulation and no document found online. The pvscsi driver is

Re: [Xen-devel] [PATCH 1/2] pci: Include asm/numa.h in pci.h

2015-03-24 Thread Ian Campbell
On Mon, 2015-03-23 at 15:54 -0400, Boris Ostrovsky wrote: > Commit 4fa6b0bacf9c ("pci: stash device's PXM information in struct > pci_dev") added node field to xen/include/xen/pci.h. Its type, > nodeid_t, is defined in asm/numa.h and we should include this file > explicitly in pci.h > > Signed-off

Re: [Xen-devel] One question to lowlevel/xl/xl.c and lowlevel/xc/xc.c

2015-03-24 Thread Chen, Tiejun
On 2015/3/24 17:51, Ian Campbell wrote: On Tue, 2015-03-24 at 16:47 +0800, Chen, Tiejun wrote: All guys, Thanks for your reply. Sorry to bother you. I have a question to two files, tools/python/xen/lowlevel/xc/xc.c and tools/python/xen/lowlevel/xl/xl.c. Who is a caller to those methods lik

Re: [Xen-devel] One question to lowlevel/xl/xl.c and lowlevel/xc/xc.c

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 18:15 +0800, Chen, Tiejun wrote: > On 2015/3/24 17:51, Ian Campbell wrote: > > On Tue, 2015-03-24 at 16:47 +0800, Chen, Tiejun wrote: > >> All guys, > >> > > Thanks for your reply. > > >> Sorry to bother you. > >> > >> I have a question to two files, tools/python/xen/lowleve

Re: [Xen-devel] One question to lowlevel/xl/xl.c and lowlevel/xc/xc.c

2015-03-24 Thread Chen, Tiejun
On 2015/3/24 18:20, Ian Campbell wrote: On Tue, 2015-03-24 at 18:15 +0800, Chen, Tiejun wrote: On 2015/3/24 17:51, Ian Campbell wrote: On Tue, 2015-03-24 at 16:47 +0800, Chen, Tiejun wrote: All guys, Thanks for your reply. Sorry to bother you. I have a question to two files, tools/python

Re: [Xen-devel] One question to lowlevel/xl/xl.c and lowlevel/xc/xc.c

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 18:31 +0800, Chen, Tiejun wrote: > > NB, the libxl ones are broken and not even compiled right now, you can > > ignore them. > > Looks this is still compiled now. xc is, xl is not, I am sure of that. > > I don't know what the semantics of flag is, if it is per SBDF then I >

Re: [Xen-devel] [PATCH 01/29] libxl: Further fix exit paths from libxl_device_events_handler

2015-03-24 Thread Ian Campbell
On Wed, 2015-02-18 at 17:23 +0100, Roger Pau Monné wrote: > El 10/02/15 a les 21.09, Ian Jackson ha escrit: > > On the success path, do not call GC_FREE explicitly. Instead, call > > AO_INPROGRESS. > > > > GC_FREE will free the gc underlying the long-term ao, which is then > > subsequently refere

Re: [Xen-devel] [PATCH 02/29] libxl: Comment cleanups

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > * Add two comments in libxl_remus_disk_drbd documenting buggy handling > of the hotplug script exit status. > > * Add a section heading for async exec in libxl_aoutils.c > > * Mention the right function name (libxl__ev_child_fork, not >

Re: [Xen-devel] [PATCH 03/29] libxl: suspend: switch_logdirty_done takes rc

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > +int broke; > +if (rc) { > +broke = -1; > +} else { > +broke = 0; > +} int broke = rc ? -1 : 0; ? But it looks like perhaps you are preparing to add other code in one or the other case, or maybe you just pref

Re: [Xen-devel] [PATCH 04/29] libxl: suspend: common suspend callbacks take rc

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > Change the following functions to take a libxl error code rather than > a boolean "ok" value, and translate that value to the boolean expected > by libxc at the last moment: > domain_suspend_callback_common_done} dss->callback_common

Re: [Xen-devel] [PATCH 05/29] libxl: suspend: Return correct error from callbacks

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > If a suspend callback fails, it has a libxl error code in its hand. > However we must return to libxc the values that libxc expects. So we > stash the libxl error code in dss->rc and fish it out again after > libxc returns from the suspend ca

[Xen-devel] March Advisory Board Meeting Minutes

2015-03-24 Thread Lars Kurth
See http://wiki.xenproject.org/wiki/AB_Meeting/March_2015_Minutes ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 06/29] libxl: Use libxl__xswait* in libxl__ao_device

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > @@ -1164,7 +1136,7 @@ static void device_hotplug_clean(libxl__gc *gc, > libxl__ao_device *aodev) > { > /* Clean events and check reentrancy */ > libxl__ev_time_deregister(gc, &aodev->timeout); You seem to have removed the initiali

Re: [Xen-devel] [PATCH 07/29] libxl: xswait/devstate: Move xswait to before devstate

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > Pure code motion. We are going to make devstate use xswait. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-d

Re: [Xen-devel] [PATCH v2] xen/passthrough: Support a single iommu_domain per xen domain per SMMU

2015-03-24 Thread Manish Jaggi
On Monday 23 March 2015 07:16 PM, Robbie VanVossen wrote: If multiple devices are being passed through to the same domain and they share a single SMMU, then they only require a single iommu_domain. In arm_smmu_assign_dev, before a new iommu_domain is created, the xen_domain->contexts is checked

Re: [Xen-devel] [PATCH 09/29] libxl: New error codes CANCELLED etc.

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > We introduce ERROR_CANCELLED now, so that we can write code to handle > it, and decreee that functions might return it, even though currently > there is nowhere where this error is generated. > > While we're here, provide ERROR_NOTFOUND and E

Re: [Xen-devel] [PATCH 08/29] libxl: devstate: Use libxl__xswait*

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 10/29] libxl: events: Make timeout and async exec setup take an ao, not a gc

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > Change the timeout setup functions to take a libxl__ao, not a > libxl__gc. This is going to be needed for ao cancellation, because > timeouts are going to be a main hook for ao cancellation - so the > timeouts need to be associated with an ao

Re: [Xen-devel] [PATCH 11/29] libxl: events: Make libxl__async_exec_* pass caller an rc

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > The internal user of libxl__async_exec_start et al now gets an rc as > well as the process's exit status. > > For now this is always either 0 or ERROR_FAIL, but with ao > cancellation this will possibly be CANCELLED or TIMEDOUT too. > > Sign

Re: [Xen-devel] [PATCH v2 7/7] tools: add tools support for Intel CAT

2015-03-24 Thread Chao Peng
On Thu, Mar 19, 2015 at 12:51:19PM +, Ian Campbell wrote: > On Thu, 2015-03-19 at 18:41 +0800, Chao Peng wrote: > > This is the xc/xl changes to support Intel Cache Allocation > > Technology(CAT). Two commands are introduced: > > - xl psr-cat-cbm-set [-s socket] > > Set cache capacity bitma

Re: [Xen-devel] [PATCH 12/29] libxl: events: Permit timeouts to signal cancellation

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:09 +, Ian Jackson wrote: > The callback functions provided by users must take an rc value. This > rc value can be ERROR_TIMEDOUT or ERROR_CANCELLED. > > Users of xswait are now expected to deal correctly with > ERROR_CANCELLED. If they experience this, it hasn't been

Re: [Xen-devel] [PATCH 13/29] libxl: domain create: Do not destroy on cancellation

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > If we cancelled the domain creation, do not try to tear it down again > Document this. > > This is a backwards-compatible API change since old libxl users will > never cancel any operations. > > In the current code, there is no functional ch

Re: [Xen-devel] [PATCH 14/29] libxl: ao: Record ultimate parent of a nested ao

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > This will be used by the cancellation machinery. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 15/29] libxl: ao: Count the nested progeny of an ao

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > This will detect any "escaped" nested aos. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 16/29] libxl: ao: Provide manip_refcnt

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > +/* > + * A "manip" is a libxl public function manipulating this ao, which > + * has a pointer to it. We have to not destroy it while that's the > + * case, obviously. It might be nice to the reader to make a reference to the "An ao and its

Re: [Xen-devel] [OSSTEST Nested PATCH 2/6] Add and expose some testsupport APIs

2015-03-24 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST Nested PATCH 2/6] Add and expose some testsupport APIs"): > On Tue, 2015-03-24 at 03:34 +, Hu, Robert wrote: > > > Make it an option to the function which creates the configuration in the > > > first place. Quoting myself from earlier in this very thread: >

Re: [Xen-devel] [PATCH 17/29] libxl: cancellation: Provide public ao cancellation API

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > +/* > + * It is sometimes possible to cancel an asynchronous operation. > + * > + * libxl_ao_cancel searches for an ongoing asynchronous operation whose > + * ao_how is identical to *how, and tries to cancel it. I can see that you have arrang

Re: [Xen-devel] [PATCH 18/29] libxl: cancellation: Provide explicit internal cancel check API

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > Some places in libxl which can't handle cancellation via a > libxl__ao_cancellable callback might nevertheless benefit from being > able to explicitly check for cancellation. > > Provide the (fairly trivial) internal API function to do this.

[Xen-devel] [PATCH v4] xen/arm: Add RESERVE option to reserve non-leaf level table entries

2015-03-24 Thread vijay . kilari
From: Vijaya Kumar K On x86, for the pages mapped with PAGE_HYPERVISOR attribute non-leaf page tables are allocated with valid pte entries. and with MAP_SMALL_PAGES attribute only non-leaf page tables are allocated with invalid (valid bit set to 0) pte entries. However on arm this is not the case

[Xen-devel] [PATCH OSSTEST 0/2] Uboot patches for XSM test case

2015-03-24 Thread Wei Liu
Wei Liu (2): uboot: use "readlink -f" uboot: make flask loading address host property Osstest/Debian.pm | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) -- 1.9.1 ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.x

[Xen-devel] [PATCH OSSTEST 2/2] uboot: make flask loading address host property

2015-03-24 Thread Wei Liu
Signed-off-by: Wei Liu --- Please set host property 'UBootSetFlaskAddrR' to 0x120. --- Osstest/Debian.pm | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index b829878..9f7705e 100644 --- a/Osstest/Debian.pm +++ b/Osstest/Debian.

[Xen-devel] [PATCH OSSTEST 1/2] uboot: use "readlink -f"

2015-03-24 Thread Wei Liu
If the path is not a symlink, readlink by default returns empty string. Use "-f" to always return canonical path. This fixes the problem that xenpolicy file not getting loaded (because it is not a symlink). Also change another spot that calls readlink to get xen binary path in case in the future w

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-03-24 Thread Julien Grall
Hello Vijay, More questions/remarks about command processing. On 19/03/2015 14:38, vijay.kil...@gmail.com wrote: +int vgic_its_process_cmd(struct vcpu *v, struct vgic_its *vits) +{ +struct its_cmd_block virt_cmd; + +/* XXX: Currently we are processing one cmd at a time */ +ASSERT(sp

Re: [Xen-devel] [PATCH 17/29] libxl: cancellation: Provide public ao cancellation API

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > +/* > + * For nested aos: > + * Semantically, cancellation affects the whole tree of aos, > + *not just the parent. > + * libxl__ao_cancellable.ao refers to the child, so > + *that the child callback sees the

Re: [Xen-devel] [PATCH 19/29] libxl: cancellation: Make timeouts cancellable

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > Make libxl__ev_time* register with the cancellation machinery, so that > libxl_ao_cancel can cancel any operation which has a timeout. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___

Re: [Xen-devel] [PATCH 20/29] libxl: cancellation: Note that driver domain task cannot be usefully cancelled

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > In practice, cancelling this task will cause all subsequent actual > backend operations to fail, but will not actually cause the > libxl_device_events_handler operation to complete. > > Signed-off-by: Ian Jackson > CC: Roger Pau Monne Acke

Re: [Xen-devel] [PATCH 21/29] libxl: cancellation: Make spawns cancellable

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > The libxl__spawn_spawn internal API permits the caller to specify > .timeout_ms==-1, meaning to wait forever. Provide an explicit > cancellable to allow spawns to be cancelled. AIUI this also lets spawns with timeout_ms > -1 to be cancelled,

Re: [Xen-devel] Design and Question: Eliminate Xen (RTDS) scheduler overhead on dedicated CPU

2015-03-24 Thread George Dunlap
On Tue, Mar 24, 2015 at 3:50 AM, Meng Xu wrote: > Hi Dario and George, > > I'm exploring the design choice of eliminating the Xen scheduler overhead on > the dedicated CPU. A dedicated CPU is a PCPU that has a full capacity VCPU > pinned onto it and no other VCPUs will run on that PCPU. Hey Meng!

Re: [Xen-devel] [PATCH 22/29] libxl: Introduce DOMAIN_DESTROYED error code

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > This is currently reported only by the bootloader code, if the domain > is destroyed while the bootloader is running. > > In the future it would be nice to return it for other circumstances > where the domain existed when the operation starte

Re: [Xen-devel] [PATCH 23/29] libxl: cancellation: Support cancellation where we spot domain death

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > Make an active libxl__domaindeathcheck contain an active > libxl__ao_cancellable. > > Consequential changes are: > * domaindeath callbacks now take an rc value. > * libxl__domaindeathcheck_start takes an ao, not a gc. > * bootloader_domain

Re: [Xen-devel] [PATCH 24/29] libxl: Introduce FILLZERO

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > FILLZERO is a macro for memset(&foo,0,sizeof(foo)). It eliminates the > possiblity to make the error memset(&foo,0,sizeof(&foo)). but not: foo *p = allocate_a_foo() memset(p, 0, sizeof(p)) although that's probably less likely to

Re: [Xen-devel] [PATCH 25/29] libxl: cancellation: Preparations for save/restore cancellation

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > Two unrelated non-functional changes, broken out into a pre-patch for > easier review: > > Break out a function sendsig() in libxl_save_callout.c. > > Move io_fd to be a global variable in libxl_save_helper.c. > > Signed-off-by: Ian Jackson

Re: [Xen-devel] [PATCH 26/29] libxl: cancellation: Handle SIGTERM in save/restore helper

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > During startup of the save/restore helper, set the disposition of > SIGTERM appropriately. > > For restore, we can simply die immediately - there is no point trying > to do any kind of cleanup on what is now going to be a trashed domain. > >

Re: [Xen-devel] [PATCH 27/29] libxl: cancellation: Cancel libxc save/restore

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > Register the the save/restore helper interface with the cancellation > machinery. When we are informed that save/restore should be > cancelled, we make a note of the that in our rc variable, and send the > helper a SIGTERM. It will die in du

Re: [Xen-devel] [PATCH 28/29] libxl: ao: datacopier callback gets an rc

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > libxl__datacopier_* now provides its caller's callback function with > an rc value. This relieves the caller of the need to figure out an > appropriate rc value. > > Arrange that the `other internal failure' cases now get a valid > positive

Re: [Xen-devel] [PATCH 29/29] libxl: cancellation: Make datacopiers cancellable

2015-03-24 Thread Ian Campbell
On Tue, 2015-02-10 at 20:10 +, Ian Jackson wrote: > libxl__datacopier_* can now actually generate a callback with > rc==CANCELLED. > > This provides cancellation during some corner cases, including (at > least) copying the device model data during the end of domain save. > > Signed-off-by: Ia

Re: [Xen-devel] [PATCH 1/5] tools/mkrpm: improve version.release handling

2015-03-24 Thread Olaf Hering
On Mon, Mar 23, George Dunlap wrote: > On 03/23/2015 06:49 PM, Olaf Hering wrote: > > release will be anything after a dash, or 0. And an optional ".$PKG_RELEASE" > > will be appended. > Just trying to decode this: So if I were using PKG_RELEASE=$(date ...), > then I'd get something like > > xen-

Re: [Xen-devel] ocaml libxl bindings and KeyedUnion

2015-03-24 Thread Olaf Hering
On Fri, Mar 20, Ian Campbell wrote: > On Fri, 2015-03-20 at 17:44 +0100, Olaf Hering wrote: > > + ("dev", Struct(None, [("m", libxl_vscsi_hctl)])), > > + ("wwn", Struct(None, [("m", string)])), > > + ("hctl", Struct(None, [("m", libxl_vscsi_hctl)])), > > > > Aside: What is

Re: [Xen-devel] [PATCH v2 7/7] tools: add tools support for Intel CAT

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 19:20 +0800, Chao Peng wrote: > On Thu, Mar 19, 2015 at 12:51:19PM +, Ian Campbell wrote: > > On Thu, 2015-03-19 at 18:41 +0800, Chao Peng wrote: > > > This is the xc/xl changes to support Intel Cache Allocation > > > Technology(CAT). Two commands are introduced: > > > - x

Re: [Xen-devel] [PATCH OSSTEST 1/2] uboot: use "readlink -f"

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 11:45 +, Wei Liu wrote: > If the path is not a symlink, readlink by default returns empty string. > Use "-f" to always return canonical path. This fixes the problem that > xenpolicy file not getting loaded (because it is not a symlink). > > Also change another spot that c

Re: [Xen-devel] [PATCH OSSTEST 2/2] uboot: make flask loading address host property

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 11:45 +, Wei Liu wrote: > Signed-off-by: Wei Liu Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH OSSTEST] README.dev: Document steps after restarting ms-ownerdaemon

2015-03-24 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] README.dev: Document steps after restarting ms-ownerdaemon"): > +After restarting ms-ownerdaemon > +=== > + > +... including after an (unexpected) restart of the controller VM. Acked-by: Ian Jackson __

Re: [Xen-devel] [PATCH OSSTEST] start/stop the guest 10 times in the standard test jobs

2015-03-24 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] start/stop the guest 10 times in the standard test jobs"): > We already arrange by more adhoc means to repeat a localhost migration > 10 times. Assuming you have tested this and observed that it generates the right testid, Acked-by: Ian Jackson Ian. _

Re: [Xen-devel] [PATCH OSSTEST] Arrange for core dumps to be placed in /var/core and collect them

2015-03-24 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] Arrange for core dumps to be placed in /var/core and collect them"): > Refactor the $kvp_replace helper in ts-xen-install into a generic > helper (which requires using ::EO and ::EI for namespacing) for use > with target_editfile [etc.] Acked-by: Ian Jackson

Re: [Xen-devel] [PATCH OSSTEST] start/stop the guest 10 times in the standard test jobs

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 12:37 +, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST] start/stop the guest 10 times in the > standard test jobs"): > > We already arrange by more adhoc means to repeat a localhost migration > > 10 times. > > Assuming you have tested this and observed that i

Re: [Xen-devel] [PATCH OSSTEST] Arrange for core dumps to be placed in /var/core and collect them

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 12:40 +, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST] Arrange for core dumps to be placed in > /var/core and collect them"): > > Refactor the $kvp_replace helper in ts-xen-install into a generic > > helper (which requires using ::EO and ::EI for namespacing)

Re: [Xen-devel] [PATCH OSSTEST] start/stop the guest 10 times in the standard test jobs

2015-03-24 Thread Ian Campbell
On Tue, 2015-03-24 at 12:42 +, Ian Campbell wrote: > On Tue, 2015-03-24 at 12:37 +, Ian Jackson wrote: > > Ian Campbell writes ("[PATCH OSSTEST] start/stop the guest 10 times in the > > standard test jobs"): > > > We already arrange by more adhoc means to repeat a localhost migration > > >

Re: [Xen-devel] [PATCH OSSTEST] start/stop the guest 10 times in the standard test jobs

2015-03-24 Thread Ian Jackson
Ian Campbell writes ("Re: [Xen-devel] [PATCH OSSTEST] start/stop the guest 10 times in the standard test jobs"): > From: Ian Campbell > Date: Wed, 11 Mar 2015 10:08:57 + > Subject: [PATCH] start/stop the guest 10 times in the standard test jobs Acked-by: Ian Jackson Thanks, Ian. _

[Xen-devel] [PATCH OSSTEST] standalone: Add --dry-run option for run-job.

2015-03-24 Thread Ian Campbell
Signed-off-by: Ian Campbell --- standalone | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/standalone b/standalone index caf3fd5..d8113e4 100755 --- a/standalone +++ b/standalone @@ -17,7 +17,7 @@ Operations: Setup dist path runvars for JOB from existing build resul

Re: [Xen-devel] [PATCH OSSTEST] standalone: Add --dry-run option for run-job.

2015-03-24 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] standalone: Add --dry-run option for run-job."): > Signed-off-by: Ian Campbell Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH V13 5/7] xen/arm: Instruction prefetch abort (X) mem_event handling

2015-03-24 Thread Tamas K Lengyel
On Mon, Mar 23, 2015 at 5:47 PM, Tamas K Lengyel wrote: > On Mon, Mar 23, 2015 at 5:22 PM, Ian Campbell > wrote: > > On Mon, 2015-03-23 at 16:47 +0100, Tamas K Lengyel wrote: > >> > >> > >> On Mon, Mar 23, 2015 at 4:18 PM, Ian Campbell > >> wrote: > >> On Mon, 2015-03-23 at 15:15 +,

[Xen-devel] [PATCH 0/3] Automatically derive soft affinity from vnuma information

2015-03-24 Thread Dario Faggioli
Hi, This small series is meant at achieving the following two goals: - when building a vnuma enabled guest, if any vcpu hard or soft affinity is specified, and that does not match with the vnuma configuration supplied (e.g., vcpu X, belonging to vnode Y on pnode W, has affinity with pcpus

[Xen-devel] [PATCH 1/3] libxl: check whether vcpu affinity and vnuma info match

2015-03-24 Thread Dario Faggioli
More specifically, vcpus are assigned to a vnode, which in turn is associated with a pnode. If a vcpu also has, in its (hard or soft) affinity, some pcpus that are not part of the said pnode, print a warning to the user. Signed-off-by: Dario Faggioli Cc: Ian Campbell Cc: Ian Jackson Cc: Stefano

[Xen-devel] [PATCH 2/3] libxl: automatically set soft affinity after vnuma info

2015-03-24 Thread Dario Faggioli
More specifically, vcpus are assigned to a vnode, which in turn is associated with a pnode. If a vcpu does not have any soft affinity, automatically build up one, matching the pcpus of the said pnode. Signed-off-by: Dario Faggioli Cc: Ian Campbell Cc: Ian Jackson Cc: Stefano Stabellini Cc: Wei

[Xen-devel] [PATCH 3/3] libxl: cleanup some misuse of 'cpumap' as parameter

2015-03-24 Thread Dario Faggioli
in favour of the more generic 'bitmap', which is better since these are generic libxl_bitmap_* functions. Also fix a typo, and remove a stale (and wrong) comment. No functional change intended. Signed-off-by: Dario Faggioli Cc: Ian Campbell Cc: Ian Jackson Cc: Stefano Stabellini Cc: Wei Liu

Re: [Xen-devel] [PATCH 2/2] x86: Don't use BAD_APICID for non-APICID fields

2015-03-24 Thread Boris Ostrovsky
On 03/24/2015 04:07 AM, Jan Beulich wrote: On 23.03.15 at 20:54, wrote: BAD_APICID is used by cpuinfo_x86's phys_proc_id, cpu_core_id and compute_unit_id even though these fields don't hold an APIC ID itself but rather its derivative. Provide appropriate macros for each of those three (and mak

Re: [Xen-devel] Xen unstability on HP Moonshot m400

2015-03-24 Thread Mark Salter
On Mon, 2015-03-23 at 23:58 +, Stefano Stabellini wrote: > On Mon, 23 Mar 2015, Christoffer Dall wrote: > > On Mon, Mar 23, 2015 at 1:36 PM, Ian Campbell > > wrote: > > On Sat, 2015-03-21 at 13:34 +0100, Christoffer Dall wrote: > > > Hi, > > > > > > I have been experie

Re: [Xen-devel] [PATCH v19 05/14] x86/VPMU: Interface for setting PMU mode and flags

2015-03-24 Thread Jan Beulich
>>> On 17.03.15 at 15:54, wrote: > Add runtime interface for setting PMU mode and flags. Three main modes are > provided: > * XENPMU_MODE_OFF: PMU is not virtualized > * XENPMU_MODE_SELF: Guests can access PMU MSRs and receive PMU interrupts. > * XENPMU_MODE_HV: Same as XENPMU_MODE_SELF for non-p

Re: [Xen-devel] Xen unstability on HP Moonshot m400

2015-03-24 Thread Mark Salter
On Tue, 2015-03-24 at 09:54 -0400, Mark Salter wrote: > On Mon, 2015-03-23 at 23:58 +, Stefano Stabellini wrote: > > On Mon, 23 Mar 2015, Christoffer Dall wrote: > > > On Mon, Mar 23, 2015 at 1:36 PM, Ian Campbell > > > wrote: > > > On Sat, 2015-03-21 at 13:34 +0100, Christoffer Dall wr

Re: [Xen-devel] [PATCH v2] xen/passthrough: Support a single iommu_domain per xen domain per SMMU

2015-03-24 Thread Robert VanVossen
On 3/24/2015 7:07 AM, Manish Jaggi wrote: > > On Monday 23 March 2015 07:16 PM, Robbie VanVossen wrote: >> If multiple devices are being passed through to the same domain and they >> share a single SMMU, then they only require a single iommu_domain. >> >> In arm_smmu_assign_dev, before a new iom

Re: [Xen-devel] [PATCH v19 07/14] x86/VPMU: Initialize PMU for PV(H) guests

2015-03-24 Thread Jan Beulich
>>> On 17.03.15 at 15:54, wrote: > @@ -263,14 +264,14 @@ void vpmu_initialise(struct vcpu *v) > BUILD_BUG_ON(sizeof(struct xen_pmu_intel_ctxt) > XENPMU_CTXT_PAD_SZ); > BUILD_BUG_ON(sizeof(struct xen_pmu_amd_ctxt) > XENPMU_CTXT_PAD_SZ); > > -if ( is_pvh_vcpu(v) ) > -return;

Re: [Xen-devel] [PATCH v19 08/14] x86/VPMU: Save VPMU state for PV guests during context switch

2015-03-24 Thread Jan Beulich
>>> On 17.03.15 at 15:54, wrote: > Save VPMU state during context switch for both HVM and PV(H) guests. > > A subsequent patch ("x86/VPMU: NMI-based VPMU support") will make it possible > for vpmu_switch_to() to call vmx_vmcs_try_enter()->vcpu_pause() which needs > is_running to be correctly set/

Re: [Xen-devel] [PATCH 2/2] make {,g}dprintk() a no‑op in non‑debug builds

2015-03-24 Thread Ian Campbell
On Fri, 2015-03-20 at 15:13 +, Jan Beulich wrote: > As discussed before, code absolutely wanting to get something into the > log should use {,g}printk() instead. > > Signed-off-by: Jan Beulich Acked-by: Ian Campbell ___ Xen-devel mailing list X

Re: [Xen-devel] [PATCH 1/2] introduce gprintk()

2015-03-24 Thread Ian Campbell
On Fri, 2015-03-20 at 15:12 +, Jan Beulich wrote: > ... and convert several gdprintk()-s to it, as the next patch will make > them no-ops in non-debug builds. > > Note that as a non-debug facility this does not print file name and > line number of the origin, to people are expected to use mean

[Xen-devel] [PATCH OSSTEST v2] Arrange for core dumps to be placed in /var/core and collect them

2015-03-24 Thread Ian Campbell
Refactor the $kvp_replace helper in ts-xen-install into a generic helper (which requires using ::EO and ::EI for namespacing) for use with target_editfile and use it to edit /etc/sysctl.conf to set kernel.core_pattern on boot. Tested in standalone mode by installing and running a C program contain

Re: [Xen-devel] [PATCH] hvmloader: don't treat ROM BAR like other BARs

2015-03-24 Thread Ian Campbell
On Fri, 2015-03-20 at 15:20 +, Jan Beulich wrote: > Its low 11 bits have different meaning. > > Signed-off-by: Jan Beulich Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [linux-3.16 test] 36613: regressions - trouble: broken/fail/pass

2015-03-24 Thread xen . org
flight 36613 linux-3.16 real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/36613/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-credit2 15 guest-localmigrate/x10fail REGR. vs. 34167 test-amd64-i386-pair

Re: [Xen-devel] [PATCH v19 11/14] x86/VPMU: Handle PMU interrupts for PV guests

2015-03-24 Thread Jan Beulich
>>> On 17.03.15 at 15:54, wrote: > Changes in v19: > * Adjusted for new ops interfaces (passing vcpu vs. vpmu) > * Test for domain->max_cpu in choose_hwdom_vcpu() instead of > 'domain->vcpu!=NULL' I suppose that's something that then should also be done in patch 7? > --- a/xen/arch/x86/hvm/vpmu

Re: [Xen-devel] [PATCH v3 1/5] libxl: introduce libxl__device_model_xs_path

2015-03-24 Thread Ian Campbell
On Fri, 2015-03-20 at 16:19 +, Wei Liu wrote: > +_hidden char *libxl__device_model_xs_path(libxl__gc *gc, uint32_t dm_domid, > + uint32_t domid, > + const char *format, ...); This should have the PRINTF_ATTRIBUT

Re: [Xen-devel] [PATCH 2/3] libxl: automatically set soft affinity after vnuma info

2015-03-24 Thread Wei Liu
On Tue, Mar 24, 2015 at 02:42:15PM +0100, Dario Faggioli wrote: > More specifically, vcpus are assigned to a vnode, which in > turn is associated with a pnode. If a vcpu does not have any > soft affinity, automatically build up one, matching the pcpus > of the said pnode. > What about hard affini

[Xen-devel] [PATCH v2] tools/mkrpm: improve version.release handling

2015-03-24 Thread Olaf Hering
An increasing version and/or release number helps to update existing packages without --force as in "rpm Uvh --force xen.rpm". Instead its possible to do "rpm -Fvh *.rpm" to update only already installed packages. The usage of --force disables essentials checks such as file conflict detection. As

Re: [Xen-devel] [PATCH 1/3] libxl: check whether vcpu affinity and vnuma info match

2015-03-24 Thread Wei Liu
On Tue, Mar 24, 2015 at 02:41:48PM +0100, Dario Faggioli wrote: > More specifically, vcpus are assigned to a vnode, which in > turn is associated with a pnode. If a vcpu also has, in its > (hard or soft) affinity, some pcpus that are not part of the > said pnode, print a warning to the user. > Cu

Re: [Xen-devel] [PATCH 3/3] libxl: cleanup some misuse of 'cpumap' as parameter

2015-03-24 Thread Wei Liu
On Tue, Mar 24, 2015 at 02:42:27PM +0100, Dario Faggioli wrote: > in favour of the more generic 'bitmap', which is better > since these are generic libxl_bitmap_* functions. > > Also fix a typo, and remove a stale (and wrong) comment. > > No functional change intended. > > Signed-off-by: Dario F

  1   2   3   >