Re: [Xen-devel] [PATCH] tools: fix python install with "xentoollog"

2016-01-15 Thread Ian Campbell
On Tue, 2016-01-12 at 22:16 -0500, Konrad Rzeszutek Wilk wrote: > commit 5d3dc8671521ea4a4f753e77d3e7fb3a3a6f5f80 > "tools: Refactor "xentoollog" into its own library" > with older python versions (2.6.4) will fail to > the build if attempted to be done twice (which > happens due to pygrub

Re: [Xen-devel] [PATCH v2 1/2] libxl: fix _SC_GETPW_R_SIZE_MAX usage

2016-01-15 Thread Ian Campbell
On Thu, 2016-01-14 at 16:06 +0100, Roger Pau Monne wrote: > If sysconf(_SC_GETPW_R_SIZE_MAX) fails for any reason just use an initial > buffer size of 2048. This is not a critical failure, and the code that > makes use of this buffer is able to expand it later if required. > > Signed-off-by:

Re: [Xen-devel] [PATCH v4 1/2] memory-hotplug: add automatic onlining policy for the newly added memory

2016-01-15 Thread Vitaly Kuznetsov
David Rientjes writes: > On Thu, 14 Jan 2016, Vitaly Kuznetsov wrote: > >> > My suggestion is to just simply document that auto-onlining can add the >> > memory but fail to online it and the failure is silent to userspace. If >> > userspace cares, it can check the online

[Xen-devel] [PATCH XEN v8 12/29] tools/libs/foreignmemory: provide xenforeignmemory_unmap.

2016-01-15 Thread Ian Campbell
And require it be used instead of direct munmap. This will allow e.g. Valgrind hooks to help track incorrect use of foreign mappings. Switch all uses of xenforeignmemory_map to use xenforeignmemory_unmap, not that foreign mappings via the libxc compat xc_map_foreign_* interface will not take

[Xen-devel] [PATCH XEN v8 01/29] tools/libxc: Remove osdep indirection for xc_evtchn

2016-01-15 Thread Ian Campbell
The alternative backend (a xen-api/xapi shim) is no longer around and so this stuff is now just baggage which is getting in the way of refactoring libxenctrl. Note that the intention is to move this into a separate library shortly. Nested virt probably suffices for this use case now. One

[Xen-devel] [PATCH XEN v8 03/29] tools: Arrange to check public headers for ANSI compatiblity

2016-01-15 Thread Ian Campbell
Using the same rune as we use for the Xen public headers, except we do not need stdint.h here. Signed-off-by: Ian Campbell Acked-by: Wei Liu --- v7: Drop -pendantic: Some library headers include parts of xen/include/public, which in turn are not

[Xen-devel] [PATCH XEN v8 10/29] tools: Implement xc_map_foreign_range(s) in terms of common helper

2016-01-15 Thread Ian Campbell
Both Linux and FreeBSD already implemented these functions using identical helpers based on xc_map_foreign_pages. Make one copy of these common helpers and switch all OSes to use them, even those which previously had a specific lower level implementation of this functionality. This is makes two

[Xen-devel] [PATCH XEN v8 04/29] tools/libxc: Remove osdep indirection for xc_gnt{shr, tab}

2016-01-15 Thread Ian Campbell
The alternative backend (a xen-api/xapi shim) is no longer around and so this stuff is now just baggage which is getting in the way of refactoring libxenctrl. Nested virt probably suffices for this use case now. It is now necessary to provide explicit versions of things for platforms which do

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

2016-01-15 Thread Ian Campbell
We intend to stabilise some parts of the libxenctrl interface by splitting out some functionality into separate stable libraries. This is the xen part of the first phase of that change. This mail is (or is intended to be) a reply to a "0/" super-intro mail covering all of the related patch

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

2016-01-15 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 v8 of that set of series against: xen qemu-xen qemu-xen-traditional mini-os NB: Samuel+minios-devel will only get the mini-os

[Xen-devel] [PATCH XEN v8 08/29] tools/libxc: drop xc_map_foreign_bulk_compat wrappers

2016-01-15 Thread Ian Campbell
On Solaris and NetBSD xc_map_foreign_bulk is implemented by calling xc_map_foreign_bulk_compat and xc_map_foreign_bulk_compat is exposed as a symbol by libxenctrl.so. Remove these wrappers and turn the compat function into the real thing surrounded by the appropriate ifdef. As this is a compat

[Xen-devel] [PATCH XEN v8 09/29] tools: Remove xc_map_foreign_batch

2016-01-15 Thread Ian Campbell
It can trivially be replaced by xc_map_foreign_pages which is the interface I want to move to going forward (by standardising on _bulk but handling err=NULL as _pages does). The callers of _batch are checking a mixture of a NULL return or looking to see if the top nibble of the (usually sole) mfn

[Xen-devel] [PATCH XEN v8 17/29] tools/libs/evtchn: Review and update doc comments.

2016-01-15 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 Acked-by: Ian Jackson --- tools/libs/evtchn/include/xenevtchn.h | 6 ++ 1 file

[Xen-devel] [PATCH XEN v8 16/29] tools/libs/foreignmemory: pull array length argument to map forward

2016-01-15 Thread Ian Campbell
By having the "num" argument before the page and error arrays we can potentially use a variable-length-array argument ("int pages[num]") in the function prototype. However VLAs are a C99 feature and we are currently targetting C89 and later, so we don't actually make use of this here, merely

[Xen-devel] [PATCH XEN v8 13/29] tools/libs/foreignmemory: use size_t for size arguments.

2016-01-15 Thread Ian Campbell
Surprisingly it appears no callers need updating. Signed-off-by: Ian Campbell Acked-by: Wei Liu Acked-by: Ian Jackson --- v4: New patch --- tools/libs/foreignmemory/compat.c | 2 +-

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

2016-01-15 Thread Ian Campbell
v4: Config.mk instead of .config --- Config.mk | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/Config.mk b/Config.mk index d654af8..364e790 100644 --- a/Config.mk +++ b/Config.mk @@ -238,22 +238,22 @@ endif ifeq ($(GIT_HTTP),y) OVMF_UPSTREAM_URL ?=

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

2016-01-15 Thread Ian Campbell
We intend to stabilise some parts of the libxenctrl interface by splitting out some functionality into separate stable libraries. This is the qemu-xen part of the first phase of that change. This mail is (or is intended to be) a reply to a "0/" super-intro mail covering all of the related patch

[Xen-devel] [PATCH XEN v8 19/29] tools/libs: Clean up hard tabs.

2016-01-15 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

Re: [Xen-devel] [BUG] Assertion '(sp == 0) || (peoi[sp-1].vector < vector)' failed at irq.c:1163

2016-01-15 Thread Andrew Cooper
On 15/01/16 10:58, Håkon Alstadheim wrote: > This is just a preliminary report, mostly just for the record. > > I will report again if this keeps happening after 4.7 is out, or upon > request. Anyone working on this, please mail me and request more > information. I have available logs from dom0

Re: [Xen-devel] [BUG] Assertion '(sp == 0) || (peoi[sp-1].vector < vector)' failed at irq.c:1163

2016-01-15 Thread Håkon Alstadheim
On 01/15/2016 12:05 PM, Andrew Cooper wrote: On 15/01/16 10:58, Håkon Alstadheim wrote: This is just a preliminary report, mostly just for the record. I will report again if this keeps happening after 4.7 is out, or upon request. Anyone working on this, please mail me and request more

Re: [Xen-devel] [BUG] Assertion '(sp == 0) || (peoi[sp-1].vector < vector)' failed at irq.c:1163

2016-01-15 Thread Håkon Alstadheim
On 01/15/2016 01:42 PM, Jan Beulich wrote: On 15.01.16 at 13:32, wrote: placed microcode.bin in /boot/microcode.bin booted with : --- xen_commandline: ssd-xen-debug-marker console_timestamps=date loglvl=all guest_loglvl=all sync_console

[Xen-devel] [PATCH XEN v8 26/29] tools/libs/*: Use O_CLOEXEC on Linux and FreeBSD

2016-01-15 Thread Ian Campbell
In some cases this replaces an FD_CLOEXEC dance, in others it is new. Linux has had O_CLOEXEC since 2.6.23 (October 2007), so we can rely on it from Xen 4.7 I think. Some libc headers may still lack the definition, so we take care of that if need be by defining to 0 (on the premise that such an

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

2016-01-15 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

[Xen-devel] [PATCH XEN v8 28/29] tools/libs/*: Introduce APIs to restrict handles to a specific domain.

2016-01-15 Thread Ian Campbell
These are intended to allow user space processes (in particular QEMU) to lock down all the handles at start of day and then drop the privileges which would allow them to open any new unrestricted handles (e.g. setuid or similar). This will reduce the privileges which taking over such a process

[Xen-devel] [PATCH XEN v8 21/29] tools/libs/call: Update some log messages to not refer to xc.

2016-01-15 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 55e1e83..3641e41 100644 ---

[Xen-devel] [PATCH XEN v8 05/29] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2016-01-15 Thread Ian Campbell
libxengnttab will provide a stable API and ABI for accessing the grant table devices. The functions are moved into the xengnt{tab,shr} namespace to make a clean break from libxc and avoid ambiguity regarding which interfaces are stable. All in-tree users are updated to use the new names. Upon

[Xen-devel] [PATCH XEN v8 22/29] tools/libs/call: Describe return values and error semantics for xencall*

2016-01-15 Thread Ian Campbell
This behaviour has been confirmed by inspection on: - Linux - NetBSD & FreeBSD (NB: hcall->retval is the hypercall return value only for values >= 0. For negative values the underlying privcmd driver translates the value from Xen to {Net,Free}BSD errno space and returns it as the

[Xen-devel] [PATCH XEN v8 07/29] tools: Refactor hypercall calling wrappers into libxencall.

2016-01-15 Thread Ian Campbell
libxencall will provide a stable API and ABI for calling hypercalls (although those hypercalls themselves may not have a stable API). As well as the hypercall buffer infrastructure needed in order to safely provide pointer arguments to hypercalls. libxenctrl encapsulates a instance of this

[Xen-devel] [PATCH XEN v8 06/29] tools/libxc: Remove osdep indirection for privcmd

2016-01-15 Thread Ian Campbell
The alternative backend (a xen-api/xapi shim) is no longer around and so this stuff is now just baggage which is getting in the way of refactoring libxenctrl. Nested virt probably suffices for this use case now. This was the last component of the osdep infrastructure, so all the dynamic loading

[Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-15 Thread Ian Campbell
This avoids a potential issue with a fork after allocation but before madvise. Signed-off-by: Ian Campbell --- v7: New, replacing "tools/libs/call: linux: avoid forking between mmap and madvise". --- tools/libs/call/linux.c | 14 +- 1 file changed, 13

[Xen-devel] [PATCH QEMU-XEN v8 5/8] xen: Switch uses of xc_map_foreign_{pages, bulk} to use libxenforeignmemory API.

2016-01-15 Thread Ian Campbell
In Xen 4.7 we are refactoring parts libxenctrl into a number of separate libraries which will provide backward and forward API and ABI compatiblity. One such library will be libxenforeignmemory which provides access to privileged foreign mappings and which will provide an interface equivalent to

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

2016-01-15 Thread Ian Campbell
libxenctrl links against this library Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- 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

[Xen-devel] [PATCH XEN v8 11/29] tools: Refactor foreign memory mapping into libxenforeignmemory

2016-01-15 Thread Ian Campbell
libxenforeignmemory will provide a stable API and ABI for mapping foreign domain memory (subject to appropriate privileges). The new library exposes an interface equivalent to xc_map_foreign_memory_bulk, which all the other xc_map_foreign_memory_* functions (which remain in libxc) are implemented

[Xen-devel] [PATCH XEN v8 20/29] tools/libs/gnttab: Extensive updates to API documentation.

2016-01-15 Thread Ian Campbell
In particular around error handling, behaviour on fork and the unmap notification mechanism. Behaviour of xengnttab_map_*grant_refs and xengntshr_share_pages on partial failure has been confirmed/inferred (by inspection) on Linux and Mini-os (the only two known implementations. Likewise the

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

2016-01-15 Thread Ian Campbell
We intend to stabilise some parts of the libxenctrl interface by splitting out some functionality into separate stable libraries. This is the mini-os part of the first phase of that change. This mail is (or is intended to be) a reply to a "0/" super-intro mail covering all of the related patch

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

2016-01-15 Thread Ian Campbell
In Xen 4.7 we are refactoring parts libxenctrl into a number of separate libraries which will provide backward and forward API and ABI compatiblity. One such library will be libxenevtchn which provides access to event channels. In preparation for this switch the compatibility layer in

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

2016-01-15 Thread Ian Campbell
Until the previous patch this relied on xc_fd(), which was only implemented for Xen 4.0 and earlier. Given this wasn't working since Xen 4.0 I have marked this as disabled by default. Removing this support drops the use of a bunch of symbols from libxenctrl, specifically: - xc_domain_create

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

2016-01-15 Thread Ian Campbell
libxenctrl links against this library. Also, request the compat xc_map_foreign API from libxc. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- v3: Library moved to tools/libs/ --- xen-hooks.mak | 2 ++ 1 file changed, 2 insertions(+)

[Xen-devel] [PATCH XEN v8 25/29] tools/libs/{call, evtchn}: Document requirements around forking.

2016-01-15 Thread Ian Campbell
Much like for gnttab and foreignmemory xencall hypercall buffers need care. Evtchn is a bit simpler (no magic mappings) but may not work from parent + child simultaneously, document "parent only" since it is consistent with the others. Signed-off-by: Ian Campbell ---

[Xen-devel] [PATCH XEN v8 15/29] tools/libs/foreignmemory: Support err == NULL to map.

2016-01-15 Thread Ian Campbell
The existing xc_map_foreign_bulk-like interface encourages callers to miss error checking for partial failure (by forgetting to scan the err array). Add support for passing err==NULL which behaves in a xc_map_foreign_pages-like manner and returns a global error for any failure. While documenting

[Xen-devel] [PATCH XEN v8 27/29] tools: Update CFLAGS for qemu-xen to allow it to use new libraries

2016-01-15 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

[Xen-devel] [PATCH QEMU-XEN v8 6/8] xen: Use stable library interfaces when they are available.

2016-01-15 Thread Ian Campbell
In Xen 4.7 we are refactoring parts libxenctrl into a number of separate libraries which will provide backward and forward API and ABI compatiblity. Specifically libxenevtchn, libxengnttab and libxenforeignmemory. Previous patches have already laid the groundwork for using these by switching the

[Xen-devel] [PATCH XEN v8 14/29] tools/libs/foreignmemory: Mention restrictions on fork in docs.

2016-01-15 Thread Ian Campbell
Signed-off-by: Ian Campbell --- v6: Also discuss recovering the memory. v7: Further clarifications regarding forking based on ML discussions. (Dropped Wei's ack) --- .../libs/foreignmemory/include/xenforeignmemory.h | 33 +- 1 file changed, 32

[Xen-devel] [PATCH XEN v8 18/29] tools/libs/evtchn: Use uint32_t for domid arguments

2016-01-15 Thread Ian Campbell
Signed-off-by: Ian Campbell --- v8: New --- tools/libs/evtchn/freebsd.c | 4 ++-- tools/libs/evtchn/include/xenevtchn.h | 4 ++-- tools/libs/evtchn/linux.c | 4 ++-- tools/libs/evtchn/minios.c| 4 ++-- tools/libs/evtchn/netbsd.c

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

2016-01-15 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

Re: [Xen-devel] [PATCH v3] x86/p2m: use large pages for MMIO mappings

2016-01-15 Thread Ian Campbell
On Thu, 2016-01-14 at 03:04 -0700, Jan Beulich wrote: > - ARM side unimplemented (and hence libxc for now made cope with both >   models), So, one model is the one described in the commit message: > - zero (success, everything done) > - positive (success, this many done, more to do: re-invoke) > 

Re: [Xen-devel] [PATCH v1] xen/arm: Add r1p12 to the list of supported Cadence UARTs

2016-01-15 Thread Edgar E. Iglesias
On Fri, Jan 15, 2016 at 04:20:18AM -0700, Jan Beulich wrote: > >>> On 14.01.16 at 19:44, wrote: > > --- a/xen/drivers/char/cadence-uart.c > > +++ b/xen/drivers/char/cadence-uart.c > > Considering the wider than expected Cc list - is there an entry missing > from the ARM

[Xen-devel] [PATCH OSSTEST] Allow longer timeout when creating backing file for a raw disk.

2016-01-15 Thread Ian Campbell
I noticed this dd timiung out when recommissioning the 3 cubietrucks (picasso, metzinger, gleizes) but looking at the log shows this has been happening on braque too. The current code assumes 65MB/s arriving at a timeout of 153s for the 10G file. On arndale-* the logs indicate that it is

Re: [Xen-devel] [PATCH v1] xen/arm: Add r1p12 to the list of supported Cadence UARTs

2016-01-15 Thread Ian Campbell
On Thu, 2016-01-14 at 19:44 +0100, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" Where are the bindings for cdns,uart-r1p12 defined? I don't see it mentioned in the Documentation/devicetree/bindings/serial/cdns,uart.txt in mainline Linux. Does the driver need

Re: [Xen-devel] [PATCH v1 1/1] MAINTAINERS: Add xen/drivers/char/cadence-uart.c

2016-01-15 Thread Ian Campbell
On Fri, 2016-01-15 at 13:14 +0100, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > Add xen/drivers/char/cadence-uart.c to the ARM section. > > Signed-off-by: Edgar E. Iglesias Acked-by: Ian Campbell >

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

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

[Xen-devel] [PATCH] libxl: create 'drivers', 'feature' and 'attr' xenstore paths

2016-01-15 Thread Paul Durrant
My recent patch series 'docs: Document xenstore paths' included 3 patches documenting new xenstore paths to allow PV drivers/agents in guests to advertise version information, significant features and attributes (such as assigned IP addresses). This patch adds the necessary code to libxl to

Re: [Xen-devel] [PATCHv1 net] xen-netback: use skb to determine number of required guest Rx requests

2016-01-15 Thread David Vrabel
On 14/01/16 21:54, David Miller wrote: > From: David Vrabel > Date: Thu, 14 Jan 2016 15:18:30 + > >> -needed = xenvif_rx_ring_slots_needed(queue->vif); >> +skb = skb_peek(>rx_queue); >> +if (!skb) >> +return false; >> + >> +needed =

Re: [Xen-devel] [PATCH v3] x86/p2m: use large pages for MMIO mappings

2016-01-15 Thread Jan Beulich
>>> On 15.01.16 at 11:09, wrote: > On Thu, 2016-01-14 at 03:04 -0700, Jan Beulich wrote: >> - ARM side unimplemented (and hence libxc for now made cope with both >> models), > > So, one model is the one described in the commit message: > >> - zero (success, everything

Re: [Xen-devel] [PATCH] gitignore: update stubdom entries

2016-01-15 Thread Ian Campbell
On Thu, 2016-01-14 at 16:17 +, Wei Liu wrote: > Add stubdom/libs-* to gitignore, then sort stubdom/* entries > alphabetically. > > Signed-off-by: Wei Liu Acked + applied. I have discarded the previous "gitignore: ignore stubdom/libs-*".

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

2016-01-15 Thread osstest service owner
flight 78136 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/78136/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds 9 debian-install fail like 78082 Tests which did not

Re: [Xen-devel] [BUG] Assertion '(sp == 0) || (peoi[sp-1].vector < vector)' failed at irq.c:1163

2016-01-15 Thread Jan Beulich
>>> On 15.01.16 at 13:32, wrote: > placed microcode.bin in /boot/microcode.bin > > booted with : > --- > xen_commandline: ssd-xen-debug-marker console_timestamps=date > loglvl=all guest_loglvl=all sync_console iommu=1,verbose,debug >

[Xen-devel] [PATCH QEMU-XEN-TRADITIONAL v8 1/4] qemu-xen-traditional: Use libxenevtchn

2016-01-15 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 Acked-by: Ian

[Xen-devel] [PATCH QEMU-XEN v8 4/8] xen: Switch uses of xc_map_foreign_range into xc_map_foreign_pages

2016-01-15 Thread Ian Campbell
In Xen 4.7 we are refactoring parts libxenctrl into a number of separate libraries which will provide backward and forward API and ABI compatiblity. One such library will be libxenforeignmemory which provides access to privileged foreign mappings and which will provide an interface equivalent to

[Xen-devel] [PATCH MINI-OS v8 1/4] mini-os: Include libxenevtchn with libxc

2016-01-15 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

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

2016-01-15 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

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

2016-01-15 Thread Ian Campbell
In Xen 4.7 we are refactoring parts libxenctrl into a number of separate libraries which will provide backward and forward API and ABI compatiblity. One such library will be libxengnttab which provides access to grant tables. In preparation for this switch the compatibility layer in xen_common.h

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

2016-01-15 Thread Ian Campbell
All of the work in con_disconnect applies to the primary console case (when xendev->dev is NULL). Therefore remove the early check and bail and allow it to fall through. All of the existing code is correctly conditional already. The ->dev and ->gnttabdev handles are either both set or neither.

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

2016-01-15 Thread Ian Campbell
We intend to stabilise some parts of the libxenctrl interface by splitting out some functionality into separate stable libraries. This is the qemu-xen-traditional part of the first phase of that change. This mail is (or is intended to be) a reply to a "0/" super-intro mail covering all of the

[Xen-devel] [PATCH MINI-OS v8 2/4] mini-os: Include libxengnttab with libxc

2016-01-15 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

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

2016-01-15 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

[Xen-devel] [seabios baseline-only test] 38639: tolerable FAIL

2016-01-15 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 38639 seabios real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38639/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-qemuu-nested-intel 13 xen-boot/l1

[Xen-devel] [BUG] Assertion '(sp == 0) || (peoi[sp-1].vector < vector)' failed at irq.c:1163

2016-01-15 Thread Håkon Alstadheim
This is just a preliminary report, mostly just for the record. I will report again if this keeps happening after 4.7 is out, or upon request. Anyone working on this, please mail me and request more information. I have available logs from dom0 boot (I dump dmesg and xl dmesg to disk after

Re: [Xen-devel] [PATCH] Config.mk: Update SEABIOS_UPSTREAM_TAG to 442502

2016-01-15 Thread Ian Campbell
On Thu, 2016-01-14 at 17:56 +0100, Roger Pau Monné wrote: >  > > commit a661f70a8aad3fe7b2c828632714785a04b1d96b > > Author: Ian Campbell > > Date:   Thu Jan 14 16:43:42 2016 + > > > > SEABIOS_UPSTREAM_REVISION Update > > > > Signed-off-by: Ian Campbell

[Xen-devel] [PATCH v1 1/1] MAINTAINERS: Add xen/drivers/char/cadence-uart.c

2016-01-15 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Add xen/drivers/char/cadence-uart.c to the ARM section. Signed-off-by: Edgar E. Iglesias --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 09fe823..0e8af01 100644

Re: [Xen-devel] [PATCH 1/2] libxl: fix _SC_GETPW_R_SIZE_MAX usage

2016-01-15 Thread Ian Campbell
On Wed, 2016-01-13 at 10:08 +0100, Roger Pau Monné wrote: > El 13/01/16 a les 4.19, Doug Goldstein ha escrit: > > On 1/12/16 7:14 AM, Roger Pau Monne wrote: > > > According to the FreeBSD sysconf man page [0] if the variable is > > > associated > > > with functionality that is not supported, -1 is

Re: [Xen-devel] [PATCH v1] xen/arm: Add r1p12 to the list of supported Cadence UARTs

2016-01-15 Thread Jan Beulich
>>> On 14.01.16 at 19:44, wrote: > --- a/xen/drivers/char/cadence-uart.c > +++ b/xen/drivers/char/cadence-uart.c Considering the wider than expected Cc list - is there an entry missing from the ARM section in ./MAINTAINERS here? Jan

[Xen-devel] [PATCHv1 net] xen-netfront: request Tx response events more often

2016-01-15 Thread David Vrabel
From: Malcolm Crossley Trying to batch Tx response events results in poor performance because this delays freeing the transmitted skbs. Instead use the standard RING_FINAL_CHECK_FOR_RESPONSES() macro to be notified once the next Tx response is placed on the ring.

Re: [Xen-devel] [BUG] Assertion '(sp == 0) || (peoi[sp-1].vector < vector)' failed at irq.c:1163

2016-01-15 Thread Håkon Alstadheim
On 01/15/2016 02:09 PM, Ian Campbell wrote: On Fri, 2016-01-15 at 13:49 +0100, Håkon Alstadheim wrote: On 01/15/2016 01:42 PM, Jan Beulich wrote: On 15.01.16 at 13:32, wrote: placed microcode.bin in /boot/microcode.bin booted with : --- xen_commandline

Re: [Xen-devel] [PATCH v4 3/3] VT-d: Fix vt-d Device-TLB flush timeout issue.

2016-01-15 Thread Jan Beulich
>>> On 23.12.15 at 09:25, wrote: > --- a/xen/drivers/passthrough/vtd/qinval.c > +++ b/xen/drivers/passthrough/vtd/qinval.c > @@ -190,9 +190,19 @@ static int queue_invalidate_wait(struct iommu *iommu, > static int invalidate_sync(struct iommu *iommu) > { > struct qi_ctrl

[Xen-devel] [PATCH OSSTEST] ts-debian-install: increase time allowed for xen-create-image

2016-01-15 Thread Ian Campbell
This step is consistently timing out when run on cubietruck-*. Judging from the logs it appears to be completing during the 30s slack added by tcmdex (i.e. after the timeout message the rest of the output appears in the test step log). Looking at the results on arndale-* (which looks to pass

Re: [Xen-devel] [PATCH v1] xen/arm: Add r1p12 to the list of supported Cadence UARTs

2016-01-15 Thread Ian Campbell
On Fri, 2016-01-15 at 14:30 +0100, Edgar E. Iglesias wrote: > On Fri, Jan 15, 2016 at 12:30:27PM +, Ian Campbell wrote: > > On Thu, 2016-01-14 at 19:44 +0100, Edgar E. Iglesias wrote: > > > From: "Edgar E. Iglesias" > > > > Where are the bindings for 

[Xen-devel] [PATCHv2 3/3] xen-netback: free queues after freeing the net device

2016-01-15 Thread David Vrabel
If a queue still has a NAPI instance added to the net device, freeing the queues early results in a use-after-free. The shouldn't ever happen because we disconnect and tear down all queues before freeing the net device, but doing this makes it obviously safe. Signed-off-by: David Vrabel

Re: [Xen-devel] [PATCH v2 0/3] tools: remove 1024 domain limit at some places

2016-01-15 Thread Juergen Gross
On 04/01/16 15:55, Juergen Gross wrote: > There are some places in Xen tools which will work for only up to > 1024 domains. Remove this limit. > > Changes in V2: > - corrected a little error in patch 1 at end of loop (index -1 used > in array) > - added patches 2 and 3 > > Juergen Gross (3): >

[Xen-devel] [PATCH v2] xen/arm64: correctly emulate the {w, x}zr registers

2016-01-15 Thread Ian Campbell
From: Julien Grall On AArch64, encoding 31 for an R in the HSR is used to represent either {w,x}sp or {w,x}zr (See C1.2.4 in ARM DDI 0486A.d) depending on how the register field is interpreted by the instruction. All the instructions trapped by Xen (either via a sysreg

[Xen-devel] [PATCHv2 0/3 net] xen-netback: use skb to determine number of required (etc.)

2016-01-15 Thread David Vrabel
"xen-netback: use skb to determine number of required" plus two other minor fixes I found down the back of the sofa. David ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [PATCHv2 1/3] xen-netback: use skb to determine number of required guest Rx requests

2016-01-15 Thread David Vrabel
Using the MTU or GSO size to determine the number of required guest Rx requests for an skb was subtly broken since these value may change at runtime. After 1650d5455bd2dc6b5ee134bd6fc1a3236c266b5b (xen-netback: always fully coalesce guest Rx packets) we always fully pack a packet into its guest

Re: [Xen-devel] [PATCH QEMU-XEN v8 0/8] Begin to disentangle libxenctrl and provide some stable libraries

2016-01-15 Thread Ian Campbell
On Fri, 2016-01-15 at 14:44 +, Stefano Stabellini wrote: > What's the status of the libxc side changes? Is the interface stable > enough for me to commit this series? I'd recommend waiting. I'll ping you when it looks appropriate to apply this series. Thanks for the final acks on this

Re: [Xen-devel] [PATCH 2/2] libxl: fix UUID usage on FreeBSD

2016-01-15 Thread Ian Campbell
On Fri, 2016-01-15 at 16:11 +0100, Roger Pau Monné wrote: > El 15/01/16 a les 11.26, Ian Campbell ha escrit: > > On Tue, 2016-01-12 at 14:14 +0100, Roger Pau Monne wrote: > > > libxl makes the assumtion that libxl_uuid == uuid_t, > > > > ("assumption") > > > > >  and that uuid_t can be > > >

Re: [Xen-devel] [PATCH v3] x86/p2m: use large pages for MMIO mappings

2016-01-15 Thread Ian Campbell
On Fri, 2016-01-15 at 03:47 -0700, Jan Beulich wrote: > > > > On 15.01.16 at 11:09, wrote: > > On Thu, 2016-01-14 at 03:04 -0700, Jan Beulich wrote: > > > - ARM side unimplemented (and hence libxc for now made cope with both > > >   models), > > > > So, one model is the

Re: [Xen-devel] [PATCH] QEMU as non-root and PCI passthrough do not mix

2016-01-15 Thread Stefano Stabellini
On Thu, 14 Jan 2016, Ian Jackson wrote: > Stefano Stabellini writes ("Re: [PATCH] QEMU as non-root and PCI passthrough > do not mix"): > > On Thu, 14 Jan 2016, Ian Campbell wrote: > > > What if b_info->device_model_user is NULL or == "root"? Doesn't this warn > > > even then? > > > > I meant to

Re: [Xen-devel] [Qemu-devel] [PATCH 2/2] xen-hvm: Clean up xen_ram_alloc() error handling

2016-01-15 Thread Stefano Stabellini
On Fri, 15 Jan 2016, Markus Armbruster wrote: > Stefano Stabellini writes: > > > On Thu, 14 Jan 2016, Markus Armbruster wrote: > >> xen_ram_alloc() dies with hw_error() on error, even though its caller > >> ram_block_add() handles errors just fine. Add an Error

Re: [Xen-devel] [BUG] Assertion '(sp == 0) || (peoi[sp-1].vector < vector)' failed at irq.c:1163

2016-01-15 Thread Håkon Alstadheim
On 01/15/2016 01:49 PM, Håkon Alstadheim wrote: On 01/15/2016 01:42 PM, Jan Beulich wrote: On 15.01.16 at 13:32, wrote: placed microcode.bin in /boot/microcode.bin booted with : --- xen_commandline: ssd-xen-debug-marker console_timestamps=date

Re: [Xen-devel] [PATCH v3] x86/p2m: use large pages for MMIO mappings

2016-01-15 Thread Jan Beulich
>>> On 15.01.16 at 14:57, wrote: > On Fri, 2016-01-15 at 03:47 -0700, Jan Beulich wrote: >> > > > On 15.01.16 at 11:09, wrote: >> > On Thu, 2016-01-14 at 03:04 -0700, Jan Beulich wrote: >> > > - ARM side unimplemented (and hence libxc for now

Re: [Xen-devel] [PATCH v1] xen/arm: Add r1p12 to the list of supported Cadence UARTs

2016-01-15 Thread Ian Campbell
On Fri, 2016-01-15 at 15:36 +0100, Edgar E. Iglesias wrote: > On Fri, Jan 15, 2016 at 01:46:05PM +, Ian Campbell wrote: > > On Fri, 2016-01-15 at 14:30 +0100, Edgar E. Iglesias wrote: > > > On Fri, Jan 15, 2016 at 12:30:27PM +, Ian Campbell wrote: > > > > On Thu, 2016-01-14 at 19:44 +0100,

[Xen-devel] [PATCH RFC] xen/Kconfig: Use olddefconfig not silentoldconfig to update .config

2016-01-15 Thread Ian Campbell
RFC RFC RFC RFC RFC RFC RFC RFC RFC RFC RFC BECAUSE THIS BREAKS CLEAN BUILDS WITH: In file included from :0:0: /local/scratch/ianc/devel/arm/xen.git/xen/include/xen/config.h:10:32: fatal error: generated/autoconf.h: No such file or directory #include I'm not sure why given the descriptions of

Re: [Xen-devel] [PATCH 03/28] libxl: Provide libxl__dm_support_*

2016-01-15 Thread Jim Fehlig
On 01/15/2016 02:56 AM, Ian Campbell wrote: > On Thu, 2016-01-14 at 11:31 -0700, Jim Fehlig wrote: >> Ian Campbell wrote: >>> On Mon, 2016-01-11 at 10:00 -0700, Jim Fehlig wrote: On 01/07/2016 10:13 AM, Ian Campbell wrote: > On Tue, 2015-12-22 at 18:44 +, Ian Jackson wrote: >>

[Xen-devel] [PATCH v2 2/4] libelf: rewrite symtab/strtab loading for Dom0

2016-01-15 Thread Roger Pau Monne
Current implementation of elf_load_bsdsyms is broken when loading inside of a HVM guest, because it assumes elf_memcpy_safe is able to write into guest memory space, which it is not. Take the oportunity to do some cleanup and properly document how elf_{parse/load}_bsdsyms works. The new

[Xen-devel] [PATCH v2 0/4] HVMlite: minor fixes and Dom0 preparatory patches

2016-01-15 Thread Roger Pau Monne
Hello, This series contains some bug fixes for HVMlite DomU and a preparatory patch for HVMlite Dom0 support (2/4). Thanks, Roger. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [PATCH v2 4/4] x86/PV: enable the emulated PIT

2016-01-15 Thread Roger Pau Monne
The HVMlite series removed the initialization of the emulated PIT for PV guests, this patch re-enables it. Signed-off-by: Roger Pau Monné --- Cc: Ian Jackson Cc: Ian Campbell Cc: Wei Liu Cc: Jan

[Xen-devel] [PATCH v2 3/4] x86/hvm: don't set the BSP as initialised in hvm_vcpu_initialise

2016-01-15 Thread Roger Pau Monne
The BSP will be marked as initialised after hvm_load_cpu_ctxt has loaded the initial state, which is called from the toolstack during domain creation. Previous to my HVMlite series HVM guests were started without setting any explicit CPU state (in fact we placed that horrible jmp at 0x0, because

[Xen-devel] [PATCH v2 1/4] xen/elfnotes: check phys_entry against UNSET_ADDR32

2016-01-15 Thread Roger Pau Monne
And introduce UNSET_ADDR32. Signed-off-by: Roger Pau Monné Acked-by: Jan Beulich --- Cc: Ian Jackson Cc: Ian Campbell Cc: Wei Liu --- Changes since v1: - Fix commit title. ---

Re: [Xen-devel] [PATCH v3 00/15] xenstore: make it easier to run xenstore in a domain

2016-01-15 Thread Juergen Gross
On 08/01/16 14:08, Juergen Gross wrote: > Xen supports to run xenstore in a dedicated domain. It is, however, a > setup which isn't easy to configure. Today flask is required for full > functionality and the resulting xenstore domain is not configurable in > the same way as the xenstore daemon. >

[Xen-devel] [PATCH v6 0/2] memory-hotplug: add automatic onlining policy for the newly added memory

2016-01-15 Thread Vitaly Kuznetsov
Changes since v5: Patch 1: - Mention possible failures during automatic onlining in memory-hotplug.txt [David Rientjes] - Add Daniel's Reviewed-by: (hope it stands) Patch2: - Change the last 'domU' -> 'target domain' in Kconfig [Daniel Kiper] - Add Daniel's Reviewed-by: - Add David's Acked-by:

Re: [Xen-devel] [PATCH v1] xen/arm: Add r1p12 to the list of supported Cadence UARTs

2016-01-15 Thread Edgar E. Iglesias
On Fri, Jan 15, 2016 at 12:30:27PM +, Ian Campbell wrote: > On Thu, 2016-01-14 at 19:44 +0100, Edgar E. Iglesias wrote: > > From: "Edgar E. Iglesias" > > Where are the bindings for cdns,uart-r1p12 defined? I don't see it > mentioned in the

  1   2   3   >