Re: [Xen-devel] [PATCH v2] x86/hvm: Add MSR old value

2017-12-05 Thread Jan Beulich
>>> On 04.12.17 at 16:45, wrote: >> On Fri, Oct 13, 2017 at 03:50:57PM +0300, Alexandru Isaila wrote: >>> This patch adds the old value param and the onchangeonly option >>> to the VM_EVENT_REASON_MOV_TO_MSR event. >>> >>> The param was added to the vm_event_mov_to_msr

Re: [Xen-devel] [RFC v3 2/4] xen/linux_compat: Add a Linux compat header

2017-12-05 Thread Jan Beulich
>>> On 05.12.17 at 04:59, wrote: > For porting files from Linux it is useful to have a Linux API to Xen API > mapping header at a common location. Looking at what you add here I really think "no, please don't". But let's see what other maintainers thinks. Jan

Re: [Xen-devel] [PATCH 4/5] x86: use switch() in _put_page_type()

2017-12-05 Thread Jan Beulich
>>> On 04.12.17 at 16:40, wrote: > On 04/12/17 10:46, Jan Beulich wrote: >> Use this to cheaply add another assertion. >> >> Signed-off-by: Jan Beulich >> --- >> TBD: Would it perhaps be better to return after the assertion? > > Yes, otherwise we

Re: [Xen-devel] [PATCH v9 00/16] Enable Memory Bandwidth Allocation in Xen

2017-12-05 Thread Yi Sun
On 17-12-05 02:09:14, Jan Beulich wrote: > >>> On 05.12.17 at 03:53, wrote: > > This patch set has got all Acked-by/Reviewed-by months ago. So, I am > > wondering when it can be integrated? Any plan? Thanks! > > It is not being forgotten. But the tree is not fully open

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

2017-12-05 Thread osstest service owner
flight 116832 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/116832/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-win7-amd64 10 windows-install fail REGR. vs. 116744 Tests which did

Re: [Xen-devel] [RFC v3 1/4] Port WARN_ON_ONCE() from Linux

2017-12-05 Thread Jan Beulich
>>> On 05.12.17 at 04:59, wrote: > Port WARN_ON_ONCE macro from Linux. A return value is expected from this > macro, so the implementation does not follow the Xen convention of wrapping > macros in a do..while. There's no such convention for macros producing a value. >

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

2017-12-05 Thread osstest service owner
flight 116849 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/116849/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt 14 saverestore-support-checkfail like 116770 test-armhf-armhf-libvirt-raw 13

Re: [Xen-devel] [RFC PATCH 04/31] cpufreq: make turbo settings to be configurable

2017-12-05 Thread Oleksandr Tyshchenko
Hi Stefano On Tue, Dec 5, 2017 at 12:18 AM, Stefano Stabellini wrote: > On Sat, 2 Dec 2017, Oleksandr Tyshchenko wrote: >> On Sat, Dec 2, 2017 at 3:06 AM, Stefano Stabellini >> wrote: >> > On Thu, 9 Nov 2017, Oleksandr Tyshchenko wrote: >> >>

Re: [Xen-devel] [RFC] WIP: optee: add OP-TEE mediator

2017-12-05 Thread Julien Grall
Hi, On 04/12/17 22:04, Stefano Stabellini wrote: On Mon, 4 Dec 2017, Volodymyr Babchuk wrote: Hi Julien, On Mon, Dec 04, 2017 at 04:27:14PM +, Julien Grall wrote: [...] = Error checking / DOS protection = We need powerful checks on arguments passed by the caller and evaluated by the

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

2017-12-05 Thread Jan Beulich
>>> On 05.12.17 at 11:03, wrote: > On 05/12/2017 09:30, Jan Beulich wrote: > On 05.12.17 at 09:49, wrote: >>> flight 116832 xen-unstable real [real] >>> http://logs.test-lab.xenproject.org/osstest/logs/116832/ >>> >>> Regressions :-(

Re: [Xen-devel] [RFC PATCH 13/31] xen/arm: Add driver_data field to struct device

2017-12-05 Thread Julien Grall
On 09/11/17 17:10, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko Please explain the rationale behind adding a new field in struct device. Signed-off-by: Oleksandr Tyshchenko CC: Stefano Stabellini

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

2017-12-05 Thread Andrew Cooper
On 05/12/2017 10:03, Andrew Cooper wrote: > On 05/12/2017 09:30, Jan Beulich wrote: > On 05.12.17 at 09:49, wrote: >>> flight 116832 xen-unstable real [real] >>> http://logs.test-lab.xenproject.org/osstest/logs/116832/ >>> >>> Regressions :-( >>> >>> Tests which

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

2017-12-05 Thread Jan Beulich
>>> On 05.12.17 at 09:49, wrote: > flight 116832 xen-unstable real [real] > http://logs.test-lab.xenproject.org/osstest/logs/116832/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: >

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

2017-12-05 Thread Andrew Cooper
On 05/12/2017 09:30, Jan Beulich wrote: On 05.12.17 at 09:49, wrote: >> flight 116832 xen-unstable real [real] >> http://logs.test-lab.xenproject.org/osstest/logs/116832/ >> >> Regressions :-( >> >> Tests which did not succeed and are blocking, >> including

Re: [Xen-devel] [PATCH v2] x86/hvm: fix interaction between internal and external emulation

2017-12-05 Thread Jan Beulich
>>> On 05.12.17 at 15:00, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 05 December 2017 13:53 >> >>> On 28.11.17 at 15:05, wrote: >> rc = x86_emulate(_ctxt->ctxt, ops); >> >> if ( rc != X86EMUL_RETRY ) >> { >>

Re: [Xen-devel] [PATCH v2] x86/hvm: fix interaction between internal and external emulation

2017-12-05 Thread Jan Beulich
>>> On 05.12.17 at 15:11, wrote: >> From: Xen-devel [mailto:xen-devel-boun...@lists.xenproject.org] On Behalf >> Of Paul Durrant >> Sent: 05 December 2017 14:00 >> To: 'Jan Beulich' >> > From: Jan Beulich [mailto:jbeul...@suse.com] >> > Sent: 05

Re: [Xen-devel] [PATCH v2] x86/hvm: fix interaction between internal and external emulation

2017-12-05 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 05 December 2017 14:32 > To: Paul Durrant > Cc: xen-devel@lists.xenproject.org > Subject: RE: [PATCH v2] x86/hvm: fix interaction between internal and > external emulation > > >>> On

[Xen-devel] [PATCH] xen/pvcalls: check for xenbus_read() errors

2017-12-05 Thread Dan Carpenter
Smatch complains that "len" is uninitialized if xenbus_read() fails so let's add some error handling. Signed-off-by: Dan Carpenter diff --git a/drivers/xen/pvcalls-front.c b/drivers/xen/pvcalls-front.c index 40caa92bff33..afa3f1b5d807 100644 ---

[Xen-devel] [PATCH 2/2] xen/pvcalls: Fix a check in pvcalls_front_remove()

2017-12-05 Thread Dan Carpenter
bedata->ref can't be less than zero because it's unsigned. This affects certain error paths in probe. We first set ->ref = -1 and then we set it to a valid value later. Fixes: 219681909913 ("xen/pvcalls: connect to the backend") Signed-off-by: Dan Carpenter diff

Re: [Xen-devel] [RFC PATCH 06/31] cpufreq: make cpufreq driver more generalizable

2017-12-05 Thread Oleksandr Tyshchenko
Hi Stefano On Tue, Dec 5, 2017 at 12:46 AM, Stefano Stabellini wrote: > On Mon, 4 Dec 2017, Oleksandr Tyshchenko wrote: >> Hi, Stefano >> >> On Sat, Dec 2, 2017 at 3:37 AM, Stefano Stabellini >> wrote: >> > On Thu, 9 Nov 2017, Oleksandr Tyshchenko

Re: [Xen-devel] [RFC PATCH 16/31] arm: add SMC wrapper that is compatible with SMCCC

2017-12-05 Thread Volodymyr Babchuk
Hi Julien, On 05.12.17 16:58, Julien Grall wrote: Hi Oleksandr, On 09/11/17 17:10, Oleksandr Tyshchenko wrote: From: Volodymyr Babchuk Existing SMC wrapper call_smc() allows only 4 parameters and returns only one value. This is enough for existing use in PSCI

Re: [Xen-devel] [RFC PATCH 16/31] arm: add SMC wrapper that is compatible with SMCCC

2017-12-05 Thread Stefano Stabellini
On Tue, 5 Dec 2017, Volodymyr Babchuk wrote: > Hi Stefano, > > On Mon, Dec 04, 2017 at 06:30:13PM -0800, Stefano Stabellini wrote: > > On Thu, 9 Nov 2017, Oleksandr Tyshchenko wrote: > > > From: Volodymyr Babchuk > > > > > > Existing SMC wrapper call_smc() allows

Re: [Xen-devel] [PATCH] x86/HVM: don't retain emulated insn cache when exiting back to guest

2017-12-05 Thread Jan Beulich
>>> On 05.12.17 at 17:44, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 05 December 2017 16:14 >> --- a/xen/arch/x86/hvm/emulate.c >> +++ b/xen/arch/x86/hvm/emulate.c >> @@ -2109,20 +2109,22 @@ static int _hvm_emulate_one(struct hvm_e >> >>

Re: [Xen-devel] [PATCH] xen/pvcalls: check for xenbus_read() errors

2017-12-05 Thread Stefano Stabellini
On Tue, 5 Dec 2017, Dan Carpenter wrote: > Smatch complains that "len" is uninitialized if xenbus_read() fails so > let's add some error handling. > > Signed-off-by: Dan Carpenter Reviewed-by: Stefano Stabellini > diff --git

Re: [Xen-devel] [PATCH v17 19/19] osstest: use -DWITHOUT_AUTO_OBJ with FreeBSD release targets

2017-12-05 Thread Roger Pau Monné
On Tue, Dec 05, 2017 at 03:09:47PM +, Ian Jackson wrote: > Roger Pau Monne writes ("[PATCH v17 19/19] osstest: use -DWITHOUT_AUTO_OBJ > with FreeBSD release targets"): > > Due to a recent FreeBSD change the default output directory of the release > > targets is changed to the object directory

Re: [Xen-devel] [RFC PATCH 16/31] arm: add SMC wrapper that is compatible with SMCCC

2017-12-05 Thread Oleksandr Tyshchenko
On Tue, Dec 5, 2017 at 7:08 PM, Volodymyr Babchuk wrote: > Hi Julien, Hi Julien, Volodymyr. > > On 05.12.17 16:58, Julien Grall wrote: >> >> Hi Oleksandr, >> >> On 09/11/17 17:10, Oleksandr Tyshchenko wrote: >>> >>> From: Volodymyr Babchuk

Re: [Xen-devel] [OSSTEST PATCH] linux-arm-xen: Get from shared arm/linux.git xenbits tree

2017-12-05 Thread Ian Jackson
Stefano Stabellini writes ("Re: [OSSTEST PATCH] linux-arm-xen: Get from shared arm/linux.git xenbits tree"): > On Tue, 5 Dec 2017, Julien Grall wrote: > > Acked-by: Julien Grall > > Acked-by: Stefano Stabellini Thanks. This queued for my next

Re: [Xen-devel] [PATCH v6 00/16] xen: common: rbtree: ported updates from Linux tree

2017-12-05 Thread Praveen Kumar
Hi All, Can you please provide your comments over the changes shared. Thanks in advance. Regards, ~Praveen. On Tue, Nov 21, 2017 at 8:49 PM, Praveen Kumar wrote: > Hi All, > > The patch imports the changes and updates of the rbtree implementaiton > from Linux tree.

Re: [Xen-devel] [for-4.10] Re: [PATCH v2] tools/libxl: mark special pages as reserved in e820 map for PVH

2017-12-05 Thread Juergen Gross
On 05/12/17 16:23, Julien Grall wrote: > Hi Juergen, > > On 04/12/17 15:49, Juergen Gross wrote: >> On 21/11/17 12:06, Juergen Gross wrote: >>> The "special pages" for PVH guests include the frames for console and >>> Xenstore ring buffers. Those have to be marked as "Reserved" in the >>> guest's

Re: [Xen-devel] [OSSTEST PATCH] linux-arm-xen: Get from shared arm/linux.git xenbits tree

2017-12-05 Thread Julien Grall
Hi Ian, On 05/12/17 18:41, Ian Jackson wrote: Stefano Stabellini writes ("Re: [OSSTEST PATCH] linux-arm-xen: Get from shared arm/linux.git xenbits tree"): On Tue, 5 Dec 2017, Julien Grall wrote: Acked-by: Julien Grall Acked-by: Stefano Stabellini

Re: [Xen-devel] [PATCH v17 19/19] osstest: use -DWITHOUT_AUTO_OBJ with FreeBSD release targets

2017-12-05 Thread Ian Jackson
Roger Pau Monné writes ("Re: [PATCH v17 19/19] osstest: use -DWITHOUT_AUTO_OBJ with FreeBSD release targets"): > On Tue, Dec 05, 2017 at 03:09:47PM +, Ian Jackson wrote: > > Is that why you're not just updating osstest to expect the new > > behaviour ? > > Yes, I think it's more sensible to

Re: [Xen-devel] [OSSTEST PATCH] linux-arm-xen: Get from shared arm/linux.git xenbits tree

2017-12-05 Thread Stefano Stabellini
On Tue, 5 Dec 2017, Julien Grall wrote: > Hi Ian, > > On 05/12/17 18:28, Ian Jackson wrote: > > This drops the dependency on Stefano's personal git tree. > > The osstest output branch remains unmoved. > > > > CC: Stefano Stabellini > > CC: Julien Grall

Re: [Xen-devel] [PATCH] x86/HVM: don't retain emulated insn cache when exiting back to guest

2017-12-05 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 05 December 2017 16:14 > To: xen-devel > Cc: Julien Grall ; Andrew Cooper > ; Paul Durrant > Subject:

Re: [Xen-devel] [PATCH] x86/HVM: don't retain emulated insn cache when exiting back to guest

2017-12-05 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 05 December 2017 17:19 > To: Paul Durrant > Cc: Julien Grall ; Andrew Cooper > ; xen-devel de...@lists.xenproject.org> > Subject: RE:

Re: [Xen-devel] [OSSTEST PATCH] linux-arm-xen: Get from shared arm/linux.git xenbits tree

2017-12-05 Thread Julien Grall
Hi Ian, On 05/12/17 18:28, Ian Jackson wrote: This drops the dependency on Stefano's personal git tree. The osstest output branch remains unmoved. CC: Stefano Stabellini CC: Julien Grall Signed-off-by: Ian Jackson

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

2017-12-05 Thread osstest service owner
flight 116840 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/116840/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm 7 xen-boot fail REGR. vs. 115643

Re: [Xen-devel] linux-arm-xen branch, commit access, etc.

2017-12-05 Thread Julien Grall
Hi Ian, On 05/12/17 17:42, Ian Jackson wrote: Stefano Stabellini writes ("Re: [Xen-devel] linux-arm-xen branch, commit access, etc."): let's create a /arm/linux.git tree on xenbits where both Julien and I can push. The idea is that we'll try to use vanilla kernel releases but we'll keep it

Re: [Xen-devel] [RFC] WIP: optee: add OP-TEE mediator

2017-12-05 Thread Stuart Yoder
There are limit on pCPUs, though. But this is not a problem, because XEN scheduler will decide which guest will access OP-TEE right now. OP-TEE don't have own scheduler at all, by the way. It is scheduled by normal world. Do you mind to give a bit more explanation here? Do you plan to add

[Xen-devel] [OSSTEST PATCH] linux-arm-xen: Get from shared arm/linux.git xenbits tree

2017-12-05 Thread Ian Jackson
This drops the dependency on Stefano's personal git tree. The osstest output branch remains unmoved. CC: Stefano Stabellini CC: Julien Grall Signed-off-by: Ian Jackson --- ap-common | 4 ++-- 1 file changed, 2

Re: [Xen-devel] [RFC PATCH 16/31] arm: add SMC wrapper that is compatible with SMCCC

2017-12-05 Thread Julien Grall
On 05/12/17 17:08, Volodymyr Babchuk wrote: Hi Julien, On 05.12.17 16:58, Julien Grall wrote: Hi Oleksandr, On 09/11/17 17:10, Oleksandr Tyshchenko wrote: From: Volodymyr Babchuk Existing SMC wrapper call_smc() allows only 4 parameters and returns only one

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

2017-12-05 Thread osstest service owner
flight 116858 seabios real [real] http://logs.test-lab.xenproject.org/osstest/logs/116858/ 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] [RFC] xen/arm: Handling cache maintenance instructions by set/way

2017-12-05 Thread Julien Grall
Hi all, Even though it is an Arm failure, I have CCed x86 folks to get feedback on the approach. I have a WIP branch I could share if that interest people. Few months ago, we noticed an heisenbug on jobs run by osstest on the cubietrucks (see [1]). From the log, we figured out that the guest

[Xen-devel] [PATCH] x86/HVM: don't retain emulated insn cache when exiting back to guest

2017-12-05 Thread Jan Beulich
vio->mmio_retry is being set when a repeated string insn is being split up. In that case we'll exit to the guest, expecting immediate re-entry. Interruptions, however, may be serviced by the guest before re-entry from the repeated string insn. Any emulation needed in the course of handling the

Re: [Xen-devel] [PATCH 2/2] xen/pvcalls: Fix a check in pvcalls_front_remove()

2017-12-05 Thread Juergen Gross
On 05/12/17 15:38, Dan Carpenter wrote: > bedata->ref can't be less than zero because it's unsigned. This affects > certain error paths in probe. We first set ->ref = -1 and then we set > it to a valid value later. > > Fixes: 219681909913 ("xen/pvcalls: connect to the backend") > Signed-off-by:

Re: [Xen-devel] [PATCH] xen/pvcalls: check for xenbus_read() errors

2017-12-05 Thread Juergen Gross
On 05/12/17 15:38, Dan Carpenter wrote: > Smatch complains that "len" is uninitialized if xenbus_read() fails so > let's add some error handling. > > Signed-off-by: Dan Carpenter Reviewed-by: Juergen Gross Juergen

[Xen-devel] [OSSTEST PATCH] Revert "make-flight etc.: drop arm64"

2017-12-05 Thread Ian Jackson
We have the laxtons back and they seem to be working. This reverts commit 329ef52c9866b2a8608a598836b7d5813de1e979. Signed-off-by: Ian Jackson --- make-flight | 2 +- make-hosts-flight | 2 +- mfi-common| 6 +++--- 3 files changed, 5 insertions(+), 5

Re: [Xen-devel] [PATCH for-next 04/16] xen/arm: Extend copy_to_guest to support zeroing guest VA and use it

2017-12-05 Thread Stefano Stabellini
On Thu, 23 Nov 2017, Julien Grall wrote: > The function copy_to_guest can easily be extended to support zeroing > guest VA. To avoid using a new bit, it is considered that a NULL buffer > (i.e buf == NULL) means the guest memory will be zeroed. > > Lastly, reimplement raw_clear_guest using

Re: [Xen-devel] [PATCH for-next 05/16] xen/arm: guest_copy: Extend the prototype to pass the vCPU

2017-12-05 Thread Stefano Stabellini
On Thu, 23 Nov 2017, Julien Grall wrote: > Currently, guest_copy assumes the copy will only be done for the current > vCPU. A follow-up patch will require to use a different vCPU. > > So extend the prototype to pass the vCPU. > > Signed-off-by: Julien Grall

[Xen-devel] [linux-4.9 test] 116861: regressions - FAIL

2017-12-05 Thread osstest service owner
flight 116861 linux-4.9 real [real] http://logs.test-lab.xenproject.org/osstest/logs/116861/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs. 116754

Re: [Xen-devel] [PATCH for-next 07/16] xen/arm: Introduce copy_to_guest_phys_flush_dcache

2017-12-05 Thread Stefano Stabellini
On Thu, 23 Nov 2017, Julien Grall wrote: > Hi Andrew, > > On 23/11/17 18:49, Andrew Cooper wrote: > > On 23/11/17 18:32, Julien Grall wrote: > > > This new function will be used in a follow-up patch to copy data to the > > > guest > > > using the IPA (aka guest physical address) and then clean

Re: [Xen-devel] [RFC] xen/arm: Handling cache maintenance instructions by set/way

2017-12-05 Thread Julien Grall
On 05/12/2017 22:35, Stefano Stabellini wrote: On Tue, 5 Dec 2017, Julien Grall wrote: Hi all, Even though it is an Arm failure, I have CCed x86 folks to get feedback on the approach. I have a WIP branch I could share if that interest people. Few months ago, we noticed an heisenbug on jobs

[Xen-devel] [linux-3.18 test] 116862: regressions - trouble: blocked/broken/fail/pass

2017-12-05 Thread osstest service owner
flight 116862 linux-3.18 real [real] http://logs.test-lab.xenproject.org/osstest/logs/116862/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-xsm broken build-armhf-xsm 5

Re: [Xen-devel] [RFC] xen/arm: Handling cache maintenance instructions by set/way

2017-12-05 Thread Stefano Stabellini
On Tue, 5 Dec 2017, Julien Grall wrote: > Hi all, > > Even though it is an Arm failure, I have CCed x86 folks to get feedback on the > approach. I have a WIP branch I could share if that interest people. > > Few months ago, we noticed an heisenbug on jobs run by osstest on the > cubietrucks (see

Re: [Xen-devel] [PATCH for-next 06/16] xen/arm: Extend copy_to_guest to support copying from/to guest physical address

2017-12-05 Thread Stefano Stabellini
On Tue, 5 Dec 2017, Stefano Stabellini wrote: > On Thu, 23 Nov 2017, Julien Grall wrote: > > The only differences between copy_to_guest and access_guest_memory_by_ipa > > are: > > - The latter does not support copying data crossing page boundary > > - The former is copying from/to guest

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

2017-12-05 Thread osstest service owner
flight 116857 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/116857/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-win7-amd64 10 windows-install fail REGR. vs. 116744 Tests which are

Re: [Xen-devel] [PATCH for-next 03/16] xen/arm: Extend copy_to_guest to support copying from guest VA and use it

2017-12-05 Thread Stefano Stabellini
On Thu, 23 Nov 2017, Julien Grall wrote: > The only differences between copy_to_guest (formerly called > raw_copy_to_guest_helper) and raw_copy_from_guest is: > - The direction of the memcpy > - The permission use for translating the address > > Extend copy_to_guest to support copying

Re: [Xen-devel] [PATCH for-next 06/16] xen/arm: Extend copy_to_guest to support copying from/to guest physical address

2017-12-05 Thread Stefano Stabellini
On Thu, 23 Nov 2017, Julien Grall wrote: > The only differences between copy_to_guest and access_guest_memory_by_ipa are: > - The latter does not support copying data crossing page boundary > - The former is copying from/to guest VA whilst the latter from > guest PA > > copy_to_guest

Re: [Xen-devel] [PATCH for-next 01/16] xen/arm: raw_copy_to_guest_helper: Rename flush_dcache to flags

2017-12-05 Thread Stefano Stabellini
On Thu, 23 Nov 2017, Julien Grall wrote: > In a follow-up patch, it will be necessary to pass more flags to the > function. > > Rename flush_dcache to flags and introduce a define to tell whether the > cache needs to be flushed after the copy. > > Signed-off-by: Julien Grall

Re: [Xen-devel] [PATCH for-next 08/16] xen/arm: kernel: Rework kernel_zimage_load to use the generic copy helper

2017-12-05 Thread Stefano Stabellini
On Tue, 5 Dec 2017, Stefano Stabellini wrote: > On Thu, 23 Nov 2017, Julien Grall wrote: > > The function kernel_zimage is dealing with IPA but uses gvirt_to_maddr to > > do the translation. This is currently working fine because the stage-1 MMU > > is disabled. > > > > Furthermore, the function

[Xen-devel] [PATCH] xen-netback: Fix logging message with spurious period after newline

2017-12-05 Thread Joe Perches
Using a period after a newline causes bad output. Signed-off-by: Joe Perches --- drivers/net/xen-netback/interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/xen-netback/interface.c b/drivers/net/xen-netback/interface.c index

Re: [Xen-devel] [PATCH for-next 09/16] xen/arm: domain_build: Rework initrd_load to use the generic copy helper

2017-12-05 Thread Stefano Stabellini
On Thu, 23 Nov 2017, Julien Grall wrote: > The function initrd_load is dealing with IPA but uses gvirt_to_maddr to > do the translation. This is currently working fine because the stage-1 MMU > is disabled. > > Furthermore, the function is implementing its own copy to guest resulting > in code

[Xen-devel] [linux-linus bisection] complete test-amd64-i386-xl-raw

2017-12-05 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job test-amd64-i386-xl-raw testid xen-boot 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 Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.git

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

2017-12-05 Thread osstest service owner
flight 116868 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/116868/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs. 116683 Tests

[Xen-devel] [xen-unstable baseline-only test] 72519: regressions - FAIL

2017-12-05 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 72519 xen-unstable real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72519/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt-raw 15

Re: [Xen-devel] [RFC PATCH 06/31] cpufreq: make cpufreq driver more generalizable

2017-12-05 Thread Jan Beulich
>>> On 05.12.17 at 21:48, wrote: > You are right. We need to define a new struct for internal usage, for > example: > > struct xen_processor_performance_internal { > uint32_t flags; /* flag for Px sub info type */ > uint32_t platform_limit; /* Platform

Re: [Xen-devel] Ping#3: [PATCH v3] x86/HVM: don't #GP/#SS on wrapping virt->linear translations

2017-12-05 Thread Jan Beulich
>>> On 04.12.17 at 17:39, wrote: > On 04/12/17 10:16, Jan Beulich wrote: > On 25.08.17 at 16:59, wrote: >> On 10.08.17 at 09:19, wrote: >>> On 10.07.17 at 12:39, wrote: > Real hardware wraps silently in most

[Xen-devel] [RFC Patch v4 3/8] xl/acpi: unify the computation of lapic_id

2017-12-05 Thread Chao Gao
There were two places where the lapic_id is computed, one in hvmloader and one in libacpi. Unify them by defining LAPIC_ID in a header file and incluing it in both places. To address compilation issue and make libacpi.h self-contained, include stdint.h in libacpi.h. Signed-off-by: Chao Gao

[Xen-devel] [RFC Patch v4 6/8] hvmload: Add x2apic entry support in the MADT and SRAT build

2017-12-05 Thread Chao Gao
From: Lan Tianyu This patch contains the following changes: 1. add x2apic entry support for ACPI MADT table according to ACPI spec 5.2.12.12 Processor Local x2APIC Structure. 2. add x2apic entry support for ACPI SRAT table according to ACPI spec 5.2.16.3 Processor Local

[Xen-devel] [RFC Patch v4 8/8] x86/hvm: bump the maximum number of vcpus to 512

2017-12-05 Thread Chao Gao
Signed-off-by: Chao Gao --- xen/include/public/hvm/hvm_info_table.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/include/public/hvm/hvm_info_table.h b/xen/include/public/hvm/hvm_info_table.h index 08c252e..6833a4c 100644 ---

[Xen-devel] [RFC Patch v4 0/8] Extend resources to support more vcpus in single VM

2017-12-05 Thread Chao Gao
This series is based on Paul Durrant's "x86: guest resource mapping" (https://lists.xenproject.org/archives/html/xen-devel/2017-11/msg01735.html) and "add vIOMMU support with irq remapping function of virtual VT-d" (https://lists.xenproject.org/archives/html/xen-devel/2017-11/msg01063.html).

[Xen-devel] [RFC Patch v4 1/8] ioreq: remove most 'buf' parameter from static functions

2017-12-05 Thread Chao Gao
It is a preparation to support multiple IOREQ pages. No functional change. Signed-off-by: Chao Gao --- v4: -new --- xen/arch/x86/hvm/ioreq.c | 48 +++- 1 file changed, 23 insertions(+), 25 deletions(-) diff --git

[Xen-devel] Xen 4.10 RC8

2017-12-05 Thread Julien Grall
Hi all, Xen 4.10 RC8 is tagged. You can check that out from xen.git: git://xenbits.xen.org/xen.git 4.10.0-rc8 For your convenience, there is also a tarball at: https://downloads.xenproject.org/release/xen/4.10.0-rc8/xen-4.10.0-rc8.tar.gz And the signature is at:

Re: [Xen-devel] [patch 01/60] x86/entry/64/paravirt: Use paravirt-safe macro to access eflags

2017-12-05 Thread Juergen Gross
On 04/12/17 15:07, Thomas Gleixner wrote: > From: Boris Ostrovsky > > Commit 1d3e53e8624a ("x86/entry/64: Refactor IRQ stacks and make them > NMI-safe") added DEBUG_ENTRY_ASSERT_IRQS_OFF macro that acceses eflags > using 'pushfq' instruction when testing for IF bit.

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

2017-12-05 Thread Andrew Cooper
On 05/12/17 11:16, Jan Beulich wrote: On 05.12.17 at 11:03, wrote: >> On 05/12/2017 09:30, Jan Beulich wrote: >> On 05.12.17 at 09:49, wrote: flight 116832 xen-unstable real [real]

Re: [Xen-devel] [RFC PATCH 13/31] xen/arm: Add driver_data field to struct device

2017-12-05 Thread Oleksandr Tyshchenko
Hi, Julien. On Tue, Dec 5, 2017 at 1:26 PM, Julien Grall wrote: > > > On 09/11/17 17:10, Oleksandr Tyshchenko wrote: >> >> From: Oleksandr Tyshchenko > > > Please explain the rationale behind adding a new field in struct device. Basically

Re: [Xen-devel] [RFC] WIP: optee: add OP-TEE mediator

2017-12-05 Thread Volodymyr Babchuk
Hi Stefano, On 05.12.17 00:06, Stefano Stabellini wrote: On Mon, 4 Dec 2017, Volodymyr Babchuk wrote: = Xen command forwarding = In the code below, it looks like Xen is forwarding everything to OP-TEE. Are there some commands Xen should avoid forwarding? Should we have a whitelist or a

Re: [Xen-devel] [RFC v3 2/4] xen/linux_compat: Add a Linux compat header

2017-12-05 Thread Julien Grall
Hi Sameer, On 05/12/17 03:59, Sameer Goel wrote: For porting files from Linux it is useful to have a Linux API to Xen API mapping header at a common location. This file adds common API functions and other defines that are needed for porting arm SMMU drivers. ---

Re: [Xen-devel] [xen-unstable test] 116832: regressions - FAIL [and 1 more messages]

2017-12-05 Thread Ian Jackson
Jan Beulich writes ("Re: [Xen-devel] [xen-unstable test] 116832: regressions - FAIL [and 1 more messages]"): > Oh, that's pretty unfortunate. I think we'll then need a custom flight > tied to the box that this failure occurred on, to have a way to tell > whether the fix I'm about to prepare has

Re: [Xen-devel] linux-arm-xen branch, commit access, etc.

2017-12-05 Thread Julien Grall
Hi, Ping? This is going to be very helpful for bringing up osstest on Thunder-X. Cheers, On 02/11/17 13:34, Julien Grall wrote: Hi, On 23/10/17 22:33, Stefano Stabellini wrote: On Fri, 20 Oct 2017, Julien Grall wrote:    Julien, do you think we need to keep a special linux tree around

Re: [Xen-devel] [xen-unstable test] 116832: regressions - FAIL [and 1 more messages]

2017-12-05 Thread Andrew Cooper
On 05/12/17 15:31, Jan Beulich wrote: On 05.12.17 at 16:05, wrote: >> Jan Beulich writes ("Re: [Xen-devel] [xen-unstable test] 116832: regressions >> - >> FAIL"): >>> This is a blue screen, recurring, and has first been reported in flight >>> 116779, i.e. was

[Xen-devel] [OSSTEST PATCH] README.dev: Say how to do a commissioning flight for a new architecture

2017-12-05 Thread Ian Jackson
Signed-off-by: Ian Jackson --- README.dev | 7 +++ 1 file changed, 7 insertions(+) diff --git a/README.dev b/README.dev index dc0dafd..37e2953 100644 --- a/README.dev +++ b/README.dev @@ -132,6 +132,13 @@ If that works, a more thorough test: This should show no

Re: [Xen-devel] [PATCH v17 17/19] osstest: remove the loader timeout from the install image

2017-12-05 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v17 17/19] osstest: remove the loader timeout from the install image"): > When the FreeBSD installer is booted on the godello{0/1} boxes it > receives spurious key strokes. This doesn't happen so far when booted > from disk, or with any other boxes. > > In order

Re: [Xen-devel] [PATCH v17 19/19] osstest: use -DWITHOUT_AUTO_OBJ with FreeBSD release targets

2017-12-05 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v17 19/19] osstest: use -DWITHOUT_AUTO_OBJ with FreeBSD release targets"): > Due to a recent FreeBSD change the default output directory of the release > targets is changed to the object directory instead of the source > directory. Use WITHOUT_AUTO_OBJ to restore

[Xen-devel] [for-4.10] Re: [PATCH v2] tools/libxl: mark special pages as reserved in e820 map for PVH

2017-12-05 Thread Julien Grall
Hi Juergen, On 04/12/17 15:49, Juergen Gross wrote: On 21/11/17 12:06, Juergen Gross wrote: The "special pages" for PVH guests include the frames for console and Xenstore ring buffers. Those have to be marked as "Reserved" in the guest's E820 map, as otherwise conflicts might arise later e.g.

Re: [Xen-devel] [RFC PATCH 16/31] arm: add SMC wrapper that is compatible with SMCCC

2017-12-05 Thread Volodymyr Babchuk
Hi Stefano, On Mon, Dec 04, 2017 at 06:30:13PM -0800, Stefano Stabellini wrote: > On Thu, 9 Nov 2017, Oleksandr Tyshchenko wrote: > > From: Volodymyr Babchuk > > > > Existing SMC wrapper call_smc() allows only 4 parameters and > > returns only one value. This is

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

2017-12-05 Thread Ian Jackson
osstest service owner writes ("[xen-unstable test] 116832: regressions - FAIL"): > flight 116832 xen-unstable real [real] > http://logs.test-lab.xenproject.org/osstest/logs/116832/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: >

Re: [Xen-devel] [RFC PATCH 16/31] arm: add SMC wrapper that is compatible with SMCCC

2017-12-05 Thread Julien Grall
Hi Oleksandr, On 09/11/17 17:10, Oleksandr Tyshchenko wrote: From: Volodymyr Babchuk Existing SMC wrapper call_smc() allows only 4 parameters and returns only one value. This is enough for existing use in PSCI code, but TEE mediator will need a call that is fully

Re: [Xen-devel] [xen-unstable test] 116832: regressions - FAIL [and 1 more messages]

2017-12-05 Thread Jan Beulich
>>> On 05.12.17 at 16:05, wrote: > Jan Beulich writes ("Re: [Xen-devel] [xen-unstable test] 116832: regressions > - > FAIL"): >> This is a blue screen, recurring, and has first been reported in flight >> 116779, i.e. was likely introduced in the batch ending in commit

[Xen-devel] [for-4.10] Re: [PATCH v2] tools/libxl: mark special pages as reserved in e820 map for PVH

2017-12-05 Thread Ian Jackson
Julien Grall writes ("[for-4.10] Re: [Xen-devel] [PATCH v2] tools/libxl: mark special pages as reserved in e820 map for PVH"): > I would recommend to tag your patch is 4.10 to help reviewers prioritize > review on your patch. I have done it now. Thanks. Looking at the thread, I would have

Re: [Xen-devel] [PATCH v2] x86/hvm: fix interaction between internal and external emulation

2017-12-05 Thread Jan Beulich
>>> On 28.11.17 at 15:05, wrote: > --- a/xen/arch/x86/hvm/io.c > +++ b/xen/arch/x86/hvm/io.c > @@ -88,7 +88,7 @@ bool hvm_emulate_one_insn(hvm_emulate_validate_t *validate, > const char *descr) > > rc = hvm_emulate_one(); > > -if (

Re: [Xen-devel] [RFC v3 4/4] xen/iommu: smmu-v3: Add Xen specific code to enable the ported driver

2017-12-05 Thread Julien Grall
Hello, On 05/12/17 03:59, Sameer Goel wrote: This driver follows an approach similar to smmu driver. The intent here is to reuse as much Linux code as possible. - Glue code has been introduced in headers to bridge the API calls. - Called Linux functions from the Xen IOMMU function calls. - Xen

Re: [Xen-devel] [PATCH v2] x86/hvm: fix interaction between internal and external emulation

2017-12-05 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xenproject.org] On Behalf > Of Paul Durrant > Sent: 05 December 2017 14:00 > To: 'Jan Beulich' > Cc: xen-devel@lists.xenproject.org > Subject: Re: [Xen-devel] [PATCH v2] x86/hvm: fix interaction

Re: [Xen-devel] [RFC PATCH 09/31] xen/device-tree: Add dt_property_for_each_string macros

2017-12-05 Thread Oleksandr Tyshchenko
Hi, Stefano On Tue, Dec 5, 2017 at 1:24 AM, Stefano Stabellini wrote: > On Thu, 9 Nov 2017, Oleksandr Tyshchenko wrote: >> From: Oleksandr Tyshchenko >> >> This is a port from Linux. > > When you port stuff from Linux you have to retain the