Re: [Xen-devel] Running Xen on Nvidia Jetson-TK1

2016-04-08 Thread Ian Campbell
On Fri, 2016-04-08 at 11:10 +0100, Julien Grall wrote: > > In your patch for *Hacky* support for Jetsok-TK1 you said that you > > were able to run guests on > > Jetson-tk1 board with Xen. Can I know which kernel version you used as > > dom0 (and possibly domU guests)? I'm afraid I don't remember.

Re: [Xen-devel] [xen-unstable test] 63540: regressions - FAIL

2015-11-05 Thread Ian Campbell
On Thu, 2015-11-05 at 03:49 -0700, Jan Beulich wrote: > > > > On 05.11.15 at 04:01, wrote: > > flight 63540 xen-unstable real [real] > > http://logs.test-lab.xenproject.org/osstest/logs/63540/ > > > > Regressions :-( > > > > Tests which did not succeed and are blocking, > > including tests whic

[Xen-devel] [PATCH] tools: pygrub: if partition table is empty, try treating as a whole disk

2015-11-05 Thread Ian Campbell
=512 count=1 to add an MBR (with 0xaa55 signature) to it, which after this patch also works. Signed-off-by: Ian Campbell Cc: 745419-forwar...@bugs.debian.org --- tools/pygrub/src/pygrub | 5 + 1 file changed, 5 insertions(+) diff --git a/tools/pygrub/src/pygrub b/tools/pygrub/src/pygrub ind

Re: [Xen-devel] Xen on Arm Arndale Exynos5250

2015-11-06 Thread Ian Campbell
On Thu, 2015-11-05 at 12:59 -0600, ed sandberg wrote: > > My initial thought was that I needed to add a getty process on COM3, so > that when I switched to Dom0 mode I could login. This page seems to > agree with me: > > http://wiki.xenproject.org/wiki/Xen_Serial_Console > > I tried to do so b

Re: [Xen-devel] [PATCH 4/4] xen/public: arm: rework the macro set_xen_guest_handle_raw

2015-11-06 Thread Ian Campbell
On Wed, 2015-11-04 at 16:50 +, Ian Jackson wrote: > Jan Beulich writes ("Re: [PATCH 4/4] xen/public: arm: rework the > macro set_xen_guest_handle_raw"): > > All quite interesting, but pretty moot with there not being any > > get_xen_guest_handle() anymore. > > If we don't provide a get_xen_gue

Re: [Xen-devel] Xen on Arm Arndale Exynos5250

2015-11-06 Thread Ian Campbell
On Fri, 2015-11-06 at 08:38 -0600, ed sandberg wrote: Please don't top post. > Thanks for the help Ian, > # setenv xen_bootargs 'console=dtuart dtuart=/serial@12C2 > dom0_mem=512M' > # setenv dom0_bootargs 'console=hvc0 console=ttySAC2,115200n8 > ignore_loglevel psci=enable clk_ignore_unused

[Xen-devel] [Minios-devel] [PATCH v4 0/] Begin to disentangle libxenctrl and provide some stable libraries

2015-11-09 Thread Ian Campbell
In <1431963008.4944.80.ca...@citrix.com> I proposed stabilising some parts of the libxenctrl API/ABI by disaggregating into separate libraries. This is v5 of that set of series against: xen qemu-xen qemu-xen-traditional mini-os NB: Samuel+minios-devel will only get the mini-os sid

[Xen-devel] [PATCH XEN v5 01/23] tools/Rules.mk: Properly handle libraries with recursive dependencies.

2015-11-09 Thread Ian Campbell
dependencies more than 1 step deep. Introduce a SHDEPS idiom to contain all the recursive deps for a library and include those in both LDLIBS (for linking) and SHLIB (for recursive uses). Try and document the whole thing. Signed-off-by: Ian Campbell Acked-by: Ian Jackson Acked-by: Wei Liu --- v5

[Xen-devel] [PATCH XEN v5 08/23] tools/libxc: Remove osdep indirection for privcmd

2015-11-09 Thread Ian Campbell
, which could have been achieved by using some ifdefs around a renamed function. This will fall out in the wash when these functions move to their own library. Signed-off-by: Ian Campbell Acked-by: David Scott Acked-by: Wei Liu Cc: David Scott --- config/FreeBSD.mk | 2

[Xen-devel] [PATCH XEN v5 00/23] Begin to disentangle libxenctrl and provide some stable libraries

2015-11-09 Thread Ian Campbell
ated patch series and which contains more details. Ian Campbell (23): tools/Rules.mk: Properly handle libraries with recursive dependencies. tools: Refactor "xentoollog" into its own library tools/libxc: Remove osdep indirection for xc_evtchn tools: Refactor /dev/xen/evtchn wrappe

[Xen-devel] [PATCH XEN v5 05/23] tools: Arrange to check public headers for ANSI compatiblity

2015-11-09 Thread Ian Campbell
Using the same rune as we use for the Xen public headers, except we do not need stdint.h here and we use -pedantic too. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- .gitignore | 2 ++ tools/Rules.mk | 8 tools/libs/evtchn/Makefile | 4 +++- tools

[Xen-devel] [PATCH XEN v5 14/23] tools: foreignmemory: provide xenforeignmemory_unmap.

2015-11-09 Thread Ian Campbell
advantage of this and will need converting. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- v4: xenforeignmemory_unmap takes pages not bytes, adjust callers. --- tools/libs/foreignmemory/freebsd.c | 6 ++ tools/libs/foreignmemory/include/xenforeignmemory.h | 7 +-- tools

[Xen-devel] [PATCH XEN v5 02/23] tools: Refactor "xentoollog" into its own library

2015-11-09 Thread Ian Campbell
ainst libxenctrl. The new library uses a version script to ensure that only expected symbols are exported and to version them such that ABI guarantees can be kept in the future. Signed-off-by: Ian Campbell Acked-by: Ian Jackson Acked-by: Wei Liu --- Must be applied with: - "qemu-xen-tradit

[Xen-devel] [PATCH XEN v5 03/23] tools/libxc: Remove osdep indirection for xc_evtchn

2015-11-09 Thread Ian Campbell
incorrect instance of using xc_interface where xc_evtchn (in ocaml stubs) is removed, this used to work because they were typedefs to the same struct, but is no longer permitted. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/libxc/include/xenctrl.h | 2 +- tools/libxc

[Xen-devel] [PATCH XEN v5 19/23] tools/libs/call: Update some log messages to not refer to xc.

2015-11-09 Thread Ian Campbell
Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/libs/call/linux.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/libs/call/linux.c b/tools/libs/call/linux.c index 906ca7e..80b505c 100644 --- a/tools/libs/call/linux.c +++ b/tools/libs/call/linux.c @@ -88,7

[Xen-devel] [PATCH XEN v5 15/23] foreignmemory: use size_t for size arguments.

2015-11-09 Thread Ian Campbell
Surprisingly it appears no callers need updating. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- v4: New patch --- tools/libs/foreignmemory/compat.c | 2 +- tools/libs/foreignmemory/freebsd.c | 4 ++-- .../libs/foreignmemory/include/xenforeignmemory.h

[Xen-devel] [PATCH XEN v5 17/23] tools/libs: Clean up hard tabs.

2015-11-09 Thread Ian Campbell
These were wrong in the context of libxc before this code was extracted, clean them up. Also add some emacs magic blocks Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/libs/call/buffer.c | 4 ++-- tools/libs/call/core.c| 2 +- tools/libs/call/minios.c

[Xen-devel] [PATCH QEMU-XEN v5 2/9] xen: Switch to libxenevtchn interface for compat shims.

2015-11-09 Thread Ian Campbell
ins in libxenctrl, since that functionality is not exposed by /dev/xen/evtchn. Signed-off-by: Ian Campbell Reviewed-by: Stefano Stabellini --- v4: Ran checkpatch, fixed all errors Allocate correct size for handle (i.e. not size of the ptr) --- hw/xen/xen_backend.c |

[Xen-devel] [PATCH XEN v5 11/23] tools: Remove xc_map_foreign_batch

2015-11-09 Thread Ian Campbell
always implement xc_map_foreign_bulk instead. Signed-off-by: Ian Campbell Acked-by: George Dunlap Acked-by: Wei Liu Cc: George Dunlap --- tools/libxc/include/xenctrl.h | 10 --- tools/libxc/xc_foreign_memory.c | 4 ++- tools/libxc/xc_linux_osdep.c| 59

[Xen-devel] [PATCH QEMU-XEN v5 7/9] xen: Use stable library interfaces when they are available.

2015-11-09 Thread Ian Campbell
time. The only non-obvious bit is that we now open a proper xenforeignmemory handle for xen_fmem instead of reusing the xen_xc handle. Build tested with 4.0 .. 4.6 (inclusive) and the patches targetting 4.7 which adds these libraries. Signed-off-by: Ian Campbell --- v5: Remove ifdef check when

[Xen-devel] [PATCH XEN v5 16/23] tools/libs/evtchn: Review and update doc comments.

2015-11-09 Thread Ian Campbell
Remove the reference to pre-4.1, since this is now a new library. Fixup references to xc. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/libs/evtchn/include/xenevtchn.h | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/tools/libs/evtchn/include/xenevtchn.h b

[Xen-devel] [PATCH MINI-OS v5 4/5] mini-os: Include libxencall with libxc

2015-11-09 Thread Ian Campbell
libxencall has just been split out from libxc. From mini-os's point of view we don't care about the distinction, so keep things simple by just including libxencall if libxc is enabled. Signed-off-by: Ian Campbell Acked-by: Samuel Thibault Acked-by: Wei Liu --- v2: Adjust for libs/$

[Xen-devel] [PATCH QEMU-XEN v5 6/9] xen: Switch uses of xc_map_foreign_bulk to use libxenforeignmemory API.

2015-11-09 Thread Ian Campbell
s is thought to be a problem then we could consider adding an extra API to the libxenforeignmemory interface to replace a foreign mapping with anonymous shared memory, but I'd prefer not to. Signed-off-by: Ian Campbell Reviewed-by: Stefano Stabellini --- I noticed in xen_console.c that the

[Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-09 Thread Ian Campbell
such that ABI guarantees can be kept in the future. After this change libxenvchan no longer needs to link against libxenctrl. It still needs xenctrl.h in one file for xen_mb and friends. Signed-off-by: Ian Campbell --- Must be applied with: - "qemu-xen-traditional: Use libxengnt

[Xen-devel] [PATCH QEMU-XEN-TRADITIONAL v5 4/5] qemu-xen-traditional: Add libxencall to rpath-link

2015-11-09 Thread Ian Campbell
libxenctrl links against this library Signed-off-by: Ian Campbell --- v3: Library moved to tools/libs --- xen-hooks.mak | 1 + 1 file changed, 1 insertion(+) diff --git a/xen-hooks.mak b/xen-hooks.mak index 179a6b7..229d642 100644 --- a/xen-hooks.mak +++ b/xen-hooks.mak @@ -25,6 +25,7 @@ LIBS

[Xen-devel] [PATCH QEMU-XEN-TRADITIONAL v5 3/5] qemu-xen-traditional: Use libxengnttab

2015-11-09 Thread Ian Campbell
/dev/xen/gntdev related wrappers have been moved out of libxenctrl into their own library. Signed-off-by: Ian Campbell --- v3: Library moved to tools/libs/ --- hw/xen_backend.c | 4 ++-- hw/xen_backend.h | 2 +- hw/xen_common.h | 1 + hw/xen_console.c | 4 ++-- hw/xen_disk.c| 24

[Xen-devel] [PATCH XEN v5 23/23] HACK: Update Config.mk to pull all the right bits from my xenbits trees

2015-11-09 Thread Ian Campbell
v4: Config.mk instead of .config --- Config.mk | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/Config.mk b/Config.mk index 5db7ca5..7258065 100644 --- a/Config.mk +++ b/Config.mk @@ -242,20 +242,20 @@ endif ifeq ($(GIT_HTTP),y) OVMF_UPSTREAM_URL ?= ht

[Xen-devel] [PATCH XEN v5 06/23] tools/libxc: Remove osdep indirection for xc_gnt{shr, tab}

2015-11-09 Thread Ian Campbell
should never be called. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/libxc/Makefile | 10 ++-- tools/libxc/include/xenctrl.h | 4 +- tools/libxc/include/xenctrlosdep.h | 23 --- tools/libxc/xc_gnttab.c| 57 -- tools/libxc

[Xen-devel] [PATCH QEMU-XEN v5 1/9] xen_console: correctly cleanup primary console on teardown.

2015-11-09 Thread Ian Campbell
r neither. For consistency with con_initialise() with to the former here too. With this con_initialise and con_disconnect now mirror each other. Fix up a hard tab in the function while editing. Signed-off-by: Ian Campbell Reviewed-by: Stefano Stabellini --- v4: New patch based on feedback to &qu

[Xen-devel] [PATCH XEN v5 12/23] tools: Implement xc_map_foreign_range(s) in terms of common helper

2015-11-09 Thread Ian Campbell
fewer low level interfaces to think about. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/libxc/xc_foreign_memory.c | 50 + tools/libxc/xc_freebsd_osdep.c | 50 - tools/libxc/xc_linux_osdep.c| 49

[Xen-devel] [PATCH XEN v5 13/23] tools: Refactor foreign memory mapping into libxenforeignmemory

2015-11-09 Thread Ian Campbell
). Signed-off-by: Ian Campbell Acked-by: Wei Liu --- Must be applied with: - "qemu-xen-traditional: qemu-xen-traditional: Add libxenforeignmemory to rpath-link" and a corresponding QEMU_TAG update folded here. - "mini-os: mini-os: Include libxenforeignmemory wi

[Xen-devel] [PATCH QEMU-XEN v5 5/9] xen: Switch uses of xc_map_foreign_pages into xc_map_foreign_bulk

2015-11-09 Thread Ian Campbell
xc_map_foreign_bulk. In preparation for this switch both uses of xc_map_foreign_pages (which both happen to be in xenfb_map_fb) to xc_map_foreign_bulk. This simply requires allocating and passing a new err array (the same one for both calls). Signed-off-by: Ian Campbell Acked-by: Stefano

[Xen-devel] [PATCH XEN v5 10/23] tools/libxc: drop xc_map_foreign_bulk_compat wrappers

2015-11-09 Thread Ian Campbell
function all new ports should instead implement xc_map_foreign_bulk properly, hence the ifdef should never be expanded. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/libxc/xc_foreign_memory.c | 13 + tools/libxc/xc_netbsd.c | 7 --- tools/libxc/xc_private.h

[Xen-devel] [PATCH QEMU-XEN v5 4/9] xen: Switch uses of xc_map_foreign_range into xc_map_foreign_bulk

2015-11-09 Thread Ian Campbell
what circumstances xendev->dev can be NULL or if anything elsewhere ensures the value fits into an int. For now I just use a temporary xen_pfn_t to in effect upcast the pointer from int* to xen_pfn_t*. Signed-off-by: Ian Campbell Reviewed-by: Stefano Stabellini --- v4: Ran checkpatch, fixed

[Xen-devel] [PATCH MINI-OS v5 3/5] mini-os: Include libxengnttab with libxc

2015-11-09 Thread Ian Campbell
libxengnttab has just been split out from libxc. From mini-os's point of view we don't care about the distinction, so keep things simple by just including libxengnttab if libxc is enabled. Signed-off-by: Ian Campbell Acked-by: Samuel Thibault Acked-by: Wei Liu --- v2: Adjust for

[Xen-devel] [PATCH XEN v5 18/23] tools/libs/gnttab: Review and update doc comments.

2015-11-09 Thread Ian Campbell
Remove some stray xc references. While I'm not convinced by javadoc/doxygen cause the existing comments which appear to use that syntax to have the appropriate /** marker. Also fix a typo in a code comment. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/libs/gnttab/in

[Xen-devel] [PATCH QEMU-XEN v5 0/9] Begin to disentangle libxenctrl and provide some stable libraries

2015-11-09 Thread Ian Campbell
ated patch series and which contains more details. Ian Campbell (9): xen_console: correctly cleanup primary console on teardown. xen: Switch to libxenevtchn interface for compat shims. xen: Switch to libxengnttab interface for compat shims. xen: Switch uses of xc_map_foreign_

[Xen-devel] [PATCH QEMU-XEN-TRADITIONAL v5 0/5] Begin to disentangle libxenctrl and provide some stable libraries

2015-11-09 Thread Ian Campbell
all of the related patch series and which contains more details. Ian Campbell (5): qemu-xen-traditional: Use xentoollog as a separate library qemu-xen-traditional: Use libxenevtchn qemu-xen-traditional: Use libxengnttab qemu-xen-traditional: Add libxencall to rpath-link qemu-xen-tradit

[Xen-devel] [PATCH XEN v5 09/23] tools: Refactor hypercall calling wrappers into libxencall.

2015-11-09 Thread Ian Campbell
. Foreign memory mapping is logically separate functionality and will be moved into its own library. The new library uses a version script to ensure that only expected symbols are exported and to version them such that ABI guarantees can be kept in the future. Signed-off-by: Ian Campbell Acked-by: Wei

[Xen-devel] [PATCH XEN v5 22/23] tools: Update CFLAGS for qemu-xen to allow it to use new libraries

2015-11-09 Thread Ian Campbell
This means adding -L for libxen{evtchn,gnttab,foreignmemory} so that it can link them directly (rather than using the libxenctrl compat layer exposed via -rpath-link). Also add -I for libxenforeignmemory. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/Makefile | 4 1 file changed

[Xen-devel] [PATCH QEMU-XEN-TRADITIONAL v5 1/5] qemu-xen-traditional: Use xentoollog as a separate library

2015-11-09 Thread Ian Campbell
This has been split out of libxenctrl, so the build needs to be able to find the header and the library. QEMU does not use xtl_* itself so -rpath-link is sufficient to allow linking to libxenctrl.so (which links against libxentoollog). Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- v3

[Xen-devel] [PATCH QEMU-XEN v5 8/9] xen: domainbuild: reopen libxenctrl interface after forking for domain watcher.

2015-11-09 Thread Ian Campbell
Using an existing libxenctrl handle after a fork was never particularly safe (especially if foreign mappings existed at the time of the fork) and the xc fd has been unavailable for many releases. Reopen the handle after fork and therefore do away with xc_fd(). Signed-off-by: Ian Campbell Acked

[Xen-devel] [PATCH MINI-OS v5 1/5] mini-os: Include libxentoollog with libxc

2015-11-09 Thread Ian Campbell
libxentoollog has just been split out from libxc. From mini-os's point of view we don't care about the distinction, so keep things simple by just including libxentoollog if libxc is enabled. Signed-off-by: Ian Campbell Acked-by: Samuel Thibault Acked-by: Wei Liu --- v2: Adjust for

[Xen-devel] [PATCH XEN v5 21/23] tools/libs/foreignmemory: Mention restrictions on fork in docs.

2015-11-09 Thread Ian Campbell
Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/libs/foreignmemory/include/xenforeignmemory.h | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/tools/libs/foreignmemory/include/xenforeignmemory.h b/tools/libs/foreignmemory/include/xenforeignmemory.h index

[Xen-devel] [PATCH MINI-OS v5 2/5] mini-os: Include libxenevtchn with libxc

2015-11-09 Thread Ian Campbell
libxenevtchn has just been split out from libxc. From mini-os's point of view we don't care about the distinction, so keep things simple by just including libxenevtchn if libxc is enabled. Signed-off-by: Ian Campbell Acked-by: Samuel Thibault Acked-by: Wei Liu --- v2: Adjust for

[Xen-devel] [PATCH MINI-OS v5 0/5] Begin to disentangle libxenctrl and provide some stable libraries

2015-11-09 Thread Ian Campbell
ated patch series and which contains more details. Ian Campbell (5): mini-os: Include libxentoollog with libxc mini-os: Include libxenevtchn with libxc mini-os: Include libxengnttab with libxc mini-os: Include libxencall with libxc mini-os: Include libxenforeignmemory with libxc Ma

[Xen-devel] [PATCH QEMU-XEN v5 3/9] xen: Switch to libxengnttab interface for compat shims.

2015-11-09 Thread Ian Campbell
is always a pointer. This leads to less typedef headaches and the need for XC_HANDLER_INITIAL_VALUE etc for these interfaces. Note that this patch does not add any support for actually using libxengnttab, it just adjusts the existing shims. Signed-off-by: Ian Campbell --- v4: Ran checkpatch, fi

Re: [Xen-devel] [Minios-devel] [PATCH v4 0/] Begin to disentangle libxenctrl and provide some stable libraries

2015-11-09 Thread Ian Campbell
I've just sent out v5. Anyone got any thoughts on how to proceed with qemu- xen? On Wed, 2015-10-21 at 16:47 +0100, Ian Campbell wrote: > (Trimming CCs a bit) > > On Wed, 2015-10-21 at 16:22 +0100, Ian Campbell wrote: > > > [...] > > Still to come would be libra

[Xen-devel] [PATCH QEMU-XEN v5 9/9] xen: make it possible to build without the Xen PV domain builder

2015-11-09 Thread Ian Campbell
reate - xc_domain_destroy - xc_domain_getinfo - xc_domain_max_vcpus - xc_domain_setmaxmem - xc_domain_unpause - xc_evtchn_alloc_unbound - xc_linux_build This is another step towards only using Xen libraries which provide a stable inteface. Signed-off-by: Ian Campbell --- v5: XEN_C

[Xen-devel] [PATCH QEMU-XEN-TRADITIONAL v5 2/5] qemu-xen-traditional: Use libxenevtchn

2015-11-09 Thread Ian Campbell
/dev/xen/evtchn related wrappers have been moved out of libxenctrl into their own library. Note that i386-dm/helper2.c's xc_interface * was always really an xc_evtchn *, it's just they used to be typedefs to the same thing... Signed-off-by: Ian Campbell --- v3: Library moved to

[Xen-devel] [PATCH MINI-OS v5 5/5] mini-os: Include libxenforeignmemory with libxc

2015-11-09 Thread Ian Campbell
libxenforeignmemory has just been split out from libxc. From mini-os's point of view we don't care about the distinction, so keep things simple by just including libxenforeignmemory if libxc is enabled. Signed-off-by: Ian Campbell Acked-by: Samuel Thibault Acked-by: Wei Liu --- v2:

[Xen-devel] [PATCH XEN v5 20/23] tools/libs/call: Avoid xc_memalign in netbsd and solaris backends

2015-11-09 Thread Ian Campbell
These are already arch specific, so just use the appropriate interfaces (as determined by looking at the xc_memalign backend). Signed-off-by: Ian Campbell Acked-by: Wei Liu --- tools/libs/call/netbsd.c | 4 ++-- tools/libs/call/solaris.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions

[Xen-devel] [PATCH QEMU-XEN-TRADITIONAL v5 5/5] qemu-xen-traditional: Add libxenforeignmemory to rpath-link

2015-11-09 Thread Ian Campbell
libxenctrl links against this library. Also, request the compat xc_map_foreign API from libxc. Signed-off-by: Ian Campbell --- v3: Library moved to tools/libs/ --- xen-hooks.mak | 2 ++ 1 file changed, 2 insertions(+) diff --git a/xen-hooks.mak b/xen-hooks.mak index 229d642..c1ea4be 100644

Re: [Xen-devel] Xen on Arm Arndale Exynos5250

2015-11-09 Thread Ian Campbell
On Fri, 2015-11-06 at 12:51 -0600, ed sandberg wrote: > > Does your kernel have the Xen drivers (CONFIG_XEN and all which is > > gated on that) enabled. In particular CONFIG_HVC_XEN. > > > Yes the config file I used to compile has CONFIG_XEN=y and > CONFIG_HVC_XEN=y.  Config file is attached. Tha

[Xen-devel] [PATCH] perfc: count number of explicit vcpu yields

2015-11-09 Thread Ian Campbell
Signed-off-by: Ian Campbell --- xen/common/schedule.c| 1 + xen/include/xen/perfc_defn.h | 1 + 2 files changed, 2 insertions(+) diff --git a/xen/common/schedule.c b/xen/common/schedule.c index 292e9a0..86d6cc0 100644 --- a/xen/common/schedule.c +++ b/xen/common/schedule.c @@ -896,6

[Xen-devel] [PATCH] xen: arm: perfc: count individual SGIs and PPIs

2015-11-09 Thread Ian Campbell
SPIs are too numerous and not indivcidually as interesting. Make the exists #IPIS counter x86 specific. Signed-off-by: Ian Campbell Cc: jbeul...@suse.com --- xen/arch/arm/gic.c | 2 +- xen/arch/arm/irq.c | 4 ++-- xen/include/asm-arm/perfc_defn.h | 3 ++- xen

Re: [Xen-devel] [PATCH] xen: arm: perfc: count individual SGIs and PPIs

2015-11-09 Thread Ian Campbell
On Mon, 2015-11-09 at 07:11 -0700, Jan Beulich wrote: > > > > On 09.11.15 at 15:01, wrote: > > SPIs are too numerous and not indivcidually as interesting. > > > > Make the exists #IPIS counter x86 specific. > > I'd recommend not to - even if ARM doesn't want to use it, it's still a > pretty gene

Re: [Xen-devel] [PATCH] perfc: count number of explicit vcpu yields

2015-11-09 Thread Ian Campbell
On Mon, 2015-11-09 at 16:14 +0100, Dario Faggioli wrote: >  > > As far as xl accessibility --- doesn't xenperf already read them out? > > > Mmm... ISTR having tried without much luck, and having heard that it > wasn't functional any longer, but maybe I'm confusing it with something > else. I'll tr

[Xen-devel] Using IRQ_GUEST|IRQ_PER_CPU to signal "delivery to current vcpu"?

2015-11-10 Thread Ian Campbell
Hi arch maintainers, The ARM hardware has a concept called a "Peripheral Private Interrupt" (PPI), an IRQ which is per-core as opposed to a "Shared Peripheral Interrupt" (SPI) which is per-SOC (routable to all cores). (For completeness the other classes of IRQ are "Software Generated Interrupt" (

Re: [Xen-devel] Runtime services support for Xen on ARM

2015-11-10 Thread Ian Campbell
On Tue, 2015-11-10 at 12:26 +, Stefano Stabellini wrote: > CC'ing xen-devel and Jan > > On Tue, 10 Nov 2015, Shannon Zhao wrote: > > Hi Stefano, > > > > I'm working on adding Runtime services support at Xen side. Most of > > work > > is adding the ARM part in xen/common/efi/runtime.c. > > >

[Xen-devel] [PATCH v2 2/8] xen: arm: fix typo in the description of struct pending_irq->desc

2015-11-10 Thread Ian Campbell
s/it/if/ makes more sense. Signed-off-by: Ian Campbell --- xen/include/asm-arm/vgic.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/include/asm-arm/vgic.h b/xen/include/asm-arm/vgic.h index cb51a9e..7d580cc 100644 --- a/xen/include/asm-arm/vgic.h +++ b/xen/include/asm

[Xen-devel] [PATCH v2 7/8] xen: arm: context switch vtimer PPI state.

2015-11-10 Thread Ian Campbell
e and restore the active state of the interrupt, which prevents the nested invocations which the current masking works around. Signed-off-by: Ian Campbell --- v2: Rebased, in particular over Julien's passthrough stuff which reworked a bunch of IRQ related stuff. Also largely rewri

[Xen-devel] [PATCH v2] xen: arm: context switch vtimer PPI state.

2015-11-10 Thread Ian Campbell
... instead of artificially masking the timer interrupt in the timer state and relying on the guest to unmask which it isn't required to do per the h/w spec, although Linux does, other OSes (FreeRTOS?) needed changes. Making the vtimer follow the h/w spec is the main driving force here although it

[Xen-devel] [PATCH v2 6/8] xen: arm: supporting routing a PPI to the current vcpu.

2015-11-10 Thread Ian Campbell
save and restore the PPI state. The vtimer driver will do so shortly. Signed-off-by: Ian Campbell --- xen/arch/arm/gic.c| 24 ++ xen/arch/arm/irq.c| 84 +-- xen/include/asm-arm/gic.h | 2 ++ xen/include/asm-arm/irq.h | 2 +- 4

[Xen-devel] [PATCH v2 3/8] xen: arm: Refactor route_irq_to_guest

2015-11-10 Thread Ian Campbell
aller). I don't think this is an issue (and avoiding this would make things more complex) Signed-off-by: Ian Campbell --- v2: New patch (maybe, it's been a while...) --- xen/arch/arm/irq.c | 104 +++-- 1 file changed, 61 insertions(+), 43 del

[Xen-devel] [PATCH v2 5/8] xen: arm: add interfaces to save/restore the state of a PPI.

2015-11-10 Thread Ian Campbell
h/w state changes), therefore we do not want to context switch the pending state for level PPIs -- instead we rely on the context switch of the peripheral to restore the correct level. Tested on GIC v2 only. Unused as yet, will be used by the vtimer code shortly. Signed-off-by: Ian Campbell

[Xen-devel] [PATCH v2 8/8] HACK: Force virt timer to PPI0 (IRQ16)

2015-11-10 Thread Ian Campbell
Just for testing so the guest vtimer ppi it isn't the same as the physical virt timer PPI on my platform, and therefore allows to exercise the non-1:1 bits of the code. --- xen/include/public/arch-arm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/include/public/arch-arm

[Xen-devel] [PATCH v2 1/8] xen: arm: fix indendation of struct vtimer

2015-11-10 Thread Ian Campbell
Signed-off-by: Ian Campbell --- xen/include/asm-arm/domain.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/xen/include/asm-arm/domain.h b/xen/include/asm-arm/domain.h index e7e40da..c56f06e 100644 --- a/xen/include/asm-arm/domain.h +++ b/xen/include/asm-arm

[Xen-devel] [PATCH v2 4/8] xen: arm: remove vgic_vcpu_inject_spi()

2015-11-10 Thread Ian Campbell
Currently this has only a single caller, which I intend to teach about injecting PPIs shortly. This helper doesn't add much so remove it. Drop a stray tab from the comment immediately preceding this change. Signed-off-by: Ian Campbell --- xen/arch/arm/irq.c | 7 +-- xen/arc

Re: [Xen-devel] Critique of the Xen Security Process

2015-11-11 Thread Ian Campbell
On Mon, 2015-11-09 at 15:48 -0600, Doug Goldstein wrote: >  > I'll echo this sentiment as well. Most distro packagers will dislike > this and need to work around some of this behavior in their respective > distros. This is something we have been working upstream to address as well. As it stands I

Re: [Xen-devel] [OSSTEST PATCH v16 00/22] Nested HVM

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: >  > Ian: I'm not sure whether or not you want to review these.  About half > of the patches are mine; the other half are Robert's but have been > reviewed by me.  All but the last three have been tested by Robert. I'll have a skim. _

Re: [Xen-devel] [OSSTEST PATCH 01/22] cs-adjust-flight: Add some missing doc comment info

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 02/22] cs-adjust-flight: Allow adjusting "this" flight

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: > This allows cs-adjust-flight to be run by hand to adjust runvars, in a > flight being used with hand-invocation of ./ts-* scripts. > > Signed-off-by: Ian Jackson > --- > v14: New patch > --- >  cs-adjust-flight |8 >  1 file chan

Re: [Xen-devel] [OSSTEST PATCH 03/22] selecthost: Minor cleanups

2015-11-11 Thread Ian Campbell
OR > +# can be = > +# which means ignore except for logging purposes etc. > +# and use > +# > +# uis which means use that host (and all                     ^is o/w: Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 04/22] selecthost: Support nested hosts (guests which are also hosts)

2015-11-11 Thread Ian Campbell
1 is up and has obtained its IP address. > > `Power cycling' the L1 will be done by VM control operations in the > L0; this will come in a subsequent patch. > > `Serial access' to the L1 guest will likewise need to be done via the > console arrangement

Re: [Xen-devel] [OSSTEST PATCH 06/22] DhcpWatch::leases: Fix a reporting message

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: > This talks about `guest_check_ip', but this code is now factored out > into a method.  Use the correct method name in reporting. > > Signed-off-by: Ian Jackson > Tested-by: Robert Ho Ac

Re: [Xen-devel] [OSSTEST PATCH 05/22] Nested hosts: Provide PDU power method

2015-11-11 Thread Ian Campbell
/off method shall be > its host invoke $(toolstack)->create/destroy method. > > Signed-off-by: Robert Ho > Signed-off-by: Ian Jackson > Tested-by: Robert Ho Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 07/22] target_check_ip: Rename and improve from guest_check_ip

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: > Make this function suitable for running on targets with static IP > addresses.  (Ie, on physical hosts.)  Accordingly, rename it and > adjust all call sites. > > Signed-off-by: Ian Jackson > Tested-by: Robert Ho Acke

Re: [Xen-devel] [OSSTEST PATCH 08/22] await_tcp(): Run check_ip on each loop iteration

2015-11-11 Thread Ian Campbell
. > Fix this by running target_check_ip on each iteration. > > Signed-off-by: Robert Ho > Signed-off-by: Ian Jackson > Tested-by: Robert Ho Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 09/22] LVM: Break out lv_create

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: >  > +sub lv_create () { > +my ($ho, $vg, $lv, $mb) = @_; > +my $lvdev = "/dev/$vg/$lv"; In the original code it was using $gho->{Lvdev}, is this semantic change deliberate or a rebase-o? If the former then I think it warrants a com

Re: [Xen-devel] [OSSTEST PATCH 10/22] Toolstack::xl: Provide block_attach method

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: > It is possible that this may work some of the time with xm, so I have > taken no measures to prevent it running then. > > Signed-off-by: Ian Jackson > Signed-off-by: Robert Hu > Tested-by: Robert Hu Acke

Re: [Xen-devel] [OSSTEST PATCH 11/22] sg-run-job: Break out per-host-prep and per-host-finish

2015-11-11 Thread Ian Campbell
> > Signed-off-by: Ian Jackson > Tested by: Robert Ho Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 13/22] Nested HVM: Provide ts-nested-setup to help make L1 usable as a host

2015-11-11 Thread Ian Campbell
to run ts-xen-install etc. > > Signed-off-by: longtao.pang > Signed-off-by: Robert Ho > Signed-off-by: Ian Jackson > Tested-by: Robert Ho Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 14/22] Nested HVM: Provide test-nested recipe

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: > From: Robert Ho > > Signed-off-by: Robert Ho > Signed-off-by: Ian Jackson > Tested-by: Robert Ho Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lis

Re: [Xen-devel] [OSSTEST PATCH 15/22] Nested HVM: Add test job to appropriate flights

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: > From: Robert Ho > > Signed-off-by: longtao.pang > Signed-off-by: Robert Ho > Signed-off-by: Ian Jackson > Tested-by: Robert Ho Acked-by: Ian Campbell +            l1_image=debian-7.

Re: [Xen-devel] [OSSTEST PATCH 17/22] ts-xen-install: networking: Rename `nodhcp' to `ensurebridge'

2015-11-11 Thread Ian Campbell
Jackson > Tested-by: Robert Ho Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 16/22] ts-xen-install: Properly handle hosts without a static IP address

2015-11-11 Thread Ian Campbell
t have a > static IP address. > > In principle this makes matters more correct for physical hosts > without static IP addresses, but these are currently not supported > by selecthost(). > > Signed-off-by: Robert Ho > Signed-off-by: Ian Jack

Re: [Xen-devel] [OSSTEST PATCH 18/22] Osstest/Testsupport.pm: change target's default kernkind to 'pvops'

2015-11-11 Thread Ian Campbell
gt; Signed-off-by:  Ian Jackson > Signed-off-by:  Robert Ho > Tested-by:  Robert Ho Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 19/22] Osstest/Testsupport.pm: use get_target_property() for some host setup

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: > From: Robert Ho > > For nested cases, nested host can inherit its host's property for > dhcp watch setup and ether_prefix property setup. > > Signed-off-by: Robert Ho > Tested-by: Robert H

Re: [Xen-devel] [OSSTEST PATCH 20/22] HVM guests: Use qemu "pipe:" for serial output logging

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: > Modern qemu has the "pipe:/PATH" character driver.  This opens > /PATH.in for reading and /PATH.out for writing. Coo... > Signed-off-by: Ian Jackson Acked-by: Ian Campbell __

Re: [Xen-devel] [OSSTEST PATCH 21/22] Serial: Factor out Osstest::Serial::keys_real

2015-11-11 Thread Ian Campbell
ailure to send the > first debug key, sending the remainder will not be attempted. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH v16 00/22] Nested HVM

2015-11-11 Thread Ian Campbell
On Wed, 2015-11-11 at 09:45 +, Ian Campbell wrote: > On Tue, 2015-11-10 at 19:53 +, Ian Jackson wrote: > >   > > Ian: I'm not sure whether or not you want to review these.  About half > > of the patches are mine; the other half are Robert's but have been &

Re: [Xen-devel] RFC: PV vfb's and the handling of global vnc options

2015-11-11 Thread Ian Campbell
On Mon, 2015-11-09 at 15:03 +, George Dunlap wrote: Adding other tools maintainers. > So I had a user report that he couldn't get the vnclisten option to > work.  It turns out he was using a PV guest, and had the following in > his config file: > > vfb=[ 'type=vnc' ] > vnclisten='0.0.0.0' >

Re: [Xen-devel] [xen-4.3-testing test] 63948: regressions - FAIL

2015-11-11 Thread Ian Campbell
On Wed, 2015-11-11 at 03:58 -0700, Jan Beulich wrote: > > > > On 10.11.15 at 18:59, wrote: > > flight 63948 xen-4.3-testing real [real] > > http://logs.test-lab.xenproject.org/osstest/logs/63948/ > > > > Regressions :-( > > > > Tests which did not succeed and are blocking, > > including tests w

Re: [Xen-devel] [xen-4.3-testing test] 63948: regressions - FAIL

2015-11-11 Thread Ian Campbell
On Wed, 2015-11-11 at 04:35 -0700, Jan Beulich wrote: > > > > On 11.11.15 at 12:25, wrote: > > On Wed, 2015-11-11 at 03:58 -0700, Jan Beulich wrote: > > > > > > On 10.11.15 at 18:59, wrote: > > > > flight 63948 xen-4.3-testing real [real] > > > > http://logs.test-lab.xenproject.org/osstest/logs/6

Re: [Xen-devel] [CALL-FOR-AGENDA] Monthly Xen.org Technical Call (2015-11-11)

2015-11-11 Thread Ian Campbell
On Wed, 2015-11-04 at 15:37 +, Ian Campbell wrote: > The next Xen technical call will be at: > Wed 11 Nov 17:00:00 GMT 2015 > `date -d @1447261200` No-one has anything to discuss, thus cancelled. Ian. ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH v2] xen: arm: context switch vtimer PPI state.

2015-11-11 Thread Ian Campbell
On Tue, 2015-11-10 at 16:20 +, Ian Campbell wrote: > ... instead of artificially masking the timer interrupt in the timer > state Julien pointed out IRL that this now causes vgic_{enable,disable}_irqs to call into the h/w driver for that irq, which is new for a PPI. The GICv3 hook for

Re: [Xen-devel] [PATCH v2 5/8] xen: arm: add interfaces to save/restore the state of a PPI.

2015-11-11 Thread Ian Campbell
On Wed, 2015-11-11 at 12:49 +, Julien Grall wrote: >  > > +if ( s->enabled ) > > +gicv2_irq_disable(desc); > > Don't we want to disable the IRQ first and then saving the state? We need at least the enable/disable state first. Perhaps some of the other stuff might be better done af

[Xen-devel] [PATCH] tools: migration: Use PRIpfn when printing frame numbers.

2015-11-11 Thread Ian Campbell
This avoids various printf formatting warnings when building on arm32. While touching the affected lines make them consistently use %#. Signed-off-by: Ian Campbell Cc: Andrew Cooper --- tools/libxc/xc_sr_restore.c | 15 --- tools/libxc/xc_sr_save.c| 2 +- 2 files changed, 9

Re: [Xen-devel] Xen on Arm Arndale Exynos5250

2015-11-11 Thread Ian Campbell
On Wed, 2015-11-11 at 07:26 -0600, ed sandberg wrote: > > So does it still not work if you add a getty onto "hvc0" instead of > > "ttySA2"? > > Correct, adding getty to hvc0 also does not work. And you are either using udev or have otherwise arranged for /dev/hvc0 to exist? Looking back at your

  1   2   3   4   5   6   7   8   9   10   >