Re: [Xen-devel] [PATCH v4 3/7] libxl: introduce a new structure to represent static shared memory regions

2018-02-06 Thread Julien Grall
On 02/06/2018 04:06 PM, Zhongze Liu wrote: Hi, Hi, 2018-02-06 23:46 GMT+08:00 Julien Grall : Hi, On 02/06/2018 03:41 PM, Zhongze Liu wrote: Thanks for reviewing. 2018-02-06 19:27 GMT+08:00 Julien Grall : Hi, On 01/30/2018 05:50 PM,

Re: [Xen-devel] [PATCH 3/7] xen/arm: vpsci: Add support for PSCI 1.1

2018-02-06 Thread Julien Grall
On 02/06/2018 04:07 PM, Volodymyr Babchuk wrote: Hi, Hi Volodymyr, Thank you for the review. On 5 February 2018 at 15:20, Julien Grall wrote: At the moment, Xen provides virtual PSCI interface compliant with 0.1 and 0.2. Since them, the specification has been

Re: [Xen-devel] [PATCH 4/7] xen/arm: vsmc: Implement SMCCC 1.1

2018-02-06 Thread Julien Grall
Hi, On 02/06/2018 04:18 PM, Volodymyr Babchuk wrote: On 5 February 2018 at 15:20, Julien Grall wrote: The new SMC Calling Convention (v1.1) allows for a reduced overhead when calling into the firmware, and provides a new feature discovery mechanism. See ARM DEN 00070A.

Re: [Xen-devel] [PATCH] x86: reduce Meltdown band-aid IPI overhead

2018-02-06 Thread George Dunlap
On 01/26/2018 12:07 PM, Jan Beulich wrote: > In case we can detect single-threaded guest processes (by checking > whether we can account for all root page table uses locally on the vCPU > that's running), there's no point in issuing a sync IPI upon an L4 entry > update, as no other vCPU of the

Re: [Xen-devel] [PATCH v4 4/7] libxl: support mapping static shared memory areas during domain creation

2018-02-06 Thread Wei Liu
On Tue, Feb 06, 2018 at 05:30:50PM +, Julien Grall wrote: > > > On 02/06/2018 03:59 PM, Zhongze Liu wrote: > > Hi Julien, > > Hi, > > > > 2018-02-06 21:07 GMT+08:00 Julien Grall : > > > Hi, > > > > > > On 01/30/2018 05:50 PM, Zhongze Liu wrote: > > > > > > > > Add

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

2018-02-06 Thread osstest service owner
flight 118620 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118620/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 13 migrate-support-checkfail never pass test-arm64-arm64-xl-xsm

Re: [Xen-devel] [PATCH v4 4/7] libxl: support mapping static shared memory areas during domain creation

2018-02-06 Thread Julien Grall
On 02/06/2018 03:59 PM, Zhongze Liu wrote: Hi Julien, Hi, 2018-02-06 21:07 GMT+08:00 Julien Grall : Hi, On 01/30/2018 05:50 PM, Zhongze Liu wrote: Add libxl__sshm_add to map shared pages from one DomU to another, The mapping process involves the follwing steps:

Re: [Xen-devel] [PATCH v4 5/7] libxl: support unmapping static shared memory areas during domain destruction

2018-02-06 Thread Wei Liu
On Tue, Feb 06, 2018 at 01:24:30PM +, Julien Grall wrote: > > if (libxl__device_pci_destroy_all(gc, domid) < 0) > > LOGD(ERROR, domid, "Pci shutdown failed"); > > rc = xc_domain_pause(ctx->xch, domid); > > diff --git a/tools/libxl/libxl_internal.h

Re: [Xen-devel] [PATCH 5/7] xen/arm: vsmc: Implement SMCCC_ARCH_WORKAROUND_1 BP hardening support

2018-02-06 Thread Julien Grall
On 02/06/2018 04:23 PM, Volodymyr Babchuk wrote: Hi, Hi, On 5 February 2018 at 15:20, Julien Grall wrote: SMCCC 1.1 offers firmware-based CPU workarounds. In particular, SMCCC_ARCH_WORKAROUND_1 provides BP hardening for variant 2 of XSA-254 (CVE-2017-5715). If the

Re: [Xen-devel] [PATCH] x86/nmi: lower initial watchdog frequency to avoid boot hangs

2018-02-06 Thread Alexey G
On Tue, 6 Feb 2018 17:21:19 + Igor Druzhinin wrote: >On 06/02/18 17:08, Alexey G wrote: >> The major concern here is the possiblity of SMI being triggered _not_ >> by some specific I/O port access. Primarily, if it actually was a >> periodic SMI. >> >> If the

Re: [Xen-devel] [PATCH v5 5/8] ARM: VGIC: factor out vgic_connect_hw_irq()

2018-02-06 Thread Julien Grall
Hi Andre, On 02/06/2018 05:09 PM, Andre Przywara wrote: At the moment we happily access VGIC internal data structures like the rank and struct pending_irq in gic.c, which should be VGIC agnostic. Factor out a new function vgic_connect_hw_irq(), which allows a virtual IRQ to be connected to a

Re: [Xen-devel] [PATCH 2/7] xen/arm: psci: Rework the PSCI definitions

2018-02-06 Thread Volodymyr Babchuk
Hi, On 5 February 2018 at 15:20, Julien Grall wrote: > Some PSCI functions are only available in the 32-bit version. After > recent changes, Xen always needs to know whether the call was made using > 32-bit id or 64-bit id. So we don't emulate reserved one. > > With the

Re: [Xen-devel] [PATCH v4 3/7] libxl: introduce a new structure to represent static shared memory regions

2018-02-06 Thread Zhongze Liu
Hi, 2018-02-06 23:46 GMT+08:00 Julien Grall : > Hi, > > On 02/06/2018 03:41 PM, Zhongze Liu wrote: >> >> Thanks for reviewing. >> >> 2018-02-06 19:27 GMT+08:00 Julien Grall : >>> >>> Hi, >>> >>> On 01/30/2018 05:50 PM, Zhongze Liu wrote:

Re: [Xen-devel] [PATCH 6/7] xen/arm: Adapt smccc.h to be able to use it in assembly code

2018-02-06 Thread Volodymyr Babchuk
Hi, On 5 February 2018 at 15:20, Julien Grall wrote: > Signed-off-by: Julien Grall Reviewed-by: Volodymyr Babchuk > --- > xen/include/asm-arm/smccc.h | 16 ++-- > 1 file changed, 10 insertions(+), 6

Re: [Xen-devel] Xen ARM community call Tuesday 13th February 5PM UTC

2018-02-06 Thread Lars Kurth
> As we do not have Jira access, it would be better to do it on Wiki or > "something else". The project has Jira access and can give selected community members write access Read access is not an issue Lars On 06/02/2018, 11:22, "Artem Mygaiev" wrote: Hi

Re: [Xen-devel] [PATCH v2 2/3] xen/arm: vsmc: Don't implement function ID that doesn't exist

2018-02-06 Thread Julien Grall
Hi Volodymyr, On 02/02/2018 01:46 PM, Volodymyr Babchuk wrote: On 02.02.18 13:41, Julien Grall wrote: The current implementation of SMCCC relies on the fact only function number (bits [15:0]) is enough to identify what to implement. However, PSCI call are only available in the range

Re: [Xen-devel] Xen Project Spectre / Meltdown FAQ (Jan 22 Update)

2018-02-06 Thread Rich Persaud
> On Jan 22, 2018, at 11:18, Lars Kurth wrote: > > Hi all, > I will post the following new version of the FAQ on > https://blog.xenproject.org/ in a moment. As there are tables in it, I will > post as PDF rather than text. This thread is primarily a placeholder to

Re: [Xen-devel] [PATCH 1/7] xen/arm: vpsci: Remove parameter 'ver' from do_common_cpu

2018-02-06 Thread Volodymyr Babchuk
Hi Julien, On 5 February 2018 at 15:20, Julien Grall wrote: > Currently, the behavior of do_common_cpu will slightly change depending > on the PSCI version passed in parameter. Looking at the code, more the > specific 0.2 behavior could move out of the function or adapted

Re: [Xen-devel] [PATCH] x86/spec_ctrl: Fix determination of when to use IBRS

2018-02-06 Thread Jan Beulich
>>> On 06.02.18 at 14:48, wrote: > The original version of this logic was: > > /* > * On Intel hardware, we'd like to use retpoline in preference to > * IBRS, but only if it is safe on this hardware. > */ > else if (

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

2018-02-06 Thread osstest service owner
flight 118607 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/118607/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-rumprun-i386 17 rumprun-demo-xenstorels/xenstorels.repeat fail REGR. vs. 118582 Tests

Re: [Xen-devel] [PATCH RFC 2/8] public/io/netif: add directory for backend parameters

2018-02-06 Thread Wei Liu
(Three months after you sent this, sorry...) On Mon, Nov 06, 2017 at 12:33:06PM +, Joao Martins wrote: > On Mon, Nov 06, 2017 at 10:33:59AM +, Paul Durrant wrote: > > > -Original Message- > > > From: Joao Martins [mailto:joao.m.mart...@oracle.com] > > > Sent: 02 November 2017

Re: [Xen-devel] [PATCH v4 3/7] libxl: introduce a new structure to represent static shared memory regions

2018-02-06 Thread Julien Grall
Hi, On 02/06/2018 03:41 PM, Zhongze Liu wrote: Thanks for reviewing. 2018-02-06 19:27 GMT+08:00 Julien Grall : Hi, On 01/30/2018 05:50 PM, Zhongze Liu wrote: Add a new structure to the IDL familiy to represent static shared memory regions [...] +libxl_static_shm =

Re: [Xen-devel] [PATCH v2 3/3] xen/arm: vpsci: Move PSCI function dispatching from vsmc.c to vpsci.c

2018-02-06 Thread Volodymyr Babchuk
On 06.02.18 17:39, Julien Grall wrote: On 02/06/2018 03:28 PM, Volodymyr Babchuk wrote: Hi Julien, On 06.02.18 16:45, Julien Grall wrote: [...] +/* + * PSCI 0.2 or later calls. It will return false if the function ID is + * not handled. + */ +bool do_vpsci_0_2_call(struct cpu_user_regs

Re: [Xen-devel] [PATCH] x86/nmi: lower initial watchdog frequency to avoid boot hangs

2018-02-06 Thread Igor Druzhinin
On 06/02/18 16:23, Jan Beulich wrote: On 06.02.18 at 17:14, wrote: >> On 06/02/18 16:07, Jan Beulich wrote: >> On 05.02.18 at 22:18, wrote: --- a/xen/arch/x86/nmi.c +++ b/xen/arch/x86/nmi.c @@ -34,7 +34,8 @@

Re: [Xen-devel] [PATCH v1 3/6] libxl: add backend type and id to vkb

2018-02-06 Thread Oleksandr Grytsov
On Tue, Feb 6, 2018 at 4:25 PM, Wei Liu wrote: > On Wed, Nov 01, 2017 at 05:05:04PM +0200, Oleksandr Grytsov wrote: > > From: Oleksandr Grytsov > > > > New field backend_type is added to vkb device > > in order to have QEMU and user space backend

Re: [Xen-devel] [PATCH 7/7] xen/arm64: Implement a fast path for handling SMCCC_ARCH_WORKAROUND_1

2018-02-06 Thread Julien Grall
Hi, On 02/06/2018 04:36 PM, Volodymyr Babchuk wrote: On 5 February 2018 at 15:20, Julien Grall wrote: The function SMCCC_ARCH_WORKAROUND_1 will be called by the guest for hardening the branch predictor. So we want the handling to be as fast as possible. As the

Re: [Xen-devel] [PATCH] Choose retpoline only when it is safe to use

2018-02-06 Thread Andrew Cooper
On 06/02/2018 09:13, Zhenzhong Duan wrote: > 在 2018/2/6 16:59, Andrew Cooper 写道: >> On 06/02/2018 08:43, Zhenzhong Duan wrote: >>> When ( ibrs && thunk == THUNK_DEFAULT && !retpoline_safe() ) is true, >>> thunk is set to THUNK_JMP rather than THUNK_RETPOLINE. >>> >>> When (!ibrs && thunk ==

Re: [Xen-devel] [PATCH] x86/boot: Make alternative patching NMI-safe

2018-02-06 Thread Jan Beulich
>>> On 05.02.18 at 19:04, wrote: > But as you correctly pointed out, it was a very long way from being > complete. We currently have no idea whether we are in NMI context, so > arranging not to not execute an iret is hard. As long as we don't mean to patch extremely

Re: [Xen-devel] [PATCH 2/4] x86: eliminate most XPTI entry/exit code when it's not in use

2018-02-06 Thread Jan Beulich
>>> On 05.02.18 at 18:28, wrote: > On 30/01/18 13:51, Jan Beulich wrote: --- a/xen/arch/x86/x86_64/compat/entry.S +++ b/xen/arch/x86/x86_64/compat/entry.S @@ -189,7 +189,7 @@ ENTRY(compat_post_handle_exception) /* See lstar_enter for entry

Re: [Xen-devel] [PATCH] x86/NMI: invert condition in nmi_show_execution_state()

2018-02-06 Thread Andrew Cooper
On 06/02/18 10:41, Jan Beulich wrote: > We want to decode the symbol when _not_ in guest mode. > > Signed-off-by: Jan Beulich Nice.  Figuring out why this wasn't working was on my todo list. Reviewed-by: Andrew Cooper > > ---

Re: [Xen-devel] Xen ARM community call Tuesday 13th February 5PM UTC

2018-02-06 Thread Robin Randhawa
Hi Julien. Thanks for looping me in. On Tue, 2018-02-06 at 10:11 +, Julien Grall wrote: > Hi all, > > I would suggest to have the next community call on Tuesday 13th > February > 5pm GMT. Does it sound good? I'm out of office from the 8th to the 16th unfortunately. > Do you have any

Re: [Xen-devel] Xen ARM community call Tuesday 13th February 5PM UTC

2018-02-06 Thread Lars Kurth
Adding Rich > I think it would be hugely beneficial if there could be an open > repository of information that describes in clear terms what the > specific engineering items are which are needed to make Xen viable for > assessment towards safety certification. Very good idea If

[Xen-devel] [seabios test] 118601: regressions - FAIL

2018-02-06 Thread osstest service owner
flight 118601 seabios real [real] http://logs.test-lab.xenproject.org/osstest/logs/118601/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ws16-amd64 17 guest-stop fail REGR. vs. 115539 Tests which did not

[Xen-devel] [PATCH v3] x86/boot: Make alternative patching NMI-safe

2018-02-06 Thread Andrew Cooper
During patching, there is a very slim risk that an NMI or MCE interrupt in the middle of altering the code in the NMI/MCE paths, in which case bad things will happen. The NMI risk can be eliminated by running the patching loop in NMI context, at which point the CPU will defer further NMIs until

Re: [Xen-devel] Xen ARM community call Tuesday 13th February 5PM UTC

2018-02-06 Thread Artem Mygaiev
Hi Lars As we do not have Jira access, it would be better to do it on Wiki or "something else". -- Artem On 06.02.18 12:53, Lars Kurth wrote: Adding Rich I think it would be hugely beneficial if there could be an open repository of information that describes in clear terms what the

Re: [Xen-devel] [PATCH v4 3/7] libxl: introduce a new structure to represent static shared memory regions

2018-02-06 Thread Julien Grall
Hi, On 01/30/2018 05:50 PM, Zhongze Liu wrote: Add a new structure to the IDL familiy to represent static shared memory regions s/familiy/family/ as proposed in the proposal "Allow setting up shared memory areas between VMs from xl config file" (see [1]). And deleted some trailing white

Re: [Xen-devel] 回复: Re: [PATCH] Choose retpoline only when it is safe to use

2018-02-06 Thread Zhenzhong Duan
On 2018/2/6 18:50, Andrew Cooper wrote: On 06/02/18 10:29, zhenzhong.duan wrote: 2018年2月6日 17:20于 Andrew Cooper >写道: > > On 06/02/2018 09:13, Zhenzhong Duan wrote: > > 在 2018/2/6 16:59, Andrew Cooper 写道: > >> On 06/02/2018 08:43,

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

2018-02-06 Thread osstest service owner
flight 118600 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/118600/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt-xsm 14 saverestore-support-checkfail like 118548 test-armhf-armhf-libvirt 14

Re: [Xen-devel] [PATCH 4/4] x86: avoid double CR3 reload when switching to guest user mode

2018-02-06 Thread Jan Beulich
>>> On 05.02.18 at 18:37, wrote: > On 31/01/18 10:12, Jan Beulich wrote: >> --- a/xen/arch/x86/pv/domain.c +++ b/xen/arch/x86/pv/domain.c @@ -220,10 +220,20 @@ int pv_domain_initialise(struct domain * return rc; } -static void

Re: [Xen-devel] [PATCH] xen/livepatch: Drop stray tabs and fix indentation

2018-02-06 Thread Ross Lagerwall
On 02/05/2018 11:04 AM, Andrew Cooper wrote: Signed-off-by: Andrew Cooper --- CC: Konrad Rzeszutek Wilk CC: Ross Lagerwall --- xen/common/livepatch.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

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

2018-02-06 Thread osstest service owner
flight 118605 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/118605/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt 14 saverestore-support-checkfail like 118557 test-armhf-armhf-libvirt-xsm 14

Re: [Xen-devel] 回复: Re: [PATCH] Choose retpoline only when it is safe to use

2018-02-06 Thread Andrew Cooper
On 06/02/18 10:29, zhenzhong.duan wrote: > > > 2018年2月6日 17:20于 Andrew Cooper >写道: > > > > On 06/02/2018 09:13, Zhenzhong Duan wrote: > > > 在 2018/2/6 16:59, Andrew Cooper 写道: > > >> On 06/02/2018 08:43, Zhenzhong Duan wrote: > > >>>

Re: [Xen-devel] [PATCH v2] x86/boot: Make alternative patching NMI-safe

2018-02-06 Thread Jan Beulich
>>> On 06.02.18 at 02:07, wrote: > On 05/02/2018 19:23, Konrad Rzeszutek Wilk wrote: >> On Mon, Feb 05, 2018 at 07:10:33PM +, Andrew Cooper wrote: >>> -apply_alternatives(__alt_instructions, __alt_instructions_end); >>> +/* Send ourselves an NMI to trigger

[Xen-devel] Xen ARM community call Tuesday 13th February 5PM UTC

2018-02-06 Thread Julien Grall
Hi all, I would suggest to have the next community call on Tuesday 13th February 5pm GMT. Does it sound good? Do you have any specific topic you would like to discuss? Cheers, -- Julien Grall ___ Xen-devel mailing list

Re: [Xen-devel] [xen-unstable test] 118594: regressions - FAIL

2018-02-06 Thread Jan Beulich
>>> On 06.02.18 at 06:40, wrote: > flight 118594 xen-unstable real [real] > http://logs.test-lab.xenproject.org/osstest/logs/118594/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: >

[Xen-devel] [PATCH] x86/NMI: invert condition in nmi_show_execution_state()

2018-02-06 Thread Jan Beulich
We want to decode the symbol when _not_ in guest mode. Signed-off-by: Jan Beulich --- a/xen/arch/x86/traps.c +++ b/xen/arch/x86/traps.c @@ -632,7 +632,7 @@ static int nmi_show_execution_state(cons show_execution_state(regs); else printk(XENLOG_ERR

Re: [Xen-devel] [PATCH v4 2/7] xen: xsm: flask: introduce XENMAPSPACE_gmfn_share for memory sharing

2018-02-06 Thread Julien Grall
Hi, On 01/30/2018 05:50 PM, Zhongze Liu wrote: diff --git a/xen/arch/arm/mm.c b/xen/arch/arm/mm.c index 3c328e2df5..8e385d62a8 100644 --- a/xen/arch/arm/mm.c +++ b/xen/arch/arm/mm.c @@ -1251,6 +1251,7 @@ int xenmem_add_to_physmap_one( break; case XENMAPSPACE_gmfn_foreign: +

Re: [Xen-devel] 回复: Re: [PATCH] Choose retpoline only when it is safe to use

2018-02-06 Thread Andrew Cooper
On 06/02/18 11:41, Zhenzhong Duan wrote: > On 2018/2/6 18:50, Andrew Cooper wrote: >> On 06/02/18 10:29, zhenzhong.duan wrote: >>> >>> >>> 2018年2月6日 17:20于 Andrew Cooper >> >写道: >>> > >>> > On 06/02/2018 09:13, Zhenzhong Duan wrote: >>>

Re: [Xen-devel] 回复: Re: [PATCH] Choose retpoline only when it is safe to use

2018-02-06 Thread Zhenzhong Duan
在 2018/2/6 19:56, Andrew Cooper 写道: On 06/02/18 11:41, Zhenzhong Duan wrote: On 2018/2/6 18:50, Andrew Cooper wrote: On 06/02/18 10:29, zhenzhong.duan wrote: 2018年2月6日 17:20于 Andrew Cooper >写道: > > On 06/02/2018 09:13, Zhenzhong

Re: [Xen-devel] [PATCH v3] libxc: don't fail domain creation when unpacking initrd fails

2018-02-06 Thread Wei Liu
On Thu, Feb 01, 2018 at 12:22:43AM -0700, Jan Beulich wrote: > At least Linux kernels have been able to work with gzip-ed initrd for > quite some time; initrd compressed with other methods aren't even being > attempted to unpack. Furthermore the unzip-ing routine used here isn't > capable of

Re: [Xen-devel] Problem with IOMEM and domain reboot

2018-02-06 Thread Wei Liu
On Wed, Dec 20, 2017 at 06:27:02PM +0200, Oleksandr Andrushchenko wrote: > Hi, all! > > While trying to reboot a domain which has iomem configured > (we are passing through some devices), I found an issue, > that after domain reboot those iomem's are incorrectly re-mapped, > e.g. for the

Re: [Xen-devel] [PATCH RFC] firmware/shim: fix Xen tree setup

2018-02-06 Thread Wei Liu
On Wed, Jan 31, 2018 at 09:55:21AM -0700, Jan Beulich wrote: > There are multiple issues here, and I'm happy to split the patch up if > that's what it takes: > - "set -e" on a separate Makefile line is meaningless. Glue together all > the lines that this is supposed to cover. > - I have no idea

Re: [Xen-devel] [PATCH] libxl: add libxl__is_driver_domain function

2018-02-06 Thread Wei Liu
On Thu, Dec 14, 2017 at 04:14:12PM +0200, Oleksandr Grytsov wrote: > From: Oleksandr Grytsov > > We have following arm-based setup: > > - Dom0 with xen and xen tools; > - Dom1 with device backends (but it is not the driver domain); What is your definition of a

Re: [Xen-devel] Problem with IOMEM and domain reboot

2018-02-06 Thread Oleksandr Andrushchenko
Hi, Wei! On 02/06/2018 02:36 PM, Wei Liu wrote: On Wed, Dec 20, 2017 at 06:27:02PM +0200, Oleksandr Andrushchenko wrote: Hi, all! While trying to reboot a domain which has iomem configured (we are passing through some devices), I found an issue, that after domain reboot those iomem's are

Re: [Xen-devel] [PATCH 1/2] make xen ocaml safe-strings compliant

2018-02-06 Thread Michael Young
On Tue, 6 Feb 2018, Wei Liu wrote: On Tue, Jan 30, 2018 at 10:55:47PM +, Michael Young wrote: Xen built with ocaml 4.06 gives errors such as Error: This expression has type bytes but an expression was expected of type string as Byte and safe-strings which were introduced in 4.02

Re: [Xen-devel] [PATCH 1/3] Make credit2 the default scheduler

2018-02-06 Thread Dario Faggioli
On Tue, 2018-02-06 at 17:02 +, George Dunlap wrote: > On 02/06/2018 06:18 AM, Juergen Gross wrote: > > On 05/02/18 17:53, Dario Faggioli wrote: > > > > > > Considering we're releasing in June, but freezing in March, do we > > > think > > > it is still early enough? > > > The 4.11 release is

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

2018-02-06 Thread osstest service owner
flight 118613 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/118613/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt-xsm 14 saverestore-support-checkfail like 118600 test-armhf-armhf-libvirt 14

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

2018-02-06 Thread osstest service owner
flight 118626 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/118626/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 13 migrate-support-checkfail never pass test-arm64-arm64-xl-xsm

Re: [Xen-devel] [GSOC] Xen on ARM: create multiple guests from device tree

2018-02-06 Thread Stefano Stabellini
On Tue, 6 Feb 2018, Denis Obrezkov wrote: > > Hello Denis, > > > Hello Stefano, > > it is great to see interest in Xen on ARM and this project! > > > > Unfortunately RPi3 can't run Xen as far as I know due to their non-ARM > > interrupt controller without virtualization support. Otherwise it would

[Xen-devel] [PATCH 1/2] xen/arm: Extend the number of memory banks supported

2018-02-06 Thread Julien Grall
When booting using Grub on Thunder-X, the number of memory available is greater than 64. Bump the number to 128, so we can take advantage of all the memory. Signed-off-by: Julien Grall --- Note that I wasn't able to boot without this patch, because EFI stub is

[Xen-devel] [PATCH 0/2] Fixup for booting Xen on Thunder-X using Grub

2018-02-06 Thread Julien Grall
Hi all, This small series will help to boot Xen on Thunder-X using Grub. This part of my work to use Thunder-X in Osstest Cheers, Julien Grall (2): xen/arm: Extend the number of memory banks supported xen/arm: Blacklist SMMU on Thunder-X xen/arch/arm/platforms/Makefile | 1 +

Re: [Xen-devel] [PATCH] x86/nmi: lower initial watchdog frequency to avoid boot hangs

2018-02-06 Thread Igor Druzhinin
On 06/02/18 16:23, Jan Beulich wrote: On 06.02.18 at 17:14, wrote: >> On 06/02/18 16:07, Jan Beulich wrote: >> On 05.02.18 at 22:18, wrote: --- a/xen/arch/x86/nmi.c +++ b/xen/arch/x86/nmi.c @@ -34,7 +34,8 @@

[Xen-devel] [linux-linus bisection] complete build-armhf-pvops

2018-02-06 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job build-armhf-pvops testid kernel-build Tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git *** Found and reproduced problem changeset *** Bug is in

Re: [Xen-devel] [PATCH v2 3/3] xen/arm: vpsci: Move PSCI function dispatching from vsmc.c to vpsci.c

2018-02-06 Thread Volodymyr Babchuk
Hi Julien, On 06.02.18 16:45, Julien Grall wrote: [...] +/* + * PSCI 0.2 or later calls. It will return false if the function ID is + * not handled. + */ +bool do_vpsci_0_2_call(struct cpu_user_regs *regs, uint32_t fid) +{ +    /* + * /!\ VPSCI_NR_FUNCS (in asm-arm/vpsci.h) should be

Re: [Xen-devel] [PATCH 4/7] xen/arm: vsmc: Implement SMCCC 1.1

2018-02-06 Thread Volodymyr Babchuk
On 5 February 2018 at 15:20, Julien Grall wrote: > The new SMC Calling Convention (v1.1) allows for a reduced overhead when > calling into the firmware, and provides a new feature discovery > mechanism. See ARM DEN 00070A. Сould you please use also a human-readable document

Re: [Xen-devel] [PATCH v1 3/6] libxl: add backend type and id to vkb

2018-02-06 Thread Wei Liu
On Tue, Feb 06, 2018 at 06:45:15PM +0200, Oleksandr Grytsov wrote: > > > +static int libxl__set_xenstore_vkb(libxl__gc *gc, uint32_t domid, > > > + libxl_device_vkb *vkb, > > > + flexarray_t *back, flexarray_t > > *front, > > > +

Re: [Xen-devel] [PATCH 1/3] Make credit2 the default scheduler

2018-02-06 Thread George Dunlap
On 02/06/2018 06:18 AM, Juergen Gross wrote: > On 05/02/18 17:53, Dario Faggioli wrote: >> On Mon, 2018-02-05 at 13:01 +, George Dunlap wrote: >>> And in any case, making those improvements >>> on credit2 will be easier than on credit. >>> >> And, if possible, I agree with George on this even

Re: [Xen-devel] [PATCH v2 2/3] xen/arm: vsmc: Don't implement function ID that doesn't exist

2018-02-06 Thread Volodymyr Babchuk
Hi Julien, On 06.02.18 16:53, Julien Grall wrote: Hi Volodymyr, On 02/02/2018 01:46 PM, Volodymyr Babchuk wrote: On 02.02.18 13:41, Julien Grall wrote: The current implementation of SMCCC relies on the fact only function number (bits [15:0]) is enough to identify what to implement.

Re: [Xen-devel] [PATCH v2 2/3] xen/arm: vsmc: Don't implement function ID that doesn't exist

2018-02-06 Thread Julien Grall
On 02/06/2018 03:15 PM, Volodymyr Babchuk wrote: Hi Julien, Hi, On 06.02.18 16:53, Julien Grall wrote: On 02/02/2018 01:46 PM, Volodymyr Babchuk wrote: On 02.02.18 13:41, Julien Grall wrote: The current implementation of SMCCC relies on the fact only function number (bits [15:0]) is

[Xen-devel] [PATCH v3 3/3] xen/arm: vpsci: Move PSCI function dispatching from vsmc.c to vpsci.c

2018-02-06 Thread Julien Grall
At the moment PSCI function dispatching is done in vsmc.c and the function implementation in vpsci.c. Some bits of the implementation is even done in vsmc.c (see PSCI_SYSTEM_RESET). This means that it is difficult to follow the implementation and also requires to export functions for each PSCI

[Xen-devel] [PATCH v3 0/3] xen/arm: SMCCC fixes and PSCI clean-up

2018-02-06 Thread Julien Grall
Hi all, This small patch series contains SMCCC fixes (see #2) and PSCI clean-up. Cheers, Julien Grall (3): xen/arm: vpsci: Removing dummy MIGRATE and MIGRATE_INFO_UP_CPU xen/arm: vsmc: Don't implement function ID that doesn't exist xen/arm: vpsci: Move PSCI function dispatching from

Re: [Xen-devel] [PATCH 3/7] xen/arm: vpsci: Add support for PSCI 1.1

2018-02-06 Thread Volodymyr Babchuk
Hi, On 5 February 2018 at 15:20, Julien Grall wrote: > At the moment, Xen provides virtual PSCI interface compliant with 0.1 > and 0.2. Since them, the specification has been updated and the latest > version is 1.1 (see ARM DEN 0022D). > > From an implementation point of

Re: [Xen-devel] [PATCH] x86/nmi: lower initial watchdog frequency to avoid boot hangs

2018-02-06 Thread Jan Beulich
>>> On 05.02.18 at 22:18, wrote: > --- a/xen/arch/x86/nmi.c > +++ b/xen/arch/x86/nmi.c > @@ -34,7 +34,8 @@ > #include > > unsigned int nmi_watchdog = NMI_NONE; > -static unsigned int nmi_hz = HZ; > +/* initial watchdog frequency - shouldn't be too high to avoid

Re: [Xen-devel] [PATCH 7/7] xen/arm64: Implement a fast path for handling SMCCC_ARCH_WORKAROUND_1

2018-02-06 Thread Volodymyr Babchuk
Hi, On 5 February 2018 at 15:20, Julien Grall wrote: > The function SMCCC_ARCH_WORKAROUND_1 will be called by the guest for > hardening the branch predictor. So we want the handling to be as fast as > possible. > > As the mitigation is applied on every guest exit, we can

Re: [Xen-devel] [PATCH v1 5/5] docs: add PV sound device config

2018-02-06 Thread Wei Liu
On Wed, Nov 01, 2017 at 05:04:47PM +0200, Oleksandr Grytsov wrote: > From: Oleksandr Grytsov > > Update documentation with virtual sound device > > Signed-off-by: Oleksandr Grytsov Acked-by: Wei Liu

Re: [Xen-devel] [PATCH v2 3/3] xen/arm: vpsci: Move PSCI function dispatching from vsmc.c to vpsci.c

2018-02-06 Thread Julien Grall
On 02/06/2018 03:28 PM, Volodymyr Babchuk wrote: Hi Julien, On 06.02.18 16:45, Julien Grall wrote: [...] +/* + * PSCI 0.2 or later calls. It will return false if the function ID is + * not handled. + */ +bool do_vpsci_0_2_call(struct cpu_user_regs *regs, uint32_t fid) +{ +    /* + *

Re: [Xen-devel] [PATCH v4 4/7] libxl: support mapping static shared memory areas during domain creation

2018-02-06 Thread Zhongze Liu
Hi Julien, 2018-02-06 21:07 GMT+08:00 Julien Grall : > Hi, > > On 01/30/2018 05:50 PM, Zhongze Liu wrote: >> >> Add libxl__sshm_add to map shared pages from one DomU to another, The >> mapping >> process involves the follwing steps: [...] >> + >> +/* Set default values for

[Xen-devel] CfP 13th Virtualization in High­-Performance Cloud Computing Workshop (VHPC '18)

2018-02-06 Thread VHPC 18
*CALL FOR PAPERS 13th Workshop on Virtualization in High­-Performance Cloud Computing (VHPC '18)held in conjunction with the International Supercomputing Conference - High Performance,June 24-28, 2018, Frankfurt,

Re: [Xen-devel] [PATCH 5/7] xen/arm: vsmc: Implement SMCCC_ARCH_WORKAROUND_1 BP hardening support

2018-02-06 Thread Volodymyr Babchuk
Hi, On 5 February 2018 at 15:20, Julien Grall wrote: > SMCCC 1.1 offers firmware-based CPU workarounds. In particular, > SMCCC_ARCH_WORKAROUND_1 provides BP hardening for variant 2 of XSA-254 > (CVE-2017-5715). > > If the hypervisor has some mitigation for this issue,

[Xen-devel] [PATCH v5 7/8] ARM: VGIC: rework gicv[23]_update_lr to not use pending_irq

2018-02-06 Thread Andre Przywara
The functions to actually populate a list register were accessing the VGIC internal pending_irq struct, although they should be abstracting from that. Break the needed information down to remove the reference to pending_irq from gic-v[23].c. Signed-off-by: Andre Przywara

Re: [Xen-devel] [PATCH] x86/nmi: lower initial watchdog frequency to avoid boot hangs

2018-02-06 Thread Igor Druzhinin
On 06/02/18 17:08, Alexey G wrote: > On Tue, 6 Feb 2018 14:21:12 + > Andrew Cooper wrote: > >> On 06/02/18 03:10, Alexey G wrote: >>> I/O port 61h normally is not emulated by SMI legacy kbd handling code >>> in BIOS, only ports like 60h, 64h, etc. >>> Contrary to

Re: [Xen-devel] [PATCH v3 3/3] xen/arm: vpsci: Move PSCI function dispatching from vsmc.c to vpsci.c

2018-02-06 Thread Volodymyr Babchuk
Hi, On 06.02.18 17:53, Julien Grall wrote: At the moment PSCI function dispatching is done in vsmc.c and the function implementation in vpsci.c. Some bits of the implementation is even done in vsmc.c (see PSCI_SYSTEM_RESET). This means that it is difficult to follow the implementation and also

[Xen-devel] [PATCH v5 2/8] ARM: VGIC: split gic.c to observe hardware/virtual GIC separation

2018-02-06 Thread Andre Przywara
Currently gic.c holds code to handle hardware IRQs as well as code to bridge VGIC requests to the GIC virtualization hardware. Despite being named gic.c, this file reaches into the VGIC and uses data structures describing virtual IRQs. To improve abstraction, move the VGIC functions into a

[Xen-devel] [PATCH v5 0/8] ARM: VGIC/GIC separation cleanups

2018-02-06 Thread Andre Przywara
Hi, a minor update over the last post. Added tags, extended one commit message and added an error return in patch 5/8. Please test and apply. Cheers, Andre Changelog v4->v5 01: unchanged 02: reordered headers, added tag 03, 04: unchanged 05: return an error if irq_desc does not match, extended

[Xen-devel] [PATCH v5 1/8] ARM: VGIC: drop unneeded gic_restore_pending_irqs()

2018-02-06 Thread Andre Przywara
In gic_restore_pending_irqs() we push our pending virtual IRQs into the list registers. This function is called once from gic_inject(), just before we return to the guest, but also in gic_restore_state(), when we context-switch a VCPU. Having a closer look it turns out that the later call is not

[Xen-devel] [PATCH v5 5/8] ARM: VGIC: factor out vgic_connect_hw_irq()

2018-02-06 Thread Andre Przywara
At the moment we happily access VGIC internal data structures like the rank and struct pending_irq in gic.c, which should be VGIC agnostic. Factor out a new function vgic_connect_hw_irq(), which allows a virtual IRQ to be connected to a hardware IRQ (using the hw bit in the LR). This removes

Re: [Xen-devel] [PATCH] x86/nmi: lower initial watchdog frequency to avoid boot hangs

2018-02-06 Thread Alexey G
On Tue, 6 Feb 2018 14:21:12 + Andrew Cooper wrote: >On 06/02/18 03:10, Alexey G wrote: >> I/O port 61h normally is not emulated by SMI legacy kbd handling code >> in BIOS, only ports like 60h, 64h, etc. >> Contrary to USB legacy emulation, it has to intercept port

[Xen-devel] [PATCH v5 4/8] ARM: VGIC: rework events_need_delivery()

2018-02-06 Thread Andre Przywara
In event.h we very deeply dive into the VGIC to learn if an event for a guest is pending. Rework that function to abstract the VGIC specific part out. Also reorder the queries there, as we only actually need to check for the event channel if there are no other pending IRQs. Signed-off-by: Andre

[Xen-devel] [PATCH v5 3/8] ARM: VGIC: split up gic_dump_info() to cover virtual part separately

2018-02-06 Thread Andre Przywara
Currently gic_dump_info() not only dumps the hardware state of the GIC, but also the VGIC internal virtual IRQ lists. Split the latter off and move it into gic-vgic.c to observe the abstraction. Signed-off-by: Andre Przywara Reviewed-by: Stefano Stabellini

[Xen-devel] [PATCH v5 6/8] ARM: VGIC: factor out vgic_get_hw_irq_desc()

2018-02-06 Thread Andre Przywara
At the moment we happily access the VGIC internal struct pending_irq (which describes a virtual IRQ) in irq.c. Factor out the actually needed functionality to learn the associated hardware IRQ and move that into gic-vgic.c to improve abstraction. Signed-off-by: Andre Przywara

[Xen-devel] [PATCH v5 8/8] ARM: make nr_irqs a constant

2018-02-06 Thread Andre Przywara
On ARM the maximum number of IRQs is a constant, but we share it being a variable to match x86. Since we are not supposed to alter it, let's mark it as "const" to avoid accidental change. Suggested-by: Julien Grall Signed-off-by: Andre Przywara

Re: [Xen-devel] [PATCH v1 2/5] libxl: add vsnd list and info

2018-02-06 Thread Wei Liu
On Wed, Nov 01, 2017 at 05:04:44PM +0200, Oleksandr Grytsov wrote: > From: Oleksandr Grytsov > > Add getting vsnd list amd info API > > Signed-off-by: Oleksandr Grytsov > --- > tools/libxl/libxl.h | 10 ++ >

Re: [Xen-devel] [PATCH v4 5/8] ARM: VGIC: factor out vgic_connect_hw_irq()

2018-02-06 Thread Andre Przywara
Hi, On 06/02/18 14:21, Julien Grall wrote: > Hi Andre, > > On 02/05/2018 04:19 PM, Andre Przywara wrote: >> At the moment we happily access VGIC internal data structures like >> the rank and struct pending_irq in gic.c, which should be VGIC agnostic. >> >> Factor out a new function

[Xen-devel] [PATCH v3 2/3] xen/arm: vsmc: Don't implement function ID that doesn't exist

2018-02-06 Thread Julien Grall
The current implementation of SMCCC relies on the fact only function number (bits [15:0]) is enough to identify what to implement. However, PSCI call are only available in the range 0x8400-0x841F and 0xC400-0xC41F. Furthermore, not all SMC32 functions have equivalent in the SMC64.

[Xen-devel] [PATCH v3 1/3] xen/arm: vpsci: Removing dummy MIGRATE and MIGRATE_INFO_UP_CPU

2018-02-06 Thread Julien Grall
The PSCI call MIGRATE and MIGRATE_INFO_UP_CPU are optional and implemented as just returning PSCI_NOT_SUPPORTED (aka UNKNOWN_FUNCTION for SMCCC). The new SMCCC framework is able to deal with unimplemented function and return the proper error code. So remove the implementations for both function.

Re: [Xen-devel] [PATCH] x86/nmi: lower initial watchdog frequency to avoid boot hangs

2018-02-06 Thread Jan Beulich
>>> On 06.02.18 at 17:14, wrote: > On 06/02/18 16:07, Jan Beulich wrote: > On 05.02.18 at 22:18, wrote: >>> --- a/xen/arch/x86/nmi.c >>> +++ b/xen/arch/x86/nmi.c >>> @@ -34,7 +34,8 @@ >>> #include >>> >>> unsigned int nmi_watchdog =

Re: [Xen-devel] [PATCH 1/2] make xen ocaml safe-strings compliant

2018-02-06 Thread Wei Liu
On Tue, Jan 30, 2018 at 10:55:47PM +, Michael Young wrote: > Xen built with ocaml 4.06 gives errors such as > Error: This expression has type bytes but an expression was > expected of type string > as Byte and safe-strings which were introduced in 4.02 are the > default in 4.06. > This

Re: [Xen-devel] [PATCH v1 3/5] xl: add PV sound condif parser

2018-02-06 Thread Wei Liu
On Wed, Nov 01, 2017 at 05:04:45PM +0200, Oleksandr Grytsov wrote: > From: Oleksandr Grytsov > > Add config parser for virtual sound devices > > Signed-off-by: Oleksandr Grytsov > + > +int parse_vsnd_item(libxl_device_vsnd *vsnd, const

Re: [Xen-devel] [PATCH v1 1/5] libxl: add PV sound device

2018-02-06 Thread Wei Liu
On Wed, Nov 01, 2017 at 05:04:43PM +0200, Oleksandr Grytsov wrote: > From: Oleksandr Grytsov > > Add PV sound device described in sndif.h > > Signed-off-by: Oleksandr Grytsov Acked-by: Wei Liu (I haven't checked if

Re: [Xen-devel] [PATCH v4 5/8] ARM: VGIC: factor out vgic_connect_hw_irq()

2018-02-06 Thread Julien Grall
On 02/06/2018 03:44 PM, Andre Przywara wrote: Hi, On 06/02/18 14:21, Julien Grall wrote: Hi Andre, On 02/05/2018 04:19 PM, Andre Przywara wrote: At the moment we happily access VGIC internal data structures like the rank and struct pending_irq in gic.c, which should be VGIC agnostic.

  1   2   >