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

2017-11-27 Thread Julien Grall
Hi Sameer, On 22/11/17 02:17, Goel, Sameer wrote: On 11/20/2017 7:25 AM, Julien Grall wrote: On 19/11/17 07:45, Goel, Sameer wrote: On 10/12/2017 10:36 AM, Julien Grall wrote: Please use #if 0 rather than removing the code + comment on top. But I am not sure why you drop the S2 free code

Re: [Xen-devel] [PATCH] arm64: ITS: fix cacheability adjustment

2017-11-28 Thread Julien Grall
.ja...@linaro.org> Manish, can you please test this patch and confirm that it works? Also how does the bug manifest for you? Julien, Stefano: Are there any objections against taking this patch for > 4.10? This was introduced with the ITS emulation. Reviewed-by: Julien Grall <julien.gr..

Re: [Xen-devel] [PATCH] xen/arm: domain_builder: irq sanity check logic fix

2017-11-28 Thread Julien Grall
. Well spotted! Reviewed-by: Julien Grall <julien.gr...@linaro.org> Also, I think it would be good to get the patch merge in Xen 4.10. It is boot code and low risk. So: Release-acked-by: Julien Grall <julien.gr...@linaro.org> Cheers, --- xen/arch/arm/domain_build.c | 5 +++--

Re: [Xen-devel] [PATCH 2/3 v3] xen: Add support for initializing 16550 UART using ACPI

2017-11-24 Thread Julien Grall
e Moonshot gives you in this table? Is it "7"? What is the point to use the baud rate from the table? It should have been configured by the firmware and there are no point for the driver to reconfigure it. It will likely make it worst as AFAICT we don't have the clock f

Re: [Xen-devel] [PATCH 1/3 v3] xen: Refactor 16550 UART code

2017-11-24 Thread Julien Grall
gt; CC: Stefano Stabellini <sstabell...@kernel.org> CC: Tim Deegan <t...@xen.org> CC: Wei Liu <wei.l...@citrix.com> CC: Julien Grall <julien.gr...@arm.com> xen/drivers/char/ns16550.c | 53 ++ 1 file changed, 40 insertions(+), 13

Re: [Xen-devel] [PATCH 2/3 v3] xen: Add support for initializing 16550 UART using ACPI

2017-11-24 Thread Julien Grall
> CC: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> CC: Stefano Stabellini <sstabell...@kernel.org> CC: Tim Deegan <t...@xen.org> CC: Wei Liu <wei.l...@citrix.com> CC: Julien Grall <julien.gr...@arm.com> xen/drivers/char/ns16550.c | 67 ++

Re: [Xen-devel] [PATCH v3 09/17] SUPPORT.md: Add ARM-specific virtual hardware

2017-11-23 Thread Julien Grall
ix.com> CC: Andrew Cooper <andrew.coop...@citrix.com> CC: Jan Beulich <jbeul...@suse.com> CC: Stefano Stabellini <sstabell...@kernel.org> CC: Konrad Wilk <konrad.w...@oracle.com> CC: Tim Deegan <t...@xen.org> CC: Julien Grall <julien.gr...@arm.com> --- SUPPORT.md | 2

[Xen-devel] Xen 4.10 RC6

2017-11-23 Thread Julien Grall
://downloads.xenproject.org/release/xen/4.10.0-rc6/xen-4.10.0-rc6.tar.gz.sig Please send bug reports and test reports to xen-devel@lists.xenproject.org. When sending bug reports, please CC relevant maintainers and me (julien.gr...@linaro.org). Thanks, -- Julien Grall

Re: [Xen-devel] [PATCH] Use non-debug build for Xen 4.10

2017-11-29 Thread Julien Grall
Hi Jan, On 11/29/2017 01:29 PM, Jan Beulich wrote: On 29.11.17 at 13:09, <julien.gr...@linaro.org> wrote: Modify Config.mk and Kconfig.debug to disable debug by default in preparation for late RCs and eventual release. Signed-off-by: Julien Grall <julien.gr...@linaro.org> ---

Re: [Xen-devel] [PATCH] xen/arm: domain_builder: irq sanity check logic fix

2017-11-29 Thread Julien Grall
On 11/28/2017 07:03 PM, Julien Grall wrote: Hi Stewart, Hi, On 11/28/2017 02:42 PM, Stewart Hildebrand wrote: It's not possible for an irq to be both below 16 and greater/equal than 32. Also fix the reference to linux documentation while we're at it. Signed-off-by: Stewart Hildebrand

Re: [Xen-devel] [PATCH] migration, xen: Fix block image lock issue on live migration

2017-11-29 Thread Julien Grall
+ Stefano On 11/27/2017 03:00 PM, Anthony PERARD wrote: Hi Julien, Hi Anthony, Can I get a release-ack for this patch? This fix local live migration of HVM guest when the disk backend is qdisk. osstest doesn't report a regression because the kernel or the glibc is just a bit too old.

[Xen-devel] [PATCH] Use non-debug build for Xen 4.10

2017-11-29 Thread Julien Grall
Modify Config.mk and Kconfig.debug to disable debug by default in preparation for late RCs and eventual release. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- I would like this to get included before branching. So we can cut the RC right after branching. --- tools/Rules.mk

[Xen-devel] [PATCH] arm64/xen: Add missing #address-cells and #size-cells properties

2017-11-29 Thread Julien Grall
are never set by GRUB and will result to later failure when the device-tree is generated by GRUB or contain different values. To prevent this failure, create the both properties in the chosen nodes. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- grub-core/loader/arm64

[Xen-devel] [PATCH] xen/arm: Surround HSR_SYSREG macro value with ()

2017-11-29 Thread Julien Grall
The value of the macro HCR_SYSREG is not surrounded by (). This means the behavior may change depend on how it is used. Thanksfully recent GCC will issue a warning for that. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- I am not aware of any "bad" usage today in Xen

Re: [Xen-devel] [PATCH] arm64/xen: Add missing #address-cells and #size-cells properties

2017-11-30 Thread Julien Grall
On 30 November 2017 at 21:22, Daniel Kiper <daniel.ki...@oracle.com> wrote: > On Thu, Nov 30, 2017 at 01:22:37PM +0000, Julien Grall wrote: >> Hi Daniel, >> >> On 30/11/17 13:06, Daniel Kiper wrote: >> >On Wed, Nov 29, 2017 at 05:08:12PM +, Julien Grall

Re: [Xen-devel] [PATCH 3/3 v3] xen: Fix 16550 UART console for HP Moonshot (Aarch64) platform

2017-11-27 Thread Julien Grall
for both.) I believe the problem here is the firmware table does not describe correctly the hardware. I have CCed Graeme which might be able to confirm. Cheers, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.

Re: [Xen-devel] [BUG] win2008 guest cannot get ip through sriov

2017-11-27 Thread Julien Grall
-acked-by: Julien Grall <julien.gr...@linaro.org> Note that I think we would need to update xen also to point to the commit with this backport included. Confirmed Xen 4.10 RC5 cleans away this issue. Thank you for the testing! Cheers, -- Julien

[Xen-devel] [PATCH for-next 02/16] xen/arm: raw_copy_to_guest_helper: Rework the prototype and rename it

2017-11-23 Thread Julien Grall
. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- xen/arch/arm/guestcopy.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/xen/arch/arm/guestcopy.c b/xen/arch/arm/guestcopy.c index 2620e659b4..d1cfbe922c 100644 --- a/xen/arch/arm/guestcopy.c +++

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

2017-11-23 Thread Julien Grall
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 <julien.gr...@linaro.org> --- xen/arch/arm/guestcopy.c | 8 +

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

2017-11-23 Thread Julien Grall
the logic in page-tables (such support for Populate On Demand). The new copy_to_guest_phys_flush_dcache could be used here by temporarily mapping the full initrd in the virtual space. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- xen/arch/arm/domain_build.

[Xen-devel] [PATCH for-next 10/16] xen/arm: domain_build: Use copy_to_guest_phys_flush_dcache in dtb_load

2017-11-23 Thread Julien Grall
. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- xen/arch/arm/domain_build.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c index 42c2e16ef6..9245753a6b 100644 --- a/xen/arch/arm/domain_build.c

[Xen-devel] [PATCH for-next 12/16] xen/arm: p2m: Introduce p2m_tlb_flush_sync, export it and use it

2017-11-23 Thread Julien Grall
Multiple places in the code requires to flush the TLBs wonly when p2m->need_flush is set. Rather than open-coding it, introduce a new helper p2m_tlb_flush_sync to do it. Note that p2m_tlb_flush_sync is exported as it might be used by other part of Xen. Signed-off-by: Julien Grall <jul

[Xen-devel] [PATCH for-next 11/16] xen/arm: p2m: Rename p2m_flush_tlb and p2m_flush_tlb_sync

2017-11-23 Thread Julien Grall
with. For p2m_flush_tlb_sync, the 'force' was added because the TLBs are flush unconditionally. A follow-up patch will add an helper to flush TLBs only in certain cases. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- xen/arch/arm/p2m.c | 18 +- 1 file changed, 9 insertions

[Xen-devel] [PATCH for-next 13/16] xen/arm: p2m: Fold p2m_tlb_flush into p2m_force_tlb_flush_sync

2017-11-23 Thread Julien Grall
for flushing and avoid misusage in the P2M code. So update p2m_alloc_table to use p2m_force_tlb_flush_sync and fold p2m_tlb_flush in p2m_force_tlb_flush_sync. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- xen/arch/arm/p2m.c | 24 +++- 1 file changed, 11 insertions(

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

2017-11-23 Thread Julien Grall
-by: Julien Grall <julien.gr...@linaro.org> --- xen/arch/arm/guestcopy.c | 86 ++-- 1 file changed, 39 insertions(+), 47 deletions(-) diff --git a/xen/arch/arm/guestcopy.c b/xen/arch/arm/guestcopy.c index 487f5ab82d..be53bee559 100644 --- a/xen/ar

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

2017-11-23 Thread Julien Grall
the direction of the copy. Lastly, reimplement raw_copy_from_guest using copy_to_guest. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- xen/arch/arm/guestcopy.c | 46 +- 1 file changed, 13 insertions(+), 33 deletions(-) diff --git a/xe

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

2017-11-23 Thread Julien Grall
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 the cache. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- xen/arch/arm/guestcopy.c | 10 ++ xen/include/asm-arm/guest_access.

[Xen-devel] [PATCH for-next 00/16] xen/arm: Stage-2 handling cleanup

2017-11-23 Thread Julien Grall
Hi all, This patch series is a collection of cleanup around stage-2 handling. They are consolidating different pieces of the hypervisor. This will make easier to maintain and update stage-2 change in the future. Cheers, Julien Grall (16): xen/arm: raw_copy_to_guest_helper: Rename flush_dcache

[Xen-devel] [PATCH for-next 15/16] xen/arm: traps: Move the definition of mmio_info_t in try_handle_mmio

2017-11-23 Thread Julien Grall
mmio_info_t is currently filled by do_trap_data_guest_abort but only important when emulation an MMIO region. A follow-up patch will merge stage-2 prefetch abort and stage-2 data abort in a single helper. To prepare that, mmio_info_t is now filled by try_handle_mmio. Signed-off-by: Julien Grall

[Xen-devel] [PATCH for-next 16/16] xen/arm: traps: Merge do_trap_instr_abort_guest and do_trap_data_abort_guest

2017-11-23 Thread Julien Grall
to maintain stage-2 abort handling. So consolidate the two helpers in a new helper do_trap_stage2_abort. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- xen/arch/arm/traps.c | 133 --- 1 file changed, 41 insertions(+), 92 deletions(-)

[Xen-devel] [PATCH for-next 14/16] xen/arm: traps: Remove the field gva from mmio_info_t

2017-11-23 Thread Julien Grall
mmio_info_t is used to gather information in order do emulation a region. Guest virtual address is unlikely to be a useful information and not currently used. So remove the field gva from mmio_info_t and replace by a local variable. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --

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

2017-11-23 Thread Julien Grall
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 copy_to_guest. Signed-off-by: Julien Grall <julien

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

2017-11-23 Thread Julien Grall
the logic in page-tables (such support for Populate On Demand). The new copy_to_guest_phys_flush_dcache could be used here by temporarily mapping the full kernel in the virtual space. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- xen/arch/arm/domain_build.c | 1 + xen/arch/arm/ke

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

2017-11-23 Thread Julien Grall
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 the cache. Signed-off-by: Julien Grall <julien.gr...@linaro.

Re: [Xen-devel] [PATCH] arm64/xen: Add missing #address-cells and #size-cells properties

2017-11-30 Thread Julien Grall
Hi Daniel, On 30/11/17 13:06, Daniel Kiper wrote: On Wed, Nov 29, 2017 at 05:08:12PM +, Julien Grall wrote: The properties #address-cells and #size-cells are used to know the number of cells for ranges provided by "regs". If they don't exist, the value are resp. 2 and 1. Curre

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

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

2017-12-05 Thread Julien Grall
ni <sstabell...@kernel.org> CC: Julien Grall <julien.gr...@linaro.org> --- xen/include/asm-arm/device.h | 1 + 1 file changed, 1 insertion(+) diff --git a/xen/include/asm-arm/device.h b/xen/include/asm-arm/device.h index 6734ae8..3e2f34a 100644 --- a/xen/include/asm-arm/device.h +++ b/xen/

[Xen-devel] Backport request for Arm

2017-12-18 Thread Julien Grall
f73bfc77 xen/arm: fix smpboot barriers -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

[Xen-devel] Xen 4.10.0 is released

2017-12-14 Thread Julien Grall
without all the awesome contributions from around the globe. Regards, Julien Grall (on behalf of the Xen Project Hypervisor team) ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] Xen release cycle revisited

2017-12-14 Thread Julien Grall
On 14/12/17 11:38, Juergen Gross wrote: On 14/12/17 12:28, Julien Grall wrote: On 14/12/17 07:56, Juergen Gross wrote: Hi all, Hi Juergen, I would recommend to CC committers on that thread, so your thread don't get lost in the xen-devel meanders :). with 4.10 more or less finished

[Xen-devel] [v2 16/16] xen/arm: traps: Merge do_trap_instr_abort_guest and do_trap_data_abort_guest

2017-12-12 Thread Julien Grall
to maintain stage-2 abort handling. So consolidate the two helpers in a new helper do_trap_stage2_abort. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- Changes in v2 - Fix the way to compute npfec.write_access --- xen/arch/arm/traps.c

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

2017-12-12 Thread Julien Grall
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 copy_to_guest. Signed-off-by: Julien Grall <julien

[Xen-devel] [v2 15/16] xen/arm: traps: Move the definition of mmio_info_t in try_handle_mmio

2017-12-12 Thread Julien Grall
mmio_info_t is currently filled by do_trap_data_guest_abort but only important when emulation an MMIO region. A follow-up patch will merge stage-2 prefetch abort and stage-2 data abort in a single helper. To prepare that, mmio_info_t is now filled by try_handle_mmio. Signed-off-by: Julien Grall

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

2017-12-12 Thread Julien Grall
the logic in page-tables (such support for Populate On Demand). The new copy_to_guest_phys_flush_dcache could be used here by temporarily mapping the full initrd in the virtual space. Signed-off-by: Julien Grall <julien.gr...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@k

[Xen-devel] [v2 02/16] xen/arm: raw_copy_to_guest_helper: Rework the prototype and rename it

2017-12-12 Thread Julien Grall
. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- Changes in v2: - Use vaddr_t - Use uint64_t for addr in copy_guest - Add a BUILD_BUG_ON to make sure vaddr_t fit in uint64_t. --- xen/arch/arm/guestcopy.c | 20 +++- 1 file changed, 11 inse

[Xen-devel] [v2 14/16] xen/arm: traps: Remove the field gva from mmio_info_t

2017-12-12 Thread Julien Grall
mmio_info_t is used to gather information in order do emulation of a region. Guest virtual address is unlikely to be a useful information and not currently used. So remove the field gva from mmio_info_t and replace by a local variable. Signed-off-by: Julien Grall <julien.gr...@linaro.

[Xen-devel] [v2 00/16] xen/arm: Stage-2 handling cleanup

2017-12-12 Thread Julien Grall
Hi all, This patch series is a collection of cleanup around stage-2 handling. They are consolidating different pieces of the hypervisor. This will make easier to maintain and update stage-2 change in the future. For all the changes see in each patch. Cheers, Julien Grall (16): xen/arm

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

2017-12-12 Thread Julien Grall
-by: Julien Grall <julien.gr...@linaro.org> --- Changes in v2: - Rework the patch after the interface changes in the previous patch. - Use uint64_t rather than paddr_t in translate_get_page - Add a BUILD_BUG_ON to check whether paddr_t fits in uint64_t --- xen/ar

[Xen-devel] [v2 07/16] xen/arm: Introduce copy_to_guest_phys_flush_dcache

2017-12-12 Thread Julien Grall
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 the cache. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- Changes in v2: - Use the new interface --- xen/arch/arm/guest

[Xen-devel] [v2 13/16] xen/arm: p2m: Fold p2m_tlb_flush into p2m_force_tlb_flush_sync

2017-12-12 Thread Julien Grall
for flushing and avoid misusage in the P2M code. So update p2m_alloc_table to use p2m_force_tlb_flush_sync and fold p2m_tlb_flush in p2m_force_tlb_flush_sync. Signed-off-by: Julien Grall <julien.gr...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> --- C

[Xen-devel] [v2 11/16] xen/arm: p2m: Rename p2m_flush_tlb and p2m_flush_tlb_sync

2017-12-12 Thread Julien Grall
with. For p2m_flush_tlb_sync, the 'force' was added because the TLBs are flush unconditionally. A follow-up patch will add an helper to flush TLBs only in certain cases. Signed-off-by: Julien Grall <julien.gr...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> --- C

[Xen-devel] [v2 10/16] xen/arm: domain_build: Use copy_to_guest_phys_flush_dcache in dtb_load

2017-12-12 Thread Julien Grall
. Signed-off-by: Julien Grall <julien.gr...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> --- Changes in v2: - Add Stefano's reviewed-by --- xen/arch/arm/domain_build.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/

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

2017-12-12 Thread Julien Grall
the direction of the copy. Lastly, reimplement raw_copy_from_guest using copy_to_guest. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- Changes in v2: - Use vaddr_t --- xen/arch/arm/guestcopy.c | 46 +- 1 file chang

[Xen-devel] [v2 01/16] xen/arm: raw_copy_to_guest_helper: Rename flush_dcache to flags

2017-12-12 Thread Julien Grall
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 <julien.gr...@linaro.org> Reviewed-by: Stefano Stabellini <

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

2017-12-12 Thread Julien Grall
the logic in page-tables (such support for Populate On Demand). The new copy_to_guest_phys_flush_dcache could be used here by temporarily mapping the full kernel in the virtual space. Signed-off-by: Julien Grall <julien.gr...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@k

Re: [Xen-devel] [PATCH] docs/process/xen-release-management: Lesson to learn

2017-12-13 Thread Julien Grall
need to get the tree completely frozen 2-3 weeks before the actual date. Cheers, Evidently our docs makes it insufficiently clear not to do that. CC: Lars Kurth <lars.ku...@citrix.com> CC: Julien Grall <julien.gr...@arm.com> CC: Juergen Gross <jgr...@suse.com> Signed-o

Re: [Xen-devel] Xen release cycle revisited

2017-12-14 Thread Julien Grall
, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] Xen release cycle revisited

2017-12-18 Thread Julien Grall
Hi George, On 18/12/17 14:56, George Dunlap wrote: On Fri, Dec 15, 2017 at 2:54 PM, Juergen Gross <jgr...@suse.com> wrote: On 14/12/17 14:13, Juergen Gross wrote: On 14/12/17 13:43, Julien Grall wrote: On 14/12/17 11:38, Juergen Gross wrote: On 14/12/17 12:28, Julien Grall wrote:

Re: [Xen-devel] Xen release cycle revisited

2017-12-18 Thread Julien Grall
Hi Juergen, On 18/12/17 16:10, Juergen Gross wrote: On 18/12/17 16:57, Julien Grall wrote: Hi George, On 18/12/17 14:56, George Dunlap wrote: On Fri, Dec 15, 2017 at 2:54 PM, Juergen Gross <jgr...@suse.com> wrote: On 14/12/17 14:13, Juergen Gross wrote: On 14/12/17 13:43, Julien

Re: [Xen-devel] Xen release cycle revisited

2017-12-18 Thread Julien Grall
Hi Juergen, On 18/12/2017 18:32, Juergen Gross wrote: On 18/12/17 17:38, Julien Grall wrote: Hi Juergen, On 18/12/17 16:10, Juergen Gross wrote: On 18/12/17 16:57, Julien Grall wrote: Hi George, On 18/12/17 14:56, George Dunlap wrote: On Fri, Dec 15, 2017 at 2:54 PM, Juergen Gross <

Re: [Xen-devel] Memory usage on Aarch64

2017-12-15 Thread Julien Grall
/domain_build.c. Further below, it mentions that each domU has an artificial memory map. But I'm not sure what this looks like in memory. Could someone clarify how this is done? I am not sure to understand what you mean by "what this looks like in memory". Could you clarify? Cheers, -- Ju

Re: [Xen-devel] [OSSTEST PATCH v2 07/19] ts-host-install: don't use the new nic naming scheme

2017-12-15 Thread Julien Grall
On 12/12/17 15:15, Julien Grall wrote: Hi Wei/Ian, Hi, I have tried this series on Arm64 hardware. I am able to boot and install Debian on AMD Seattle (laxton{0,1}). But I don't get network when using Cavium Thunder-X (rochester{0,1}) after reboot. Looking into more details

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

2017-12-15 Thread Julien Grall
On 15/12/2017 22:32, Goel, Sameer wrote: On 12/5/2017 5:31 AM, Julien Grall wrote: 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

[Xen-devel] Please welcome our new release manager

2017-12-14 Thread Julien Grall
ing up. Regards, Julien Grall (on behalf of the Xen Project Hypervisor team) ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] [PATCH for-next 16/16] xen/arm: traps: Merge do_trap_instr_abort_guest and do_trap_data_abort_guest

2017-12-07 Thread Julien Grall
Hi Stefano, On 7 December 2017 at 22:43, Stefano Stabellini <sstabell...@kernel.org> wrote: > On Thu, 23 Nov 2017, Julien Grall wrote: >> @@ -2039,10 +1982,11 @@ static void do_trap_data_abort_guest(struct >> cpu_user_regs *regs, >> case FSC_FLT_PERM: >>

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 <julien.gr...@linaro.org> Acked-by: Stefano Stab

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

2017-12-05 Thread Julien Grall
On 05/12/17 18:59, Ian Jackson wrote: We have the laxtons back and they seem to be working. Finally! This reverts commit 329ef52c9866b2a8608a598836b7d5813de1e979. > Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> FWIW: Acked-by: Julien Grall <julien.gr...@linaro.

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 <sstabell...@kernel.org> CC: Julien Grall <julien.gr...@linaro.org> Signed-off-by: Ian Jackson <ian.jack..

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

2017-12-05 Thread Julien Grall
.git based on the current linux-arm-xen branch. /arm/linux.git will replace linux-arm-xen in OSSTest. Let me know when you've populated it and I'll change osstest. The tree has been populated based on Stefano's tree. Branch linux-arm-xen. Cheers, -- Ju

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 <volodymyr_babc...@epam.com> Existing SMC wrapper call_smc() allows only 4 parameters and returns on

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

2017-12-05 Thread Julien Grall
W are expected to be used. Again, a command line option could be introduced here. Any feedbacks on the approach will be welcomed. Cheers, [1] https://lists.xenproject.org/archives/html/xen-devel/2017-09/msg03191.html -- Julien Grall ___ Xen-deve

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

2017-12-08 Thread Julien Grall
Hi, On 06/12/17 12:27, Julien Grall wrote: On 12/06/2017 01:26 AM, Stefano Stabellini wrote: 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

Re: [Xen-devel] [PATCH] xen/arm: gic-v3: Bail out if gicv3_cpu_init fail

2017-12-08 Thread Julien Grall
Hi Stefano, On 07/12/17 23:05, Stefano Stabellini wrote: On Wed, 6 Dec 2017, Julien Grall wrote: From: Julien Grall <julien.gr...@arm.com> When system registers are not enabled, all the access to them will trap ^ accesses in EL2.

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

2017-12-08 Thread Julien Grall
On 08/12/17 08:03, Tim Deegan wrote: Hi, Hi Tim, Somehow your e-mail was marked as spam by gmail. At 12:58 + on 06 Dec (1512565090), Julien Grall wrote: On 12/06/2017 12:28 PM, George Dunlap wrote: 2. It sounds like rather than using PoD, you could use the "misconfigured p2m

[Xen-devel] linux-xen-arm branch update

2017-12-08 Thread Julien Grall
On 05/12/17 18:42, Julien Grall wrote: 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 <julien.gr...@linaro

Re: [Xen-devel] [PATCH] xen/arm: bootfdt: Use proper default for #address-cells and #size-cells

2017-12-08 Thread Julien Grall
Hi, On 29/11/17 18:12, Stefano Stabellini wrote: On Wed, 29 Nov 2017, Julien Grall wrote: Per the device-tree specific [1], when the property #address-cells and #size-cells are not present, the default value should be resp. 1 and 2. [1] https://www.devicetree.org/downloads/devicetree

Re: [Xen-devel] [PATCH] xen/arm: Surround HSR_SYSREG macro value with ()

2017-12-08 Thread Julien Grall
Hi, Ping? Cheers, On 29/11/17 17:46, Julien Grall wrote: The value of the macro HCR_SYSREG is not surrounded by (). This means the behavior may change depend on how it is used. Thanksfully recent GCC will issue a warning for that. Signed-off-by: Julien Grall <julien.gr...@linaro.

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

2017-12-08 Thread Julien Grall
Hi Stefano, On 07/12/17 23:01, Stefano Stabellini wrote: On Wed, 6 Dec 2017, Julien Grall wrote: Hi Stefano, On 12/06/2017 01:22 AM, Stefano Stabellini wrote: On Thu, 23 Nov 2017, Julien Grall wrote: The only differences between copy_to_guest and access_guest_memory_by_ipa

Re: [Xen-devel] [PATCH v2 02/10] ARM: vGIC: fix nr_irq definition

2017-12-08 Thread Julien Grall
,7 +31,7 @@ struct arch_irq_desc { /* LPIs are always numbered starting at 8192, so 0 is a good invalid case. */ #define INVALID_LPI 0 -#define nr_irqs NR_IRQS +extern unsigned int nr_irqs; #define nr_static_irqs NR_IRQS #define arch_hwdom_irqs(domid) NR_IR

Re: [Xen-devel] Commit moratorium for branching Xen 4.10

2017-12-01 Thread Julien Grall
Hi, On 01/12/17 15:23, Ian Jackson wrote: Julien Grall writes ("Commit moratorium for branching Xen 4.10"): Xen tree is going to branch at RC7. I don't want to branch when master != staging, so please avoid committing new patches to staging now to let master catch up with stagin

Re: [Xen-devel] [PATCH v2 for-4.10] x86: Avoid corruption on migrate for vcpus using CPUID Faulting

2017-12-01 Thread Julien Grall
ome of the earlier requested changes to not be done), as indicated in a late response to v1 Reviewed-by: Jan Beulich <jbeul...@suse.com> I thought that was already clear from the second paragraph. Either way, how about this? Yes, I like this new version better. Thanks. Release-acked-by:

Re: [Xen-devel] [PATCH] migration, xen: Fix block image lock issue on live migration

2017-12-01 Thread Julien Grall
Hi Anthony, On 29/11/17 15:06, Anthony PERARD wrote: On Wed, Nov 29, 2017 at 12:28:39PM +, Julien Grall wrote: + Stefano On 11/27/2017 03:00 PM, Anthony PERARD wrote: Hi Julien, Hi Anthony, Can I get a release-ack for this patch? This fix local live migration of HVM guest when

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

2017-12-01 Thread Julien Grall
Release-acked-by: Julien Grall <julien.gr...@linaro.org> Could this be committed today? Cheers, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] [PATCH v2 01/17] xen: x86: expose SGX to HVM domain in CPU featureset

2017-12-04 Thread Julien Grall
X512DQ, AVX512IFMA, AVX512PF, AVX512ER, AVX512CD, AVX512BW, AVX512VL, AVX512VBMI, AVX512_4VNNIW, AVX512_4FMAPS, AVX512_VPOPCNTDQ], + +# SGX Launch Control depends on SGX +SGX: [SGX_LC], } deep_features = tuple(sorted(deps.keys()))

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

2017-12-04 Thread Julien Grall
+return false; + +cmd_mfn = lookup_guest_ram_addr(cmd_gaddr); +if ( mfn_eq(cmd_mfn, INVALID_MFN) ) + return false; + +ctx->guest_arg = map_domain_page(cmd_mfn); +if ( !ctx->guest_arg ) +return false; + +ctx->xen_arg = alloc_xenheap_page(); +if ( !ctx->xen_arg ) +return false; + +memcpy(ctx->xen_arg, ctx->guest_arg, OPTEE_MSG_NONCONTIG_PAGE_SIZE); Have a look a guest copy helpers. Cheers, [1] https://lists.xenproject.org/archives/html/xen-devel/2017-11/msg01481.html -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

[Xen-devel] Xen 4.10 RC7

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

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

2017-12-07 Thread Julien Grall
hat region on a guest access. Getting back to the hypercall case, I am still trying to figure out if we need to clean & invalidate the buffer used when the guest entry is "misconfigured". I can't convince myself why this would not be necessary. I need

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

Re: [Xen-devel] [RFC] arm: Handle SiP Service Calls

2017-12-07 Thread Julien Grall
CCC. How do you recognize that on yours? Now the next question, is it safe to let the hardware domain to access all SiP service? Can you explain the kind of service present in SiP for your platform? Cheers, -- Julien Grall ___ Xen-devel mailing list Xen-

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

2017-12-07 Thread Julien Grall
in virtualization context. The difficulty is clearly spell out in the Arm Arm. So the main goal here is to workaround those software. On 06/12/17 17:49, George Dunlap wrote: On 12/06/2017 12:58 PM, Julien Grall wrote: Hi George, On 12/06/2017 12:28 PM, George Dunlap wrote: On 12/05/2017 06

[Xen-devel] [PATCH] xen/arm: Remove unused fixmap slots

2017-12-07 Thread Julien Grall
There are quite a few fixmap slots that have not been used for a while. Remove them. Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- xen/include/asm-arm/config.h | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/xen/include/asm-arm/config.h b/xen/inclu

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

2017-12-07 Thread Julien Grall
Hi Jan, On 07/12/17 15:45, Jan Beulich wrote: On 07.12.17 at 15:53, <marc.zyng...@arm.com> wrote: On 07/12/17 13:52, Julien Grall wrote: There is exactly one case where set/way makes sense, and that's when you're the only CPU left in the system, your MMU is off, and you're about to g

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

2017-12-11 Thread Julien Grall
Hi, On 12/10/2017 03:22 PM, Tim Deegan wrote: At 14:38 + on 08 Dec (1512743913), Julien Grall wrote: On 08/12/17 08:03, Tim Deegan wrote: +1 for avoiding the full majesty of PoD if you don't need it. It should be possible to do something like the misconfigured-entry bit trick

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

2017-12-11 Thread Julien Grall
s fine because S/W are not easily virtualizable, I take that as a hint for "All the features may not be available when using S/W in a guest". Cheers, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://li

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

2017-12-11 Thread Julien Grall
u use them, then it is the price to pay. It is better than not been able to boot current kernel or randomly crashing. Cheers, -- Julien Grall, ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

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

2017-12-06 Thread Julien Grall
Hi Andrew, On 12/06/2017 03:08 PM, Andrew Cooper wrote: On 06/12/17 13:04, Julien Grall wrote: Hi Jan, On 12/06/2017 12:58 PM, Jan Beulich wrote: On 06.12.17 at 12:47, <julien.gr...@linaro.org> wrote: On 12/06/2017 11:45 AM, Jan Beulich wrote: On 06.12.17 at 10:47, <julien.gr...@l

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

2017-12-06 Thread Julien Grall
Hi Jan, I guess I have been CCed because you would like this patch is fixing the regression you mentioned on IRC? Cheers, On 12/05/2017 04:13 PM, Jan Beulich wrote: 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

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

2017-12-06 Thread Julien Grall
On 12/05/2017 11:26 PM, Goel, Sameer wrote: On 12/5/2017 7:17 AM, Julien Grall wrote: 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

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

2017-12-06 Thread Julien Grall
4TB. Whether we will support 4TB is not decided, but I think 512MB should be. Cheers, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

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

2017-12-06 Thread Julien Grall
On 11/15/2017 04:46 PM, Julien Grall wrote: > Hi, > > On 11/15/2017 11:29 AM, osstest service owner wrote: >> flight 116178 xen-unstable real [real] >> http://logs.test-lab.xenproject.org/osstest/logs/116178/ >> >> Regressions :-( >> >> Tests which d

  1   2   3   4   5   6   7   8   9   10   >