[Xen-devel] [xen-unstable test] 33215: trouble: blocked/broken/fail/pass

2015-01-08 Thread xen . org
flight 33215 xen-unstable real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/33215/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-libvirt 3 host-install(3) broken REGR. vs. 33112

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

2015-01-08 Thread Xu, Quan
> -Original Message- > From: Daniel De Graaf [mailto:dgde...@tycho.nsa.gov] > Sent: Wednesday, January 07, 2015 3:47 AM > To: Xu, Quan; xen-devel@lists.xen.org > Cc: samuel.thiba...@ens-lyon.org; stefano.stabell...@eu.citrix.com > Subject: Re: [PATCH v2 1/5] vTPM: event channel bind inter

[Xen-devel] Xen-ringwatch generating many alerts

2015-01-08 Thread vishalfaldu
Hi , We have xen-ringwatch which is sending below alert frequently, Any one can help ? Subject: Cron xen-ringwatch check -T3 --kick | logger -p daemon.crit -t RINGWATCH-ALERT To: root Traceback (most recent call last): File "/usr/sbin/xen-ringwatch", line 593, in ? watches.update() Fil

Re: [Xen-devel] Mapping Data between Dom0 and DomU

2015-01-08 Thread Ian Campbell
On Thu, 2015-01-08 at 02:01 +, Andrew Cooper wrote: > A relevant bit of code to look at would be tools/libvchan/ , which as > far as I am aware does pretty much what you describe (but without the > windows support). Adding a libvchan front and/or backend driver to the Windows PV drivers would

Re: [Xen-devel] [PATCH v2 4/5] vTPM: add vTPM device for HVM virtual machine

2015-01-08 Thread Xu, Quan
> -Original Message- > From: Wei Liu [mailto:wei.l...@citrix.com] > Sent: Monday, January 05, 2015 9:14 PM > To: Xu, Quan > Cc: xen-devel@lists.xen.org; ian.jack...@eu.citrix.com; > stefano.stabell...@eu.citrix.com; ian.campb...@citrix.com; wei.l...@citrix.com > Subject: Re: [PATCH v2 4/5

Re: [Xen-devel] [PATCH v2 3/4] sysctl: Add sysctl interface for querying PCI topology

2015-01-08 Thread Jan Beulich
>>> On 07.01.15 at 18:55, wrote: > There probably would not be too serious issues in converting everything > to PXM, or adding duplicates, but I don't see the reason why we should > do such a thing... Perhaps I'm missing what using PXM would actually buy > us? As long as those node IDs don't get

Re: [Xen-devel] [PATCH v4] x86/hvm: Add per-vcpu evtchn upcalls

2015-01-08 Thread Jan Beulich
>>> On 07.01.15 at 19:06, wrote: > --- a/xen/arch/x86/hvm/hvm.c > +++ b/xen/arch/x86/hvm/hvm.c > @@ -5462,6 +5462,40 @@ static int hvmop_destroy_ioreq_server( > return rc; > } > > +static int hvmop_set_evtchn_upcall_vector( > +XEN_GUEST_HANDLE_PARAM(xen_hvm_set_evtchn_upcall_vector_t)

Re: [Xen-devel] Mapping Data between Dom0 and DomU

2015-01-08 Thread Paul Durrant
> -Original Message- > From: xen-devel-boun...@lists.xen.org [mailto:xen-devel- > boun...@lists.xen.org] On Behalf Of Ian Campbell > Sent: 08 January 2015 08:59 > To: Andrew Cooper > Cc: xen-de...@lists.xenproject.org; win-pv-de...@lists.xenproject.org; > Christian Refvik > Subject: Re: [Xe

Re: [Xen-devel] Xen-ringwatch generating many alerts

2015-01-08 Thread Ian Campbell
On Wed, 2015-01-07 at 22:54 -0700, vishalfaldu wrote: > Hi , > > We have xen-ringwatch which is sending below alert frequently, Any one can > help ? > > Subject: Cron xen-ringwatch check -T3 --kick | logger -p > daemon.crit -t RINGWATCH-ALERT > To: root > > > Traceback (most recent call last):

[Xen-devel] [qemu-mainline test] 33239: regressions - trouble: blocked/broken/fail/pass

2015-01-08 Thread xen . org
flight 33239 qemu-mainline real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/33239/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-win7-amd64 7 windows-install fail REGR. vs. 32598 test-amd64-i386-xl-

Re: [Xen-devel] [OSSTEST PATCH 3/4] Add nested testcase of installing L2 guest VM

2015-01-08 Thread Wei Liu
On Thu, Jan 08, 2015 at 07:34:39AM +, Pang, LongtaoX wrote: [...] > > > ts-debian-install installs a L2 PV guest, which should work even > > > without nested HVM enabled for your L1 HVM guest. You're testing > > > nested HVM I think it makes more sense to install a L2 HVM guest. > > > > > [Pang

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

2015-01-08 Thread Ian Campbell
Coverity complains: > /tools/libxl/_libxl_types.c: 9194 in libxl__device_channel_parse_json() > 9188 } > 9189 x = libxl__json_map_get("connection.socket", o, JSON_MAP); > 9190 if (x) { > 9191 libxl_device_channel_init_connection(p, > LIBXL_CHANNE

Re: [Xen-devel] [OSSTEST PATCH 3/4] Add nested testcase of installing L2 guest VM

2015-01-08 Thread Robert Hu
On Thu, 2015-01-08 at 10:48 +, Wei Liu wrote: > On Thu, Jan 08, 2015 at 07:34:39AM +, Pang, LongtaoX wrote: > [...] > > > > ts-debian-install installs a L2 PV guest, which should work even > > > > without nested HVM enabled for your L1 HVM guest. You're testing > > > > nested HVM I think it

Re: [Xen-devel] [OSSTEST PATCH 3/4] Add nested testcase of installing L2 guest VM

2015-01-08 Thread Ian Campbell
On Thu, 2015-01-08 at 10:48 +, Wei Liu wrote: > On Thu, Jan 08, 2015 at 07:34:39AM +, Pang, LongtaoX wrote: > [...] > > > > ts-debian-install installs a L2 PV guest, which should work even > > > > without nested HVM enabled for your L1 HVM guest. You're testing > > > > nested HVM I think it

Re: [Xen-devel] [PATCH v2 4/5] vTPM: add vTPM device for HVM virtual machine

2015-01-08 Thread Wei Liu
On Thu, Jan 08, 2015 at 09:44:11AM +, Xu, Quan wrote: [...] > > > > +flexarray_append(back, "resume"); > > > +flexarray_append(back, "False"); > > > + > > > +flexarray_append(front, "backend-id"); > > > +flexarray_append(front, GCSPRINTF("%d", vtpm->backend_domid)); > > > +

Re: [Xen-devel] [OSSTEST PATCH 3/4] Add nested testcase of installing L2 guest VM

2015-01-08 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH 3/4] Add nested testcase of installing L2 guest VM"): > Once the L1 hypervisor is up and running shouldn't we mostly be treating > it as if it were an L0 hypervisor for the purposes of testing its > functionality, up to and including the types of guest we t

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

2015-01-08 Thread Wei Liu
On Thu, Jan 08, 2015 at 10:57:47AM +, Ian Campbell wrote: > Coverity complains: > > /tools/libxl/_libxl_types.c: 9194 in libxl__device_channel_parse_json() > > 9188 } > > 9189 x = libxl__json_map_get("connection.socket", o, JSON_MAP); > > 9190 if (x) { > > 91

Re: [Xen-devel] Writable page tables questions

2015-01-08 Thread Tim Deegan
At 09:55 + on 06 Jan (1420534536), Ian Campbell wrote: > The tlb flushes involved in the above are reasonably expensive, IIRC Xen > flip flopped a bit (years ago now) on whether it is worthwhile doing > this or not, which is why I'm not sure if it still does or not. The current "writable paget

Re: [Xen-devel] [OSSTEST PATCH 3/4] Add nested testcase of installing L2 guest VM

2015-01-08 Thread Ian Campbell
On Thu, 2015-01-08 at 19:22 +0800, Robert Hu wrote: > On Thu, 2015-01-08 at 10:48 +, Wei Liu wrote: > > On Thu, Jan 08, 2015 at 07:34:39AM +, Pang, LongtaoX wrote: > > [...] > > > > > ts-debian-install installs a L2 PV guest, which should work even > > > > > without nested HVM enabled for y

Re: [Xen-devel] Writable page tables questions

2015-01-08 Thread Ian Campbell
On Thu, 2015-01-08 at 12:19 +0100, Tim Deegan wrote: > At 09:55 + on 06 Jan (1420534536), Ian Campbell wrote: > > The tlb flushes involved in the above are reasonably expensive, IIRC Xen > > flip flopped a bit (years ago now) on whether it is worthwhile doing > > this or not, which is why I'm n

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

2015-01-08 Thread Ian Campbell
ePAPR v1.1 section 3.5 defines the /chosen/stdout-path property to refer to the device to be used for boot console output, so if no dtuart property is given try to use that instead. This will make Xen find a suitable console by default on DT platforms which include this property. As it happens the

[Xen-devel] [PATCH v2 2/4] dt-uart: Clarify log messages at init time.

2015-01-08 Thread Ian Campbell
- Don't log at all if console=dtuart (the default) was not present, in that case the user has asked for something else, no need for every other driver to tell them this. - Use "dtuart" in all other messages, rather than just "console" or "uart". - Be more explicit if we are exiting because dt

[Xen-devel] [PATCH v2 3/4] dt-uart: use ':' as separator between path and options

2015-01-08 Thread Ian Campbell
',' is a valid character in a device-tree path (see ePAPR v1.1 Table 2-1), in fact ',' is actually pretty common in node names. Using ',' as a separator breaks for example on fast models. If you use the full path (/smb/motherboard/iofpga@3,/uart@09) rather than the alias then earlyprin

[Xen-devel] [PATCH v2 0/4] dt-uart: cleanups, bugfixes and /chosen/stdout-path support

2015-01-08 Thread Ian Campbell
The two main changes here are: * a bugfix to deal with DT paths which contain a common, which are perfectly valid and quite common. * support for the /chosen/stdout-path device tree property, which allows Xen to find a boot console without manual configuration i

[Xen-devel] [PATCH v2 1/4] dt-uart: add an emacs magic block

2015-01-08 Thread Ian Campbell
Signed-off-by: Ian Campbell Reviewed-by: Julien Grall --- xen/drivers/char/dt-uart.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/xen/drivers/char/dt-uart.c b/xen/drivers/char/dt-uart.c index fa92b5c..45a87a6 100644 --- a/xen/drivers/char/dt-uart.c +++ b/xen/drivers/char/dt-

Re: [Xen-devel] [RFC V9 3/4] domain snapshot design: xl

2015-01-08 Thread Ian Campbell
On Mon, 2014-12-22 at 01:52 -0700, Chun Yan Liu wrote: > > >>> On 12/19/2014 at 06:27 PM, in message > >>> <1418984856.20028.17.ca...@citrix.com>, > Ian Campbell wrote: > > On Fri, 2014-12-19 at 00:03 -0700, Chun Yan Liu wrote: > > > > > '--name' meant to give a meaningful name (like: newins

Re: [Xen-devel] [RFC V9 4/4] domain snapshot design: libxl/libxlu

2015-01-08 Thread Ian Campbell
On Mon, 2014-12-22 at 02:36 -0700, Chun Yan Liu wrote: > > > b). For internal snapshot, like qcow2, lvm too. For lvm, it doesn't > > > support > > > snapshot of snapshot, so out of scope. For qcow2, delete any disk > > > snapshot > > > won't affect others. > > > > For either internal or exte

Re: [Xen-devel] Significant network performance difference when different NIC MQ handles the packet

2015-01-08 Thread Zoltan Kiss
On 04/01/15 06:15, Zhangleiqiang (Trump) wrote: On 31/12/14 09:06, Zhangleiqiang (Trump) wrote: Hi, all: I have used pkt-gen to send udp packet (1400 bytes) from one Dom0 A to another Dom0 B each of which is connected by 10GE network. On the receive side (B),different "ksoftirqd/x" p

[Xen-devel] [PATCH] x86/apic: Remove redundant LVR read in init_bsp_APIC()

2015-01-08 Thread Andrew Cooper
'ver' became unused by c/s 0b964c0c7 "x86/LAPIC: drop support for non-integrated APIC", which in turn makes the LVR read redundant. Signed-off-by: Andrew Cooper CC: Keir Fraser CC: Jan Beulich --- xen/arch/x86/apic.c |5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/xe

Re: [Xen-devel] [PATCH v2 3/4] dt-uart: use ':' as separator between path and options

2015-01-08 Thread Julien Grall
Hi Ian, On 08/01/15 11:53, Ian Campbell wrote: > diff --git a/xen/drivers/char/dt-uart.c b/xen/drivers/char/dt-uart.c > index 04dbb97..b9a0956 100644 > --- a/xen/drivers/char/dt-uart.c > +++ b/xen/drivers/char/dt-uart.c > @@ -31,7 +31,7 @@ > * doesn't start with '/', we assuming that it's an ali

Re: [Xen-devel] [RFC V9 2/4] domain snapshot overview

2015-01-08 Thread Ian Campbell
On Mon, 2014-12-22 at 20:42 -0700, Chun Yan Liu wrote: > > >>> On 12/19/2014 at 06:25 PM, in message > >>> <1418984720.20028.15.ca...@citrix.com>, > Ian Campbell wrote: > > On Thu, 2014-12-18 at 22:45 -0700, Chun Yan Liu wrote: > > > > > > >>> On 12/18/2014 at 11:10 PM, in message > > <141

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

2015-01-08 Thread Ian Campbell
CCing Stefano, who was involved in the original xl ballooning stuff and the other toolstack maintainers. On Wed, 2015-01-07 at 18:11 -0700, Mike Latimer wrote: > On Wednesday, January 07, 2015 09:38:31 AM Ian Campbell wrote: > > That's exactly what I was about to suggest as I read the penultimate

Re: [Xen-devel] [PATCH v2 3/4] dt-uart: use ':' as separator between path and options

2015-01-08 Thread Ian Campbell
On Thu, 2015-01-08 at 12:26 +, Julien Grall wrote: > Hi Ian, > > On 08/01/15 11:53, Ian Campbell wrote: > > diff --git a/xen/drivers/char/dt-uart.c b/xen/drivers/char/dt-uart.c > > index 04dbb97..b9a0956 100644 > > --- a/xen/drivers/char/dt-uart.c > > +++ b/xen/drivers/char/dt-uart.c > > @@ -3

Re: [Xen-devel] (v2) Design proposal for RMRR fix

2015-01-08 Thread Tim Deegan
Hi Kevin, Thanks for sending out this design document. I think Jan will have the most to say about this. Looking just at the hypervisor side of things, and leaving the tools desig to others... At 11:23 + on 26 Dec (1419589382), Tian, Kevin wrote: > c) in Xen hypervisor it is reasonable to f

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

2015-01-08 Thread Ian Campbell
ping. On Mon, 2014-12-15 at 13:18 +, Ian Campbell wrote: > TDB provides us with a callback for this purpose. Use it in both > xenstored and xs_tdb_dump. > > While at it make the existing log() macro tollerate memory failures. > > Signed-off-by: Ian Campbell > --- > tools/xenstore/xenstored

Re: [Xen-devel] One question about the hypercall to translate gfn to mfn.

2015-01-08 Thread Tim Deegan
Hi, At 08:56 + on 06 Jan (1420530995), Tian, Kevin wrote: > > From: Tim Deegan [mailto:t...@xen.org] > > At 07:24 + on 12 Dec (1418365491), Tian, Kevin wrote: > > > but just to confirm one point. from my understanding whether it's a > > > mapping operation doesn't really matter. We can inv

[Xen-devel] [PATCH v2] x86/stack: Avoid peeking into unmapped guard pages when dumping Xens stack

2015-01-08 Thread Andrew Cooper
Currently, Xens stack tracing and dumping of its own stacks will always attempt to continue to the top of the primary stack. While this is fine for 99% of cases, it is incorrect when the stack pointer starts on an IST stack. In particular, the stack analysis functions will wander up from the IST

Re: [Xen-devel] [PATCH] x86/apic: Remove redundant LVR read in init_bsp_APIC()

2015-01-08 Thread Tim Deegan
At 12:20 + on 08 Jan (1420716041), Andrew Cooper wrote: > 'ver' became unused by c/s 0b964c0c7 "x86/LAPIC: drop support for > non-integrated APIC", which in turn makes the LVR read redundant. > > Signed-off-by: Andrew Cooper Reviewed-by: Tim Deegan _

Re: [Xen-devel] (v2) Design proposal for RMRR fix

2015-01-08 Thread George Dunlap
On Fri, Dec 26, 2014 at 11:23 AM, Tian, Kevin wrote: > (please note some proposal is different from last sent version after more > discussions. But I tried to summarize previous discussions and explained why > we choose a different way. Sorry if I may miss some opens/conclusions > discussed in pas

Re: [Xen-devel] [RFC 2/2] x86, vdso, pvclock: Simplify and speed up the vdso pvclock reader

2015-01-08 Thread David Vrabel
On 23/12/2014 00:39, Andy Lutomirski wrote: The pvclock vdso code was too abstracted to understand easily and excessively paranoid. Simplify it for a huge speedup. This opens the door for additional simplifications, as the vdso no longer accesses the pvti for any vcpu other than vcpu 0. Before

Re: [Xen-devel] (v2) Design proposal for RMRR fix

2015-01-08 Thread George Dunlap
On Thu, Jan 8, 2015 at 12:49 PM, George Dunlap wrote: > If RMRRs almost always happen up above 2G, for example, then a simple > solution that wouldn't require too much work would be to make sure > that the PCI MMIO hole we specify to libxc and to qemu-upstream is big > enough to include all RMRRs.

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

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

Re: [Xen-devel] [PATCH v2] x86/stack: Avoid peeking into unmapped guard pages when dumping Xens stack

2015-01-08 Thread Tim Deegan
At 12:43 + on 08 Jan (1420717392), Andrew Cooper wrote: > Currently, Xens stack tracing and dumping of its own stacks will always > attempt to continue to the top of the primary stack. While this is fine for > 99% of cases, it is incorrect when the stack pointer starts on an IST stack. > > In

Re: [Xen-devel] (v2) Design proposal for RMRR fix

2015-01-08 Thread Jan Beulich
>>> On 08.01.15 at 13:49, wrote: > One question: where are these RMRRs typically located in memory? Are > they normally up in the MMIO region? Or can they occur anywhere (even > in really low areas, say, under 1GiB)? They would typically sit in the MMIO hole or below 1Mb; that latter case is pa

Re: [Xen-devel] (v2) Design proposal for RMRR fix

2015-01-08 Thread Jan Beulich
>>> On 08.01.15 at 13:54, wrote: > On Thu, Jan 8, 2015 at 12:49 PM, George Dunlap > wrote: >> If RMRRs almost always happen up above 2G, for example, then a simple >> solution that wouldn't require too much work would be to make sure >> that the PCI MMIO hole we specify to libxc and to qemu-upstr

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

2015-01-08 Thread Ian Campbell
On Thu, 2015-01-08 at 12:57 +, Wei Liu wrote: > On Thu, Jan 08, 2015 at 12:33:29PM +, Ian Campbell wrote: > > ping. > > > > On Mon, 2014-12-15 at 13:18 +, Ian Campbell wrote: > > > TDB provides us with a callback for this purpose. Use it in both > > > xenstored and xs_tdb_dump. > > >

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

2015-01-08 Thread Wei Liu
On Thu, Jan 08, 2015 at 01:04:26PM +, Ian Campbell wrote: > On Thu, 2015-01-08 at 12:57 +, Wei Liu wrote: > > On Thu, Jan 08, 2015 at 12:33:29PM +, Ian Campbell wrote: > > > ping. > > > > > > On Mon, 2014-12-15 at 13:18 +, Ian Campbell wrote: > > > > TDB provides us with a callback

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

2015-01-08 Thread Ian Campbell
On Thu, 2015-01-08 at 13:04 +, Ian Campbell wrote: > On Thu, 2015-01-08 at 12:57 +, Wei Liu wrote: > > On Thu, Jan 08, 2015 at 12:33:29PM +, Ian Campbell wrote: > > > ping. > > > > > > On Mon, 2014-12-15 at 13:18 +, Ian Campbell wrote: > > > > TDB provides us with a callback for th

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

2015-01-08 Thread Julien Grall
Hi Ian, On 08/01/15 11:53, Ian Campbell wrote: > +ret = dt_property_read_string(chosen, "stdout-path", &stdout); > +if ( ret >= 0 ) > +{ > +printk("Taking dtuart configuration from > /chosen/stdout-path\n"); > +if ( strlcpy(opt_d

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

2015-01-08 Thread Ian Campbell
On Thu, 2015-01-08 at 13:15 +, Julien Grall wrote: > Hi Ian, > > On 08/01/15 11:53, Ian Campbell wrote: > > +ret = dt_property_read_string(chosen, "stdout-path", &stdout); > > +if ( ret >= 0 ) > > +{ > > +printk("Taking dtuart configuration f

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

2015-01-08 Thread Wei Liu
On Thu, Jan 08, 2015 at 01:10:38PM +, Ian Campbell wrote: > On Thu, 2015-01-08 at 13:04 +, Ian Campbell wrote: > > On Thu, 2015-01-08 at 12:57 +, Wei Liu wrote: > > > On Thu, Jan 08, 2015 at 12:33:29PM +, Ian Campbell wrote: > > > > ping. > > > > > > > > On Mon, 2014-12-15 at 13:18

[Xen-devel] [PATCH] x86/traps: Avoid the #GP slowpath for guest #DB exceptions

2015-01-08 Thread Andrew Cooper
do_debug() is capable of correctly dealing with #DB exceptions in guest context, and indeed needs to be as the 'icebp' instruction skips the DPL check anyway. Signed-off-by: Andrew Cooper CC: Jan Beulich CC: Keir Fraser CC: Tim Deegan --- xen/arch/x86/traps.c |2 +- 1 file changed, 1 inse

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

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

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

2015-01-08 Thread Andrew Cooper
The existing makefile was awkward with needing to express conditional inclusion for both the build and install rules, and contained both split and unsplit long lines. The INSTALL_* rules now contain the conditional inclusion information, while the TARGET_* rules generate the build list from the co

[Xen-devel] [PATCH] tools/misc: Remove sbdf2devicepath

2015-01-08 Thread Andrew Cooper
This script has become orphaned from the build system, and depends on removed Xend functionality (xen.util.pci) so can't possibly function now. Signed-off-by: Andrew Cooper CC: Ian Campbell CC: Ian Jackson CC: Wei Liu --- tools/misc/sbdf2devicepath | 82 -

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

2015-01-08 Thread Wei Liu
This patch resurrects 11a63a166. The previous patch had a bug that wrong "physical-device" was written to xenstore causing block script execution fail. This patch fixes that problem. Following configurations have been tested: 1. Raw file and PV 2. Raw file and HVM 3. Block device and PV 4. Block

Re: [Xen-devel] [PATCH v2 for-4.5] libxl: account for romfile memory

2015-01-08 Thread Ian Campbell
On Tue, 2014-11-25 at 16:49 +, Stefano Stabellini wrote: > Actually Wei suggested a better alternative: I could call > xc_domain_setmaxmem directly from QEMU. That makes much more sense. Do you have links to the patches for the final approach you decided to take here? Or maybe just resend? Ei

Re: [Xen-devel] (v2) Design proposal for RMRR fix

2015-01-08 Thread Jan Beulich
>>> On 26.12.14 at 12:23, wrote: > [Issue-2] Being lacking of goal-b), existing device assignment with > RMRR works only when reserved regions happen to not conflicting with > other valid allocations in the guest physical address space. This could > lead to unpredicted failures in various deploym

Re: [Xen-devel] [PATCH] x86/traps: Avoid the #GP slowpath for guest #DB exceptions

2015-01-08 Thread Jan Beulich
>>> On 08.01.15 at 14:25, wrote: > do_debug() is capable of correctly dealing with #DB exceptions in guest > context, and indeed needs to be as the 'icebp' instruction skips the DPL > check anyway. I don't follow: ICEBP doesn't check DPL, right, but what does setting DPL to 3 buy us? Other than f

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

2015-01-08 Thread Ian Campbell
On Wed, 2014-11-26 at 16:55 +, Wei Liu wrote: > Modify libxl and hotplug script to allow raw format file to use phy > backend. > > The block script now tests the path and determine the actual type of > file (block device or regular file) then use the actual type to > determine which branch to

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

2015-01-08 Thread Ian Campbell
oops, I was literally reviewing the previous send as you sent this... On Thu, 2015-01-08 at 13:46 +, Wei Liu wrote: > This patch resurrects 11a63a166. The previous patch had a bug that > wrong "physical-device" was written to xenstore causing block script > execution fail. This patch fixes tha

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

2015-01-08 Thread Fabio Fantoni
Il 08/01/2015 14:46, Wei Liu ha scritto: This patch resurrects 11a63a166. The previous patch had a bug that wrong "physical-device" was written to xenstore causing block script execution fail. This patch fixes that problem. Following configurations have been tested: 1. Raw file and PV 2. Raw fi

Re: [Xen-devel] [PATCH v2] x86/xen: avoid freeing static 'name' when kasprintf() fails

2015-01-08 Thread David Vrabel
On 05/01/15 15:27, Vitaly Kuznetsov wrote: > In case kasprintf() fails in xen_setup_timer() we assign name to the static > string "". We, however, don't check that fact before > issuing kfree() in xen_teardown_timer(), kernel is supposed to crash with > 'kernel BUG at mm/slub.c:3341!' > > Solve th

[Xen-devel] Linux'es ESPFIX

2015-01-08 Thread Jan Beulich
All, after having decided with the rest of the security team that we ought to be dealing with this in public considering it's been public on the Linux side for quite a while, here's an issue we should try to find a solution for, or decide whether a partial solution like Linux is using (see below)

[Xen-devel] [linux-linus test] 33240: trouble: blocked/broken/fail/pass

2015-01-08 Thread xen . org
flight 33240 linux-linus real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/33240/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 3 host-install(3) broken REGR. vs. 32879

Re: [Xen-devel] [PATCH v2] x86/xen: Free bootmem in free_p2m_page() during early boot

2015-01-08 Thread David Vrabel
On 07/01/15 18:27, Boris Ostrovsky wrote: > With recent changes in p2m we now have legitimate cases when > p2m memory needs to be freed during early boot (i.e. before > slab is initialized). > > Signed-off-by: Boris Ostrovsky > Reviewed-by: Juergen Gross > --- > > v2: Add __ref annotation Than

Re: [Xen-devel] [v4] libxc: Expose the 1GB pages cpuid flag to guest

2015-01-08 Thread Ian Campbell
On Fri, 2014-11-28 at 11:31 +, Jan Beulich wrote: > >>> On 28.11.14 at 11:52, wrote: > > If hardware support the 1GB pages, expose the feature to guest by > > default. Users don't have to use a 'cpuid= ' option in config fil > > e to turn it on. > > > > If guest use shadow mode, the 1GB pages

Re: [Xen-devel] [PATCH v1] xen/arm: Manage pl011 uart TX interrupt correctly

2015-01-08 Thread Ian Campbell
On Thu, 2014-12-11 at 13:15 +0100, Tim Deegan wrote: > At 10:09 +0530 on 09 Dec (1418116195), vijay.kil...@gmail.com wrote: > > From: Vijaya Kumar K > > > > In pl011.c, when TX interrupt is received > > serial_tx_interrupt() is called to push next > > characters. If TX buffer is empty, serial_tx_

Re: [Xen-devel] [PATCH RFC] xen-time: decreasing the rating of the xen clocksource below that of the tsc clocksource for dom0's

2015-01-08 Thread Imre Palik
On 01/07/15 17:30, Ian Campbell wrote: > On Wed, 2015-01-07 at 17:16 +0100, Imre Palik wrote: >> From: "Palik, Imre" >> >> In Dom0's the use of the TSC clocksource (whenever it is stable enough to >> be used) instead of the Xen clocksource should not cause any issues, as >> Dom0 VMs never live-mig

Re: [Xen-devel] (v2) Design proposal for RMRR fix

2015-01-08 Thread George Dunlap
On Thu, Jan 8, 2015 at 1:00 PM, Jan Beulich wrote: On 08.01.15 at 13:54, wrote: >> On Thu, Jan 8, 2015 at 12:49 PM, George Dunlap >> wrote: >>> If RMRRs almost always happen up above 2G, for example, then a simple >>> solution that wouldn't require too much work would be to make sure >>> th

[Xen-devel] [PATCH 0/3] x86: XSA-111 follow-ups

2015-01-08 Thread Jan Beulich
1: streamline hypercall_create_continuation() 2: HVM: clobber hypercall arguments just like for PV 3: HVM: make hvm_efer_valid() honor guest features Signed-off-by: Jan Beulich ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xe

Re: [Xen-devel] (v2) Design proposal for RMRR fix

2015-01-08 Thread Jan Beulich
>>> On 08.01.15 at 16:15, wrote: > On Thu, Jan 8, 2015 at 1:00 PM, Jan Beulich wrote: > On 08.01.15 at 13:54, wrote: >>> On Thu, Jan 8, 2015 at 12:49 PM, George Dunlap >>> wrote: If RMRRs almost always happen up above 2G, for example, then a simple solution that wouldn't require t

[Xen-devel] [xen-4.3-testing test] 33246: trouble: blocked/broken/fail/pass

2015-01-08 Thread xen . org
flight 33246 xen-4.3-testing real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/33246/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-pvops 3 host-install(3) broken REGR. vs. 322

[Xen-devel] [PATCH 1/3] x86: streamline hypercall_create_continuation()

2015-01-08 Thread Jan Beulich
- drop clearing of excessive multicall arguments in compat case (no longer needed now that hypercall_xlat_continuation() only checks the actual arguments) - latch current into a local variable - use the cached value of hvm_guest_x86_mode() instead of re-executing it - scope restrict "regs" -

[Xen-devel] [PATCH 2/3] x86/HVM: clobber hypercall arguments just like for PV

2015-01-08 Thread Jan Beulich
Unused arguments get clobbered before the call (not affecting caller visible state), while used arguments get clobbered afterwards unless a continuation is needed (affecting caller visible state). Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -4818,6 +48

[Xen-devel] [PATCH 3/3] x86/HVM: make hvm_efer_valid() honor guest features

2015-01-08 Thread Jan Beulich
Following the earlier similar change validating CR4 modifications. Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -1672,20 +1672,53 @@ static int hvm_save_cpu_ctxt(struct doma return 0; } -static bool_t hvm_efer_valid(struct domain *d, -

[Xen-devel] [PATCH v2] libxl: Spice image compression setting support for upstream qemu

2015-01-08 Thread Fabio Fantoni
Usage: spice_image_compression=[auto_glz|auto_lz|quic|glz|lz|off] Specifies what image compression is to be used by spice (if given), otherwise the qemu default will be used. Signed-off-by: Fabio Fantoni --- Changes in v2: - refresh --- docs/man/xl.cfg.pod.5 | 5 + tools/libxl/libx

[Xen-devel] [PATCH v2] libxl: Spice streaming video setting support for upstream qemu

2015-01-08 Thread Fabio Fantoni
Usage: spice_streaming_video=[filter|all|off] Specifies what streaming video setting is to be used by spice (if given), otherwise the qemu default will be used. Signed-off-by: Fabio Fantoni --- Changes in v2: - refresh --- docs/man/xl.cfg.pod.5 | 5 + tools/libxl/libxl.h |

[Xen-devel] [PATCHv1 0/2] mm: infrastructure for correctly handling foreign pages on Xen

2015-01-08 Thread David Vrabel
These two patches are the common parts of a larger Xen series[1] fixing several long-standing bugs the handling of foreign[2] pages in Xen guests. The first patch is required to fix get_user_pages[_fast]() with userspace space mappings of such foreign pages. Basically, pte_page() doesn't work so

[Xen-devel] [PATCH 1/2] mm: allow for an alternate set of pages for userspace mappings

2015-01-08 Thread David Vrabel
Add an optional array of pages to struct vm_area_struct that can be used find the page backing a VMA. This is useful in cases where the normal mechanisms for finding the page don't work. This array is only inspected if the PTE is special. Splitting a VMA with such an array of pages is trivially

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

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

Re: [Xen-devel] Question about significant network performance difference after pin RX netback to vcpu0

2015-01-08 Thread Zoltan Kiss
On 08/01/2015 02:33, trump_zhang wrote: Hi, I am trying to test the single-queue networking performance for netback/netfront in upstream, my testing environment is as follows: 1. Using pkt-gen to send a single UDP flow from one host to a vm which runs on another XEN host. Two hosts ar

[Xen-devel] preparing for 4.4.2 / 4.3.4

2015-01-08 Thread Jan Beulich
All, now that 4.5 is almost out the door, I'd like to get stable releases prepared on the other two active branches. 4.3.4 is expected to be the last xen.org managed release on the 4.3 branch. Aiming at RC1s some time next week, please indicate commits you consider relevant to backport but find mi

[Xen-devel] [PATCH] hvm/vmx: WRITE_MSR() macro hygene

2015-01-08 Thread Andrew Cooper
Use the standard "do { } while ( 0 )" semantics, and don't hide the break statement, incase this macro wants to be used anywhere outside of a switch. No functional change, but it is now clear that long_mode_do_msr_write() doesn't have fallthough for the STAR, LSTAR and SYSCALL_MASK cases. Signed-

[Xen-devel] [PATCH 0/3] x86: XSA-112 follow-ups

2015-01-08 Thread Jan Beulich
1: also allow REP STOS emulation acceleration 2: HVM: drop pointless parameters from vIOAPIC internal routines 3: HVM: vMSI simplification Signed-off-by: Jan Beulich ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 3/4] sysctl: Add sysctl interface for querying PCI topology

2015-01-08 Thread Boris Ostrovsky
On 01/08/2015 04:50 AM, Jan Beulich wrote: On 07.01.15 at 18:55, wrote: There probably would not be too serious issues in converting everything to PXM, or adding duplicates, but I don't see the reason why we should do such a thing... Perhaps I'm missing what using PXM would actually buy us? As

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

2015-01-08 Thread Jan Beulich
While the REP MOVS acceleration appears to have helped qemu-traditional based guests, qemu-upstream (or really the respective video BIOSes) doesn't appear to benefit from that. Instead the acceleration added here provides a visible performance improvement during very early HVM guest boot. Signed-o

[Xen-devel] [PATCH 2/3] x86/HVM: drop pointless parameters from vIOAPIC internal routines

2015-01-08 Thread Jan Beulich
Also simplify a few other operations (without funtional change). Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/vioapic.c +++ b/xen/arch/x86/hvm/vioapic.c @@ -46,11 +46,9 @@ static void vioapic_deliver(struct hvm_hw_vioapic *vioapic, int irq); -static unsigned long vioapic_read_indirect(

[Xen-devel] [PATCH 3/3] x86/HVM: vMSI simplification

2015-01-08 Thread Jan Beulich
- struct msixtbl_entry's table_len field can be unsigned int, and by moving it down a little the structure size can be reduced slightly - a disjoint xmalloc()/memset() pair can be converted to xzalloc() - a pointless local variable can be dropped Signed-off-by: Jan Beulich --- unstable.orig/xe

Re: [Xen-devel] [PATCH ARM v8 1/4] mini-os: arm: time

2015-01-08 Thread Ian Campbell
On Tue, 2014-10-21 at 11:50 +0100, Ian Campbell wrote: > On Fri, 2014-10-03 at 10:20 +0100, Thomas Leonard wrote: > > Based on an initial patch by Karim Raslan. > > > > Signed-off-by: Karim Allah Ahmed > > Signed-off-by: Thomas Leonard > > Acked-by: Ian Campbell Applied (for 4.6), thanks. #2

Re: [Xen-devel] [PATCH 0/3] x86: XSA-111 follow-ups

2015-01-08 Thread Tim Deegan
At 15:15 + on 08 Jan (1420726540), Jan Beulich wrote: > 1: streamline hypercall_create_continuation() > 2: HVM: clobber hypercall arguments just like for PV > 3: HVM: make hvm_efer_valid() honor guest features > > Signed-off-by: Jan Beulich Reviewed-by: Tim Deegan _

Re: [Xen-devel] [PATCH v2 3/4] sysctl: Add sysctl interface for querying PCI topology

2015-01-08 Thread Jan Beulich
>>> On 08.01.15 at 16:49, wrote: > On 01/08/2015 04:50 AM, Jan Beulich wrote: > On 07.01.15 at 18:55, wrote: >>> There probably would not be too serious issues in converting everything >>> to PXM, or adding duplicates, but I don't see the reason why we should >>> do such a thing... Perhaps I'

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

2015-01-08 Thread Daniel De Graaf
On 01/08/2015 03:20 AM, Xu, Quan wrote: -Original Message- From: Daniel De Graaf [mailto:dgde...@tycho.nsa.gov] Sent: Wednesday, January 07, 2015 3:47 AM To: Xu, Quan; xen-devel@lists.xen.org Cc: samuel.thiba...@ens-lyon.org; stefano.stabell...@eu.citrix.com Subject: Re: [PATCH v2 1/5]

Re: [Xen-devel] [PATCH 3/3] x86/HVM: make hvm_efer_valid() honor guest features

2015-01-08 Thread Tim Deegan
At 15:23 + on 08 Jan (1420727005), Jan Beulich wrote: > +if ( (value & EFER_LMSLE) && !cpu_has_lmsl ) > +return 0; I see this bit has no CPUID flag, and the docs don't seem to suggest that it would ever not be valid. Are there real CPUs where it can't be set? My reviewed-by: stan

Re: [Xen-devel] [PATCH] hvm/vmx: WRITE_MSR() macro hygene

2015-01-08 Thread Tim Deegan
At 15:38 + on 08 Jan (1420727918), Andrew Cooper wrote: > Use the standard "do { } while ( 0 )" semantics, and don't hide the break > statement, incase this macro wants to be used anywhere outside of a switch. > > No functional change, but it is now clear that long_mode_do_msr_write() > doesn'

Re: [Xen-devel] [PATCH ARM v8 1/4] mini-os: arm: time

2015-01-08 Thread Thomas Leonard
On 8 January 2015 at 15:52, Ian Campbell wrote: > On Tue, 2014-10-21 at 11:50 +0100, Ian Campbell wrote: >> On Fri, 2014-10-03 at 10:20 +0100, Thomas Leonard wrote: >> > Based on an initial patch by Karim Raslan. >> > >> > Signed-off-by: Karim Allah Ahmed >> > Signed-off-by: Thomas Leonard >> >>

Re: [Xen-devel] (v2) Design proposal for RMRR fix

2015-01-08 Thread George Dunlap
On Thu, Jan 8, 2015 at 1:54 PM, Jan Beulich wrote: > Ideally, rather than detecting conflicts, hvmloader would just > consume what libxc set up. Obviously that would require awareness > in libxc of things it currently doesn't care about (like fitting PCI BARs > into the MMIO hole, enlarging it as

[Xen-devel] [PATCH 0/3] XSA-114 follow-ups

2015-01-08 Thread Jan Beulich
1: spinlock: use local_irq_disable() instead of local_irq_save() where possible 2: rwlock: allow arch to override read_unlock() atomic 3: rwlock: allow arch to override write_unlock() atomic Signed-off-by: Jan Beulich ___ Xen-devel mailing list Xen-de

Re: [Xen-devel] [PATCH 1/3] x86: streamline hypercall_create_continuation()

2015-01-08 Thread Andrew Cooper
On 08/01/15 15:22, Jan Beulich wrote: > - drop clearing of excessive multicall arguments in compat case (no > longer needed now that hypercall_xlat_continuation() only checks the > actual arguments) > - latch current into a local variable > - use the cached value of hvm_guest_x86_mode() instead

Re: [Xen-devel] [PATCH 3/3] x86/HVM: make hvm_efer_valid() honor guest features

2015-01-08 Thread Jan Beulich
>>> On 08.01.15 at 16:56, wrote: > At 15:23 + on 08 Jan (1420727005), Jan Beulich wrote: >> +if ( (value & EFER_LMSLE) && !cpu_has_lmsl ) >> +return 0; > > I see this bit has no CPUID flag, and the docs don't seem to suggest > that it would ever not be valid. Are there real CPUs

  1   2   >