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

2019-08-16 Thread Wieczorkiewicz, Pawel
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] [PATCH] libxlu: Handle += in config files

2019-08-16 Thread Wei Liu
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

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

2019-08-16 Thread Wei Liu
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

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

2019-08-16 Thread Christoph Hellwig
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

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

2019-08-16 Thread Christoph Hellwig
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 ---

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

2019-08-16 Thread Christoph Hellwig
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 ---

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

2019-08-16 Thread Christoph Hellwig
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 ---

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

2019-08-16 Thread Christoph Hellwig
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

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

2019-08-16 Thread osstest service owner
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

2019-08-16 Thread Christoph Hellwig
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 --

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

2019-08-16 Thread Lars Kurth
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

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

2019-08-16 Thread Lars Kurth
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

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

2019-08-16 Thread Paul Durrant
> -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

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

2019-08-16 Thread Wei Liu
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 > *

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

2019-08-16 Thread Julien Grall
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

2019-08-16 Thread Julien Grall
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

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

2019-08-16 Thread David Gibson
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

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

2019-08-16 Thread Lars Kurth
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] [PATCH v2] Allow get_maintainer.pl / add_maintainers.pl scripts to be called outside of xen.git

2019-08-16 Thread Lars Kurth
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: > *

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

2019-08-16 Thread Julien Grall
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

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

2019-08-16 Thread Jim Fehlig
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

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

2019-08-16 Thread Anthony PERARD
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

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

2019-08-16 Thread Wieczorkiewicz, Pawel
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

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

2019-08-16 Thread Ross Lagerwall
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

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

2019-08-16 Thread Ross Lagerwall
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] [Qemu-devel] [PATCH v7 00/42] Invert Endian bit in SPARCv9 MMU TTE

2019-08-16 Thread tony.nguyen
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

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

2019-08-16 Thread osstest service owner
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

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

2019-08-16 Thread Wieczorkiewicz, Pawel
> 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 >

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

2019-08-16 Thread Lars Kurth
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

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

2019-08-16 Thread osstest service owner
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

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

2019-08-16 Thread Wieczorkiewicz, Pawel
> 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 >>

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

2019-08-16 Thread Peter Maydell
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

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

2019-08-16 Thread Wei Liu
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] [livepatch-build-tools part3 v2 2/3] create-diff-object: Extend patchability verification: STN_UNDEF

2019-08-16 Thread Ross Lagerwall
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] [PATCH lp-metadata 3/3] livepatch: Add metadata runtime retrieval mechanism

2019-08-16 Thread Wieczorkiewicz, Pawel
> 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] [PATCH 03/11] xen/arm: pass one less argument to dma_cache_maint

2019-08-16 Thread Robin Murphy
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

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

2019-08-16 Thread Paul Durrant
> -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

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

2019-08-16 Thread Wei Liu
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

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

2019-08-16 Thread Anthony PERARD
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 > > >

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

2019-08-16 Thread Lars Kurth
> 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] [RFC] Code of Conduct

2019-08-16 Thread George Dunlap
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 >>

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

2019-08-16 Thread Christoph Hellwig
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

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

2019-08-16 Thread Christoph Hellwig
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

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

2019-08-16 Thread Christoph Hellwig
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 ---

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

2019-08-16 Thread Christoph Hellwig
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 04/11] xen/arm: remove xen_dma_ops

2019-08-16 Thread Christoph Hellwig
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

[Xen-devel] swiotlb-xen cleanups

2019-08-16 Thread Christoph Hellwig
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

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

2019-08-16 Thread osstest service owner
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

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

2019-08-16 Thread Paul Durrant
...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

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

2019-08-16 Thread Paul Durrant
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

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

2019-08-16 Thread Paul Durrant
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-devel] [PATCH v6 00/10] use stashed domain create flags...

2019-08-16 Thread Paul Durrant
...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

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

2019-08-16 Thread Paul Durrant
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

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

2019-08-16 Thread Paul Durrant
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

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

2019-08-16 Thread Paul Durrant
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

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

2019-08-16 Thread Paul Durrant
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

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

2019-08-16 Thread Paul Durrant
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] [linux-linus bisection] complete test-amd64-i386-xl-xsm

2019-08-16 Thread osstest service owner
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

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

2019-08-16 Thread Christoph Hellwig
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 v6 08/10] remove late (on-demand) construction of IOMMU page tables

2019-08-16 Thread Razvan Cojocaru
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

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

2019-08-16 Thread Volodymyr Babchuk
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? > --- >

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

2019-08-16 Thread Julien Grall
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

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

2019-08-16 Thread Volodymyr Babchuk
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. > --- >

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

2019-08-16 Thread Julien Grall
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

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

2019-08-16 Thread Volodymyr Babchuk
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

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

2019-08-16 Thread Lars Kurth
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

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

2019-08-16 Thread Volodymyr Babchuk
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

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

2019-08-16 Thread Paul Durrant
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

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

2019-08-16 Thread Paul Durrant
...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

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

2019-08-16 Thread Johnson, Ethan
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

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

2019-08-16 Thread Rich Persaud
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 >>>

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

2019-08-16 Thread Stefano Stabellini
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,

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

2019-08-16 Thread osstest service owner
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

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

2019-08-16 Thread Julien Grall
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

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

2019-08-16 Thread osstest service owner
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

[Xen-devel] [Qemu-devel] [PATCH v7 08/42] hw/vfio: Access MemoryRegion with MemOp

2019-08-16 Thread tony.nguyen
The memory_region_dispatch_{read|write} operand "unsigned size" is being converted into a "MemOp op". Convert interfaces by using no-op size_memop. After all interfaces are converted, size_memop will be implemented and the memory_region_dispatch_{read|write} operand "unsigned size" will be

[Xen-devel] [Qemu-devel] [PATCH v7 10/42] cputlb: Access MemoryRegion with MemOp

2019-08-16 Thread tony.nguyen
The memory_region_dispatch_{read|write} operand "unsigned size" is being converted into a "MemOp op". Convert interfaces by using no-op size_memop. After all interfaces are converted, size_memop will be implemented and the memory_region_dispatch_{read|write} operand "unsigned size" will be

[Xen-devel] [Qemu-devel] [PATCH v7 09/42] exec: Access MemoryRegion with MemOp

2019-08-16 Thread tony.nguyen
The memory_region_dispatch_{read|write} operand "unsigned size" is being converted into a "MemOp op". Convert interfaces by using no-op size_memop. After all interfaces are converted, size_memop will be implemented and the memory_region_dispatch_{read|write} operand "unsigned size" will be

[Xen-devel] [Qemu-devel] [PATCH v7 07/42] hw/virtio: Access MemoryRegion with MemOp

2019-08-16 Thread tony.nguyen
The memory_region_dispatch_{read|write} operand "unsigned size" is being converted into a "MemOp op". Convert interfaces by using no-op size_memop. After all interfaces are converted, size_memop will be implemented and the memory_region_dispatch_{read|write} operand "unsigned size" will be

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

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 26/42] hw/net: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

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

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

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

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 28/42] hw/sd: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 29/42] hw/ssi: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 30/42] hw/timer: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 22/42] hw/input: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

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

2019-08-16 Thread tony.nguyen
This patchset implements the IE (Invert Endian) bit in SPARCv9 MMU TTE. It is an attempt of the instructions outlined by Richard Henderson to Mark Cave-Ayland. Tested with OpenBSD on sun4u. Solaris 10 is my actual goal, but unfortunately a separate keyboard issue remains in the way. On 01/11/17

[Xen-devel] [Qemu-devel] [PATCH v7 11/42] memory: Access MemoryRegion with MemOp

2019-08-16 Thread tony.nguyen
Convert memory_region_dispatch_{read|write} operand "unsigned size" into a "MemOp op". Signed-off-by: Tony Nguyen Reviewed-by: Richard Henderson --- include/exec/memop.h | 20 ++-- include/exec/memory.h | 9 + memory.c | 7 +-- 3 files changed, 24

[Xen-devel] [Qemu-devel] [PATCH v7 13/42] target/mips: Hard code size with MO_{8|16|32|64}

2019-08-16 Thread tony.nguyen
Temporarily no-op size_memop was introduced to aid the conversion of memory_region_dispatch_{read|write} operand "unsigned size" into "MemOp op". Now size_memop is implemented, again hard coded size but with MO_{8|16|32|64}. This is more expressive and avoids size_memop calls. Signed-off-by:

[Xen-devel] [Qemu-devel] [PATCH v7 12/42] hw/s390x: Hard code size with MO_{8|16|32|64}

2019-08-16 Thread tony.nguyen
Temporarily no-op size_memop was introduced to aid the conversion of memory_region_dispatch_{read|write} operand "unsigned size" into "MemOp op". Now size_memop is implemented, again hard coded size but with MO_{8|16|32|64}. This is more expressive and avoids size_memop calls. Signed-off-by:

[Xen-devel] [Qemu-devel] [PATCH v7 15/42] hw/audio: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 14/42] exec: Hard code size with MO_{8|16|32|64}

2019-08-16 Thread tony.nguyen
Temporarily no-op size_memop was introduced to aid the conversion of memory_region_dispatch_{read|write} operand "unsigned size" into "MemOp op". Now size_memop is implemented, again hard coded size but with MO_{8|16|32|64}. This is more expressive and avoids size_memop calls. Signed-off-by:

[Xen-devel] [Qemu-devel] [PATCH v7 19/42] hw/dma: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 17/42] hw/char: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 20/42] hw/gpio: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 21/42] hw/i2c: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 23/42] hw/intc: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 16/42] hw/block: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

[Xen-devel] [Qemu-devel] [PATCH v7 18/42] hw/display: Declare device little or big endian

2019-08-16 Thread tony.nguyen
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_ENDIAN respectively. This *naive* deduction may result

  1   2   >