Re: [RFC PATCH V1 04/12] xen/arm: Introduce arch specific bits for IOREQ/DM features

2020-08-10 Thread Oleksandr
On 11.08.20 01:37, Julien Grall wrote: Hi Julien On Mon, 10 Aug 2020 at 21:29, Oleksandr wrote: On 10.08.20 22:00, Julien Grall wrote: Hi Julien @@ -2275,6 +2282,16 @@ static void check_for_vcpu_work(void) */ void leave_hypervisor_to_guest(void) { +#ifdef CONFIG_IOREQ_SERVER +

[seabios test] 152554: tolerable FAIL - PUSHED

2020-08-10 Thread osstest service owner
flight 152554 seabios real [real] http://logs.test-lab.xenproject.org/osstest/logs/152554/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-qemuu-ws16-amd64 17 guest-stopfail like 152267 test-amd64-i386-xl-qemuu-win7-amd64 17 gu

Re: [RFC PATCH V1 01/12] hvm/ioreq: Make x86's IOREQ feature common

2020-08-10 Thread Stefano Stabellini
On Mon, 10 Aug 2020, Julien Grall wrote: > On 07/08/2020 00:48, Stefano Stabellini wrote: > > On Thu, 6 Aug 2020, Julien Grall wrote: > > > On 06/08/2020 01:37, Stefano Stabellini wrote: > > > > On Wed, 5 Aug 2020, Julien Grall wrote: > > > > > On 04/08/2020 20:11, Stefano Stabellini wrote: > > > >

Re: [RFC PATCH V1 01/12] hvm/ioreq: Make x86's IOREQ feature common

2020-08-10 Thread Stefano Stabellini
On Sat, 8 Aug 2020, Oleksandr wrote: > On 08.08.20 00:50, Stefano Stabellini wrote: > > On Fri, 7 Aug 2020, Oleksandr wrote: > > > On 06.08.20 03:37, Stefano Stabellini wrote: > > > > > > Hi Stefano > > > > > > Trying to simulate IO_RETRY handling mechanism (according to model below) > > > I > >

Re: [RFC PATCH V1 05/12] hvm/dm: Introduce xendevicemodel_set_irq_level DM op

2020-08-10 Thread Stefano Stabellini
On Sat, 8 Aug 2020, Julien Grall wrote: > On Fri, 7 Aug 2020 at 22:51, Stefano Stabellini > wrote: > > > > On Fri, 7 Aug 2020, Jan Beulich wrote: > > > On 07.08.2020 01:49, Stefano Stabellini wrote: > > > > On Thu, 6 Aug 2020, Julien Grall wrote: > > > >> On 06/08/2020 01:37, Stefano Stabellini w

Re: [RFC PATCH V1 01/12] hvm/ioreq: Make x86's IOREQ feature common

2020-08-10 Thread Stefano Stabellini
On Mon, 10 Aug 2020, Oleksandr wrote: > On 08.08.20 01:19, Oleksandr wrote: > > On 08.08.20 00:50, Stefano Stabellini wrote: > > > On Fri, 7 Aug 2020, Oleksandr wrote: > > > > On 06.08.20 03:37, Stefano Stabellini wrote: > > > > > > > > Hi Stefano > > > > > > > > Trying to simulate IO_RETRY handl

Re: [RFC PATCH V1 04/12] xen/arm: Introduce arch specific bits for IOREQ/DM features

2020-08-10 Thread Julien Grall
On Mon, 10 Aug 2020 at 21:29, Oleksandr wrote: > > > On 10.08.20 22:00, Julien Grall wrote: > > Hi Julien > > > > >>> > > @@ -2275,6 +2282,16 @@ static void check_for_vcpu_work(void) > >*/ > > void leave_hypervisor_to_guest(void) > > { > > +#ifdef CONFIG_IOREQ_SERVER >

Re: [PATCH 3/4] build: also check for empty .bss.* in .o -> .init.o conversion

2020-08-10 Thread Andrew Cooper
On 07/08/2020 16:40, Jan Beulich wrote: > On 07.08.2020 17:12, Andrew Cooper wrote: >> On 07/08/2020 11:56, Jan Beulich wrote: >>> On 06.08.2020 18:16, Andrew Cooper wrote: On 06/08/2020 10:05, Jan Beulich wrote: Can't we remove all of this by having CONFIG_XEN_PE expressed/selectable >>>

Re: [PATCH 3/4] build: also check for empty .bss.* in .o -> .init.o conversion

2020-08-10 Thread Andrew Cooper
On 07/08/2020 16:40, Jan Beulich wrote: > On 07.08.2020 17:12, Andrew Cooper wrote: >> On 07/08/2020 11:56, Jan Beulich wrote: >>> On 06.08.2020 18:16, Andrew Cooper wrote: On 06/08/2020 10:05, Jan Beulich wrote: Can't we remove all of this by having CONFIG_XEN_PE expressed/selectable >>>

Re: [PATCH 4/4] EFI: free unused boot mem in at least some cases

2020-08-10 Thread Andrew Cooper
On 06/08/2020 10:06, Jan Beulich wrote: > Address at least the primary reason why 52bba67f8b87 ("efi/boot: Don't > free ebmalloc area at all") was put in place: Make xen_in_range() aware > of the freed range. This is in particular relevant for EFI-enabled > builds not actually running on EFI, as th

Re: [PATCH 3/3] x86: don't override INVALID_M2P_ENTRY with SHARED_M2P_ENTRY

2020-08-10 Thread Andrew Cooper
On 06/08/2020 10:29, Jan Beulich wrote: > While in most cases code ahead of the invocation of set_gpfn_from_mfn() > deals with shared pages, at least in set_typed_p2m_entry() I can't spot > such handling (it's entirely possible there's code missing there). Let's > try to play safe and add an extra

Re: [RFC PATCH V1 04/12] xen/arm: Introduce arch specific bits for IOREQ/DM features

2020-08-10 Thread Oleksandr
On 10.08.20 22:00, Julien Grall wrote: Hi Julien @@ -2275,6 +2282,16 @@ static void check_for_vcpu_work(void)    */   void leave_hypervisor_to_guest(void)   { +#ifdef CONFIG_IOREQ_SERVER +    /* + * XXX: Check the return. Shall we call that in + * continue_running and context_swit

Re: [RFC PATCH V1 04/12] xen/arm: Introduce arch specific bits for IOREQ/DM features

2020-08-10 Thread Oleksandr
On 10.08.20 21:25, Julien Grall wrote: Hi Julien Do you mind to provide more details? Previous patch "xen/mm: Make x86's XENMEM_resource_ioreq_server handling common" breaks build on Arm as it includes xen/hvm/ioreq.h which requires arch header to be present (asm/hvm/ioreq.h). But the mi

Re: [RFC PATCH V1 01/12] hvm/ioreq: Make x86's IOREQ feature common

2020-08-10 Thread Julien Grall
On 07/08/2020 00:48, Stefano Stabellini wrote: On Thu, 6 Aug 2020, Julien Grall wrote: On 06/08/2020 01:37, Stefano Stabellini wrote: On Wed, 5 Aug 2020, Julien Grall wrote: On 04/08/2020 20:11, Stefano Stabellini wrote: On Tue, 4 Aug 2020, Julien Grall wrote: On 04/08/2020 12:10, Oleksan

Re: [PATCH] libxl: avoid golang building without CONFIG_GOLANG=y

2020-08-10 Thread George Dunlap
> On Aug 5, 2020, at 10:55 AM, Ian Jackson wrote: > > Nick Rosbrook writes ("Re: [PATCH] libxl: avoid golang building without > CONFIG_GOLANG=y"): >> Jan - is the problem specifically that a fresh clone, or `git >> checkout`, etc. changes file timestamps in a way that triggers make to >> rebu

Re: [RFC PATCH V1 04/12] xen/arm: Introduce arch specific bits for IOREQ/DM features

2020-08-10 Thread Julien Grall
On 10/08/2020 19:09, Oleksandr wrote: On 05.08.20 12:32, Julien Grall wrote: Hi Julien @@ -2275,6 +2282,16 @@ static void check_for_vcpu_work(void)    */   void leave_hypervisor_to_guest(void)   { +#ifdef CONFIG_IOREQ_SERVER +    /* + * XXX: Check the return. Shall we call that in + 

Re: [PATCH v3 4/7] x86/paravirt: remove 32-bit support from PARAVIRT_XXL

2020-08-10 Thread Jürgen Groß
On 10.08.20 18:53, Boris Ostrovsky wrote: On 8/10/20 12:39 AM, Jürgen Groß wrote: On 09.08.20 04:34, Boris Ostrovsky wrote: On 8/7/20 4:38 AM, Juergen Gross wrote: @@ -377,10 +373,7 @@ static inline pte_t __pte(pteval_t val)   {   pteval_t ret;   -    if (sizeof(pteval_t) > sizeof(long)

Re: [PATCH v2 1/7] x86/EFI: sanitize build logic

2020-08-10 Thread Andrew Cooper
On 07/08/2020 17:33, Andrew Cooper wrote: > On 07/08/2020 12:32, Jan Beulich wrote: >> With changes done over time and as far as linking goes, the only special >> thing about building with EFI support enabled is the need for the dummy >> relocations object for xen.gz uniformly in all build stages.

Re: [RFC PATCH V1 04/12] xen/arm: Introduce arch specific bits for IOREQ/DM features

2020-08-10 Thread Julien Grall
On 06/08/2020 14:27, Oleksandr wrote: On 06.08.20 14:08, Julien Grall wrote: Hi Julien What is this function supposed to do? Agree, sounds confusing a bit. I assume it is supposed to complete Guest MMIO access after finishing emulation. Shall I rename it to something appropriate (mayb

Re: [RFC PATCH V1 04/12] xen/arm: Introduce arch specific bits for IOREQ/DM features

2020-08-10 Thread Oleksandr
Hi @@ -2275,6 +2282,16 @@ static void check_for_vcpu_work(void)    */   void leave_hypervisor_to_guest(void)   { +#ifdef CONFIG_IOREQ_SERVER +    /* + * XXX: Check the return. Shall we call that in + * continue_running and context_switch instead? + * The benefits would be to a

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

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

[linux-linus test] 152550: regressions - FAIL

2020-08-10 Thread osstest service owner
flight 152550 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/152550/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ws16-amd64 7 xen-boot fail REGR. vs. 152332 test-amd64-i386-qem

Re: [RFC PATCH V1 04/12] xen/arm: Introduce arch specific bits for IOREQ/DM features

2020-08-10 Thread Oleksandr
On 05.08.20 12:32, Julien Grall wrote: Hi Julien @@ -2275,6 +2282,16 @@ static void check_for_vcpu_work(void)    */   void leave_hypervisor_to_guest(void)   { +#ifdef CONFIG_IOREQ_SERVER +    /* + * XXX: Check the return. Shall we call that in + * continue_running and context_switch

Re: [RFC] efi/boot: Unified Xen executable for UEFI Secure Boot support

2020-08-10 Thread Andrew Cooper
On 07/08/2020 19:22, Trammell Hudson wrote: > On Thursday, August 6, 2020 8:14 PM, Andrew Cooper > wrote: >> For SecureBoot, it is important that nothing which is signed can be >> tricked into running unsigned code. >> >> That includes configuration such as xen.cfg or the command line. >> Consumi

Re: [RFC PATCH V1 01/12] hvm/ioreq: Make x86's IOREQ feature common

2020-08-10 Thread Oleksandr
On 10.08.20 19:29, Julien Grall wrote: Hi, Hi Julien On 06/08/2020 12:37, Oleksandr wrote: On 05.08.20 16:30, Julien Grall wrote: Hi, Hi Julien On 03/08/2020 19:21, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko As a lot of x86 code can be re-used on Arm later on, this

Re: [PATCH v3 4/7] x86/paravirt: remove 32-bit support from PARAVIRT_XXL

2020-08-10 Thread Boris Ostrovsky
On 8/10/20 12:39 AM, Jürgen Groß wrote: > On 09.08.20 04:34, Boris Ostrovsky wrote: >> On 8/7/20 4:38 AM, Juergen Gross wrote: >>> @@ -377,10 +373,7 @@ static inline pte_t __pte(pteval_t val) >>>   { >>>   pteval_t ret; >>>   -    if (sizeof(pteval_t) > sizeof(long)) >>> -    ret = PVOP_CAL

Re: [RFC PATCH V1 01/12] hvm/ioreq: Make x86's IOREQ feature common

2020-08-10 Thread Julien Grall
Hi, On 06/08/2020 12:37, Oleksandr wrote: On 05.08.20 16:30, Julien Grall wrote: Hi, Hi Julien On 03/08/2020 19:21, Oleksandr Tyshchenko wrote: From: Oleksandr Tyshchenko As a lot of x86 code can be re-used on Arm later on, this patch splits IOREQ support into common and arch specific

Re: [PATCH v2 4/7] bitmap: move to/from xenctl_bitmap conversion helpers

2020-08-10 Thread Julien Grall
Hi, On 07/08/2020 12:33, 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 Acked-by: Julien Grall Cheers, --- v2: Move function decls to xen/bitmap.h.

[OSSTEST PATCH] SQL: Change "... LIKE '...\_...' ..." to "... LIKE '...\\_...' ..."

2020-08-10 Thread Ian Jackson
Perl's "" quotes, and corresponding < --- Osstest/Executive.pm | 2 +- cs-bisection-step| 8 mg-force-push| 2 +- mg-report-host-usage-collect | 2 +- sg-report-flight | 4 ++-- sg-report-host-history | 8 sg-report-job-histor

Re: [PATCH] OSSTEST: Install libtirpc-dev for libvirt builds

2020-08-10 Thread Jim Fehlig
On 8/10/20 4:13 AM, Ian Jackson wrote: Jim Fehlig writes ("[PATCH] OSSTEST: Install libtirpc-dev for libvirt builds"): The check for XDR support was changed in libvirt commit d7147b3797 to use libtirpc pkg-config instead of complicated AC_CHECK_LIB, AC_COMPILE_IFELSE, et. al. logic. The libvirt

Re: [RFC PATCH V1 01/12] hvm/ioreq: Make x86's IOREQ feature common

2020-08-10 Thread Oleksandr
Hi On 08.08.20 01:19, Oleksandr wrote: On 08.08.20 00:50, Stefano Stabellini wrote: Hi Stefano On Fri, 7 Aug 2020, Oleksandr wrote: On 06.08.20 03:37, Stefano Stabellini wrote: Hi Stefano Trying to simulate IO_RETRY handling mechanism (according to model below) I continuously get IO_

Re: Xen VMI APIs

2020-08-10 Thread Wei Liu
On Mon, Aug 10, 2020 at 06:10:17PM +0530, CS17B006 ARNAB DUTTA wrote: > Respected Sir/Ma'm > I am Arnab and I am currently writing rust bindings for xen vmi APIs for a > project known as Libmicrovmi which aims to provide unified vmi APIs for > various hypervisors. I am facing some difficulty regard

Xen VMI APIs

2020-08-10 Thread CS17B006 ARNAB DUTTA
Respected Sir/Ma'm I am Arnab and I am currently writing rust bindings for xen vmi APIs for a project known as Libmicrovmi which aims to provide unified vmi APIs for various hypervisors. I am facing some difficulty regarding setting up the event system in the sense that after enabling monitoring fo

[xen-unstable test] 152548: tolerable FAIL

2020-08-10 Thread osstest service owner
flight 152548 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/152548/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-qemuu-win7-amd64 17 guest-stopfail like 152542 test-amd64-amd64-xl-qemuu-ws16-amd64

[OSSTEST PATCH 1/6] cr-publish-flight-logs: Make a proper option parser

2020-08-10 Thread Ian Jackson
I was going to add another option but changed my mind. But, let's keep this patch anyway. Signed-off-by: Ian Jackson --- cr-publish-flight-logs | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/cr-publish-flight-logs b/cr-publish-flight-logs index faae7e0e..717cb95

[OSSTEST PATCH 6/6] timing traces: cri-args-hostlists: Add some timestamps

2020-08-10 Thread Ian Jackson
Pass --report-processing-start-time, and add some calls to date >&2. Signed-off-by: Ian Jackson --- cri-args-hostlists | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/cri-args-hostlists b/cri-args-hostlists index 61572c2d..994e00c0 100644 --- a/cri-args-hostlists

[OSSTEST PATCH 5/6] timing traces: Executive: Provide processing timestamp facilities

2020-08-10 Thread Ian Jackson
Nothing uses these yet. Signed-off-by: Ian Jackson --- Osstest/Executive.pm | 13 + 1 file changed, 13 insertions(+) diff --git a/Osstest/Executive.pm b/Osstest/Executive.pm index c46a979d..1e94b282 100644 --- a/Osstest/Executive.pm +++ b/Osstest/Executive.pm @@ -51,6 +51,7 @@ BEGIN

[OSSTEST PATCH 4/6] timing traces: sg-check-tested: Add prints of (show_abs_time time)

2020-08-10 Thread Ian Jackson
More tools to see where it is faffing too much. Signed-off-by: Ian Jackson --- sg-check-tested | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sg-check-tested b/sg-check-tested index 7946e4b5..39c0d0c2 100755 --- a/sg-check-tested +++ b/sg-check-tested @@ -148,7 +148,

[OSSTEST PATCH 2/6] timing traces: cr-publish-flight-logs: Report more progress

2020-08-10 Thread Ian Jackson
More tools to see where it is faffing too much. Signed-off-by: Ian Jackson --- cr-publish-flight-logs | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/cr-publish-flight-logs b/cr-publish-flight-logs index 717cb957..ba5d3b70 100755 --- a/cr-publish-flight-logs +++ b/cr-pu

xen-devel@lists.xenproject.org

2020-08-10 Thread Ian Jackson
More tools to see where it is faffing too much. Signed-off-by: Ian Jackson --- cr-daily-branch | 12 1 file changed, 12 insertions(+) diff --git a/cr-daily-branch b/cr-daily-branch index d84dcc8e..b8f221ee 100755 --- a/cr-daily-branch +++ b/cr-daily-branch @@ -51,6 +51,7 @@ determi

Re: [PATCH] OSSTEST: Install libtirpc-dev for libvirt builds

2020-08-10 Thread Ian Jackson
Jim Fehlig writes ("[PATCH] OSSTEST: Install libtirpc-dev for libvirt builds"): > The check for XDR support was changed in libvirt commit d7147b3797 > to use libtirpc pkg-config instead of complicated AC_CHECK_LIB, > AC_COMPILE_IFELSE, et. al. logic. The libvirt OSSTEST has been > failing since thi

[linux-linus test] 152547: regressions - FAIL

2020-08-10 Thread osstest service owner
flight 152547 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/152547/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ws16-amd64 7 xen-boot fail REGR. vs. 152332 test-amd64-i386-qem