[RFC v2 0/2] Generic SMMU Bindings

2020-07-21 Thread Brian Woods
Finally had time to do a v2. Changes are in each patch. Brian Woods (2): arm,smmu: switch to using iommu_fwspec functions arm,smmu: add support for generic DT bindings xen/drivers/passthrough/arm/smmu.c| 147 -- xen/drivers/passthrough/device_tree.c |

[RFC v2 1/2] arm,smmu: switch to using iommu_fwspec functions

2020-07-21 Thread Brian Woods
Modify the smmu driver so that it uses the iommu_fwspec helper functions. This means both ARM IOMMU drivers will both use the iommu_fwspec helper functions. Signed-off-by: Brian Woods --- Interested in if combining the legacy and generic bindings paths are worth or if Xen plans to depreicate

[RFC v2 2/2] arm,smmu: add support for generic DT bindings

2020-07-21 Thread Brian Woods
Restructure some of the code and add supporting functions for adding generic device tree (DT) binding support. This will allow for using current Linux device trees with just modifying the chosen field to enable Xen. Signed-off-by: Brian Woods --- Just realized that I'm fairly sure I need to do

[xen-unstable test] 152067: regressions - trouble: fail/pass/starved

2020-07-21 Thread osstest service owner
flight 152067 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/152067/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-dom0pvh-xl-intel 18 guest-localmigrate/x10 fail REGR. vs. 152045

Re: [PATCH v2 01/11] xen/manage: keep track of the on-going suspend mode

2020-07-21 Thread Stefano Stabellini
On Tue, 21 Jul 2020, Boris Ostrovsky wrote: > >> +static int xen_setup_pm_notifier(void) > >> +{ > >> + if (!xen_hvm_domain()) > >> + return -ENODEV; > >> > >> I forgot --- what did we decide about non-x86 (i.e. ARM)? > > It would be great to support

Re: [PATCH v2 01/11] xen/manage: keep track of the on-going suspend mode

2020-07-21 Thread Anchal Agarwal
On Tue, Jul 21, 2020 at 10:30:18AM +0200, Roger Pau Monné wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you can confirm the sender and know the > content is safe. > > > > Marek: I'm adding you in case you could be

Re: [PATCH] x86/xen/time: set the X86_FEATURE_TSC_KNOWN_FREQ flag in xen_tsc_khz()

2020-07-21 Thread Boris Ostrovsky
On 7/21/20 12:12 PM, Hayato Ohhashi wrote: > If the TSC frequency is known from the pvclock page, > the TSC frequency does not need to be recalibrated. > We can avoid recalibration by setting X86_FEATURE_TSC_KNOWN_FREQ. > > Signed-off-by: Hayato Ohhashi Reviewed-by: Boris Ostrovsky

[ovmf test] 152068: all pass - PUSHED

2020-07-21 Thread osstest service owner
flight 152068 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/152068/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 02539e900854488343a1efa435d4dded1ddd66a2 baseline version: ovmf

Re: [PATCH v2 01/11] xen/manage: keep track of the on-going suspend mode

2020-07-21 Thread Boris Ostrovsky
>> +static int xen_setup_pm_notifier(void) >> +{ >> + if (!xen_hvm_domain()) >> + return -ENODEV; >> >> I forgot --- what did we decide about non-x86 (i.e. ARM)? > It would be great to support that however, its out of > scope for this patch set.

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Julien Grall
Hi Oleksandr, On 21/07/2020 19:16, Oleksandr wrote: On 21.07.20 17:27, Julien Grall wrote: On a similar topic, I am a bit surprised you didn't encounter memory exhaustion when trying to use virtio. Because on how Linux currently works (see XSA-300), the backend domain as to have a least as

Re: RFC: PCI devices passthrough on Arm design proposal

2020-07-21 Thread Stefano Stabellini
On Mon, 20 Jul 2020, Rob Herring wrote: > On Mon, Jul 20, 2020 at 5:24 PM Stefano Stabellini > wrote: > > > > + Rob Herring > > > > On Fri, 17 Jul 2020, Bertrand Marquis wrote: > > > >> Regarding the DT entry, this is not coming from us and this is already > > > >> defined this way in existing

[xen-unstable-smoke test] 152077: tolerable all pass - PUSHED

2020-07-21 Thread osstest service owner
flight 152077 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/152077/ 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

[OSSTEST PATCH 13/14] duration_estimator: Move $uptincl_testid to separate @x_params

2020-07-21 Thread Ian Jackson
This is going to be useful soon. No functional change. Signed-off-by: Ian Jackson --- Osstest/Executive.pm | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Osstest/Executive.pm b/Osstest/Executive.pm index 8e8b3d33..621153ee 100644 --- a/Osstest/Executive.pm +++

[OSSTEST PATCH 10/14] duration_estimator: Introduce some _qtxt variables

2020-07-21 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- Osstest/Executive.pm | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/Osstest/Executive.pm b/Osstest/Executive.pm index 3cd37c14..c966a1be 100644 --- a/Osstest/Executive.pm +++ b/Osstest/Executive.pm @@ -1146,7

[OSSTEST PATCH 12/14] duration_estimator: Return job column in first query

2020-07-21 Thread Ian Jackson
Right now this is pointless since the Perl code doesn't need it. But this row is going to be part of a WITH clause soon. No functional change. Diffs to two example queries (from the Perl DBI trace): SELECT f.flight AS flight, + j.job AS job,

[OSSTEST PATCH 11/14] duration_estimator: Explicitly provide null in general host q

2020-07-21 Thread Ian Jackson
Our spec. says we return nulls for started and status if we don't find a job matching the host spec. The way this works right now is that we look up the nonexistent entries in $refs->[0]. This is not really brilliant and is going to be troublesome as we continue to refactor. Provide these

[OSSTEST PATCH 14/14] duration_estimator: Move duration query loop into database

2020-07-21 Thread Ian Jackson
Stuff the two queries together: we use the firsty query as a WITH clause. This is significantly faster, perhaps because the query optimiser does a better job but probably just because it saves on round trips. No functional change. Perf: subjectively this seemed to help when the cache was cold.

[OSSTEST PATCH 09/14] duration_estimator: Ignore truncated jobs unless we know the step

2020-07-21 Thread Ian Jackson
If we are looking for a particular step then we will ignore jobs without that step, so any job which was truncated before it will be ignored. Otherwise we are looking for the whole job duration and a truncated job is not a good representative. This is a bugfix (to duration estimation), not a

[OSSTEST PATCH 08/14] Executive: Use index for report__find_test

2020-07-21 Thread Ian Jackson
After we refactor this query then we can enable the index use. (Both of these things together in this commit because I haven't perf tested the version with just the refactoring.) (We have provided an index that can answer this question really quickly if a version is specified. But the query

[OSSTEST PATCH 07/14] sg-report-flight: Use the job row from the intitial query

2020-07-21 Thread Ian Jackson
$jcheckq is redundant: we looked this up right at the start. This is not expected to speed things up very much, but it makes things somewhat cleaner and clearer. Signed-off-by: Ian Jackson --- sg-report-flight | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git

[OSSTEST PATCH 02/14] sg-report-flight: Sort failures by job name as last resort

2020-07-21 Thread Ian Jackson
This removes some nondeterminism from the output. Signed-off-by: Ian Jackson --- sg-report-flight | 1 + 1 file changed, 1 insertion(+) diff --git a/sg-report-flight b/sg-report-flight index 927ea37d..70def778 100755 --- a/sg-report-flight +++ b/sg-report-flight @@ -813,6 +813,7 @@ END

[OSSTEST PATCH 04/14] sg-report-flight: Ask the db for flights of interest

2020-07-21 Thread Ian Jackson
Specifically, we narrow the initial query to flights which have at least some job with the built_revision_foo we are looking for. This condition is strictly broader than that implemented inside the flight search loop, so there is no functional change. Perf: runtime of my test case now

[OSSTEST PATCH 06/14] sg-report-flight: Use WITH clause to use index for $anypassq

2020-07-21 Thread Ian Jackson
Perf: runtime of my test case now ~11s Example query before (from the Perl DBI trace): SELECT * FROM flights JOIN steps USING (flight) WHERE (branch='xen-unstable') AND job=? and testid=? and status='pass' AND ( (TRUE AND flight <= 151903) AND

[OSSTEST PATCH 01/14] sg-report-flight: Add a comment re same-flight search narrowing

2020-07-21 Thread Ian Jackson
In afe851ca1771e5da6395b596afa69e509dbbc278 sg-report-flight: When justifying, disregard out-of-flight build jobs we narrowed sg-report-flight's search algorith. An extensive justification is in the commit message. I think much of this information belongs in-tree, so c it (with slight edits)

[OSSTEST PATCH 00/14] Flight report performance improvements

2020-07-21 Thread Ian Jackson
osstest was taking far too long calculating what test failures were regressions, and generating the email and web reports. The slow part was analysing the test history, mostly because it ended up doing a lot of dumb scans of large tables. In this series I fix this problem for sg-report-flight: I

[OSSTEST PATCH 05/14] sg-report-flight: Use WITH to use best index use for $flightsq

2020-07-21 Thread Ian Jackson
While we're here, convert this EXISTS subquery to a JOIN. Perf: runtime of my test case now ~200-300s. Example query before (from the Perl DBI trace): SELECT * FROM ( SELECT DISTINCT flight, blessing FROM flights JOIN runvars r1 USING (flight)

[OSSTEST PATCH 03/14] schema: Provide indices for sg-report-flight

2020-07-21 Thread Ian Jackson
These indexes allow very fast lookup of "relevant" flights eg when trying to justify failures. In my ad-hoc test case, these indices (along with the subsequent changes to sg-report-flight and Executive.pm, reduce the runtime of sg-report-flight from 2-3ks (unacceptably long!) to as little as 5-7s

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Oleksandr
On 21.07.20 17:27, Julien Grall wrote: Hi, Hello On a similar topic, I am a bit surprised you didn't encounter memory exhaustion when trying to use virtio. Because on how Linux currently works (see XSA-300), the backend domain as to have a least as much RAM as the domain it serves.

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Oleksandr
On 21.07.20 17:27, Julien Grall wrote: Hi, Hello Julien On 17/07/2020 19:34, Oleksandr wrote: On 17.07.20 18:00, Roger Pau Monné wrote: requires some implementation to forward guest MMIO access to a device model. And as it turned out the Xen on x86 contains most of the pieces to be

[PATCH] x86/svm: Fold nsvm_{wr, rd}msr() into svm_msr_{read, write}_intercept()

2020-07-21 Thread Andrew Cooper
... to simplify the default cases. There are multiple errors with the handling of these three MSRs, but they are deliberately not addressed this point. This removes the dance converting -1/0/1 into X86EMUL_*, allowing for the removal of the 'ret' variable. While cleaning this up, drop the

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread André Przywara
On 21/07/2020 17:09, Oleksandr wrote: > > On 21.07.20 17:58, André Przywara wrote: >> On 21/07/2020 15:52, Oleksandr wrote: >>> On 21.07.20 17:32, André Przywara wrote: On 21/07/2020 14:43, Julien Grall wrote: >>> Hello Andre, Julien >>> >>> > (+ Andre) > > Hi Oleksandr, >

[xen-4.14-testing test] 152061: regressions - trouble: fail/pass/starved

2020-07-21 Thread osstest service owner
flight 152061 xen-4.14-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/152061/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-dom0pvh-xl-amd 16 guest-localmigratefail REGR. vs. 152043

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Stefano Stabellini
On Tue, 21 Jul 2020, Julien Grall wrote: > Hi Alex, > > Thank you for your feedback! > > On 21/07/2020 15:15, Alex Bennée wrote: > > Julien Grall writes: > > > > > (+ Andree for the vGIC). > > > > > > Hi Stefano, > > > > > > On 20/07/2020 21:38, Stefano Stabellini wrote: > > > > On Fri, 17

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Stefano Stabellini
On Tue, 21 Jul 2020, Alex Bennée wrote: > Julien Grall writes: > > > Hi Stefano, > > > > On 20/07/2020 21:37, Stefano Stabellini wrote: > >> On Mon, 20 Jul 2020, Roger Pau Monné wrote: > >>> On Mon, Jul 20, 2020 at 10:40:40AM +0100, Julien Grall wrote: > > > On 20/07/2020 10:17,

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Oleksandr
On 21.07.20 17:58, André Przywara wrote: On 21/07/2020 15:52, Oleksandr wrote: On 21.07.20 17:32, André Przywara wrote: On 21/07/2020 14:43, Julien Grall wrote: Hello Andre, Julien (+ Andre) Hi Oleksandr, On 21/07/2020 13:26, Oleksandr wrote: On 20.07.20 23:38, Stefano Stabellini

Re: [PATCH][next] x86/ioperm: initialize pointer bitmap with NULL rather than 0

2020-07-21 Thread Jürgen Groß
On 21.07.20 12:02, Colin King wrote: From: Colin Ian King The pointer bitmap is being initialized with a plain integer 0, fix this by initializing it with a NULL instead. Cleans up sparse warning: arch/x86/xen/enlighten_pv.c:876:27: warning: Using plain integer as NULL pointer Signed-off-by:

[xen-unstable-smoke test] 152074: tolerable all pass - PUSHED

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

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread André Przywara
On 21/07/2020 15:52, Oleksandr wrote: > > On 21.07.20 17:32, André Przywara wrote: >> On 21/07/2020 14:43, Julien Grall wrote: > > Hello Andre, Julien > > >>> (+ Andre) >>> >>> Hi Oleksandr, >>> >>> On 21/07/2020 13:26, Oleksandr wrote: On 20.07.20 23:38, Stefano Stabellini wrote: >

Re: [PATCH 1/2] Partially revert "Cross-compilation fixes."

2020-07-21 Thread Wei Liu
On Tue, Jul 21, 2020 at 07:44:10AM -0700, Elliott Mitchell wrote: > On Tue, Jul 21, 2020 at 12:26:45PM +, Wei Liu wrote: > > On Fri, Jul 17, 2020 at 08:31:21PM -0700, Elliott Mitchell wrote: > > > This partially reverts commit 16504669c5cbb8b195d20412aadc838da5c428f7. > > > > Ok, so this

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Oleksandr
On 21.07.20 17:32, André Przywara wrote: On 21/07/2020 14:43, Julien Grall wrote: Hello Andre, Julien (+ Andre) Hi Oleksandr, On 21/07/2020 13:26, Oleksandr wrote: On 20.07.20 23:38, Stefano Stabellini wrote: For instance, what's your take on notifications with virtio-mmio? How are

Re: [PATCH 1/2] Partially revert "Cross-compilation fixes."

2020-07-21 Thread Elliott Mitchell
On Tue, Jul 21, 2020 at 12:26:45PM +, Wei Liu wrote: > On Fri, Jul 17, 2020 at 08:31:21PM -0700, Elliott Mitchell wrote: > > This partially reverts commit 16504669c5cbb8b195d20412aadc838da5c428f7. > > Ok, so this commit is really old. Yup. It will still be visible in `git blame

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Julien Grall
Hi Alex, Thank you for your feedback! On 21/07/2020 15:15, Alex Bennée wrote: Julien Grall writes: (+ Andree for the vGIC). Hi Stefano, On 20/07/2020 21:38, Stefano Stabellini wrote: On Fri, 17 Jul 2020, Oleksandr wrote: *A few word about solution:* As it was mentioned at [1], in order

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread André Przywara
On 21/07/2020 14:43, Julien Grall wrote: > (+ Andre) > > Hi Oleksandr, > > On 21/07/2020 13:26, Oleksandr wrote: >> On 20.07.20 23:38, Stefano Stabellini wrote: >>> For instance, what's your take on notifications with virtio-mmio? How >>> are they modelled today? Are they good enough or do we

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Julien Grall
Hi, On 17/07/2020 19:34, Oleksandr wrote: On 17.07.20 18:00, Roger Pau Monné wrote: requires some implementation to forward guest MMIO access to a device model. And as it turned out the Xen on x86 contains most of the pieces to be able to use that transport (via existing IOREQ concept).

[qemu-mainline test] 152058: regressions - trouble: fail/pass/starved

2020-07-21 Thread osstest service owner
flight 152058 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/152058/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 10 debian-hvm-install fail REGR. vs. 151065

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Alex Bennée
Julien Grall writes: > (+ Andree for the vGIC). > > Hi Stefano, > > On 20/07/2020 21:38, Stefano Stabellini wrote: >> On Fri, 17 Jul 2020, Oleksandr wrote: > *A few word about solution:* > As it was mentioned at [1], in order to implement virtio-mmio Xen on Arm Any plans for

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Alex Bennée
Julien Grall writes: > Hi Stefano, > > On 20/07/2020 21:37, Stefano Stabellini wrote: >> On Mon, 20 Jul 2020, Roger Pau Monné wrote: >>> On Mon, Jul 20, 2020 at 10:40:40AM +0100, Julien Grall wrote: On 20/07/2020 10:17, Roger Pau Monné wrote: > On Fri, Jul 17, 2020 at

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Julien Grall
(+ Andre) Hi Oleksandr, On 21/07/2020 13:26, Oleksandr wrote: On 20.07.20 23:38, Stefano Stabellini wrote: For instance, what's your take on notifications with virtio-mmio? How are they modelled today? Are they good enough or do we need MSIs? Notifications are sent from device (backend) to

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Roger Pau Monné
On Tue, Jul 21, 2020 at 02:32:38PM +0100, Julien Grall wrote: > Hi Roger, > > On 21/07/2020 14:25, Roger Pau Monné wrote: > > On Tue, Jul 21, 2020 at 01:31:48PM +0100, Julien Grall wrote: > > > Hi Stefano, > > > > > > On 20/07/2020 21:37, Stefano Stabellini wrote: > > > > On Mon, 20 Jul 2020,

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Julien Grall
Hi Roger, On 21/07/2020 14:25, Roger Pau Monné wrote: On Tue, Jul 21, 2020 at 01:31:48PM +0100, Julien Grall wrote: Hi Stefano, On 20/07/2020 21:37, Stefano Stabellini wrote: On Mon, 20 Jul 2020, Roger Pau Monné wrote: On Mon, Jul 20, 2020 at 10:40:40AM +0100, Julien Grall wrote: On

Re: [PATCH 02/12] tools: switch XEN_LIBXEN* make variables to lower case (XEN_libxen*)

2020-07-21 Thread Wei Liu
On Wed, Jul 15, 2020 at 05:25:01PM +0100, Ian Jackson wrote: > From: Juergen Gross > > In order to harmonize names of library related make variables switch > XEN_LIBXEN* names to XEN_libxen*, as all other related variables (e.g. > CFLAGS_libxen*, SHDEPS_libxen*, ...) already use this pattern. >

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Roger Pau Monné
On Tue, Jul 21, 2020 at 01:31:48PM +0100, Julien Grall wrote: > Hi Stefano, > > On 20/07/2020 21:37, Stefano Stabellini wrote: > > On Mon, 20 Jul 2020, Roger Pau Monné wrote: > > > On Mon, Jul 20, 2020 at 10:40:40AM +0100, Julien Grall wrote: > > > > > > > > > > > > On 20/07/2020 10:17, Roger

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Julien Grall
(+ Andree for the vGIC). Hi Stefano, On 20/07/2020 21:38, Stefano Stabellini wrote: On Fri, 17 Jul 2020, Oleksandr wrote: *A few word about solution:* As it was mentioned at [1], in order to implement virtio-mmio Xen on Arm Any plans for virtio-pci? Arm seems to be moving to the PCI bus, and

RE: vPT rework (and timer mode)

2020-07-21 Thread Paul Durrant
> -Original Message- > From: Roger Pau Monné > Sent: 21 July 2020 12:53 > To: p...@xen.org > Cc: 'Andrew Cooper' ; 'Jan Beulich' > ; xen- > de...@lists.xenproject.org; 'Wei Liu' ; Igor Druzhinin > > Subject: Re: vPT rework (and timer mode) > > On Mon, Jul 06, 2020 at 09:58:53AM +0100,

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Oleksandr
On 20.07.20 23:40, Stefano Stabellini wrote: Hello Stefano On Mon, 20 Jul 2020, Roger Pau Monné wrote: On Mon, Jul 20, 2020 at 01:56:51PM +0300, Oleksandr wrote: On 20.07.20 12:17, Roger Pau Monné wrote: On Fri, Jul 17, 2020 at 09:34:14PM +0300, Oleksandr wrote: On 17.07.20 18:00, Roger

Re: [PATCH v2] mini-os: don't hard-wire xen internal paths

2020-07-21 Thread Wei Liu
On Tue, Jul 21, 2020 at 12:39:40PM +, Wei Liu wrote: > On Tue, Jul 21, 2020 at 02:24:51PM +0200, Jürgen Groß wrote: > > On 21.07.20 14:21, Wei Liu wrote: > > > On Sat, Jul 18, 2020 at 08:18:27PM +0200, Samuel Thibault wrote: > > > > Juergen Gross, le lun. 13 juil. 2020 10:42:30 +0200, a ecrit:

Re: [PATCH v2] mini-os: don't hard-wire xen internal paths

2020-07-21 Thread Wei Liu
On Tue, Jul 21, 2020 at 02:24:51PM +0200, Jürgen Groß wrote: > On 21.07.20 14:21, Wei Liu wrote: > > On Sat, Jul 18, 2020 at 08:18:27PM +0200, Samuel Thibault wrote: > > > Juergen Gross, le lun. 13 juil. 2020 10:42:30 +0200, a ecrit: > > > > Mini-OS shouldn't use Xen internal paths for building.

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Julien Grall
Hi Stefano, On 20/07/2020 21:37, Stefano Stabellini wrote: On Mon, 20 Jul 2020, Roger Pau Monné wrote: On Mon, Jul 20, 2020 at 10:40:40AM +0100, Julien Grall wrote: On 20/07/2020 10:17, Roger Pau Monné wrote: On Fri, Jul 17, 2020 at 09:34:14PM +0300, Oleksandr wrote: On 17.07.20 18:00,

Re: [PATCH] tools/xen-cpuid: use dashes consistently in feature names

2020-07-21 Thread Andrew Cooper
On 21/07/2020 13:16, Wei Liu wrote: > On Tue, Jul 21, 2020 at 02:04:59PM +0200, Jan Beulich wrote: >> We've grown to a mix of dashes and underscores - switch to consistent >> naming in the hope that future additions will play by this. >> >> Signed-off-by: Jan Beulich > Acked-by: Wei Liu

Re: Virtio in Xen on Arm (based on IOREQ concept)

2020-07-21 Thread Oleksandr
On 20.07.20 23:38, Stefano Stabellini wrote: Hello Stefano On Fri, 17 Jul 2020, Oleksandr wrote: *A few word about solution:* As it was mentioned at [1], in order to implement virtio-mmio Xen on Arm Any plans for virtio-pci? Arm seems to be moving to the PCI bus, and it would be very

Re: [PATCH 1/2] Partially revert "Cross-compilation fixes."

2020-07-21 Thread Wei Liu
On Fri, Jul 17, 2020 at 08:31:21PM -0700, Elliott Mitchell wrote: > This partially reverts commit 16504669c5cbb8b195d20412aadc838da5c428f7. Ok, so this commit is really old. > > Signed-off-by: Elliott Mitchell > --- > Doesn't look like much of 16504669c5cbb8b195d20412aadc838da5c428f7 >

Re: [PATCH v2] mini-os: don't hard-wire xen internal paths

2020-07-21 Thread Jürgen Groß
On 21.07.20 14:21, Wei Liu wrote: On Sat, Jul 18, 2020 at 08:18:27PM +0200, Samuel Thibault wrote: Juergen Gross, le lun. 13 juil. 2020 10:42:30 +0200, a ecrit: Mini-OS shouldn't use Xen internal paths for building. Import the needed paths from Xen and fall back to the current values only if

Re: [PATCH] SUPPORT.md: Spell correctly Experimental

2020-07-21 Thread Julien Grall
Hi Paul, On 21/07/2020 08:09, Paul Durrant wrote: -Original Message- From: Andrew Cooper Sent: 20 July 2020 18:49 To: Julien Grall ; xen-devel@lists.xenproject.org Cc: p...@xen.org; Julien Grall ; George Dunlap ; Ian Jackson ; Jan Beulich ; Stefano Stabellini ; Wei Liu Subject: Re:

Re: [PATCH v2] mini-os: don't hard-wire xen internal paths

2020-07-21 Thread Wei Liu
On Sat, Jul 18, 2020 at 08:18:27PM +0200, Samuel Thibault wrote: > Juergen Gross, le lun. 13 juil. 2020 10:42:30 +0200, a ecrit: > > Mini-OS shouldn't use Xen internal paths for building. Import the > > needed paths from Xen and fall back to the current values only if > > the import was not

Re: [PATCH] tools/xen-cpuid: use dashes consistently in feature names

2020-07-21 Thread Wei Liu
On Tue, Jul 21, 2020 at 02:04:59PM +0200, Jan Beulich wrote: > We've grown to a mix of dashes and underscores - switch to consistent > naming in the hope that future additions will play by this. > > Signed-off-by: Jan Beulich Acked-by: Wei Liu

Re: [PATCH v2 0/7] xen: credit2: limit the number of CPUs per runqueue

2020-07-21 Thread Jan Beulich
On 28.05.2020 23:29, Dario Faggioli wrote: > Dario Faggioli (7): > xen: credit2: factor cpu to runqueue matching in a function > xen: credit2: factor runqueue initialization in its own function. > xen: cpupool: add a back-pointer from a scheduler to its pool > xen: credit2:

[PATCH] tools/xen-cpuid: use dashes consistently in feature names

2020-07-21 Thread Jan Beulich
We've grown to a mix of dashes and underscores - switch to consistent naming in the hope that future additions will play by this. Signed-off-by: Jan Beulich --- a/tools/misc/xen-cpuid.c +++ b/tools/misc/xen-cpuid.c @@ -75,7 +75,7 @@ static const char *const str_e1d[32] = static const char

Re: vPT rework (and timer mode)

2020-07-21 Thread Roger Pau Monné
On Mon, Jul 06, 2020 at 09:58:53AM +0100, Paul Durrant wrote: > > -Original Message- > > From: Roger Pau Monné > > Sent: 06 July 2020 09:32 > > To: p...@xen.org > > Cc: 'Andrew Cooper' ; 'Jan Beulich' > > ; xen- > > de...@lists.xenproject.org; 'Wei Liu' > > Subject: Re: vPT rework (and

Re: [PATCH] x86/S3: put data segment registers into known state upon resume

2020-07-21 Thread Roger Pau Monné
On Mon, Jul 20, 2020 at 05:20:15PM +0200, Jan Beulich wrote: > wakeup_32 sets %ds and %es to BOOT_DS, while leaving %fs at what > wakeup_start did set it to, and %gs at whatever BIOS did load into it. > All of this may end up confusing the first load_segments() to run on > the BSP after resume, in

[PATCH] PCI: pciehp: Fix AB-BA deadlock between reset_lock and device_lock

2020-07-21 Thread Lukas Wunner
Back in 2013, commits 2e35afaefe64 ("PCI: pciehp: Add reset_slot() method") 608c388122c7 ("PCI: Add slot reset option to pci_dev_reset()") introduced the callback pciehp_reset_slot() to the PCIe hotplug driver and amended __pci_dev_reset() (today __pci_reset_function_locked()) to invoke it

[osstest PATCH] freebsd: remove freebsd- hostflags request from guest tests

2020-07-21 Thread Roger Pau Monne
Guest tests shouldn't care about the capabilities or firmware of the underlying hosts, so drop the request of specific freebsd- hostflags for FreeBSD guest tests. While there request the presence of the hvm hostflag since the FreeBSD guest tests are run in HVM mode. Signed-off-by: Roger Pau

Xen Security Advisory 329 v3 (CVE-2020-15852) - Linux ioperm bitmap context switching issues

2020-07-21 Thread Xen . org security team
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Xen Security Advisory CVE-2020-15852 / XSA-329 version 3 Linux ioperm bitmap context switching issues UPDATES IN VERSION 3 CVE assigned. ISSUE DESCRIPTION

Re: [PATCH v4 10/10] tools/proctrace: add proctrace tool

2020-07-21 Thread Wei Liu
On Thu, Jul 02, 2020 at 04:10:57PM +0100, Andrew Cooper wrote: [...] > > > +#include > > +#include > > +#include > > +#include > > + > > +#include > > +#include > > +#include > > + > > +#define BUF_SIZE (16384 * XC_PAGE_SIZE) > > This hardcodes the size of the buffer which is configurable

Re: [PATCH v4 09/10] tools/libxc: add xc_vmtrace_* functions

2020-07-21 Thread Wei Liu
On Tue, Jun 30, 2020 at 02:33:52PM +0200, Michał Leszczyński wrote: > From: Michal Leszczynski > > Add functions in libxc that use the new XEN_DOMCTL_vmtrace interface. > > Signed-off-by: Michal Leszczynski Acked-by: Wei Liu (Subject to acceptance of hypervisor patches)

Re: [PATCH] x86emul: support AVX512_VP2INTERSECT insns

2020-07-21 Thread Andrew Cooper
On 21/07/2020 11:47, Jan Beulich wrote: > On 21.07.2020 12:32, Andrew Cooper wrote: >> On 21/07/2020 11:23, Jan Beulich wrote: >>> --- a/tools/misc/xen-cpuid.c >>> +++ b/tools/misc/xen-cpuid.c >>> @@ -160,7 +160,7 @@ static const char *const str_7d0[32] = >>> [ 2] = "avx512_4vnniw", [ 3] =

Re: [PATCH] x86emul: support AVX512_VP2INTERSECT insns

2020-07-21 Thread Jan Beulich
On 21.07.2020 12:32, Andrew Cooper wrote: > On 21/07/2020 11:23, Jan Beulich wrote: >> --- a/tools/misc/xen-cpuid.c >> +++ b/tools/misc/xen-cpuid.c >> @@ -160,7 +160,7 @@ static const char *const str_7d0[32] = >> [ 2] = "avx512_4vnniw", [ 3] = "avx512_4fmaps", >> [ 4] = "fsrm", >> >> -

Re: Kexec and libxenctlr.so

2020-07-21 Thread Wei Liu
On Thu, Jul 02, 2020 at 06:34:48PM +0100, Julien Grall wrote: > Hi Wei, > > On 26/06/2020 12:08, Wei Liu wrote: > > On Thu, Jun 11, 2020 at 03:57:37PM +0100, Julien Grall wrote: > > > Hi all, > > > > > > kexec-tools has an option to load dynamically libxenctlr.so (not .so.4.x) > > > (see [1]). >

Re: [PATCH] compat: add a little bit of description to xlat.lst

2020-07-21 Thread Wei Liu
On Thu, Jul 16, 2020 at 02:21:33PM +0200, Jan Beulich wrote: > Requested-by: Roger Pau Monné > Signed-off-by: Jan Beulich Acked-by: Wei Liu This is much appreciated. > > --- a/xen/include/xlat.lst > +++ b/xen/include/xlat.lst > @@ -1,3 +1,22 @@ > +# There are a few fundamentally different

Re: [PATCH for-4.14] golang/xenlight: fix code generation for python 2.6

2020-07-21 Thread Wei Liu
On Tue, Jul 21, 2020 at 08:13:28AM +0100, Paul Durrant wrote: > > -Original Message- > > From: Nick Rosbrook > > Sent: 21 July 2020 00:55 > > To: xen-devel@lists.xenproject.org > > Cc: p...@xen.org; Nick Rosbrook ; George Dunlap > > ; > > Ian Jackson ; Wei Liu > > Subject: [PATCH

Re: [PATCH] x86emul: support AVX512_VP2INTERSECT insns

2020-07-21 Thread Andrew Cooper
On 21/07/2020 11:23, Jan Beulich wrote: > --- a/tools/misc/xen-cpuid.c > +++ b/tools/misc/xen-cpuid.c > @@ -160,7 +160,7 @@ static const char *const str_7d0[32] = > [ 2] = "avx512_4vnniw", [ 3] = "avx512_4fmaps", > [ 4] = "fsrm", > > -/* 8 */[ 9] = "srbds-ctrl", >

[libvirt test] 152064: regressions - FAIL

2020-07-21 Thread osstest service owner
flight 152064 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/152064/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-libvirt 6 libvirt-buildfail REGR. vs. 151777 build-i386-libvirt

Re: [PATCH v1 1/1] oxenstored: fix ABI breakage introduced in Xen 4.9.0

2020-07-21 Thread Wei Liu
On Wed, Jul 15, 2020 at 03:21:50PM +, Christian Lindig wrote: > > > From: Edwin Török > Sent: 15 July 2020 16:10 > To: xen-devel@lists.xenproject.org > Cc: Edwin Torok; Christian Lindig; David Scott; Ian Jackson; Wei Liu; Igor > Druzhinin > Subject:

[PATCH] x86emul: support AVX512_VP2INTERSECT insns

2020-07-21 Thread Jan Beulich
The standard memory access pattern once again should allow us to go without a test harness addition beyond the EVEX Disp8-scaling one. Signed-off-by: Jan Beulich --- (SDE: -tgl) --- a/tools/libxl/libxl_cpuid.c +++ b/tools/libxl/libxl_cpuid.c @@ -214,6 +214,7 @@ int

RE: [PATCH for-4.14] golang/xenlight: fix code generation for python 2.6

2020-07-21 Thread Paul Durrant
> -Original Message- > From: George Dunlap > Sent: 21 July 2020 10:54 > To: p...@xen.org > Cc: Andrew Cooper ; Nick Rosbrook > ; xen- > de...@lists.xenproject.org; Nick Rosbrook ; Ian > Jackson > ; Wei Liu > Subject: Re: [PATCH for-4.14] golang/xenlight: fix code generation for python

[PATCH][next] x86/ioperm: initialize pointer bitmap with NULL rather than 0

2020-07-21 Thread Colin King
From: Colin Ian King The pointer bitmap is being initialized with a plain integer 0, fix this by initializing it with a NULL instead. Cleans up sparse warning: arch/x86/xen/enlighten_pv.c:876:27: warning: Using plain integer as NULL pointer Signed-off-by: Colin Ian King ---

Re: RFC: PCI devices passthrough on Arm design proposal

2020-07-21 Thread Rahul Singh
> On 21 Jul 2020, at 12:23 am, Stefano Stabellini > wrote: > > On Fri, 17 Jul 2020, Bertrand Marquis wrote: >>> On 17 Jul 2020, at 16:06, Jan Beulich wrote: >>> >>> On 17.07.2020 15:59, Bertrand Marquis wrote: > On 17 Jul 2020, at 15:19, Jan Beulich wrote: > > On

Re: [PATCH for-4.14] golang/xenlight: fix code generation for python 2.6

2020-07-21 Thread George Dunlap
> On Jul 21, 2020, at 10:23 AM, Paul Durrant wrote: > >> -Original Message- >> From: Andrew Cooper >> Sent: 21 July 2020 10:21 >> To: p...@xen.org; 'Nick Rosbrook' ; >> xen-devel@lists.xenproject.org >> Cc: 'Nick Rosbrook' ; 'Ian Jackson' >> ; 'George >> Dunlap' ; 'Wei Liu' >>

[linux-linus test] 152052: regressions - trouble: blocked/fail/pass/starved

2020-07-21 Thread osstest service owner
flight 152052 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/152052/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-arm64-arm64-libvirt-xsm 16 guest-start/debian.repeat fail REGR. vs. 151214 build-armhf-pvops

RE: [PATCH for-4.14] golang/xenlight: fix code generation for python 2.6

2020-07-21 Thread Paul Durrant
> -Original Message- > From: Andrew Cooper > Sent: 21 July 2020 10:21 > To: p...@xen.org; 'Nick Rosbrook' ; > xen-devel@lists.xenproject.org > Cc: 'Nick Rosbrook' ; 'Ian Jackson' > ; 'George > Dunlap' ; 'Wei Liu' > Subject: Re: [PATCH for-4.14] golang/xenlight: fix code generation for

Re: [PATCH for-4.14] golang/xenlight: fix code generation for python 2.6

2020-07-21 Thread Andrew Cooper
On 21/07/2020 08:13, Paul Durrant wrote: >> -Original Message- >> From: Nick Rosbrook >> Sent: 21 July 2020 00:55 >> To: xen-devel@lists.xenproject.org >> Cc: p...@xen.org; Nick Rosbrook ; George Dunlap >> ; >> Ian Jackson ; Wei Liu >> Subject: [PATCH for-4.14] golang/xenlight: fix code

Re: [PATCH v2 01/11] xen/manage: keep track of the on-going suspend mode

2020-07-21 Thread Roger Pau Monné
Marek: I'm adding you in case you could be able to give this a try and make sure it doesn't break suspend for dom0. On Tue, Jul 21, 2020 at 12:17:36AM +, Anchal Agarwal wrote: > On Mon, Jul 20, 2020 at 11:37:05AM +0200, Roger Pau Monné wrote: > > CAUTION: This email originated from outside of

Re: [PATCH 5/8] bitmap: move to/from xenctl_bitmap conversion helpers

2020-07-21 Thread Jan Beulich
On 20.07.2020 18:20, Julien Grall wrote: > On 15/07/2020 11:40, Jan Beulich wrote: >> A subsequent change will exclude domctl.c from getting built for a >> particular configuration, yet the two functions get used from elsewhere. >> >> Signed-off-by: Jan Beulich >> >> --- a/xen/common/bitmap.c >>

RE: [PATCH for-4.14] golang/xenlight: fix code generation for python 2.6

2020-07-21 Thread Paul Durrant
> -Original Message- > From: Nick Rosbrook > Sent: 21 July 2020 00:55 > To: xen-devel@lists.xenproject.org > Cc: p...@xen.org; Nick Rosbrook ; George Dunlap > ; > Ian Jackson ; Wei Liu > Subject: [PATCH for-4.14] golang/xenlight: fix code generation for python 2.6 > > Before python

RE: [PATCH] SUPPORT.md: Spell correctly Experimental

2020-07-21 Thread Paul Durrant
> -Original Message- > From: Andrew Cooper > Sent: 20 July 2020 18:49 > To: Julien Grall ; xen-devel@lists.xenproject.org > Cc: p...@xen.org; Julien Grall ; George Dunlap > ; Ian > Jackson ; Jan Beulich ; Stefano > Stabellini > ; Wei Liu > Subject: Re: [PATCH] SUPPORT.md: Spell

[ovmf test] 152048: all pass - PUSHED

2020-07-21 Thread osstest service owner
flight 152048 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/152048/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf cb38ace647231076acfc0c5bdd21d3aff43e4f83 baseline version: ovmf

[xen-unstable test] 152045: tolerable trouble: fail/pass/starved - PUSHED

2020-07-21 Thread osstest service owner
flight 152045 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/152045/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt 14 saverestore-support-checkfail like 152031

Re: [PATCH v3 1/2] x86: restore pv_rtc_handler() invocation

2020-07-21 Thread Jan Beulich
On 20.07.2020 18:27, Jan Beulich wrote: > On 20.07.2020 17:28, Andrew Cooper wrote: >> On 16/07/2020 11:06, Jan Beulich wrote: >>> ACCESS_ONCE() guarantees single access, but doesn't guarantee that >>> the compiler wouldn't split this single access into multiple insns. >> >> ACCESS_ONCE() does

[qemu-mainline bisection] complete test-amd64-i386-xl-qemuu-ovmf-amd64

2020-07-21 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job test-amd64-i386-xl-qemuu-ovmf-amd64 testid debian-hvm-install Tree: linux git://xenbits.xen.org/linux-pvops.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: ovmf git://xenbits.xen.org/osstest/ovmf.git Tree: qemu