Re: [Xen-devel] [PATCH 1/2] ts-raisin-build: consolidate open coded iterations over enabled components

2015-08-05 Thread Ian Campbell
On Tue, 2015-08-04 at 14:02 +0100, Ian Campbell wrote: > +my %treerev_override = ( This is a kindofcrappy name actually. Maybe %comp2runvar or %raisin2osstest or (perhaps going too far) just %r2o? Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH v3 1/1] x86: allow to enable/disable modify_ldt at run time

2015-08-05 Thread Ingo Molnar
* Willy Tarreau wrote: > @@ -276,6 +282,15 @@ asmlinkage int sys_modify_ldt(int func, void __user *ptr, > { > int ret = -ENOSYS; > > + if (!sysctl_modify_ldt) { > + printk_ratelimited(KERN_INFO > + "Denied a call to modify_ldt() from %s[%d] (uid: %d).

Re: [Xen-devel] [PATCH v3 1/1] x86: allow to enable/disable modify_ldt at run time

2015-08-05 Thread Willy Tarreau
Hi Ingo, On Wed, Aug 05, 2015 at 10:00:37AM +0200, Ingo Molnar wrote: > > * Willy Tarreau wrote: > > > @@ -276,6 +282,15 @@ asmlinkage int sys_modify_ldt(int func, void __user > > *ptr, > > { > > int ret = -ENOSYS; > > > > + if (!sysctl_modify_ldt) { > > + printk_ratelimited

Re: [Xen-devel] [PATCH v3 1/1] x86: allow to enable/disable modify_ldt at run time

2015-08-05 Thread Ingo Molnar
* Willy Tarreau wrote: > Hi Ingo, > > On Wed, Aug 05, 2015 at 10:00:37AM +0200, Ingo Molnar wrote: > > > > * Willy Tarreau wrote: > > > > > @@ -276,6 +282,15 @@ asmlinkage int sys_modify_ldt(int func, void __user > > > *ptr, > > > { > > > int ret = -ENOSYS; > > > > > > + if (!sysctl_mo

Re: [Xen-devel] [PATCH V3 4/6] libxc: expose xsaves/xgetbv1/xsavec to hvm guest

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 09:57 +0800, Shuai Ruan wrote: > This patch exposes xsaves/xgetbv1/xsavec to hvm guest. > The reserved bits of eax/ebx/ecx/edx must be cleaned up > when call cpuid(0dh) with leaf 1 or 2..63. > > According to the spec the following bits must be reseved: "reserved" > For leaf

Re: [Xen-devel] [PATCH] tools/libxl: Correct use of phyinfo_{init, dispose}()

2015-08-05 Thread Ian Campbell
On Tue, 2015-08-04 at 19:58 +0100, Andrew Cooper wrote: > Signed-off-by: Andrew Cooper > --- > CC: Ian Campbell > CC: Ian Jackson > CC: Wei Liu > > For my cpuid work which I have just started, I added an array to physinfo. > Coverity then frowned at me when it spotted all the memory leaks. >

Re: [Xen-devel] [BUG] kernel panics with drbd

2015-08-05 Thread Ian Campbell
On Tue, 2015-08-04 at 22:24 +0100, Matthew Vernon wrote: > On 04/08/15 15:39, Matthew Vernon wrote: > > Hi, > > > > On 04/08/15 15:22, Ian Campbell wrote: > > > On Tue, 2015-08-04 at 14:52 +0100, Matthew Vernon wrote: > > > > Hi, > > > > > > Hello, > > > > > > > I'm getting dom0 kernel panics, r

Re: [Xen-devel] [RFC PATCH v3.1 2/2] xsplice: Add hook for build_id

2015-08-05 Thread Martin Pohlack
On 27.07.2015 21:20, Konrad Rzeszutek Wilk wrote: > Signed-off-by: Konrad Rzeszutek Wilk > --- > tools/libxc/xc_private.c | 3 +++ > tools/misc/xen-xsplice.c | 25 + > xen/common/kernel.c | 11 +++ > xen/common/version.c | 5 + > xen/

Re: [Xen-devel] [PATCH] tools/libxl: Correct use of phyinfo_{init, dispose}()

2015-08-05 Thread Andrew Cooper
On 05/08/15 09:40, Ian Campbell wrote: > On Tue, 2015-08-04 at 19:58 +0100, Andrew Cooper wrote: >> Signed-off-by: Andrew Cooper >> --- >> CC: Ian Campbell >> CC: Ian Jackson >> CC: Wei Liu >> >> For my cpuid work which I have just started, I added an array to physinfo. >> Coverity then frowned

[Xen-devel] Hotpatch construction and __LINE__ (was: [RFC PATCH v3.1] xSplice design.)

2015-08-05 Thread Martin Pohlack
Hi, Another high-level point to think about is how we want to handle inlined __LINE__ references. This problem is related to hotpatch construction and potentially has influence on the design of the hotpatching infrastructure in Xen. Let me try to explain the problem: We have file1.c with functi

Re: [Xen-devel] [RFC PATCH v3.1 2/2] xsplice: Add hook for build_id

2015-08-05 Thread Andrew Cooper
On 05/08/15 09:50, Martin Pohlack wrote: > On 27.07.2015 21:20, Konrad Rzeszutek Wilk wrote: >> Signed-off-by: Konrad Rzeszutek Wilk >> --- >> tools/libxc/xc_private.c | 3 +++ >> tools/misc/xen-xsplice.c | 25 + >> xen/common/kernel.c | 11 +++ >>

Re: [Xen-devel] [PATCH v3 1/1] x86: allow to enable/disable modify_ldt at run time

2015-08-05 Thread Willy Tarreau
On Wed, Aug 05, 2015 at 10:26:16AM +0200, Ingo Molnar wrote: > > +modify_ldt: (X86 only) > > + > > +Enables (1) or disables (0) the modify_ldt syscall. Modifying the LDT > > +(Local Descriptor Table) may be needed to run a 16-bit or segmented code > > s/run a/run good catch, thanks. > > +such as

Re: [Xen-devel] [PATCH v3 1/1] x86: allow to enable/disable modify_ldt at run time

2015-08-05 Thread Ingo Molnar
* Willy Tarreau wrote: > > > + if (!sysctl_modify_ldt) { > > > + printk_ratelimited(KERN_INFO > > > + "Denied a call to modify_ldt() from %s[%d] (uid: %d)." > > > + " Adjust the modify_ldt sysctl if this was not an" > > > > Would it really be so difficult

[Xen-devel] [PATCH v3] VT-d: add iommu=igfx option to workaround graphics issues

2015-08-05 Thread Ting-Wei Lan
When using Linux >= 3.19 (commit 47591df) as dom0 on some Intel Ironlake devices, It is possible to encounter graphics issues that make screen unreadable or crash the system. It was reported in freedesktop bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90037 As we still cannot find a prop

[Xen-devel] [xen 4.6 retrospective] [urgent] rename "freeze" window and make release branch as soon as possible after RC1

2015-08-05 Thread Lars Kurth
This is one item of feedback, which I believe is a quick win for us. This is one piece of feedback from a list of items that have during the last few weeks been raised with me personally, either during face-2-face conversations in a private e-mail thread. See http://lists.xenproject.org/archive

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen

2015-08-05 Thread Shannon Zhao
On 2015/8/4 22:37, Stefano Stabellini wrote: > On Tue, 4 Aug 2015, Shannon Zhao wrote: >> This document is going to explain the design details of Xen booting with >> ACPI on ARM. Maybe parts of it may not be appropriate. Any comments are >> welcome. > > Good start! > > >> To Xen itself booting

Re: [Xen-devel] [PATCH for-4.6 v3 2/6] docs/libxl: Re-specify XENSTORE_DATA as EMULATOR_XENSTORE_DATA

2015-08-05 Thread Ian Campbell
On Tue, 2015-08-04 at 18:16 +0100, Andrew Cooper wrote: > The legacy "toolstack" record as implemented in libxl turns out not to > be 32/64bit safe. As migration v2 has not shipped yet, take this > opportunity to adjust the specification and fix the incompatibility. > > Libxl shall loose all know

Re: [Xen-devel] [PATCH for-4.6 v3 3/6] tools/libxl: Save and restore EMULATOR_XENSTORE_DATA content

2015-08-05 Thread Ian Campbell
On Tue, 2015-08-04 at 18:16 +0100, Andrew Cooper wrote: > The new EMULATOR_XENSTORE_DATA content is a sequence of NUL terminated > key/value strings, with the key relative to the device model's xenstore > tree. > > A sample might look like (as decoded by verify-stream-v2): > > Emulator Xensto

Re: [Xen-devel] [PATCH for-4.6 v3 2/6] docs/libxl: Re-specify XENSTORE_DATA as EMULATOR_XENSTORE_DATA

2015-08-05 Thread Wei Liu
On Tue, Aug 04, 2015 at 06:16:32PM +0100, Andrew Cooper wrote: > The legacy "toolstack" record as implemented in libxl turns out not to > be 32/64bit safe. As migration v2 has not shipped yet, take this > opportunity to adjust the specification and fix the incompatibility. > > Libxl shall loose a

Re: [Xen-devel] [PATCH for-4.6 v3 3/6] tools/libxl: Save and restore EMULATOR_XENSTORE_DATA content

2015-08-05 Thread Wei Liu
On Tue, Aug 04, 2015 at 06:16:33PM +0100, Andrew Cooper wrote: > The new EMULATOR_XENSTORE_DATA content is a sequence of NUL terminated > key/value strings, with the key relative to the device model's xenstore > tree. > > A sample might look like (as decoded by verify-stream-v2): > > Emulator

Re: [Xen-devel] [PATCH v2 2/8] xen: Make clear that swiotlb and biomerge are dealing with DMA address

2015-08-05 Thread Stefano Stabellini
On Tue, 4 Aug 2015, Julien Grall wrote: > The swiotlb is required when programming a DMA address on ARM when a > device is not protected by an IOMMU. > > In this case, the DMA address should always be equal to the machine address. > For DOM0 memory, Xen ensure it by have an identity mapping betwee

Re: [Xen-devel] [PATCH v2 3/8] arm/xen: implement correctly pfn_to_mfn

2015-08-05 Thread Stefano Stabellini
On Tue, 4 Aug 2015, Julien Grall wrote: > After the commit introducing convertion between DMA and guest address, ^ addresses > all the callers of pfn_to_mfn are expecting to get a GFN (Guest Frame > Number). On ARM, all the guests a

Re: [Xen-devel] [PATCH v3 07/32] xen/x86: fix arch_set_info_guest for HVM guests

2015-08-05 Thread Roger Pau Monné
El 04/08/15 a les 20.08, Andrew Cooper ha escrit: > On 03/08/15 18:31, Roger Pau Monné wrote: >>> Therefore, I am leaning slightly towards the specify the internals side >>> of things, which removes some complexity from the Xen side of the hypercall. >> I've updated the proposed structure, so now i

[Xen-devel] Reminder: Urgent - Action Required - Xen Dev Summit Bof's Developer Meetings and WG Meetings on Aug 18 & 19t (print and food order deadline Friday 7th)

2015-08-05 Thread Lars Kurth
Hi folks, this email is for people planning to attend the Xen Dev Summit in Seattle (Aug 17 & 18) and the Developer Meeting on the 19th. Please read and take action before *Friday* Aug 7th Best Regards Lars = BoF's (Print Deadline Friday 7th) == If you do want to host some BoF's and/or suggest to

Re: [Xen-devel] [PATCH] libxl: remove stray declaration of libxl__hotplug_settings

2015-08-05 Thread Ian Campbell
On Tue, 2015-08-04 at 11:25 +0100, Ian Campbell wrote: > On Tue, 2015-08-04 at 11:16 +0100, Wei Liu wrote: > > That function was removed in 2ba368d1 ("libxl: Remove linux udev > > rules") > > > > Signed-off-by: Wei Liu > > Acked-by: Ian Campbell > > Trivially fine for 4.6 IMHO. Applied. _

Re: [Xen-devel] [PATCH for-4.6] libxl: increase hotplug timeout to 40s

2015-08-05 Thread Ian Campbell
On Tue, 2015-08-04 at 11:26 +0100, Wei Liu wrote: > On Tue, Aug 04, 2015 at 12:02:55PM +0200, Roger Pau Monne wrote: > > The default libxl timeout for hotplug scripts execution is too low, > > when > > launching 40 HVM guests in parallel, all using the same file as disk, > > execution times of ~20

Re: [Xen-devel] [PATCH for-4.6] x86/gdt: Drop write-only, xalloc()'d array from set_gdt()

2015-08-05 Thread Ian Campbell
On Tue, 2015-08-04 at 15:17 +0100, George Dunlap wrote: > On Mon, Aug 3, 2015 at 6:05 PM, Andrew Cooper > wrote: > > It is not used, and can cause a spurious failure of the set_gdt() > > hypercall in > > low memory situations. > > > > Signed-off-by: Andrew Cooper > > Reviewed-by: George Dunla

Re: [Xen-devel] [PATCH for-4.6 v3 0/6] Fix libxl TOOLSTACK records for migration v2

2015-08-05 Thread Ian Campbell
On Tue, 2015-08-04 at 18:16 +0100, Andrew Cooper wrote: > The libxl blob for toolstack records was not 32/64bit safe. In 64bit > builds, it had an extra 4 bytes of padding due to alignment, and because > of the pointer arithmatic used to build it, leaked 4 bytes of libxl heap > into the stream. >

Re: [Xen-devel] [PATCH v2 5/8] xen/tmem: Use page_to_gfn rather than pfn_to_gfn

2015-08-05 Thread Stefano Stabellini
On Tue, 4 Aug 2015, Julien Grall wrote: > All the caller of xen_tmem_{get,put}_page have a struct page * in hand > and call pfn_to_gfn for the only benefits of these 2 functions. > > Rather than passing the pfn in parameter, pass directly the page and use > directly page_to_gfn. > > Signed-off-by

Re: [Xen-devel] [PATCH v7 2/8] cxenstored: add support for systemd active sockets

2015-08-05 Thread George Dunlap
On Fri, Jul 18, 2014 at 12:28 AM, Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" > > This adds systemd socket activation support for the C xenstored. > Active sockets enable xenstored to be loaded only if required by a system > onto which Xen is installed on. Socket activation is handled by

Re: [Xen-devel] [PATCH v2 4/8] xen: Use the correctly the Xen memory terminologies

2015-08-05 Thread Stefano Stabellini
On Tue, 4 Aug 2015, Boris Ostrovsky wrote: > On 08/04/2015 02:12 PM, Julien Grall wrote: > > /* > >* We detect special mappings in one of two ways: > > @@ -217,9 +232,13 @@ static inline unsigned long bfn_to_local_pfn(unsigned > > long mfn) > > /* VIRT <-> MACHINE conversion */ > > #d

Re: [Xen-devel] [PATCH v2 4/8] xen: Use the correctly the Xen memory terminologies

2015-08-05 Thread Stefano Stabellini
On Tue, 4 Aug 2015, Julien Grall wrote: > Based on include/xen/mm.h [1], Linux is mistakenly using MFN when GFN > is meant, I suspect this is because the first support for Xen was for > PV. This resulted in some misimplementation of helpers on ARM and > confused developers about the expected behavi

Re: [Xen-devel] [PATCH v7 2/8] cxenstored: add support for systemd active sockets

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 11:06 +0100, George Dunlap wrote: > On Fri, Jul 18, 2014 at 12:28 AM, Luis R. Rodriguez > wrote: > > From: "Luis R. Rodriguez" > > > > This adds systemd socket activation support for the C xenstored. > > Active sockets enable xenstored to be loaded only if required by a >

[Xen-devel] [PATCH for-4.6] libxl: move calling libxl__arch_domain_construct_memmap to right place

2015-08-05 Thread Wei Liu
This function was called in the wrong place, because both libxl__vnuma_build_vmemrange_hvm and xc_hvm_build rely on its output. Move the call of said function to the right place -- before the other two functions which reply on its output. Signed-off-by: Wei Liu --- Cc: "Chen, Tiejun" Cc: Ian Ca

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen

2015-08-05 Thread Julien Grall
Hi Shannon, On 05/08/15 10:29, Shannon Zhao wrote: >> >>> 3)DOM0 how to get grant table and event channel irq informations >>> As said above, we assign the hypervisor_id be "XenVMM" to tell DOM0 >>> that it runs on Xen hypervisor. >>> Then save the start address and size >>> of grant table

Re: [Xen-devel] [PATCH v2 2/8] xen: Make clear that swiotlb and biomerge are dealing with DMA address

2015-08-05 Thread Julien Grall
Hi Stefano, On 05/08/15 10:49, Stefano Stabellini wrote: >> /* VIRT <-> MACHINE conversion */ >> #define virt_to_mfn(v) (pfn_to_mfn(virt_to_pfn(v))) >> @@ -96,7 +115,7 @@ static inline bool set_phys_to_machine(unsigned long pfn, >> unsigned long mfn) >> >> bool xen_arch_need_swi

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen

2015-08-05 Thread Stefano Stabellini
On Wed, 5 Aug 2015, Shannon Zhao wrote: > On 2015/8/4 22:37, Stefano Stabellini wrote: > > On Tue, 4 Aug 2015, Shannon Zhao wrote: > >> This document is going to explain the design details of Xen booting with > >> ACPI on ARM. Maybe parts of it may not be appropriate. Any comments are > >> welcome.

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen

2015-08-05 Thread Stefano Stabellini
On Wed, 5 Aug 2015, Julien Grall wrote: > Hi Shannon, > > On 05/08/15 10:29, Shannon Zhao wrote: > >> > >>> 3)DOM0 how to get grant table and event channel irq informations > >>> As said above, we assign the hypervisor_id be "XenVMM" to tell DOM0 > >>> that it runs on Xen hypervisor. > >>> T

Re: [Xen-devel] [PATCH for-4.6] libxl: move calling libxl__arch_domain_construct_memmap to right place

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 11:23 +0100, Wei Liu wrote: > This function was called in the wrong place, because both > libxl__vnuma_build_vmemrange_hvm and xc_hvm_build rely on its output. What is the effect of this call being in the wrong place? Presumably one or the other of those functions reaches the

Re: [Xen-devel] [PATCH for-4.6] libxl: move calling libxl__arch_domain_construct_memmap to right place

2015-08-05 Thread Wei Liu
On Wed, Aug 05, 2015 at 11:38:54AM +0100, Ian Campbell wrote: > On Wed, 2015-08-05 at 11:23 +0100, Wei Liu wrote: > > This function was called in the wrong place, because both > > libxl__vnuma_build_vmemrange_hvm and xc_hvm_build rely on its output. > > What is the effect of this call being in the

Re: [Xen-devel] [PATCH for-4.6] libxl: move calling libxl__arch_domain_construct_memmap to right place

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 11:43 +0100, Wei Liu wrote: > On Wed, Aug 05, 2015 at 11:38:54AM +0100, Ian Campbell wrote: > > On Wed, 2015-08-05 at 11:23 +0100, Wei Liu wrote: > > > This function was called in the wrong place, because both > > > libxl__vnuma_build_vmemrange_hvm and xc_hvm_build rely on its

Re: [Xen-devel] [PATCH OSSTEST v2] standalone: Extend -h to support ident=host style specifications

2015-08-05 Thread Ian Campbell
On Fri, 2015-07-31 at 16:24 +0100, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST v2] standalone: Extend -h to support > ident=host style specifications"): > > Allowing for multi-host tests. > > > > Also make reset-host reset all hosts. > > Acked-by: Ian Jackson Thanks, added to pre

Re: [Xen-devel] [PATCH OSSTEST] get_hostflags: return an empty list when there is no flight/job.

2015-08-05 Thread Ian Campbell
On Fri, 2015-07-31 at 16:39 +0100, Ian Campbell wrote: > On Fri, 2015-07-31 at 16:31 +0100, Ian Jackson wrote: > > Ian Campbell writes ("[PATCH OSSTEST] get_hostflags: return an empty > > list > > when there is no flight/job."): > > > From: Ian Campbell > > > > > > Otherwise trying to use mg-ho

Re: [Xen-devel] [PATCH v2 2/3] ts-debian-hvm-install: use di_installcmdline_core

2015-08-05 Thread Ian Campbell
On Fri, 2015-07-31 at 16:20 +0100, Ian Jackson wrote: > Ian Campbell writes ("[PATCH v2 2/3] ts-debian-hvm-install: use > di_installcmdline_core"): > > This is primarily to get DEBIAN_FRONTEND=test, for easier to read > > logging. > text > > Aside f

Re: [Xen-devel] [PATCH v2 4/8] xen: Use the correctly the Xen memory terminologies

2015-08-05 Thread Julien Grall
Hi Boris, On 05/08/15 00:16, Boris Ostrovsky wrote: > On 08/04/2015 02:12 PM, Julien Grall wrote: >> /* >>* We detect special mappings in one of two ways: >> @@ -217,9 +232,13 @@ static inline unsigned long >> bfn_to_local_pfn(unsigned long mfn) >> /* VIRT <-> MACHINE conversion */ >>

Re: [Xen-devel] [PATCH] osstest: install libnl3 packages

2015-08-05 Thread Ian Campbell
On Thu, 2015-07-02 at 16:26 +0100, Ian Jackson wrote: > Roger Pau Monne writes ("[PATCH] osstest: install libnl3 packages"): > > Install the libnl3 packages needed by the remus code. Those are > > available on > > both Wheezy and Jessie, although the Wheezy ones are too old. > > This patch implic

Re: [Xen-devel] [PATCH for-4.6] x86/gdt: Drop write-only, xalloc()'d array from set_gdt()

2015-08-05 Thread Tim Deegan
At 14:59 +0100 on 04 Aug (1438700361), Ian Campbell wrote: > On Tue, 2015-08-04 at 10:34 +0100, Wei Liu wrote: > > On Mon, Aug 03, 2015 at 06:05:43PM +0100, Andrew Cooper wrote: > > > It is not used, and can cause a spurious failure of the set_gdt() > > > hypercall in > > > low memory situations.

Re: [Xen-devel] [PATCH v7 2/8] cxenstored: add support for systemd active sockets

2015-08-05 Thread George Dunlap
On Wed, Aug 5, 2015 at 11:17 AM, Ian Campbell wrote: > On Wed, 2015-08-05 at 11:06 +0100, George Dunlap wrote: >> On Fri, Jul 18, 2014 at 12:28 AM, Luis R. Rodriguez >> wrote: >> > From: "Luis R. Rodriguez" >> > >> > This adds systemd socket activation support for the C xenstored. >> > Active so

Re: [Xen-devel] [PATCH for-4.6] libxl: move calling libxl__arch_domain_construct_memmap to right place

2015-08-05 Thread Wei Liu
On Wed, Aug 05, 2015 at 11:48:55AM +0100, Ian Campbell wrote: > On Wed, 2015-08-05 at 11:43 +0100, Wei Liu wrote: > > On Wed, Aug 05, 2015 at 11:38:54AM +0100, Ian Campbell wrote: > > > On Wed, 2015-08-05 at 11:23 +0100, Wei Liu wrote: > > > > This function was called in the wrong place, because bo

[Xen-devel] [xen-unstable test] 60541: tolerable FAIL - PUSHED

2015-08-05 Thread osstest service owner
flight 60541 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/60541/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-rumpuserxen-i386 15 rumpuserxen-demo-xenstorels/xenstorels.repeat fail REGR. vs. 60199 test

Re: [Xen-devel] [PATCH for-4.6] libxl: move calling libxl__arch_domain_construct_memmap to right place

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 11:58 +0100, Wei Liu wrote: > On Wed, Aug 05, 2015 at 11:48:55AM +0100, Ian Campbell wrote: > > On Wed, 2015-08-05 at 11:43 +0100, Wei Liu wrote: > > > On Wed, Aug 05, 2015 at 11:38:54AM +0100, Ian Campbell wrote: > > > > On Wed, 2015-08-05 at 11:23 +0100, Wei Liu wrote: > > >

Re: [Xen-devel] [PATCH v7 2/8] cxenstored: add support for systemd active sockets

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 11:56 +0100, George Dunlap wrote: > On Wed, Aug 5, 2015 at 11:17 AM, Ian Campbell > wrote: > > On Wed, 2015-08-05 at 11:06 +0100, George Dunlap wrote: > > > On Fri, Jul 18, 2014 at 12:28 AM, Luis R. Rodriguez > > > wrote: > > > > From: "Luis R. Rodriguez" > > > > > > > >

Re: [Xen-devel] [PATCH v7 2/8] cxenstored: add support for systemd active sockets

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 12:11 +0100, Ian Campbell wrote: > > I'm a bit surprised that chkconfig doesn't just to the right thing. It's > possible that the fact that our initscript and our systemd unitfiles do not > share the same names has defeated its heuristics. Perhaps as well as fixing xenstored

Re: [Xen-devel] [Patch V6 12/16] mm: provide early_memremap_ro to establish read-only mapping

2015-08-05 Thread Juergen Gross
MM maintainers, is it really so hard to comment on this patch? Juergen On 07/29/2015 11:20 AM, Juergen Gross wrote: On 07/21/2015 06:49 AM, Juergen Gross wrote: Hi MM maintainers, this patch is the last requiring an ack for the series to go in. Could you please comment? PING? Juergen

Re: [Xen-devel] [PATCH v7 2/8] cxenstored: add support for systemd active sockets

2015-08-05 Thread George Dunlap
On Wed, Aug 5, 2015 at 12:11 PM, Ian Campbell wrote: > On Wed, 2015-08-05 at 11:56 +0100, George Dunlap wrote: >> On Wed, Aug 5, 2015 at 11:17 AM, Ian Campbell >> wrote: >> > On Wed, 2015-08-05 at 11:06 +0100, George Dunlap wrote: >> > > On Fri, Jul 18, 2014 at 12:28 AM, Luis R. Rodriguez >> > >

Re: [Xen-devel] [PATCH for-4.6] libxl: move calling libxl__arch_domain_construct_memmap to right place

2015-08-05 Thread Wei Liu
On Wed, Aug 05, 2015 at 12:06:22PM +0100, Ian Campbell wrote: > On Wed, 2015-08-05 at 11:58 +0100, Wei Liu wrote: > > On Wed, Aug 05, 2015 at 11:48:55AM +0100, Ian Campbell wrote: > > > On Wed, 2015-08-05 at 11:43 +0100, Wei Liu wrote: > > > > On Wed, Aug 05, 2015 at 11:38:54AM +0100, Ian Campbell

Re: [Xen-devel] [PATCH v7 2/8] cxenstored: add support for systemd active sockets

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 12:21 +0100, George Dunlap wrote: > On Wed, Aug 5, 2015 at 12:11 PM, Ian Campbell > wrote: > > On Wed, 2015-08-05 at 11:56 +0100, George Dunlap wrote: > > > On Wed, Aug 5, 2015 at 11:17 AM, Ian Campbell < > > > ian.campb...@citrix.com> > > > wrote: > > > > On Wed, 2015-08-05

[Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-05 Thread Julien Grall
From: Stefano Stabellini ARM64 is able to support 64KB and 4KB page granularities. While the guest will support both granularities, Xen and hypercall interface will always be in 4KB. Signed-off-by: Stefano Stabellini Signed-off-by: Julien Grall --- Cc: Ian Campbell Cc: Ian Jackson Cc: Jan B

[Xen-devel] [RFC 2/2] xen/public: grant-table: Specificy the size of the grant

2015-08-05 Thread Julien Grall
The grant is always 4KB irrespectively of the page granularity of the guest. Signed-off-by: Julien Grall --- Cc: Ian Campbell Cc: Ian Jackson Cc: Jan Beulich Cc: Keir Fraser Cc: Tim Deegan --- xen/include/public/grant_table.h | 5 + 1 file changed, 5 insertions(+) diff --git a/xen/inc

[Xen-devel] [RFC 0/2] xen: Clarify the page granularity for the hypercall

2015-08-05 Thread Julien Grall
Hi all, ARM64 is able to support both 64KB and 4KB page granularity. With the upcoming support of Linux guest with 64KB page granularity, the in-tree documentation needs to be clarify in order to avoid mixing granularity. I'm not sure if the wording is clear and correct, hence, the RFC. Regards,

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-05 Thread Andrew Cooper
On 05/08/15 12:28, Julien Grall wrote: > From: Stefano Stabellini > > ARM64 is able to support 64KB and 4KB page granularities. While the guest > will support both granularities, Xen and hypercall interface will always > be in 4KB. > > Signed-off-by: Stefano Stabellini > Signed-off-by: Julien Gra

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen

2015-08-05 Thread Shannon Zhao
On 2015/8/5 18:31, Stefano Stabellini wrote: > On Wed, 5 Aug 2015, Shannon Zhao wrote: >> On 2015/8/4 22:37, Stefano Stabellini wrote: >>> On Tue, 4 Aug 2015, Shannon Zhao wrote: This document is going to explain the design details of Xen booting with ACPI on ARM. Maybe parts of it may

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 12:40 +0100, Andrew Cooper wrote: > > 64K granularity is also similar to 2M/1G superpages in their handling, > the difference being that 64K can't be subdivided if necessary? 64K is actually a separate basic "granule" (to use the ARM term), i.e. alternative to the 4K leaf pa

Re: [Xen-devel] [PATCH] osstest: install libnl3 packages

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 11:51 +0100, Ian Campbell wrote: > > There was a conflict with the addition of ebtables, the result is below. > > commit 2ff90f75e5b40152998b69900adf1985382409bf > Author: Roger Pau Monne > Date: Wed Jul 1 17:12:25 2015 +0200 > > osstest: install libnl3 packages >

[Xen-devel] [qemu-upstream-unstable test] 60546: trouble: blocked/broken/fail/pass

2015-08-05 Thread osstest service owner
flight 60546 qemu-upstream-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/60546/ 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.

Re: [Xen-devel] [PATCH v3] VT-d: add iommu=igfx option to workaround graphics issues

2015-08-05 Thread Andrew Cooper
On 05/08/15 10:11, Ting-Wei Lan wrote: > When using Linux >= 3.19 (commit 47591df) as dom0 on some Intel Ironlake > devices, It is possible to encounter graphics issues that make screen > unreadable or crash the system. It was reported in freedesktop bugzilla: > > https://bugs.freedesktop.org/show_

Re: [Xen-devel] [PATCH v2 4/8] xen: Use the correctly the Xen memory terminologies

2015-08-05 Thread Boris Ostrovsky
On 08/05/2015 06:51 AM, Julien Grall wrote: diff --git a/drivers/video/fbdev/xen-fbfront.c b/drivers/video/fbdev/xen-fbfront.c index 09dc447..25e3cce 100644 --- a/drivers/video/fbdev/xen-fbfront.c +++ b/drivers/video/fbdev/xen-fbfront.c @@ -539,7 +539,7 @@ static int xenfb_remove(struct xenbus_

Re: [Xen-devel] OSSTEST -- nested test case development, RFC: ts-guest-destroy doesn't call guest_await_dhcp_tcp() if guest has fixed IP

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 06:22 +, Hu, Robert wrote: > Hi Ians, I don't 100% recall how this is supposed to fit together. IIRC: 1# L0 is installed as usual #2 An L1 guest is installed. That L1 guest gets an IP address from DHCP in the normal way. 3# Then ts-nested setup customises the L1 guest

Re: [Xen-devel] [PATCH v2 4/8] xen: Use the correctly the Xen memory terminologies

2015-08-05 Thread Julien Grall
On 05/08/15 13:19, Boris Ostrovsky wrote: > On 08/05/2015 06:51 AM, Julien Grall wrote: >> diff --git a/drivers/video/fbdev/xen-fbfront.c b/drivers/video/fbdev/xen-fbfront.c index 09dc447..25e3cce 100644 --- a/drivers/video/fbdev/xen-fbfront.c +++ b/drivers/video/fbdev/xen-

[Xen-devel] [PATCH for-4.6] x86/mm: Make {hap, shadow}_teardown() preemptible

2015-08-05 Thread Andrew Cooper
From: Anshul Makkar A domain with sufficient shadow allocation can cause a watchdog timeout during domain destruction. Expand the existing -ERESTART logic in paging_teardown() to allow {hap/sh}_set_allocation() to become restartable during the DOMCTL_destroydomain hypercall. Signed-off-by: Ansh

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-05 Thread Julien Grall
On 05/08/15 12:40, Andrew Cooper wrote: > I think a section about granularity is worthwhile, but probably a > separate paragraph. I think it is also worth keeping Xen's idea of > memory all at 4K, and in cases where 64K is in use, require appropriate > alignment in the parameter. Which would conf

[Xen-devel] [PATCH OSSTEST] libvirt: Pass correct arguments to virsh migrate

2015-08-05 Thread Ian Campbell
$dst is a host hash/object, resulting in: 2015-08-04 22:35:25 Z executing ssh ... root@172.16.144.34 virsh migrate debian.guest.osstest HASH(0x28f4310) bash: -c: line 0: syntax error near unexpected token `(' bash: -c: line 0: `virsh migrate debian.guest.osstest HASH(0x28f4310)' Switch to using t

Re: [Xen-devel] [PATCH OSSTEST] libvirt: Pass correct arguments to virsh migrate

2015-08-05 Thread Wei Liu
On Wed, Aug 05, 2015 at 01:42:19PM +0100, Ian Campbell wrote: > $dst is a host hash/object, resulting in: > > 2015-08-04 22:35:25 Z executing ssh ... root@172.16.144.34 virsh > migrate debian.guest.osstest HASH(0x28f4310) > bash: -c: line 0: syntax error near unexpected token `(' > bash: -c: line

Re: [Xen-devel] [PATCH v2 4/8] xen: Use the correctly the Xen memory terminologies

2015-08-05 Thread Boris Ostrovsky
On 08/05/2015 08:33 AM, Julien Grall wrote: On 05/08/15 13:19, Boris Ostrovsky wrote: On 08/05/2015 06:51 AM, Julien Grall wrote: diff --git a/drivers/video/fbdev/xen-fbfront.c b/drivers/video/fbdev/xen-fbfront.c index 09dc447..25e3cce 100644 --- a/drivers/video/fbdev/xen-fbfront.c +++ b/driver

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-05 Thread Andrew Cooper
On 05/08/15 13:36, Julien Grall wrote: > On 05/08/15 12:40, Andrew Cooper wrote: >> I think a section about granularity is worthwhile, but probably a >> separate paragraph. I think it is also worth keeping Xen's idea of >> memory all at 4K, and in cases where 64K is in use, require appropriate >>

Re: [Xen-devel] [PATCH OSSTEST] libvirt: Pass correct arguments to virsh migrate

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 13:43 +0100, Wei Liu wrote: > On Wed, Aug 05, 2015 at 01:42:19PM +0100, Ian Campbell wrote: > > $dst is a host hash/object, resulting in: > > > > 2015-08-04 22:35:25 Z executing ssh ... root@172.16.144.34 virsh > > migrate debian.guest.osstest HASH(0x28f4310) > > bash: -c: li

[Xen-devel] [PATCH OSSTEST v2] libvirt: Pass correct arguments to virsh migrate

2015-08-05 Thread Ian Campbell
$dst is a host hash/object, resulting in: 2015-08-04 22:35:25 Z executing ssh ... root@172.16.144.34 virsh migrate debian.guest.osstest HASH(0x28f4310) bash: -c: line 0: syntax error near unexpected token `(' bash: -c: line 0: `virsh migrate debian.guest.osstest HASH(0x28f4310)' Switch to using t

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen

2015-08-05 Thread Julien Grall
On 05/08/15 12:49, Shannon Zhao wrote: >> That's great! >> Keep in mind that many ARM platforms have non-PCI busses, so I think >> we'll need an amba and a platform bus_notifier too, in addition to the >> existing pci bus notifier. >> > > Thanks for your reminding. I thought about amba. Since ACPI

Re: [Xen-devel] [PATCH OSSTEST v2] libvirt: Pass correct arguments to virsh migrate

2015-08-05 Thread Wei Liu
On Wed, Aug 05, 2015 at 01:48:27PM +0100, Ian Campbell wrote: > $dst is a host hash/object, resulting in: > > 2015-08-04 22:35:25 Z executing ssh ... root@172.16.144.34 virsh > migrate debian.guest.osstest HASH(0x28f4310) > bash: -c: line 0: syntax error near unexpected token `(' > bash: -c: line

Re: [Xen-devel] [PATCH OSSTEST v2] libvirt: Pass correct arguments to virsh migrate

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 13:49 +0100, Wei Liu wrote: > On Wed, Aug 05, 2015 at 01:48:27PM +0100, Ian Campbell wrote: > > $dst is a host hash/object, resulting in: > > > > 2015-08-04 22:35:25 Z executing ssh ... root@172.16.144.34 virsh > > migrate debian.guest.osstest HASH(0x28f4310) > > bash: -c: li

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen

2015-08-05 Thread Shannon Zhao
On 2015/8/5 20:48, Julien Grall wrote: On 05/08/15 12:49, Shannon Zhao wrote: >>That's great! >>Keep in mind that many ARM platforms have non-PCI busses, so I think >>we'll need an amba and a platform bus_notifier too, in addition to the >>existing pci bus notifier. >> > >Thanks for your remi

Re: [Xen-devel] [PATCH v7 2/8] cxenstored: add support for systemd active sockets

2015-08-05 Thread Wei Liu
On Wed, Aug 05, 2015 at 11:56:44AM +0100, George Dunlap wrote: > On Wed, Aug 5, 2015 at 11:17 AM, Ian Campbell wrote: > > On Wed, 2015-08-05 at 11:06 +0100, George Dunlap wrote: > >> On Fri, Jul 18, 2014 at 12:28 AM, Luis R. Rodriguez > >> wrote: > >> > From: "Luis R. Rodriguez" > >> > > >> > Th

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-05 Thread Julien Grall
Hi, On 05/08/15 13:46, Andrew Cooper wrote: > On 05/08/15 13:36, Julien Grall wrote: >> On 05/08/15 12:40, Andrew Cooper wrote: >>> I think a section about granularity is worthwhile, but probably a >>> separate paragraph. I think it is also worth keeping Xen's idea of >>> memory all at 4K, and in

Re: [Xen-devel] Hotpatch construction and __LINE__

2015-08-05 Thread Andrew Cooper
On 05/08/15 09:55, Martin Pohlack wrote: > Hi, > > Another high-level point to think about is how we want to handle inlined > __LINE__ references. This problem is related to hotpatch construction > and potentially has influence on the design of the hotpatching > infrastructure in Xen. > > Let me t

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen

2015-08-05 Thread Julien Grall
On 05/08/15 14:03, Shannon Zhao wrote: > > > On 2015/8/5 20:48, Julien Grall wrote: >> On 05/08/15 12:49, Shannon Zhao wrote: >>That's great! >>Keep in mind that many ARM platforms have non-PCI busses, so I think >>we'll need an amba and a platform bus_notifier too, in addition to

Re: [Xen-devel] [RFC PATCH v3.1 2/2] xsplice: Add hook for build_id

2015-08-05 Thread Martin Pohlack
On 05.08.2015 10:58, Andrew Cooper wrote: > On 05/08/15 09:50, Martin Pohlack wrote: >> On 27.07.2015 21:20, Konrad Rzeszutek Wilk wrote: >>> Signed-off-by: Konrad Rzeszutek Wilk >>> --- >>> tools/libxc/xc_private.c | 3 +++ >>> tools/misc/xen-xsplice.c | 25 + >>>

Re: [Xen-devel] [PATCH v3] VT-d: add iommu=igfx option to workaround graphics issues

2015-08-05 Thread Wei Liu
On Wed, Aug 05, 2015 at 01:18:05PM +0100, Andrew Cooper wrote: > On 05/08/15 10:11, Ting-Wei Lan wrote: > > When using Linux >= 3.19 (commit 47591df) as dom0 on some Intel Ironlake > > devices, It is possible to encounter graphics issues that make screen > > unreadable or crash the system. It was r

[Xen-devel] (no subject)

2015-08-05 Thread Martin Pohlack
Sending again without MUA-mangled patch. Amazon Development Center Germany GmbH Krausenstr. 38 10117 Berlin Geschaeftsfuehrer: Dr. Ralf Herbrich, Christian Schlaeger Ust-ID: DE289237879 Eingetragen am Amtsgericht Charlottenburg HRB 149173 B ___ Xen-dev

[Xen-devel] [PATCH] xsplice: Use ld-embedded build-ids

2015-08-05 Thread Martin Pohlack
Todo: * Should be moved to sysctl to only allow Dom0 access * Maybe convert to binary transport to userland instead of printable form * use ld to actually embed the build ID * convert to textual representation in hypervisor and report in printable form Signed-off-by: Martin Pohlack --- xe

Re: [Xen-devel] [PATCH for-4.6] x86/mm: Make {hap, shadow}_teardown() preemptible

2015-08-05 Thread Tim Deegan
Hi, At 13:36 +0100 on 05 Aug (1438781760), Andrew Cooper wrote: > From: Anshul Makkar > > A domain with sufficient shadow allocation can cause a watchdog timeout > during domain destruction. Expand the existing -ERESTART logic in > paging_teardown() to allow {hap/sh}_set_allocation() to become

Re: [Xen-devel] [PATCH v2 02/20] xen: Introduce a function to split a Linux page into Xen page

2015-08-05 Thread Julien Grall
Hi David, On 24/07/15 11:10, David Vrabel wrote: > On 24/07/15 10:54, Julien Grall wrote: >> On 24/07/15 10:31, David Vrabel wrote: >>> On 09/07/15 21:42, Julien Grall wrote: The Xen interface is always using 4KB page. This means that a Linux page may be split across multiple Xen page wh

[Xen-devel] [linux-4.1 test] 60547: tolerable FAIL - PUSHED

2015-08-05 Thread osstest service owner
flight 60547 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/60547/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds 11 guest-start fail like 59909 test-amd64-i386-xl-qemut-stubdom

Re: [Xen-devel] virtio on pv/pvh xen

2015-08-05 Thread Wei Liu
On Wed, Aug 05, 2015 at 11:09:41PM +0800, Lai Jiangshan wrote: > Hi, Liu > > Does pv or pvh guest support virtio devices? No. > If yes, how can I configure the guest? > If not, how can I make it support? > A new transport which makes use of xenbus and grant table needs to be developed. There

Re: [Xen-devel] [PATCH v3 07/32] xen/x86: fix arch_set_info_guest for HVM guests

2015-08-05 Thread Andrew Cooper
On 05/08/15 10:53, Roger Pau Monné wrote: > El 04/08/15 a les 20.08, Andrew Cooper ha escrit: >> On 03/08/15 18:31, Roger Pau Monné wrote: Therefore, I am leaning slightly towards the specify the internals side of things, which removes some complexity from the Xen side of the hyperc

[Xen-devel] [PATCH for-4.6 v2] x86/mm: Make {hap, shadow}_teardown() preemptible

2015-08-05 Thread Andrew Cooper
From: Anshul Makkar A domain with sufficient shadow allocation can cause a watchdog timeout during domain destruction. Expand the existing -ERESTART logic in paging_teardown() to allow {hap/sh}_set_allocation() to become restartable during the DOMCTL_destroydomain hypercall. Signed-off-by: Ansh

Re: [Xen-devel] [PATCH v2 02/20] xen: Introduce a function to split a Linux page into Xen page

2015-08-05 Thread David Vrabel
On 05/08/15 15:30, Julien Grall wrote: > Hi David, > > On 24/07/15 11:10, David Vrabel wrote: >> On 24/07/15 10:54, Julien Grall wrote: >>> On 24/07/15 10:31, David Vrabel wrote: On 09/07/15 21:42, Julien Grall wrote: > The Xen interface is always using 4KB page. This means that a Linux p

Re: [Xen-devel] [PATCH v2 02/20] xen: Introduce a function to split a Linux page into Xen page

2015-08-05 Thread Julien Grall
On 05/08/15 16:50, David Vrabel wrote: > Also perhaps make it > > int xen_for_each_gfn(struct page *page, > xen_gfn_fn_t fn, void *data); gfn standing for Guest Frame Number right? >>> >>> Yes. This suggestion is just changing the name to make it more

Re: [Xen-devel] [PATCH] osstest: install libnl3 packages

2015-08-05 Thread Ian Campbell
On Wed, 2015-08-05 at 12:54 +0100, Ian Campbell wrote: > On Wed, 2015-08-05 at 11:51 +0100, Ian Campbell wrote: > > > > There was a conflict with the addition of ebtables, the result is > > below. > > > > commit 2ff90f75e5b40152998b69900adf1985382409bf > > Author: Roger Pau Monne > > Date: We

Re: [Xen-devel] [PATCH for-4.6 v2] x86/mm: Make {hap, shadow}_teardown() preemptible

2015-08-05 Thread Wei Liu
On Wed, Aug 05, 2015 at 04:47:59PM +0100, Andrew Cooper wrote: > From: Anshul Makkar > > A domain with sufficient shadow allocation can cause a watchdog timeout > during domain destruction. Expand the existing -ERESTART logic in > paging_teardown() to allow {hap/sh}_set_allocation() to become >

Re: [Xen-devel] [BUG] kernel panics with drbd

2015-08-05 Thread Matthew Vernon
Hi, On 05/08/15 09:48, Ian Campbell wrote: > On Tue, 2015-08-04 at 22:24 +0100, Matthew Vernon wrote: >> On 04/08/15 15:39, Matthew Vernon wrote: >>> Hi, >>> >>> On 04/08/15 15:22, Ian Campbell wrote: On Tue, 2015-08-04 at 14:52 +0100, Matthew Vernon wrote: > Hi, Hello, >>>

  1   2   >