[Xen-devel] [xen-unstable test] 78129: regressions - FAIL

2016-01-15 Thread osstest service owner
flight 78129 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/78129/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-stubdom-debianhvm-amd64-xsm 5 xen-install fail REGR. vs. 77892 test-amd64

Re: [Xen-devel] [v3,11/41] mips: reuse asm-generic/barrier.h

2016-01-15 Thread Peter Zijlstra
On Thu, Jan 14, 2016 at 01:29:13PM -0800, Paul E. McKenney wrote: > So smp_mb() provides transitivity, as do pairs of smp_store_release() > and smp_read_acquire(), But they provide different grades of transitivity, which is where all the confusion lays. smp_mb() is strongly/globally transitive,

Re: [Xen-devel] [v3,11/41] mips: reuse asm-generic/barrier.h

2016-01-15 Thread Peter Zijlstra
On Fri, Jan 15, 2016 at 09:55:54AM +0100, Peter Zijlstra wrote: > On Thu, Jan 14, 2016 at 01:29:13PM -0800, Paul E. McKenney wrote: > > So smp_mb() provides transitivity, as do pairs of smp_store_release() > > and smp_read_acquire(), > > But they provide different grades of transitivity, which is

[Xen-devel] Bug 1591 - xentop truncated VM names

2016-01-15 Thread Michael Großer
Hi! Some days ago, I discovered the 'xentop' tool (somebody mentioned it here: http://discussions.citrix.com/topic/374218-what-can-we-do-to-decrease-the-time-needed-for-snapshot-reverts/ ) We work with XenServer 6.5 and we use long names for our virtual machines to distinguish between them. A

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

2016-01-15 Thread Markus Armbruster
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 **errp >> parameter and use it. >> >> Leave case RUN_STATE_INMIGRATE alone, because that

Re: [Xen-devel] [PATCH] libxl: Use libxl_strdup instead of strdup on libxl_version_info

2016-01-15 Thread Ian Campbell
On Thu, 2016-01-14 at 22:03 -0500, Konrad Rzeszutek Wilk wrote: > On January 14, 2016 9:33:49 PM EST, Konrad Rzeszutek Wilk acle.com> wrote: > > As the libxl_strdup allows us to unwind and free all > > of the allocations, while strdup requires the callers > > to remember to free (which they didn't

Re: [Xen-devel] [PATCH v3 2/5] remus: resume immediately if libxl__xc_domain_save_done() completes

2016-01-15 Thread Ian Campbell
On Fri, 2016-01-15 at 13:44 +0800, Wen Congyang wrote: > On 01/14/2016 06:21 PM, Ian Campbell wrote: > > On Tue, 2016-01-12 at 09:40 +0800, Wen Congyang wrote: > > > On 01/09/2016 12:27 AM, Ian Campbell wrote: > > > > On Fri, 2016-01-08 at 14:38 +0800, Wen Congyang wrote: > > > > > For example: if

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

2016-01-15 Thread Ian Campbell
On Thu, 2016-01-14 at 17:42 +, Ian Jackson wrote: > Ian Campbell writes ("Re: [PATCH] QEMU as non-root and PCI passthrough do > not mix"): > > That reminds me -- how does the qemu user selection work over migrate? > > Does > > it remember the specific user or does it try and pick it again on th

Re: [Xen-devel] [PATCH v3 2/5] remus: resume immediately if libxl__xc_domain_save_done() completes

2016-01-15 Thread Wen Congyang
On 01/15/2016 05:48 PM, Ian Campbell wrote: > On Fri, 2016-01-15 at 13:44 +0800, Wen Congyang wrote: >> On 01/14/2016 06:21 PM, Ian Campbell wrote: >>> On Tue, 2016-01-12 at 09:40 +0800, Wen Congyang wrote: On 01/09/2016 12:27 AM, Ian Campbell wrote: > On Fri, 2016-01-08 at 14:38 +0800, We

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

2016-01-15 Thread Ian Campbell
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: > > > > > This allows code elsewhere in libxl to find o

Re: [Xen-devel] [v3,11/41] mips: reuse asm-generic/barrier.h

2016-01-15 Thread Will Deacon
Paul, On Thu, Jan 14, 2016 at 02:20:46PM -0800, Paul E. McKenney wrote: > On Thu, Jan 14, 2016 at 01:24:34PM -0800, Leonid Yegoshin wrote: > > It is not so simple, I mean "local ordering for address and data > > dependencies". Local ordering is NOT enough. It happens that current > > MIPS R6 doesn

Re: [Xen-devel] [SeaBIOS] [SEABIOS] Plans for either 1.9.1 or 1.10.0?

2016-01-15 Thread Gerd Hoffmann
Hi, > It's been suggested (by you :)) that > 76327b9f32a009245c215f4a3c5d58a01b5310ae be cherry-picked into 1.9.1 as > well, perhaps. Yes, right. Thanks for the reminder. Picked up. cheers, Gerd ___ Xen-devel mailing list Xen-devel@lists.xen.or

[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 create

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 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

[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

Re: [Xen-devel] [v3,11/41] mips: reuse asm-generic/barrier.h

2016-01-15 Thread Will Deacon
On Thu, Jan 14, 2016 at 02:55:10PM -0800, Paul E. McKenney wrote: > On Thu, Jan 14, 2016 at 01:36:50PM -0800, Leonid Yegoshin wrote: > > On 01/14/2016 01:29 PM, Paul E. McKenney wrote: > > > > > >>On 01/14/2016 12:34 PM, Paul E. McKenney wrote: > > >>> > > >>>The WRC+addr+addr is OK because data de

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

2016-01-15 Thread Ian Campbell
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 > freely used as a byte array. This is not true on FreeBSD (and NetBSD > too, not sure about other BSD UUID implementations), where the interna

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(&queue->rx_queue); >> +if (!skb) >> +return false; >> + >> +needed = DIV_ROUND_UP(skb->len, XE

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 done) >> - positive (succ

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: Roger

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 dependen

[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 every

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 > > Acked-by: Roger Pau

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-

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 boo

[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 succeed,

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 mai

[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. Signed-off-by: Malcolm Crossley

[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 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -133,6 +133,7 @@

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 section in ./MAINTAINERS h

[Xen-devel] [linux-mingo-tip-master test] 78146: regressions - FAIL

2016-01-15 Thread osstest service owner
flight 78146 linux-mingo-tip-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/78146/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-rumpuserxen-i386 10 guest-start fail REGR. vs. 60684 test-amd64

[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 achieving

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 > --- >  MAINTAINERS | 1 + >  1 file changed, 1 insertion(+) > > diff --git a/MAINT

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 to differ at all compared wit

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 informa

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 > iommu_inclusive_mapping=1 com1=115200,8n1 console=c

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 iommu=1,verbose,debug iommu_inclusive_mappin

[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

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 *qi_ctrl = iommu_qi_

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

2016-01-15 Thread Ian Campbell
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: ssd-xen-debug-marker console_ti

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 status of the added me

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: ssd-xen-debug-marker con

[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 adva

[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 incorre

[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 -pedantic clean Retained ack, hope that's

[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 fe

[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 not

[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 series

[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 sid

[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 fu

[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 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 arran

[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 changed, 2 insertions(+), 4 deletions(-) diff --git a/tools/libs/evtchn/i

[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 +- tools/libs/foreignmemory/freebsd.c | 4 ++-- .../libs/foreignmemory/include/x

[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 s

[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| 2 +- tools/libs/call/minios.c

[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 ol

[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 files changed, 3 insertions(+), 3 deletions(-)

[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 woul

[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 --- a/tools/libs/call/linux.c +++ b/tools/libs/call/linux.c @@ -94,7 +

[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 re

[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 result

[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 interfa

[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 e

[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 --- tools/Makefile | 4 1 file changed,

[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 insertions(+), 1 deletion(-

[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| 4 ++-- tools/libs/ev

[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 insertions(+), 1 deletion(-)

[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 x

[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 179a6b7..229d642 100644 --- a/xen-hooks.mak +++ b/xen-hooks.mak @

[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 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_common.

[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(+) diff --git a/xen-hooks.mak b/xen-hooks.mak index 229d

[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 --- v7: New --- tools/libs/call

[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 Acked-by: Wei Liu --- v2: Adjust for l

[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 behavi

[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 se

[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 x

[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 Acked-by: Wei Liu --- v2: Adjust for libs/$lib layou

[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 Acked-b

[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. For

[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 rel

[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 Acked-by: Wei Liu --- v2: Adjust for libs/$lib layou

[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 Jackson --- v3: Library mov

[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 Acked-by: Wei Liu --- v2: Adjust for libs/$lib layout. -

[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 Documentation/devicetree/bindings/serial/cdns,uart.tx

[Xen-devel] [PATCH v6 2/2] xen_balloon: support memory auto onlining policy

2016-01-15 Thread Vitaly Kuznetsov
Add support for the newly added kernel memory auto onlining policy to Xen ballon driver. Suggested-by: Daniel Kiper Reviewed-by: Daniel Kiper Acked-by: David Vrabel Signed-off-by: Vitaly Kuznetsov --- Changes since v5: - Change the last 'domU' -> 'target domain' in Kconfig [Daniel Kiper] ---

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

2016-01-15 Thread Vitaly Kuznetsov
Currently, all newly added memory blocks remain in 'offline' state unless someone onlines them, some linux distributions carry special udev rules like: SUBSYSTEM=="memory", ACTION=="add", ATTR{state}=="offline", ATTR{state}="online" to make this happen automatically. This is not a great solution

[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 reaso

[Xen-devel] [libvirt test] 78148: tolerable FAIL - PUSHED

2016-01-15 Thread osstest service owner
flight 78148 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/78148/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-libvirt-vhd 9 debian-di-installfail like 77871 Tests which did not succeed, but a

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 cdns,uart-r1p12 defined? I don't see it >

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

2016-01-15 Thread Ian Campbell
/dev/xen/gntdev related wrappers have been moved out of libxenctrl into their own library. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- v3: Library moved to tools/libs/ v7: s/gnttab_munmap/gnttab_unmap/ to reflect change to the library API. --- hw/xen_backend.c | 4 ++-- hw/xen_backe

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 one described in the commi

  1   2   3   >