Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Jan Beulich
On 11.02.15 at 17:33, andrew.coop...@citrix.com wrote: On 11/02/15 13:13, Jan Beulich wrote: On 11.02.15 at 12:52, andrew.coop...@citrix.com wrote: On 11/02/15 08:28, Kai Huang wrote: We handle above two cases by flushing PML buffer at the beginning of all VMEXITs. This solves the first case

Re: [Xen-devel] [PATCH OSSTEST 08/12] Add test job for nest test case

2015-02-11 Thread Ian Jackson
Robert Ho writes ([PATCH OSSTEST 08/12] Add test job for nest test case): This patch adds creation of the nested test job; when job creation procedure is invoked. ... + job_create_test test-$xenarch$kern-$dom0arch-nested test-nested xl \ + $xenarch $dom0arch \ Have I

Re: [Xen-devel] [PATCH v4 18/21] libxlu: rework internal representation of setting

2015-02-11 Thread Ian Jackson
Wei Liu writes ([PATCH v4 18/21] libxlu: rework internal representation of setting): This patches does following things: ... +void xlu__cfg_list_append(CfgParseContext *ctx, + XLU_ConfigValue *list, + char *atom) +{ +XLU_ConfigValue

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Andrew Cooper
On 11/02/15 13:13, Jan Beulich wrote: On 11.02.15 at 12:52, andrew.coop...@citrix.com wrote: On 11/02/15 08:28, Kai Huang wrote: With PML, we don't have to use write protection but just clear D-bit of EPT entry of guest memory to do dirty logging, with an additional PML buffer full VMEXIT for

Re: [Xen-devel] [PATCH v8 4/7] xen: Add vmware_port support

2015-02-11 Thread Andrew Cooper
On 11/02/15 07:56, Jan Beulich wrote: On 10.02.15 at 20:30, dsl...@verizon.com wrote: While coding this is up I have hit issues that I need input on: As a HVM_PARAM_ item, I would assume I should be following what HVM_PARAM_VIRIDIAN does. It has this comment: case

[Xen-devel] [PATCH v2 2/3] x86/traps: Avoid interleaved writes when updating potentially-live descriptors

2015-02-11 Thread Andrew Cooper
Signed-off-by: Andrew Cooper andrew.coop...@citrix.com CC: Jan Beulich jbeul...@suse.com CC: Tim Deegan t...@xen.org --- v2: * Use _write_gate_lower() instead of opencoding write_atomic() * Drop write_atomic() in _write_gate_lower(). It doesn't appear to make a practical difference

[Xen-devel] Processed: Re: [RFC] Tweaking the release process for Xen 4.6

2015-02-11 Thread xen
Processing commands for x...@bugs.xenproject.org: create ^ Created new bug #48 rooted at `20150210150424.ga32...@zion.uk.xensource.com' Title: `Re: [RFC] Tweaking the release process for Xen 4.6' title it Tweaking the release process for Xen 4.6 Set title for #48 to `Tweaking the release

Re: [Xen-devel] [PATCH v4 19/21] libxlu: nested list support

2015-02-11 Thread Ian Jackson
Wei Liu writes ([PATCH v4 19/21] libxlu: nested list support): 1. Extend grammar of parser. 2. Adjust internal functional to accept XLU_ConfigValue instead of ^functions Otherwise, Acked-by: Ian Jackson ian.jack...@eu.citrix.com

Re: [Xen-devel] [PATCH v4 20/21] libxlu: introduce new APIs

2015-02-11 Thread Ian Jackson
Wei Liu writes ([PATCH v4 20/21] libxlu: introduce new APIs): These APIs can be used to manipulate XLU_ConfigValue and XLU_ConfigList. ... +const char *xlu_cfg_value_get_string(const XLU_ConfigValue *value) +{ +assert(value-type == XLU_STRING); +return value-u.string; +} Most of the

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

2015-02-11 Thread xen . org
flight 34426 xen-unstable real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34426/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-pvh-intel 5 xen-boot fail REGR. vs. 34341 Regressions which

Re: [Xen-devel] [PATCH OSSTEST 09/12] Add build hvm job for nested test use

2015-02-11 Thread Ian Jackson
Robert Ho writes ([PATCH OSSTEST 09/12] Add build hvm job for nested test use): Add build-debain-hvm build job. The $TREE_LINUX and $REVISION_LINU can be designaged in standalone.config. What is this for ? It seems very similar to the build-$arch-pvops job. Ian.

Re: [Xen-devel] [PATCH OSSTEST 10/12] Compose the main body of test-nested test job.

2015-02-11 Thread Ian Jackson
Robert Ho writes ([PATCH OSSTEST 10/12] Compose the main body of test-nested test job.): Compose the main body of test-nested test job. Ah, this is what I was missing earlier. You really need to order this so that things come after things which depend on them. Typically: * cleanups *

[Xen-devel] [rumpuserxen test] 34448: regressions - FAIL

2015-02-11 Thread xen . org
flight 34448 rumpuserxen real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34448/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-rumpuserxen6 xen-build fail REGR. vs. 33866

Re: [Xen-devel] [PATCH 3/3] libxl: libxl__device_from_disk should retrieve backend from xenstore

2015-02-11 Thread Jim Fehlig
Wei Liu wrote: On Tue, Feb 10, 2015 at 11:01:46AM +, Ian Jackson wrote: Wei Liu writes ([PATCH 3/3] libxl: libxl__device_from_disk should retrieve backend from xenstore): ... if backend is not set by caller. Acked-by: Ian Jackson ian.jack...@eu.citrix.com as far as it

Re: [Xen-devel] [PATCH] x86 spinlock: Fix memory corruption on completing completions

2015-02-11 Thread Oleg Nesterov
On 02/10, Jeremy Fitzhardinge wrote: On 02/10/2015 05:26 AM, Oleg Nesterov wrote: On 02/10, Raghavendra K T wrote: Unfortunately xadd could result in head overflow as tail is high. The other option was repeated cmpxchg which is bad I believe. Any suggestions? Stupid question... what

[Xen-devel] [ovmf test] 34431: regressions - FAIL

2015-02-11 Thread xen . org
flight 34431 ovmf real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34431/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 7 debian-hvm-install fail REGR. vs. 33686

Re: [Xen-devel] [PATCH] x86 spinlock: Fix memory corruption on completing completions

2015-02-11 Thread Raghavendra K T
On 02/11/2015 11:08 PM, Oleg Nesterov wrote: On 02/11, Raghavendra K T wrote: On 02/10/2015 06:56 PM, Oleg Nesterov wrote: In this case __ticket_check_and_clear_slowpath() really needs to cmpxchg the whole .head_tail. Plus obviously more boring changes. This needs a separate patch even _if_

Re: [Xen-devel] [PATCH] x86 spinlock: Fix memory corruption on completing completions

2015-02-11 Thread Oleg Nesterov
On 02/11, Raghavendra K T wrote: On 02/10/2015 06:56 PM, Oleg Nesterov wrote: In this case __ticket_check_and_clear_slowpath() really needs to cmpxchg the whole .head_tail. Plus obviously more boring changes. This needs a separate patch even _if_ this can work. Correct, but apart from

[Xen-devel] [qemu-mainline test] 34427: regressions - FAIL

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

[Xen-devel] [PATCH] x86/xen: Make sure X2APIC_ENABLE bit of MSR_IA32_APICBASE is not set

2015-02-11 Thread Boris Ostrovsky
Commit d524165cb8db (x86/apic: Check x2apic early) tests X2APIC_ENABLE bit of MSR_IA32_APICBASE when CONFIG_X86_X2APIC is off and panics the kernel when this bit is set. Xen's PV guests will pass this MSR read to the hypervisor which will return its version of the MSR, where this bit might be

[Xen-devel] Xen OVMF regression

2015-02-11 Thread Wei Liu
Hi Anthony and Laszlo The following commit caused Xen hvm guest failed to boot. commit 66b280df282ae82888d2eb416bfeda3f65afa386 Author: Laszlo Ersek ler...@redhat.com Date: Thu Nov 20 09:58:28 2014 + OvmfPkg: AcpiPlatformDxe: make dependency on PCI enumeration explicit The ACPI

Re: [Xen-devel] [PATCH OSSTEST 04/12] Just some indentation adustments.

2015-02-11 Thread Ian Jackson
Robert Ho writes ([PATCH OSSTEST 04/12] Just some indentation adustments.): - target_putfilecontents_root_stash + target_putfilecontents_root_stash This seems to be just tab/space changes. I don't think we really need to bother about these. Do you find

Re: [Xen-devel] [PATCH OSSTEST 06/12] Manipulate $ho IP assignment for nest L2 situation

2015-02-11 Thread Ian Jackson
Robert Ho writes ([PATCH OSSTEST 06/12] Manipulate $ho IP assignment for nest L2 situation): In L2 installation context, its host (L1) IP address is not queried from DNS, but from previous step of L1 installation, in which, L1 IP is stored in run var. -$ho-{IpStatic} =

Re: [Xen-devel] Query: Boot time allocation of irq descriptors

2015-02-11 Thread Vijay Kilari
On Wed, Feb 11, 2015 at 8:25 PM, Andrew Cooper andrew.coop...@citrix.com wrote: On 11/02/15 14:50, Vijay Kilari wrote: Hi , I just glaced at the x86 code, here nr_irqs are set to 1024, which includes normal irq's and MSI's. Memory for these descriptors are allocated at boot time. is it

Re: [Xen-devel] [PATCH OSSTEST v6 9/9] mfi-common, make-flight: create XSM test jobs

2015-02-11 Thread Ian Jackson
Wei Liu writes (Re: [PATCH OSSTEST v6 9/9] mfi-common, make-flight: create XSM test jobs): Here is the updated version: ... Duplicate Debian PV and HVM test jobs for XSM testing. Thanks. This series (v7, then) is currently in the osstest self-push-gate. Ian.

Re: [Xen-devel] [PATCH OSSTEST 01/12] Add support of parsing grub which has 'submenu' primitive

2015-02-11 Thread Ian Jackson
Robert Ho writes ([PATCH OSSTEST 01/12] Add support of parsing grub which has 'submenu' primitive): From a hvm kernel build from Linux stable Kernel tree, the auto generated grub2 menu will have 'submenu' primitive, upon the 'menuentry' items. Xen boot entries will be grouped into a

Re: [Xen-devel] Query: Boot time allocation of irq descriptors

2015-02-11 Thread Jan Beulich
On 11.02.15 at 16:03, vijay.kil...@gmail.com wrote: On Wed, Feb 11, 2015 at 8:25 PM, Andrew Cooper andrew.coop...@citrix.com wrote: On 11/02/15 14:50, Vijay Kilari wrote: Hi , I just glaced at the x86 code, here nr_irqs are set to 1024, which includes normal irq's and MSI's. Memory for

Re: [Xen-devel] stubdom vtpm build failure in staging

2015-02-11 Thread Olaf Hering
On Wed, Jan 28, Xu, Quan wrote: Thanks, I will check and fix it tomorrow. It is 23:12 PM Pacific time now. Any progress? These typedefs are duplicated in stubdom/vtpmmgr/tcg.h and supported compilers do not cope with current staging: # for i in `grep -w typedef stubdom/vtpmmgr/tcg.h | sed -n

[Xen-devel] [OSSTEST PATCH v2 10/10] rump kernel tests: Repeat the xenstorels test 50 times

2015-02-11 Thread Ian Jackson
Ian Campbell writes (Re: [OSSTEST PATCH 10/10] rump kernel tests: Repeat the xenstorels test 50 times): On Fri, 2015-02-06 at 19:17 +, Ian Jackson wrote: Add a new step which uses repeat-ts to run ts-rumpuserxen-demo-xenstorels many times. Acked-by: Ian Campbell

Re: [Xen-devel] [PATCH] x86: simplify non-atomic bitops

2015-02-11 Thread Andrew Cooper
On 11/02/15 13:39, Jan Beulich wrote: - being non-atomic, their pointer arguments shouldn't be volatile- qualified - their (half fake) memory operands can be a single +m instead of being both an output and an input Signed-off-by: Jan Beulich jbeul...@suse.com --- v2: Drop +m related

[Xen-devel] [PATCH] tools: require at least pixman 0.21.8 for qemu-xen

2015-02-11 Thread Olaf Hering
Avoid late build failure in openSUSE 11.4, it has just pixman-0.20: [ 211s] ERROR: pixman = 0.21.8 not present. Your options: [ 211s] (1) Preferred: Install the pixman devel package (any recent [ 211s] distro should have packages as Xorg needs pixman too). [ 211s]

Re: [Xen-devel] [PATCH v2] introduce and use relaxed cpumask bitops

2015-02-11 Thread Andrew Cooper
On 11/02/15 13:42, Jan Beulich wrote: Using atomic (LOCKed on x86) bitops for certain of the operations on cpumask_t is overkill when the variables aren't concurrently accessible (e.g. local function variables, or due to explicit locking). Introduce alternatives using non-atomic bitops and use

Re: [Xen-devel] [PATCH v3 3/3] hvmemul_do_io: Do not retry if no ioreq server exists for this I/O.

2015-02-11 Thread Paul Durrant
-Original Message- From: Jan Beulich [mailto:jbeul...@suse.com] Sent: 11 February 2015 13:37 To: Paul Durrant Cc: Andrew Cooper; Ian Campbell; Wei Liu; George Dunlap; Ian Jackson; Stefano Stabellini; xen-devel@lists.xen.org; Don Slutz; Keir (Xen.org) Subject: Re: [PATCH v3 3/3]

Re: [Xen-devel] [PATCH] x86: simplify non-atomic bitops

2015-02-11 Thread Jan Beulich
On 11.02.15 at 16:14, andrew.coop...@citrix.com wrote: On 11/02/15 13:39, Jan Beulich wrote: @@ -55,12 +54,9 @@ static inline void set_bit(int nr, volat * If it's called on the same region of memory simultaneously, the effect * may be that only one operation succeeds. */ -static

Re: [Xen-devel] [PATCH] domctl: do away with tool stack based retrying

2015-02-11 Thread Andrew Cooper
On 11/02/15 13:47, Jan Beulich wrote: XEN_DOMCTL_destroydomain so far is being special cased in libxc to reinvoke the operation when getting back EAGAIN. Quite a few other domctl-s have gained continuations, so I see no reason not to use them here too. Signed-off-by: Jan Beulich

Re: [Xen-devel] Query: Boot time allocation of irq descriptors

2015-02-11 Thread Andrew Cooper
On 11/02/15 14:50, Vijay Kilari wrote: Hi , I just glaced at the x86 code, here nr_irqs are set to 1024, which includes normal irq's and MSI's. Memory for these descriptors are allocated at boot time. is it correct? int __init init_irq_data(void) { ... for (vector = 0; vector

Re: [Xen-devel] [PATCH RFC 33/35] arm : acpi enable efi for acpi

2015-02-11 Thread Julien Grall
Hi Jan, On 11/02/2015 18:31, Jan Beulich wrote: On 11.02.15 at 10:57, julien.gr...@linaro.org wrote: Hi Ian, On 05/02/2015 20:05, Ian Campbell wrote: On Thu, 2015-02-05 at 11:58 +, Jan Beulich wrote: On 05.02.15 at 06:31, julien.gr...@linaro.org wrote: --- a/xen/common/efi/runtime.c

Re: [Xen-devel] Query: Boot time allocation of irq descriptors

2015-02-11 Thread Jan Beulich
On 11.02.15 at 15:50, vijay.kil...@gmail.com wrote: I just glaced at the x86 code, here nr_irqs are set to 1024, which includes normal irq's and MSI's. Memory for these descriptors are allocated at boot time. is it correct? int __init init_irq_data(void) { ... for (vector =

Re: [Xen-devel] [PATCH OSSTEST 02/12] Increase boot timer to accomodate to nest test

2015-02-11 Thread Ian Jackson
Robert Ho writes ([PATCH OSSTEST 02/12] Increase boot timer to accomodate to nest test): In nested test case, guest boot will take more time. Increase the timer to 200 seconds. Can we make this conditional somehow ? I think it should probably be picked up from a runvar. We don't currently

Re: [Xen-devel] [PATCH OSSTEST 03/12] Designate vif device model to e1000

2015-02-11 Thread Ian Jackson
Robert Ho writes ([PATCH OSSTEST 03/12] Designate vif device model to e1000): Designate vif model to 'e1000', otherwise, with default device model, the L1 eth0 interface disappear, hence xenbridge cannot work. Maybe this limitation can be removed later after some fix it. For now, we have to

Re: [Xen-devel] [PATCH OSSTEST 05/12] Add and expose some testsupport APIs

2015-02-11 Thread Ian Jackson
Robert Ho writes ([PATCH OSSTEST 05/12] Add and expose some testsupport APIs): When install L2 guest, we will need to invoke 'select_ether' to get guest MAC address. So here expose select_ether(). I'm not sure whether you actually need to do this. I will look at the rest of your series to see

Re: [Xen-devel] [PATCH] modify the IO_TLB_SEGSIZE to io_tlb_segsize configurable as flexible requirement about SW-IOMMU.

2015-02-11 Thread Konrad Rzeszutek Wilk
On Wed, Feb 11, 2015 at 08:38:29AM +, Wang, Xiaoming wrote: Dear David -Original Message- From: David Vrabel [mailto:david.vra...@citrix.com] Sent: Tuesday, February 10, 2015 5:46 PM To: Wang, Xiaoming; Konrad Rzeszutek Wilk Cc: linux-m...@linux-mips.org;

[Xen-devel] [PATCH] MdeModulePkg: mark completion of PCI enumeration in PciEnumeratorLight

2015-02-11 Thread Wei Liu
I had an issue when trying to boot Xen HVM guest with latest OVMF master. Guest crashed with memory violation, and the bisection pointed to 66b280df2 (OvmfPkg: AcpiPlatformDxe: make dependency on PCI enumeration explicit). That commit made AcpiPlatformDxe depend on PCI enumeration using

[Xen-devel] [PATCH v2] vsprintf: Make sure argument to %pX specifier is valid

2015-02-11 Thread Boris Ostrovsky
If invalid pointer (i.e. something smaller than HYPERVISOR_VIRT_START) is passed for %*ph/%pv/%ps/%pS format specifiers then print (NULL) Signed-off-by: Boris Ostrovsky boris.ostrov...@oracle.com --- xen/common/vsprintf.c | 23 --- 1 files changed, 16 insertions(+), 7

Re: [Xen-devel] [PATCH] x86 spinlock: Fix memory corruption on completing completions

2015-02-11 Thread Linus Torvalds
On Feb 11, 2015 3:15 PM, Jeremy Fitzhardinge jer...@goop.org wrote: Right now it needs to be a locked operation to prevent read-reordering. x86 memory ordering rules state that all writes are seen in a globally consistent order, and are globally ordered wrt reads *on the same addresses*, but

[Xen-devel] [linux-3.16 test] 34434: regressions - FAIL

2015-02-11 Thread xen . org
flight 34434 linux-3.16 real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34434/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-credit2 15 guest-localmigrate/x10fail REGR. vs. 34167 Tests which are

Re: [Xen-devel] [PATCH] virtual: Documentation: simplify and generalize paravirt_ops.txt

2015-02-11 Thread Rusty Russell
Luis R. Rodriguez mcg...@do-not-panic.com writes: From: Luis R. Rodriguez mcg...@suse.com The general documentation we have for pv_ops is currenty present on the IA64 docs, but since this documentation covers IA64 xen enablement and IA64 Xen support got ripped out a while ago through commit

Re: [Xen-devel] [PATCH] x86 spinlock: Fix memory corruption on completing completions

2015-02-11 Thread Jeremy Fitzhardinge
On 02/11/2015 09:24 AM, Oleg Nesterov wrote: I agree, and I have to admit I am not sure I fully understand why unlock uses the locked add. Except we need a barrier to avoid the race with the enter_slowpath() users, of course. Perhaps this is the only reason? Right now it needs to be a locked

[Xen-devel] [libvirt test] 34464: tolerable all pass - PUSHED

2015-02-11 Thread xen . org
flight 34464 libvirt real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34464/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-i386-libvirt 10 migrate-support-checkfail never pass test-armhf-armhf-libvirt 10

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Kai Huang
On 02/11/2015 09:13 PM, Jan Beulich wrote: On 11.02.15 at 12:52, andrew.coop...@citrix.com wrote: On 11/02/15 08:28, Kai Huang wrote: With PML, we don't have to use write protection but just clear D-bit of EPT entry of guest memory to do dirty logging, with an additional PML buffer full

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Kai Huang
On 02/11/2015 09:06 PM, Jan Beulich wrote: On 11.02.15 at 09:28, kai.hu...@linux.intel.com wrote: - PML enable/disable for particular Domain PML needs to be enabled (allocate PML buffer, initialize PML index, PML base address, turn PML on VMCS, etc) for all vcpus of the domain, as PML buffer

Re: [Xen-devel] [PATCH OSSTEST 01/12] Add support of parsing grub which has 'submenu' primitive

2015-02-11 Thread Hu, Robert
-Original Message- From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] Sent: Wednesday, February 11, 2015 10:44 PM To: Hu, Robert Cc: xen-devel@lists.xen.org; jfeh...@suse.com; wei.l...@citrix.com; ian.campb...@citrix.com; Pang, LongtaoX Subject: Re: [PATCH OSSTEST 01/12] Add

[Xen-devel] [linux-3.10 test] 34436: regressions - FAIL

2015-02-11 Thread xen . org
flight 34436 linux-3.10 real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34436/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-winxpsp3 7 windows-install fail REGR. vs. 26303 Regressions which are

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Kai Huang
On 02/11/2015 07:52 PM, Andrew Cooper wrote: On 11/02/15 08:28, Kai Huang wrote: Hi all, PML (Page Modification Logging) is a new feature on Intel's Boardwell server platfrom targeted to reduce overhead of dirty logging mechanism. Below is the design for Xen. Would you help to review and give

Re: [Xen-devel] [edk2] [PATCH] MdeModulePkg: mark completion of PCI enumeration in PciEnumeratorLight

2015-02-11 Thread Ni, Ruiyu
Wei, No you cannot install gEfiPciEnumerationCompleteProtocolGuid in PciEnumeratorLight(). For a real platform, PCI BUS is fully enumerated in PciEnumerator() and later if reconnect happens, it's light enumerated in PciEnumeratorLight(). The protocol should only be installed once in

[Xen-devel] [rumpuserxen test] 34486: regressions - FAIL

2015-02-11 Thread xen . org
flight 34486 rumpuserxen real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34486/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-rumpuserxen6 xen-build fail REGR. vs. 33866

Re: [Xen-devel] [PATCH] xen/arm: allow console=hvc0 to be omitted for guests

2015-02-11 Thread Stefano Stabellini
On Tue, 10 Feb 2015, Ian Campbell wrote: On Tue, 2015-02-10 at 15:51 +0800, Ard Biesheuvel wrote: FWIW on x86 this doesn't depend on console_set_on_cmdline, does it need to here? I didn't check the code, but it seems inappropriate to add a preferred console implicitly if the user

Re: [Xen-devel] [PATCH] xen/arm: allow console=hvc0 to be omitted for guests

2015-02-11 Thread Stefano Stabellini
On Thu, 12 Feb 2015, Stefano Stabellini wrote: On Tue, 10 Feb 2015, Ian Campbell wrote: On Tue, 2015-02-10 at 15:51 +0800, Ard Biesheuvel wrote: FWIW on x86 this doesn't depend on console_set_on_cmdline, does it need to here? I didn't check the code, but it seems

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Kai Huang
On 02/12/2015 02:25 PM, Tian, Kevin wrote: From: Kai Huang [mailto:kai.hu...@linux.intel.com] Sent: Thursday, February 12, 2015 10:35 AM On 02/11/2015 09:13 PM, Jan Beulich wrote: On 11.02.15 at 12:52, andrew.coop...@citrix.com wrote: On 11/02/15 08:28, Kai Huang wrote: With PML, we don't

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Kai Huang
On 02/12/2015 03:09 PM, Tian, Kevin wrote: From: Kai Huang [mailto:kai.hu...@linux.intel.com] Sent: Thursday, February 12, 2015 2:57 PM On 02/12/2015 02:54 PM, Tian, Kevin wrote: From: Kai Huang [mailto:kai.hu...@linux.intel.com] Sent: Thursday, February 12, 2015 10:39 AM PML needs to be

Re: [Xen-devel] [PATCH RFC 31/35] arm : acpi map status override table to dom0

2015-02-11 Thread Stefano Stabellini
On Wed, 11 Feb 2015, Julien Grall wrote: Hi Ian, On 05/02/2015 19:47, Ian Campbell wrote: On Thu, 2015-02-05 at 16:27 +0530, Parth Dixit wrote: +stao-header.length = sizeof(struct acpi_table_header) + 1; +stao-header.checksum = 0; +ACPI_MEMCPY(stao-header.oem_id,

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Tian, Kevin
From: Kai Huang [mailto:kai.hu...@linux.intel.com] Sent: Thursday, February 12, 2015 10:39 AM PML needs to be enabled (allocate PML buffer, initialize PML index, PML base address, turn PML on VMCS, etc) for all vcpus of the domain, as PML buffer and PML index are per-vcpu, but EPT table

Re: [Xen-devel] [PATCH] x86 spinlock: Fix memory corruption on completing completions

2015-02-11 Thread Jeremy Fitzhardinge
On 02/11/2015 03:28 PM, Linus Torvalds wrote: On Feb 11, 2015 3:15 PM, Jeremy Fitzhardinge jer...@goop.org mailto:jer...@goop.org wrote: Right now it needs to be a locked operation to prevent read-reordering. x86 memory ordering rules state that all writes are seen in a globally

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Tian, Kevin
From: Kai Huang [mailto:kai.hu...@linux.intel.com] Sent: Thursday, February 12, 2015 2:46 PM On 02/12/2015 02:25 PM, Tian, Kevin wrote: From: Kai Huang [mailto:kai.hu...@linux.intel.com] Sent: Thursday, February 12, 2015 10:35 AM On 02/11/2015 09:13 PM, Jan Beulich wrote: On

Re: [Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Kai Huang
On 02/12/2015 02:54 PM, Tian, Kevin wrote: From: Kai Huang [mailto:kai.hu...@linux.intel.com] Sent: Thursday, February 12, 2015 10:39 AM PML needs to be enabled (allocate PML buffer, initialize PML index, PML base address, turn PML on VMCS, etc) for all vcpus of the domain, as PML buffer and

[Xen-devel] [PATCH OSSTEST 01/12] Add support of parsing grub which has 'submenu' primitive

2015-02-11 Thread Robert Ho
From a hvm kernel build from Linux stable Kernel tree, the auto generated grub2 menu will have 'submenu' primitive, upon the 'menuentry' items. Xen boot entries will be grouped into a submenu. This patch adds capability to support such grub formats. Also, this patch adjust some indent

[Xen-devel] [PATCH OSSTEST 12/12] Changes to test step of xen install

2015-02-11 Thread Robert Ho
This patch accomodates ts-xen-install to nested L1 xen installation usage. Its change is relatively simpler than ts-debain-hvm-install. We simply alter '$ho' usage to 'w_ho', which is assigned to '$ho' in original L0 installation context, while assigned to '$gho' in L1 Xen installation

[Xen-devel] [PATCH OSSTEST 06/12] Manipulate $ho IP assignment for nest L2 situation

2015-02-11 Thread Robert Ho
In L2 installation context, its host (L1) IP address is not queried from DNS, but from previous step of L1 installation, in which, L1 IP is stored in run var. --- Osstest/TestSupport.pm | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Osstest/TestSupport.pm

[Xen-devel] [PATCH OSSTEST 11/12] Changes on test step of debain hvm guest install

2015-02-11 Thread Robert Ho
This patch is to make ts-debian-hvm-install accomodate to nested L1 and L2 guest installation context. 1. Add an input param 'nested' to indicate which installationcontext is. and manipulating 'gn' accordingly. 2. increase disk size to accomodate to nested test requirment. 3. increase

[Xen-devel] [PATCH OSSTEST 02/12] Increase boot timer to accomodate to nest test

2015-02-11 Thread Robert Ho
In nested test case, guest boot will take more time. Increase the timer to 200 seconds. --- Osstest/TestSupport.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index c4da8ce..f3c515e 100644 --- a/Osstest/TestSupport.pm +++

[Xen-devel] [PATCH OSSTEST 03/12] Designate vif device model to e1000

2015-02-11 Thread Robert Ho
Designate vif model to 'e1000', otherwise, with default device model, the L1 eth0 interface disappear, hence xenbridge cannot work. Maybe this limitation can be removed later after some fix it. For now, we have to accomodate to it. --- Osstest/TestSupport.pm | 2 +- 1 file changed, 1

[Xen-devel] [PATCH OSSTEST 07/12] For hvm guest configuration, config console to 'hvc0'

2015-02-11 Thread Robert Ho
--- Osstest/TestSupport.pm | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index c23bbc7..864805e 100644 --- a/Osstest/TestSupport.pm +++ b/Osstest/TestSupport.pm @@ -1753,7 +1753,11 @@ sub target_kernkind_check ($) {

[Xen-devel] [PATCH OSSTEST 05/12] Add and expose some testsupport APIs

2015-02-11 Thread Robert Ho
When install L2 guest, we will need to invoke 'select_ether' to get guest MAC address. So here expose select_ether(). And also, we added another function 'guest_editconfig_cd' and expose it. This function bascically changes guest boot device sequence and alter its on_reboot behavior to

[Xen-devel] [PATCH OSSTEST 08/12] Add test job for nest test case

2015-02-11 Thread Robert Ho
This patch adds creation of the nested test job; when job creation procedure is invoked. --- make-flight | 20 1 file changed, 20 insertions(+) diff --git a/make-flight b/make-flight index a91f256..4f6ce91 100755 --- a/make-flight +++ b/make-flight @@ -197,6 +197,25 @@

[Xen-devel] [PATCH OSSTEST 04/12] Just some indentation adustments.

2015-02-11 Thread Robert Ho
--- Osstest/TestSupport.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index 8f8638b..1053409 100644 --- a/Osstest/TestSupport.pm +++ b/Osstest/TestSupport.pm @@ -53,7 +53,7 @@ BEGIN {

[Xen-devel] [PATCH OSSTEST 00/12] Add nested xen on xen test case

2015-02-11 Thread Robert Ho
This patch set adds nested HVM test case for osstest. In this test case, a Xen hypervisor (L1) runs on top of another Xen hypervisor (L0). Upon L1 hypervisor, we will then create a nested guest (L2), and test if the Linux guest can then be installed and run well. About nested Xen

[Xen-devel] [PATCH OSSTEST 09/12] Add build hvm job for nested test use

2015-02-11 Thread Robert Ho
Add build-debain-hvm build job. The $TREE_LINUX and $REVISION_LINU can be designaged in standalone.config. --- mfi-common | 8 1 file changed, 8 insertions(+) diff --git a/mfi-common b/mfi-common index 27d6b31..0d43959 100644 --- a/mfi-common +++ b/mfi-common @@ -166,6 +166,14 @@

[Xen-devel] Usage of efi_enabled - Was: Re: [PATCH RFC 33/35] arm : acpi enable efi for acpi

2015-02-11 Thread Jan Beulich
On 11.02.15 at 10:51, julien.gr...@linaro.org wrote: --- a/xen/common/efi/runtime.c +++ b/xen/common/efi/runtime.c @@ -11,7 +11,13 @@ DEFINE_XEN_GUEST_HANDLE(CHAR16); #ifndef COMPAT #ifdef CONFIG_ARM /* Disabled until runtime services implemented */ + +#if defined(CONFIG_ARM_64)

Re: [Xen-devel] [PATCH RFC 33/35] arm : acpi enable efi for acpi

2015-02-11 Thread Jan Beulich
On 11.02.15 at 10:57, julien.gr...@linaro.org wrote: Hi Ian, On 05/02/2015 20:05, Ian Campbell wrote: On Thu, 2015-02-05 at 11:58 +, Jan Beulich wrote: On 05.02.15 at 06:31, julien.gr...@linaro.org wrote: --- a/xen/common/efi/runtime.c +++ b/xen/common/efi/runtime.c @@ -11,7 +11,13

Re: [Xen-devel] dprintk() and gdprintk() to be compiled out when NDEBUG

2015-02-11 Thread Andrew Cooper
On 11/02/15 10:32, Jan Beulich wrote: On 11.02.15 at 11:05, andrew.coop...@citrix.com wrote: On 11/02/15 07:50, Jan Beulich wrote: I'd like to propose to honor the 'd' in these functions' names (which I understand to mean debug) in that such functions should be no-ops in non-debug builds. I'd

Re: [Xen-devel] [PATCH] vsprintf: Make sure argument to %*ph/%pv is valid

2015-02-11 Thread Jan Beulich
On 10.02.15 at 22:49, andrew.coop...@citrix.com wrote: On 10/02/2015 21:44, Boris Ostrovsky wrote: If NULL pointer is passed for these specifiers then print '-' or 'd-v-'. Signed-off-by: Boris Ostrovsky boris.ostrov...@oracle.com Instead of special casing each custom format, I would be

Re: [Xen-devel] [PATCH 2/2] sched: credit2: consider per-vcpu soft affinity

2015-02-11 Thread Justin Weaver
Dario, I'm working on soft affinity while you review v2 of the credit 2 hard affinity patch (no rush, of course). On Tue, Jan 13, 2015 at 3:06 AM, Dario Faggioli dario.faggi...@citrix.com wrote: On Sat, 2014-11-29 at 14:33 -1000, Justin T. Weaver wrote: when deciding which run queue to assign

[Xen-devel] [linux-linus test] 34383: regressions - FAIL

2015-02-11 Thread xen . org
flight 34383 linux-linus real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34383/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt 9 guest-start fail REGR. vs. 34227

Re: [Xen-devel] [PATCH RFC 26/35] arm : acpi read mmio tables from uefi

2015-02-11 Thread Julien Grall
Hi Stefano, On 06/02/2015 22:17, Stefano Stabellini wrote: On Fri, 6 Feb 2015, Julien Grall wrote: On 06/02/2015 00:34, Stefano Stabellini wrote: diff --git a/xen/arch/arm/setup.c b/xen/arch/arm/setup.c index 93c8a8a..930746b 100644 --- a/xen/arch/arm/setup.c +++ b/xen/arch/arm/setup.c @@

Re: [Xen-devel] [PATCH 18/18] x86: add multiboot2 protocol support for EFI platforms

2015-02-11 Thread Jan Beulich
On 10.02.15 at 22:27, daniel.ki...@oracle.com wrote: After some testing we have found at least one machine on which this thing does not work. It is Dell PowerEdge R820 with latest firmware. Machine crashes/stops because early 32-bit code is not relocatable and must live under 0x10

[Xen-devel] SeaBIOS update on master?

2015-02-11 Thread Jan Beulich
Ian, I notice there is 1.7.5.2 available upstream - are there plans for us to update, or are we going to wait for 1.7.6 or 1.8 or whatever is going to be next? (I ask without having checked whether there are any fixes that would specifically concern us.) Jan

Re: [Xen-devel] [PATCH v3 2/3] hvm_complete_assist_req: We should not be able to get here on IOREQ_TYPE_PCI_CONFIG

2015-02-11 Thread Paul Durrant
-Original Message- From: Don Slutz [mailto:dsl...@verizon.com] Sent: 10 February 2015 22:52 To: xen-devel@lists.xen.org; Jan Beulich; Paul Durrant Cc: Andrew Cooper; George Dunlap; Ian Campbell; Ian Jackson; Keir (Xen.org); Stefano Stabellini; Wei Liu; Don Slutz Subject: [PATCH v3

Re: [Xen-devel] [PATCH RFC 31/35] arm : acpi map status override table to dom0

2015-02-11 Thread Julien Grall
Hi Ian, On 05/02/2015 19:47, Ian Campbell wrote: On Thu, 2015-02-05 at 16:27 +0530, Parth Dixit wrote: +stao-header.length = sizeof(struct acpi_table_header) + 1; +stao-header.checksum = 0; +ACPI_MEMCPY(stao-header.oem_id, LINARO, 6); +ACPI_MEMCPY(stao-header.oem_table_id,

[Xen-devel] Usage of efi_enabled - Was: Re: [PATCH RFC 33/35] arm : acpi enable efi for acpi

2015-02-11 Thread Julien Grall
(Adding Roy) Hi Jan and Roy, On 04/02/2015 22:02, parth.di...@linaro.org wrote: From: Parth Dixit parth.di...@linaro.org efi should be enabled to fetch the root pointer from uefi Signed-off-by: Parth Dixit parth.di...@linaro.org --- xen/common/efi/runtime.c | 6 ++ 1 file changed, 6

Re: [Xen-devel] [RFC PATCH] xen, apic: Setup our own APIC driver and validator for APIC IDs.

2015-02-11 Thread David Vrabel
On 10/02/15 20:33, Konrad Rzeszutek Wilk wrote: On Thu, Jan 22, 2015 at 10:00:55AM +, David Vrabel wrote: On 21/01/15 21:56, Konrad Rzeszutek Wilk wrote: +static struct apic xen_apic = { + .name = Xen, + .probe = probe_xen, + /* The rest is copied from the default. */ Explicitly

Re: [Xen-devel] [PATCH RFC 33/35] arm : acpi enable efi for acpi

2015-02-11 Thread Julien Grall
Hi Ian, On 05/02/2015 20:05, Ian Campbell wrote: On Thu, 2015-02-05 at 11:58 +, Jan Beulich wrote: On 05.02.15 at 06:31, julien.gr...@linaro.org wrote: --- a/xen/common/efi/runtime.c +++ b/xen/common/efi/runtime.c @@ -11,7 +11,13 @@ DEFINE_XEN_GUEST_HANDLE(CHAR16); #ifndef COMPAT

Re: [Xen-devel] dprintk() and gdprintk() to be compiled out when NDEBUG

2015-02-11 Thread Andrew Cooper
On 11/02/15 07:50, Jan Beulich wrote: All, I'd like to propose to honor the 'd' in these functions' names (which I understand to mean debug) in that such functions should be no-ops in non-debug builds. I'd then be inclined to introduce a gprintk() automatically adding XENLOG_GUEST and the

[Xen-devel] PML (Page Modification Logging) design for Xen

2015-02-11 Thread Kai Huang
Hi all, PML (Page Modification Logging) is a new feature on Intel's Boardwell server platfrom targeted to reduce overhead of dirty logging mechanism. Below is the design for Xen. Would you help to review and give comments? Background == Currently, dirty logging is done via write

Re: [Xen-devel] [PATCH] modify the IO_TLB_SEGSIZE to io_tlb_segsize configurable as flexible requirement about SW-IOMMU.

2015-02-11 Thread Wang, Xiaoming
Dear David -Original Message- From: David Vrabel [mailto:david.vra...@citrix.com] Sent: Tuesday, February 10, 2015 5:46 PM To: Wang, Xiaoming; Konrad Rzeszutek Wilk Cc: linux-m...@linux-mips.org; pebo...@tiscali.nl; Zhang, Dongxing; lau...@codeaurora.org; d.kasat...@samsung.com;

Re: [Xen-devel] [PATCH RFC 27/35] arm: acpi map mmio regions to dom0

2015-02-11 Thread Julien Grall
Hi Parth, On 04/02/2015 22:02, parth.di...@linaro.org wrote: From: Parth Dixit parth.di...@linaro.org map mmio regions described in uefi tables to dom0 address space Signed-off-by: Parth Dixit parth.di...@linaro.org --- xen/arch/arm/domain_build.c | 54

Re: [Xen-devel] [PATCH 2/3] x86/traps: Use write_atomic() when updating potentially-live descriptors

2015-02-11 Thread Jan Beulich
On 10.02.15 at 18:12, andrew.coop...@citrix.com wrote: --- a/xen/include/asm-x86/desc.h +++ b/xen/include/asm-x86/desc.h @@ -116,7 +116,8 @@ static inline void _write_gate_lower(volatile idt_entry_t *gate, const idt_entry_t *new) { ASSERT(gate-b

Re: [Xen-devel] SeaBIOS update on master?

2015-02-11 Thread Ian Campbell
On Wed, 2015-02-11 at 08:38 +, Jan Beulich wrote: Ian, I notice there is 1.7.5.2 available upstream - are there plans for us to update, or are we going to wait for 1.7.6 or 1.8 or whatever is going to be next? (I ask without having checked whether there are any fixes that would

Re: [Xen-devel] [PATCH 3/3] mini-os: sort objects in binary archives

2015-02-11 Thread Wei Liu
On Tue, Feb 03, 2015 at 01:28:54PM +0100, Olaf Hering wrote: On Tue, Feb 03, Olaf Hering wrote: +++ b/extras/mini-os/Makefile -LWO:= $(patsubst %.c,%.o,$(LWC)) +LWO:= $(patsubst %.c,%.o,$(LWC) Somehow this brace disappeared. Should I resend or can this be adjusted

Re: [Xen-devel] [PATCH 02/29] libxl: Comment cleanups

2015-02-11 Thread Ian Jackson
Hongyang Yang writes (Re: [PATCH 02/29] libxl: Comment cleanups): 在 02/11/2015 04:09 AM, Ian Jackson 写道: * Add two comments in libxl_remus_disk_drbd documenting buggy handling of the hotplug script exit status. ... Signed-off-by: Ian Jackson ian.jack...@eu.citrix.com CC: Yang Hongyang

Re: [Xen-devel] [PATCH 3/3] x86/nmi: Fix shootdown of pcpus running in VMX non-root mode

2015-02-11 Thread Andrew Cooper
On 11/02/15 11:02, Jan Beulich wrote: On 10.02.15 at 18:12, andrew.coop...@citrix.com wrote: @@ -127,38 +128,26 @@ static void nmi_shootdown_cpus(void) cpumask_andnot(waiting_to_crash, cpu_online_map, cpumask_of(cpu)); -/* Change NMI trap handlers. Non-crashing pcpus get

  1   2   >