Re: [Xen-devel] [PATCH 07/11] swiotlb-xen: provide a single page-coherent.h header

On Fri, Aug 16, 2019 at 11:40:43PM +0100, Julien Grall wrote: > I am not sure I agree with this rename. The implementation of the helpers > are very Arm specific as this is assuming Dom0 is 1:1 mapped. > > This was necessary due to the lack of IOMMU on Arm platforms back then. > But this is now a

[Xen-devel] [linux-4.14 test] 140193: regressions - FAIL

flight 140193 linux-4.14 real [real] http://logs.test-lab.xenproject.org/osstest/logs/140193/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-libvirt-xsm 7 xen-boot fail REGR. vs. 139910 test-amd64-i386-free

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

flight 140192 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/140192/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-pvshim 20 guest-start/debian.repeat fail REGR. vs. 139876 Tests which did no

Re: [Xen-devel] [PATCH v6 2/8] xen/arm: make process_memory_node a device_tree_node_func

On Fri, 16 Aug 2019, Julien Grall wrote: > Hi, > > On 16/08/2019 00:36, Stefano Stabellini wrote: > > Change the signature of process_memory_node to match > > device_tree_node_func. Thanks to this change, the next patch will be > > able to use device_tree_for_each_node to call process_memory_node

Re: [Xen-devel] [PATCH v6 1/8] xen/arm: pass node to device_tree_for_each_node

On Fri, 16 Aug 2019, Julien Grall wrote: > Hi, > > On 16/08/2019 00:36, Stefano Stabellini wrote: > > Add a new parameter to device_tree_for_each_node: node, the node to > > start the search from. Passing 0 triggers the old behavior. > > Here you say 0 triggers the old behavior but... > > > > >

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

flight 140214 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/140214/ 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 1

Re: [Xen-devel] [PATCH v6 7/8] xen/arm: don't iomem_permit_access for reserved-memory regions

On Fri, 16 Aug 2019, Julien Grall wrote: > On 16/08/2019 00:36, Stefano Stabellini wrote: > > Don't allow reserved-memory regions to be remapped into any unprivileged > > guests, until reserved-memory regions are properly supported in Xen. For > > now, do not call iomem_permit_access on them, becau

Re: [Xen-devel] [PATCH 07/11] swiotlb-xen: provide a single page-coherent.h header

Hi, On 8/16/19 2:00 PM, Christoph Hellwig wrote: Merge the various page-coherent.h files into a single one that either provides prototypes or stubs depending on the need for cache maintainance. For extra benefits alo include in the file actually implementing the interfaces provided. Signed-of

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

flight 140188 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/140188/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-examine 8 reboot fail REGR. vs. 133580 test-amd64-i386-xl-

Re: [Xen-devel] [PATCH v2 0/4] xen/console: Bug fixes and doc improvement

Hi, On 8/5/19 2:29 PM, Julien Grall wrote: Hi all, This series contains a bunch of bug fixes for the hypercall CONSOLEIO_write and some documentation. Cheers, Julien Grall (4): xen/console: Don't treat NUL character as the end of the buffer xen/console: Rework HYPERCALL_console_io inter

Re: [Xen-devel] [PATCH v2 3/4] xen/public: Document HYPERCALL_console_io()

On 8/16/19 6:42 PM, Julien Grall wrote: Hi, On 08/08/2019 15:03, Jan Beulich wrote: On 05.08.2019 15:29, Julien Grall wrote: Currently, OS developpers will have to look at Xen code in order to know the parameters of an hypercall and how it is meant to work. This is not a trivial task as you m

[Xen-devel] [freebsd-master test] 140195: all pass - PUSHED

flight 140195 freebsd-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/140195/ Perfect :-) All tests in this flight passed as required version targeted for testing: freebsd f39a1bd1426e0e5a462522b7a1de56e8233591e7 baseline version: freebsd 4bc76934c5c

[Xen-devel] More questions about Xen memory layout/usage, access to guest memory

Hi all, I have some follow-up questions about Xen's usage and layout of memory, building on the ones I asked here a few weeks ago (which were quite helpfully answered: see https://lists.xenproject.org/archives/html/xen-devel/2019-07/msg01513.html for reference). For context on why I'm asking t

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

branch xen-unstable xenbranch xen-unstable job test-amd64-i386-xl-xsm 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: ovmf git://xenbits.xen.org/osstest/ovmf.git Tree: qemu g

Re: [Xen-devel] [PATCH v2 3/4] xen/public: Document HYPERCALL_console_io()

Hi, On 08/08/2019 15:03, Jan Beulich wrote: On 05.08.2019 15:29, Julien Grall wrote: Currently, OS developpers will have to look at Xen code in order to know the parameters of an hypercall and how it is meant to work. This is not a trivial task as you may need to have a deep understanding of X

[Xen-devel] [PATCH v6 10/10] introduce a 'passthrough' configuration option to xl.cfg...

...and hence the ability to disable IOMMU mappings, and control EPT sharing. This patch introduces a new 'libxl_passthrough' enumeration into libxl_domain_create_info. The value will be set by xl either when it parses a new 'passthrough' option in xl.cfg, or implicitly if there is passthrough hard

Re: [Xen-devel] [PATCH v6 08/10] remove late (on-demand) construction of IOMMU page tables

On 8/16/19 8:19 PM, Paul Durrant wrote: > Now that there is a per-domain IOMMU enable flag, which should be enabled if > any device is going to be passed through, stop deferring page table > construction until the assignment is done. Also don't tear down the tables > again when the last device is d

[Xen-devel] [PATCH v6 03/10] x86/domain: remove the 'oos_off' flag

The flag is not needed since the domain 'options' can now be tested directly. Signed-off-by: Paul Durrant Reviewed-by: Jan Beulich --- Cc: Tim Deegan Cc: George Dunlap Cc: Andrew Cooper Cc: Wei Liu Cc: "Roger Pau Monné" v3: - Force 'oos_off' to be set for PV guests (to avoid call to is

[Xen-devel] [PATCH v6 09/10] iommu: tidy up iommu_use_hap_pt() and need_iommu_pt_sync() macros

Thes macros really ought to live in the common xen/iommu.h header rather then being distributed amongst architecture specific iommu headers and xen/sched.h. This patch moves them there. NOTE: Disabling 'sharept' in the command line iommu options should really be hard error on ARM (as opposed

[Xen-devel] [PATCH v6 02/10] x86/hvm/domain: remove the 'hap_enabled' flag

The hap_enabled() macro can determine whether the feature is available using the domain 'options'; there is no need for a separate flag. NOTE: Furthermore, by extending sanitiziing of the domain 'options', the macro can be transformed into an inline function and re-located to xen/sched

[Xen-devel] [PATCH v6 00/10] use stashed domain create flags...

...and add per-domain IOMMU control This is a combination of my previously separate series [1] and [2]. [1] https://lists.xenproject.org/archives/html/xen-devel/2019-07/msg02253.html [2] https://lists.xenproject.org/archives/html/xen-devel/2019-07/msg02267.html Paul Durrant (10): make passthro

[Xen-devel] [PATCH v6 05/10] x86/domain: remove the 's3_integrity' flag

The flag is not needed since the domain 'options' can now be tested directly. Signed-off-by: Paul Durrant Reviewed-by: "Roger Pau Monné" Reviewed-by: Jan Beulich --- Cc: Andrew Cooper Cc: Wei Liu v4: - s/TBOOT/CONFIG_TBOOT/g v3: - Also sanitise the flag against CONFIG_TBOOT being set ---

[Xen-devel] [PATCH v6 08/10] remove late (on-demand) construction of IOMMU page tables

Now that there is a per-domain IOMMU enable flag, which should be enabled if any device is going to be passed through, stop deferring page table construction until the assignment is done. Also don't tear down the tables again when the last device is de-assigned; defer that task until domain destruc

[Xen-devel] [PATCH v6 06/10] domain: introduce XEN_DOMCTL_CDF_iommu flag

This patch introduces a common domain creation flag to determine whether the domain is permitted to make use of the IOMMU. Currently the flag is always set (for both dom0 and domU) if the IOMMU is globally enabled (i.e. iommu_enabled == 1). sanitise_domain_config() is modified to reject the flag if

[Xen-devel] [PATCH v6 04/10] domain: remove the 'is_xenstore' flag

This patch introduces a convenience macro, is_xenstore_domain(), which tests the domain 'options' directly and then uses that in place of the 'is_xenstore' flag. Signed-off-by: Paul Durrant Reviewed-by: "Roger Pau Monné" Acked-by: George Dunlap --- Cc: Andrew Cooper Cc: Ian Jackson Cc: Jan Be

[Xen-devel] [PATCH v6 07/10] use is_iommu_enabled() where appropriate...

...rather than testing the global iommu_enabled flag and ops pointer. Now that there is a per-domain flag indicating whether the domain is permitted to use the IOMMU (which determines whether the ops pointer will be set), many tests of the global iommu_enabled flag and ops pointer can be translate

[Xen-devel] [PATCH v6 01/10] make passthrough/pci.c:deassign_device() static

This function is only ever called from within the same source module and really has no business being declared xen/iommu.h. This patch relocates the function ahead of the first called and makes it static. Signed-off-by: Paul Durrant Acked-by: Jan Beulich --- Previously part of series https://l

[Xen-devel] [linux-4.4 test] 140180: regressions - FAIL

flight 140180 linux-4.4 real [real] http://logs.test-lab.xenproject.org/osstest/logs/140180/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-pvshim 20 guest-start/debian.repeat fail in 140072 REGR. vs. 139698 Tests which

Re: [Xen-devel] [PATCH 03/11] xen/arm: pass one less argument to dma_cache_maint

On Fri, Aug 16, 2019 at 02:37:58PM +0100, Robin Murphy wrote: > On 16/08/2019 14:00, Christoph Hellwig wrote: >> Instead of taking apart the dma address in both callers do it inside >> dma_cache_maint itself. >> >> Signed-off-by: Christoph Hellwig >> --- >> arch/arm/xen/mm.c | 10 ++ >>

Re: [Xen-devel] [PATCH] xen/arm: domain_build: Print the correct domain in initrd_load()

On 16/08/2019 17:32, Volodymyr Babchuk wrote: Julien Grall writes: initrd_load() can be called by other domain than dom0. To avoid confusion in the log, print the correct domain. Signed-off-by: Julien Grall Reviewed-by: Volodymyr Babchuk But why didn't you gathered all those patches int

Re: [Xen-devel] [PATCH v2] xen/arm: domain_build: Print the correct domain in dtb_load()

Julien Grall writes: > dtb_load() can be called by other domain than dom0. To avoid confusion > in the log, print the correct domain. > > Signed-off-by: Julien Grall Reviewed-by: Volodymyr Babchuk > > --- > Changes in v2: > - Fix the second print in the function as well. > --- > x

Re: [Xen-devel] [PATCH] xen/arm: domain_build: Print the correct domain in initrd_load()

Julien Grall writes: > initrd_load() can be called by other domain than dom0. To avoid > confusion in the log, print the correct domain. > > Signed-off-by: Julien Grall Reviewed-by: Volodymyr Babchuk But why didn't you gathered all those patches into one patch series? > --- > xen/arch/arm/dom

Re: [Xen-devel] [PATCH] xen/arm: domain_build: Print the correct domain in construct_domain()

Julien Grall writes: > construct_domain() can be called by other domain than dom0. To avoid > confusion in the log, print the correct domain. > > Signed-off-by: Julien Grall Reviewed-by: Volodymyr Babchuk > --- > xen/arch/arm/domain_build.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletio

Re: [Xen-devel] [PATCH] xen/arm: domain_build: Remove redundant check in make_vpl011_uart_node()

Hi Julien, Julien Grall writes: > None of the code since the last check of res modify the value. So the > check can be removed. > > Coverity-ID: 1476824 > Signed-off-by: Julien Grall Reviewed-by: Volodymyr Babchuk > --- > xen/arch/arm/domain_build.c | 3 +-- > 1 file changed, 1 insertion(+),

Re: [Xen-devel] [RFC] Code of Conduct

From: Rich Persaud Date: Friday, 16 August 2019 at 16:49 To: George Dunlap Cc: Lars Kurth , xen-devel , "minios-de...@lists.xenproject.org" , "mirageos-de...@lists.xenproject.org" , "win-pv-de...@lists.xenproject.org" , "committ...@xenproject.org" Subject: Re: [Xen-devel] [RFC] Code of Co

Re: [Xen-devel] [RFC] Code of Conduct

On Aug 16, 2019, at 07:19, George Dunlap wrote: > > On 8/15/19 6:23 PM, Rich Persaud wrote: >>> On Aug 9, 2019, at 13:48, Lars Kurth wrote: >>> >>> Hi all, >> >> Hi Lars, >> >>> >>> Following the discussion we had at the Developer Summit (see >>> https://wiki.xenproject.org/wiki/Design_Sess

Re: [Xen-devel] [livepatch-build-tools part3 v2 2/3] create-diff-object: Extend patchability verification: STN_UNDEF

On 8/8/19 1:48 PM, Pawel Wieczorkiewicz wrote: During verification check if all sections do not contain any entries with undefined symbols (STN_UNDEF). This situation can happen when a section is copied over from its original object to a patched object, but various symbols related to the section

Re: [Xen-devel] [libvirt test] 140186: regressions - FAIL

On 8/16/19 7:01 AM, osstest service owner wrote: > flight 140186 libvirt real [real] > http://logs.test-lab.xenproject.org/osstest/logs/140186/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: > build-amd64-libvirt 6 libvi

Re: [Xen-devel] [livepatch-build-tools part3 v2 3/3] create-diff-object: Strip all undefined entires of known size

On 8/8/19 1:51 PM, Pawel Wieczorkiewicz wrote: The patched ELF object file contains all sections and symbols as resulted from the compilation. However, certain symbols may not be copied over to the resulting object file, due to being unchanged or not included for other reasons. In such situation

Re: [Xen-devel] [livepatch-build-tools part2 v2 5/6] create-diff-object: Add new entries to special sections array array

On 8/16/19 1:06 PM, Wieczorkiewicz, Pawel wrote: On 16. Aug 2019, at 11:40, Ross Lagerwall > wrote: On 8/8/19 1:35 PM, Pawel Wieczorkiewicz wrote: …snip...   * The rela groups in the .fixup section vary in size.  The beginning of each   * .fixup rela gro

Re: [Xen-devel] [PATCH v2] Allow get_maintainer.pl / add_maintainers.pl scripts to be called outside of xen.git

> On 16 Aug 2019, at 14:28, Julien Grall wrote: > > > > On 16/08/2019 13:17, Lars Kurth wrote: >> On 16/08/2019, 11:01, "Julien Grall" wrote: >> From my understanding, any use on mini-os.git & co will be legitimate. >> However, >> we still print the WARNING in those cases. >>

Re: [Xen-devel] [PATCH 03/11] xen/arm: pass one less argument to dma_cache_maint

On 16/08/2019 14:00, Christoph Hellwig wrote: Instead of taking apart the dma address in both callers do it inside dma_cache_maint itself. Signed-off-by: Christoph Hellwig --- arch/arm/xen/mm.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/arch/arm/xen/mm.c b

Re: [Xen-devel] [PATCH v2] Allow get_maintainer.pl / add_maintainers.pl scripts to be called outside of xen.git

On 16/08/2019 13:17, Lars Kurth wrote: On 16/08/2019, 11:01, "Julien Grall" wrote: From my understanding, any use on mini-os.git & co will be legitimate. However, we still print the WARNING in those cases. Usually WARNING means something needs attention. As most of the

Re: [Xen-devel] [PATCH] get_maintainers.pl: Enable running the script on unikraft repos

On 16/08/2019 12:41, Lars Kurth wrote: Hi Julien, Hi Lars, On 16/08/2019, 11:55, "Julien Grall" wrote: Hi Lars, On 16/08/2019 11:42, Lars Kurth wrote: > Unikraft repos follow the same syntax as xen.git with the > following exceptions: > * MAINTAINERS file

Re: [Xen-devel] [PATCH] libxlu: Handle += in config files

On Fri, Aug 16, 2019 at 12:47:07PM +, Wei Liu wrote: > On Tue, Aug 13, 2019 at 05:42:15PM +0100, Anthony PERARD wrote: > > On Tue, Aug 13, 2019 at 04:47:23PM +0100, Andrew Cooper wrote: > > > Error between user and terminal. :) > > > > > > I'd sync'd xl and libxl.so, but not libxlu.so > > > >

[Xen-devel] [libvirt test] 140186: regressions - FAIL

flight 140186 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/140186/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-libvirt 6 libvirt-buildfail REGR. vs. 139829 build-i386-libvirt

[Xen-devel] [PATCH 11/11] arm64: use asm-generic/dma-mapping.h

Now that the Xen special cases are gone nothing worth mentioning is left in the arm64 file, so switch to use the asm-generic version instead. Signed-off-by: Christoph Hellwig --- arch/arm64/include/asm/Kbuild| 1 + arch/arm64/include/asm/dma-mapping.h | 22 -- arch/

[Xen-devel] [PATCH 07/11] swiotlb-xen: provide a single page-coherent.h header

Merge the various page-coherent.h files into a single one that either provides prototypes or stubs depending on the need for cache maintainance. For extra benefits alo include in the file actually implementing the interfaces provided. Signed-off-by: Christoph Hellwig --- arch/arm/include/asm/x

[Xen-devel] [PATCH 05/11] xen: remove the exports for xen_{create, destroy}_contiguous_region

These routines are only used by swiotlb-xen, which cannot be modular. Signed-off-by: Christoph Hellwig --- arch/arm/xen/mm.c | 2 -- arch/x86/xen/mmu_pv.c | 2 -- 2 files changed, 4 deletions(-) diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c index 388a45002bad..a59980f1aa54 100644 --- a

[Xen-devel] [PATCH 09/11] swiotlb-xen: simplify cache maintainance

Now that we know we always have the dma-noncoherent.h helpers available if we are on an architecture with support for non-coherent devices, we can just call them directly, and remove the calls to the dma-direct routines, including the fact that we call the dma_direct_map_page routines but ignore th

[Xen-devel] [PATCH 08/11] swiotlb-xen: use the same foreign page check everywhere

xen_dma_map_page uses a different and more complicated check for foreign pages than the other three cache maintainance helpers. Switch it to the simpler pfn_vali method a well. Signed-off-by: Christoph Hellwig --- include/xen/page-coherent.h | 9 ++--- 1 file changed, 2 insertions(+), 7 dele

[Xen-devel] [PATCH 06/11] swiotlb-xen: always use dma-direct helpers to alloc coherent pages

x86 currently calls alloc_pages, but using dma-direct works as well there, with the added benefit of using the CMA pool if available. The biggest advantage is of course to remove a pointless bit of architecture specific code. Signed-off-by: Christoph Hellwig --- arch/x86/include/asm/xen/page-coh

[Xen-devel] [PATCH 01/11] xen/arm: use dma-noncoherent.h calls for xen-swiotlb cache maintainance

Reuse the arm64 code that uses the dma-direct/swiotlb helpers for DMA non-coherent devices. Signed-off-by: Christoph Hellwig --- arch/arm/Kconfig | 4 + arch/arm/include/asm/device.h | 3 - arch/arm/include/asm/xen/page-coherent.h | 93 -

[Xen-devel] [PATCH 10/11] swiotlb-xen: merge xen_unmap_single into xen_swiotlb_unmap_page

No need for a no-op wrapper. Signed-off-by: Christoph Hellwig --- drivers/xen/swiotlb-xen.c | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/drivers/xen/swiotlb-xen.c b/drivers/xen/swiotlb-xen.c index c3c383033ae4..b6b9c4c1b397 100644 --- a/drivers/xen/swiotlb

[Xen-devel] [PATCH 02/11] xen/arm: use dev_is_dma_coherent

Use the dma-noncoherent dev_is_dma_coherent helper instead of the home grown variant. Signed-off-by: Christoph Hellwig --- arch/arm/include/asm/dma-mapping.h | 6 -- arch/arm/xen/mm.c| 12 ++-- arch/arm64/include/asm/dma-mapping.h | 9 - 3 files change

[Xen-devel] [PATCH 03/11] xen/arm: pass one less argument to dma_cache_maint

Instead of taking apart the dma address in both callers do it inside dma_cache_maint itself. Signed-off-by: Christoph Hellwig --- arch/arm/xen/mm.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c index 90574d89d0d4..d9da24fda2f

[Xen-devel] [PATCH 04/11] xen/arm: remove xen_dma_ops

arm and arm64 can just use xen_swiotlb_dma_ops directly like x86, no need for a pointer indirection. Signed-off-by: Christoph Hellwig --- arch/arm/mm/dma-mapping.c| 3 ++- arch/arm/xen/mm.c| 4 arch/arm64/mm/dma-mapping.c | 3 ++- include/xen/arm/hypervisor.h | 2 -- 4 file

[Xen-devel] swiotlb-xen cleanups

Hi Xen maintainers and friends, please take a look at this series that cleans up the parts of swiotlb-xen that deal with non-coherent caches. ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-dev

Re: [Xen-devel] [PATCH livepatch-python 1/1] livepatch: Add python bindings for livepatch operations

> On 16. Aug 2019, at 14:37, Wei Liu wrote: > > On Thu, Aug 15, 2019 at 11:36:46AM +, Pawel Wieczorkiewicz wrote: >> Extend the XC python bindings library to support also all common >> livepatch operations and actions. >> >> …snip... >> >> Signed-off-by: Pawel Wieczorkiewicz >> Reviewe

[Xen-devel] [linux-4.19 test] 140173: regressions - FAIL

flight 140173 linux-4.19 real [real] http://logs.test-lab.xenproject.org/osstest/logs/140173/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-pvops 6 kernel-build fail REGR. vs. 129313 Tests which are fail

Re: [Xen-devel] [PATCH lp-metadata 3/3] livepatch: Add metadata runtime retrieval mechanism

On Fri, Aug 16, 2019 at 12:54:12PM +, Wieczorkiewicz, Pawel wrote: > > > On 16. Aug 2019, at 14:44, Wei Liu wrote: > > > > On Thu, Aug 15, 2019 at 11:27:50AM +, Pawel Wieczorkiewicz wrote: > >> Extend the livepatch list operation to fetch also payloads' metadata. > >> This is achieved by

Re: [Xen-devel] [PATCH lp-metadata 3/3] livepatch: Add metadata runtime retrieval mechanism

> On 16. Aug 2019, at 14:44, Wei Liu wrote: > > On Thu, Aug 15, 2019 at 11:27:50AM +, Pawel Wieczorkiewicz wrote: >> Extend the livepatch list operation to fetch also payloads' metadata. >> This is achieved by extending the sysctl list interface with 2 extra >> guest handles: >> * metadata

Re: [Xen-devel] [livepatch: independ. modules 3/3] python: Add XC binding for Xen build ID

On 16. Aug 2019, at 14:47, Wei Liu mailto:w...@xen.org>> wrote: On Thu, Aug 15, 2019 at 09:44:00AM +, Pawel Wieczorkiewicz wrote: Extend the list of xc() object methods with additional one to display Xen's buildid. The implementation follows the libxl implementation (e.g. max buildid size ass

Re: [Xen-devel] [livepatch: independ. modules 3/3] python: Add XC binding for Xen build ID

On Thu, Aug 15, 2019 at 09:44:00AM +, Pawel Wieczorkiewicz wrote: > Extend the list of xc() object methods with additional one to display > Xen's buildid. The implementation follows the libxl implementation > (e.g. max buildid size assumption being XC_PAGE_SIZE). > > Signed-off-by: Pawel Wiecz

Re: [Xen-devel] [PATCH] libxlu: Handle += in config files

On Tue, Aug 13, 2019 at 05:42:15PM +0100, Anthony PERARD wrote: > On Tue, Aug 13, 2019 at 04:47:23PM +0100, Andrew Cooper wrote: > > Error between user and terminal. :) > > > > I'd sync'd xl and libxl.so, but not libxlu.so > > I actually made the same mistake first time I tried. > > > Ok, so tha

Re: [Xen-devel] [PATCH lp-metadata 3/3] livepatch: Add metadata runtime retrieval mechanism

On Thu, Aug 15, 2019 at 11:27:50AM +, Pawel Wieczorkiewicz wrote: > Extend the livepatch list operation to fetch also payloads' metadata. > This is achieved by extending the sysctl list interface with 2 extra > guest handles: > * metadata - an array of arbitrary size strings > * metadata_le

Re: [Xen-devel] [PATCH livepatch-python 1/1] livepatch: Add python bindings for livepatch operations

On Thu, Aug 15, 2019 at 11:36:46AM +, Pawel Wieczorkiewicz wrote: > Extend the XC python bindings library to support also all common > livepatch operations and actions. > > Add the python bindings for the following operations: > - status (pyxc_livepatch_status): > Requires a payload name as

Re: [Xen-devel] [PATCH] get_maintainers.pl: Enable running the script on unikraft repos

> -Original Message- > From: Lars Kurth > Sent: 16 August 2019 13:20 > To: Paul Durrant ; Anthony Perard > ; Julien Grall > > Cc: xen-devel@lists.xenproject.org; Felipe Huici ; > Stefano Stabellini > ; Wei Liu ; Konrad Rzeszutek Wilk > ; George > Dunlap ; Andrew Cooper ; > Ian Jackson

Re: [Xen-devel] [PATCH] get_maintainers.pl: Enable running the script on unikraft repos

On 16/08/2019, 13:09, "Paul Durrant" wrote: > -Original Message- > From: Lars Kurth > Sent: 16 August 2019 13:05 > To: Anthony Perard ; Julien Grall > Cc: xen-devel@lists.xenproject.org; Felipe Huici ; Stefano Stabellini > ; Wei Liu ; Konrad Rzeszutek Wilk

Re: [Xen-devel] [PATCH v2] Allow get_maintainer.pl / add_maintainers.pl scripts to be called outside of xen.git

On 16/08/2019, 11:01, "Julien Grall" wrote: Hi Lars, Thank you for the patch. On 15/08/2019 18:22, Lars Kurth wrote: > Use-case: Allow using both scripts on xen repositories such as > mini-os.git, osstest.git, > > Tool changes: > * add_maintainers.pl:

Re: [Xen-devel] [livepatch-build-tools part2 v2 6/6] create-diff-object: Do not include all .rodata sections

> On 16. Aug 2019, at 11:57, Ross Lagerwall wrote: > > On 8/8/19 1:39 PM, Pawel Wieczorkiewicz wrote: >> …snip... >> #define inc_printf(fmt, ...) \ >> log_debug("%*s" fmt, recurselevel, "", ##__VA_ARGS__); > This patch looks good. There is a comment at the top of > should_include_str_s

Re: [Xen-devel] [PATCH] get_maintainers.pl: Enable running the script on unikraft repos

> -Original Message- > From: Lars Kurth > Sent: 16 August 2019 13:05 > To: Anthony Perard ; Julien Grall > > Cc: xen-devel@lists.xenproject.org; Felipe Huici ; > Stefano Stabellini > ; Wei Liu ; Konrad Rzeszutek Wilk > ; George > Dunlap ; Andrew Cooper ; > Ian Jackson > ; Tim (Xen.org

Re: [Xen-devel] [livepatch-build-tools part2 v2 5/6] create-diff-object: Add new entries to special sections array array

On 16. Aug 2019, at 11:40, Ross Lagerwall mailto:ross.lagerw...@citrix.com>> wrote: On 8/8/19 1:35 PM, Pawel Wieczorkiewicz wrote: …snip... * The rela groups in the .fixup section vary in size. The beginning of each * .fixup rela group is referenced by the .ex_table section. To find the

Re: [Xen-devel] [Qemu-devel] [PATCH v7 00/42] Invert Endian bit in SPARCv9 MMU TTE

On Fri, Aug 16, 2019 at 11:58:05AM +0200, Philippe Mathieu-Daudé wrote: > Hi Tony, > > On 8/16/19 8:28 AM, tony.ngu...@bt.com wrote: > > This patchset implements the IE (Invert Endian) bit in SPARCv9 MMU TTE. > > > > v7: > [...] > > - Re-declared many native endian devices as little or big endian

Re: [Xen-devel] [PATCH] get_maintainers.pl: Enable running the script on unikraft repos

Added Paul Durrant On 16/08/2019, 12:17, "Anthony PERARD" wrote: On Fri, Aug 16, 2019 at 11:55:16AM +0100, Julien Grall wrote: > Hi Lars, > > On 16/08/2019 11:42, Lars Kurth wrote: > > Unikraft repos follow the same syntax as xen.git with the > > following exceptions:

Re: [Xen-devel] [Qemu-devel] [PATCH v7 00/42] Invert Endian bit in SPARCv9 MMU TTE

On Fri, 16 Aug 2019 at 12:37, wrote: > > Hi Phillippe, > > On 8/16/19 7:58 PM, Philippe Mathieu-Daudé wrote: > >On 8/16/19 8:28 AM, tony.ngu...@bt.com wrote: > >> For each device declared with DEVICE_NATIVE_ENDIAN, find the set of > >> targets from the set of target/hw/*/device.o. > >> > >> If the

Re: [Xen-devel] [PATCH] get_maintainers.pl: Enable running the script on unikraft repos

Hi Julien, On 16/08/2019, 11:55, "Julien Grall" wrote: Hi Lars, On 16/08/2019 11:42, Lars Kurth wrote: > Unikraft repos follow the same syntax as xen.git with the > following exceptions: > * MAINTAINERS files are called MAINTAINERS.md > * M: ... etc blocks are prece

[Xen-devel] [ovmf test] 140177: all pass - PUSHED

flight 140177 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/140177/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 48d8d4d80bb299af5422312d92b044cb10a2e790 baseline version: ovmf f5892aa8d8bfbe8e6af5c

Re: [Xen-devel] [Qemu-devel] [PATCH v7 00/42] Invert Endian bit in SPARCv9 MMU TTE

Hi Phillippe, On 8/16/19 7:58 PM, Philippe Mathieu-Daudé wrote: >On 8/16/19 8:28 AM, tony.ngu...@bt.com wrote: >> This patchset implements the IE (Invert Endian) bit in SPARCv9 MMU TTE. >> >> v7: >[...] >> - Re-declared many native endian devices as little or big endian. This is why >> v7 has +1

Re: [Xen-devel] [RFC] Code of Conduct

On 8/15/19 6:23 PM, Rich Persaud wrote: >> On Aug 9, 2019, at 13:48, Lars Kurth wrote: >> >> Hi all, > > Hi Lars, > >> >> Following the discussion we had at the Developer Summit (see >> https://wiki.xenproject.org/wiki/Design_Sessions_2019#Community_Issues_.2F_Improvements_-_Communication.2C_Co

Re: [Xen-devel] [PATCH] get_maintainers.pl: Enable running the script on unikraft repos

On Fri, Aug 16, 2019 at 11:55:16AM +0100, Julien Grall wrote: > Hi Lars, > > On 16/08/2019 11:42, Lars Kurth wrote: > > Unikraft repos follow the same syntax as xen.git with the > > following exceptions: > > * MAINTAINERS files are called MAINTAINERS.md > > * M: ... etc blocks are preceded by whit

Re: [Xen-devel] [PATCH] get_maintainers.pl: Enable running the script on unikraft repos

Hi Lars, On 16/08/2019 11:42, Lars Kurth wrote: Unikraft repos follow the same syntax as xen.git with the following exceptions: * MAINTAINERS files are called MAINTAINERS.md * M: ... etc blocks are preceded by whitespaces for rendering as markup files There is an other difference. The "fall

[Xen-devel] [PATCH] get_maintainers.pl: Enable running the script on unikraft repos

Unikraft repos follow the same syntax as xen.git with the following exceptions: * MAINTAINERS files are called MAINTAINERS.md * M: ... etc blocks are preceded by whitespaces for rendering as markup files This change will - load MAINTAINERS.md if MAINTAINERS is not present - deal with indented M:

Re: [Xen-devel] More QEMU gcc-9.1 build issues

On Fri, Aug 09, 2019 at 02:32:47PM +0200, Dario Faggioli wrote: > Hey Anthony, > > There's more of those issues related to building our QEMU with recent > (the one in openSUSE Tumbleweed) GCC: > > gcc version 9.1.1 20190723 [gcc-9-branch revision 273734] (SUSE Linux) [...] > I fixed it by manua

Re: [Xen-devel] [qemu-s390x] [Qemu-devel] [PATCH v7 33/42] exec: Replace device_endian with MemOp

On 8/16/19 9:37 AM, tony.ngu...@bt.com wrote: > Simplify endianness comparisons with consistent use of the more > expressive MemOp. > > Suggested-by: Richard Henderson > Signed-off-by: Tony Nguyen > Reviewed-by: Richard Henderson > Acked-by: David Gibson This patch is *huge*, more than 800kB.

Re: [Xen-devel] [Qemu-devel] [PATCH v7 27/42] hw/pci-host: Declare device little or big endian

On 8/16/19 9:35 AM, tony.ngu...@bt.com wrote: > For each device declared with DEVICE_NATIVE_ENDIAN, find the set of > targets from the set of target/hw/*/device.o. > > If the set of targets are all little or all big endian, re-declare > the device endianness as DEVICE_LITTLE_ENDIAN or DEVICE_BIG_E

Re: [Xen-devel] [Qemu-devel] [PATCH v7 25/42] hw/misc: Declare device little or big endian

On 8/16/19 9:34 AM, tony.ngu...@bt.com wrote: > For each device declared with DEVICE_NATIVE_ENDIAN, find the set of > targets from the set of target/hw/*/device.o. > > If the set of targets are all little or all big endian, re-declare > the device endianness as DEVICE_LITTLE_ENDIAN or DEVICE_BIG_E

Re: [Xen-devel] [Qemu-devel] [PATCH v7 24/42] hw/isa: Declare device little or big endian

On 8/16/19 9:34 AM, tony.ngu...@bt.com wrote: > For each device declared with DEVICE_NATIVE_ENDIAN, find the set of > targets from the set of target/hw/*/device.o. > > If the set of targets are all little or all big endian, re-declare > the device endianness as DEVICE_LITTLE_ENDIAN or DEVICE_BIG_E

Re: [Xen-devel] [PATCH v2] Allow get_maintainer.pl / add_maintainers.pl scripts to be called outside of xen.git

Hi Lars, Thank you for the patch. On 15/08/2019 18:22, Lars Kurth wrote: Use-case: Allow using both scripts on xen repositories such as mini-os.git, osstest.git, Tool changes: * add_maintainers.pl: $get_maintainer inherits path from $0 * get_maintainer.pl: warn (instead fo die) when called

Re: [Xen-devel] [Qemu-devel] [PATCH v7 00/42] Invert Endian bit in SPARCv9 MMU TTE

Hi Tony, On 8/16/19 8:28 AM, tony.ngu...@bt.com wrote: > This patchset implements the IE (Invert Endian) bit in SPARCv9 MMU TTE. > > v7: [...] > - Re-declared many native endian devices as little or big endian. This is why > v7 has +16 patches. Why are you doing that? What is the rational? An

Re: [Xen-devel] [livepatch-build-tools part2 v2 6/6] create-diff-object: Do not include all .rodata sections

On 8/8/19 1:39 PM, Pawel Wieczorkiewicz wrote: Older versions of GCC did not split .rodata.str sections by function. Because of that, the entire section was always included. The livepatch-build-tools commit [1] fixed patch creation and kept including all .rodata.str sections, in order to maintain

Re: [Xen-devel] [PATCH v6 7/8] xen/arm: don't iomem_permit_access for reserved-memory regions

Hi, On 16/08/2019 00:36, Stefano Stabellini wrote: Don't allow reserved-memory regions to be remapped into any unprivileged guests, until reserved-memory regions are properly supported in Xen. For now, do not call iomem_permit_access on them, because giving iomem_permit_access to dom0 means that

Re: [Xen-devel] [livepatch-build-tools part2 v2 2/6] common: Add is_referenced_section() helper function

On 8/7/19 12:49 PM, Pawel Wieczorkiewicz wrote: This function checks if given section has an included corresponding RELA section and/or any of the symbols table symbols references the section. Section associated symbols are ignored here as there is always such a symbol for every section. Signed-

Re: [Xen-devel] [livepatch-build-tools part2 v2 5/6] create-diff-object: Add new entries to special sections array array

On 8/8/19 1:35 PM, Pawel Wieczorkiewicz wrote: Handle .livepatch.hooks* and .altinstr_replacement sections as the special sections with assigned group_size resolution function. By default each .livepatch.hooks* sections' entry is 8 bytes long (a pointer). The .altinstr_replacement section has und

Re: [Xen-devel] [livepatch-build-tools part2 v2 3/6] create-diff-object: Add is_special_section() helper function function

On 8/8/19 1:23 PM, Pawel Wieczorkiewicz wrote: This function determines, based on the given section name, if the sections belongs to the special sections category. Add more special sections to special_sections[] along with a new undefined_group_size() helper function returning 0 (undefined). New

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

flight 140170 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/140170/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-qemuu-win7-amd64 17 guest-stopfail like 140148 test-armhf-armhf-libvirt 14 sav

Re: [Xen-devel] [PATCH v6 4/8] xen/arm: fix indentation in early_print_info

Hi, On 16/08/2019 00:36, Stefano Stabellini wrote: No functional changes. Signed-off-by: Stefano Stabellini Acked-by: Julien Grall Cheers, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/ma

Re: [Xen-devel] [PATCH v6 2/8] xen/arm: make process_memory_node a device_tree_node_func

Hi, On 16/08/2019 00:36, Stefano Stabellini wrote: Change the signature of process_memory_node to match device_tree_node_func. Thanks to this change, the next patch will be able to use device_tree_for_each_node to call process_memory_node on all the children of a provided node. Return error if

Re: [Xen-devel] [livepatch-build-tools part2 v2 4/6] livepatch-build: detect special section group sizes

On 8/8/19 1:29 PM, Pawel Wieczorkiewicz wrote: Hard-coding the special section group sizes is unreliable. Instead, determine them dynamically by finding the related struct definitions in the DWARF metadata. This is a livepatch backport of kpatch upstream commit [1]: kpatch-build: detect special

Re: [Xen-devel] [PATCH v6 1/8] xen/arm: pass node to device_tree_for_each_node

Hi, On 16/08/2019 00:36, Stefano Stabellini wrote: Add a new parameter to device_tree_for_each_node: node, the node to start the search from. Passing 0 triggers the old behavior. Here you say 0 triggers the old behavior but... Set min_depth to depth of the current node + 1 to avoid scanning

  1   2   >