[xen-unstable-smoke test] 162955: trouble: blocked/broken/pass

2021-06-21 Thread osstest service owner
flight 162955 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/162955/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-arm64-xs

[ovmf test] 162938: trouble: blocked/broken

2021-06-21 Thread osstest service owner
flight 162938 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/162938/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-amd64-pvops

[xen-unstable-smoke test] 162950: trouble: blocked/broken/pass

2021-06-21 Thread osstest service owner
flight 162950 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/162950/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-arm64-xs

[qemu-mainline test] 162936: trouble: blocked/broken/pass

2021-06-21 Thread osstest service owner
flight 162936 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/162936/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-amd64-pvops

[xen-unstable-smoke test] 162946: trouble: blocked/broken/pass

2021-06-21 Thread osstest service owner
flight 162946 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/162946/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-arm64-xs

Re: [RESEND PATCH 12/12] golang/xenlight: add NotifyDomainDeath method to Context

2021-06-21 Thread Nick Rosbrook
On Fri, Jun 18, 2021 at 07:31:46PM +, George Dunlap wrote: > > > > On Jun 18, 2021, at 7:28 PM, George Dunlap wrote: > > > > > > > >> On May 24, 2021, at 9:36 PM, Nick Rosbrook wrote: > >> > >> Add a helper function to wait for domain death events, and then write > >> the events to a pr

Re: Linux 5.13-rc6 regression to 5.12.x: kernel OOM and panic during kernel boot in low memory Xen VM's (256MB assigned memory).

2021-06-21 Thread Sander Eikelenboom
On 21/06/2021 18:54, Rasmus Villemoes wrote: On 18/06/2021 03.06, Sander Eikelenboom wrote: On 17/06/2021 21:39, Sander Eikelenboom wrote: OK, done some experimentation and it seems with 256M assigned to the VM it was almost at the edge of OOM with the 5.12 kernel as well in the config I am

Re: [PATCH v13 01/12] swiotlb: Refactor swiotlb init functions

2021-06-21 Thread Christoph Hellwig
On Mon, Jun 21, 2021 at 10:59:20AM -0700, Stefano Stabellini wrote: > Just as a clarification: I was referring to the zeroing of "mem" in > swiotlb_late_init_with_tbl and swiotlb_init_with_tbl. While it looks > like Tom and Christoph are talking about the zeroing of "tlb". Indeed. > > The zeroi

[linux-linus test] 162932: trouble: blocked/broken/pass

2021-06-21 Thread osstest service owner
flight 162932 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/162932/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-amd64-pvops

[libvirt test] 162930: regressions - trouble: blocked/broken/fail/pass

2021-06-21 Thread osstest service owner
flight 162930 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/162930/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-amd64-pvops

[xen-unstable-smoke test] 162942: trouble: blocked/broken/pass

2021-06-21 Thread osstest service owner
flight 162942 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/162942/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-arm64-xs

Re: [PATCH v13 01/12] swiotlb: Refactor swiotlb init functions

2021-06-21 Thread Stefano Stabellini
On Fri, 18 Jun 2021, Christoph Hellwig wrote: > On Fri, Jun 18, 2021 at 09:09:17AM -0500, Tom Lendacky wrote: > > > swiotlb_init_with_tbl uses memblock_alloc to allocate the io_tlb_mem > > > and memblock_alloc[1] will do memset in memblock_alloc_try_nid[2], so > > > swiotlb_init_with_tbl is also go

Re: Linux 5.13-rc6 regression to 5.12.x: kernel OOM and panic during kernel boot in low memory Xen VM's (256MB assigned memory).

2021-06-21 Thread Rasmus Villemoes
On 18/06/2021 03.06, Sander Eikelenboom wrote: > On 17/06/2021 21:39, Sander Eikelenboom wrote: > > OK, done some experimentation and it seems with 256M assigned to the VM > it was almost at the edge of OOM with the 5.12 kernel as well in the > config I am using it. > With v5.12 when I assign 240

[xen-unstable test] 162929: trouble: blocked/broken/pass

2021-06-21 Thread osstest service owner
flight 162929 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/162929/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-amd64-prev

Re: [RESEND PATCH 00/12] golang/xenlight: domain life cycle support

2021-06-21 Thread Nick Rosbrook
On Mon, Jun 21, 2021 at 03:53:39PM +, George Dunlap wrote: > > > > On May 24, 2021, at 9:36 PM, Nick Rosbrook wrote: > > > > The primary goal of this patch series is to allow users of the xenlight > > package to manage a full domain life cycle. In particular, it provides > > support for rec

Re: [PATCH] credit2: make sure we pick a runnable unit from the runq if there is one

2021-06-21 Thread Dario Faggioli
Hi George (and sorry for the delay in replying), On Mon, 2021-06-07 at 12:10 +, George Dunlap wrote: > > On May 28, 2021, at 4:12 PM, Dario Faggioli > > wrote: > > Reported-by: Michał Leszczyński > > Reported-by: Dion Kant > > Signed-off-by: Dario Faggioli > > Thanks for tracking this dow

Re: [RESEND PATCH 03/12] golang/xenlight: fix string conversion in generated toC functions

2021-06-21 Thread Nick Rosbrook
On Fri, Jun 18, 2021 at 11:00:26AM +, George Dunlap wrote: > > > > On May 24, 2021, at 9:36 PM, Nick Rosbrook wrote: > > > > In gengotypes.py, the toC functions only set C string fields when > > the Go strings are non-empty. However, to prevent segfaults in some > > cases, these fields shou

Re: [RESEND PATCH 00/12] golang/xenlight: domain life cycle support

2021-06-21 Thread George Dunlap
> On May 24, 2021, at 9:36 PM, Nick Rosbrook wrote: > > The primary goal of this patch series is to allow users of the xenlight > package to manage a full domain life cycle. In particular, it provides > support for receiving domain death events so that domain shutdown, > reboot, destroy, etc. c

Re: [PATCH 3/6] xsm: enabling xsm to always be included

2021-06-21 Thread Jan Beulich
On 21.06.2021 12:41, Andrew Cooper wrote: > On 21/06/2021 07:58, Jan Beulich wrote: >> On 18.06.2021 22:27, Daniel P. Smith wrote: >>> On 6/18/21 8:26 AM, Jan Beulich wrote: On 18.06.2021 01:39, Daniel P. Smith wrote: > The only difference between !CONFIG_XSM and CONFIG_XSM with > !CO

[ovmf test] 162926: trouble: blocked/broken

2021-06-21 Thread osstest service owner
flight 162926 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/162926/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-amd64-pvops

Re: [PATCH 3/6] xsm: enabling xsm to always be included

2021-06-21 Thread Andrew Cooper
On 21/06/2021 07:58, Jan Beulich wrote: > On 18.06.2021 22:27, Daniel P. Smith wrote: >> On 6/18/21 8:26 AM, Jan Beulich wrote: >>> On 18.06.2021 01:39, Daniel P. Smith wrote: The only difference between !CONFIG_XSM and CONFIG_XSM with !CONFIG_XSM_SILO and !CONFIG_XSM_FLASK is wheth

Re: [PATCH] tools/xenstored: Don't crash xenstored when Live-Update is cancelled

2021-06-21 Thread Luca Fancellu
> On 17 Jun 2021, at 18:38, Julien Grall wrote: > > From: Julien GralL > > As Live-Update is asynchronous, it is possible to receive a request to > cancel it (either on the same connection or from a different one). > > Currently, this will crash xenstored because do_lu_start() assumes > lu_

Re: [PATCH 10/10] tools/xenstored: Delay new transaction while Live-Update is pending

2021-06-21 Thread Luca Fancellu
> On 16 Jun 2021, at 15:43, Julien Grall wrote: > > From: Julien Grall > > At the moment, Live-Update will, by default, not proceed if there are > in-flight transactions. It is possible force it by passing -F but this > will break any connection with in-flight transactions. > > There are PV

[qemu-mainline test] 162924: trouble: blocked/broken/pass

2021-06-21 Thread osstest service owner
flight 162924 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/162924/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 broken build-amd64-pvops

Re: [PATCH 09/10] tools/xenstored: Dump delayed requests

2021-06-21 Thread Luca Fancellu
> On 16 Jun 2021, at 15:43, Julien Grall wrote: > > From: Julien Grall > > Currently, only Live-Update request can be delayed. In a follow-up, > we will want to delay more requests (e.g. transaction start). > Therefore we want to preserve delayed requests across Live-Update. > > Delayed req

Re: [PATCH 08/10] tools/xenstored: Extend restore code to handle multiple input buffer

2021-06-21 Thread Luca Fancellu
> On 16 Jun 2021, at 15:43, Julien Grall wrote: > > From: Julien Grall > > Currently, the restore code is considering the stream will contain at > most one in-flight request per connection. In a follow-up changes, we > will want to transfer multiple in-flight requests. > > The function read

Re: [PATCH 07/10] tools/xenstored: delay_request: don't assume conn->in == in

2021-06-21 Thread Luca Fancellu
> On 16 Jun 2021, at 15:43, Julien Grall wrote: > > From: Julien Grall > > delay_request() is currently assuming that the request delayed is > always conn->in. This is currently correct, but it is a call for > a latent bug as the function allows the caller to specify any request. > > To pre

Re: [PATCH 06/10] tools/xenstored: Introduce a wrapper for conn->funcs->can_{read, write}

2021-06-21 Thread Luca Fancellu
> On 16 Jun 2021, at 15:43, Julien Grall wrote: > > From: Julien Grall > > Currently, the callbacks can_read and can_write are called directly. This > doesn't allow us to add generic check and therefore requires duplication. > > At the moment, one check that could benefit to be common is wh

Re: [PATCH 05/10] tools/xenstored: xenstored_core.h should include fcntl.h

2021-06-21 Thread Luca Fancellu
> On 16 Jun 2021, at 15:43, Julien Grall wrote: > > From: Julien Grall > > xenstored_core.h will consider live-udpate is not supported if > O_CLOEXEC doesn't exist. However, the header doesn't include the one > defining O_CLOEXEC (i.e. fcntl.h). This means that depending on > the header incl

Re: [PATCH 04/10] tools/xenstored: Limit the number of requests a connection can delay

2021-06-21 Thread Luca Fancellu
> On 16 Jun 2021, at 15:43, Julien Grall wrote: > > From: Julien Grall > > Currently, only liveupdate request can be delayed. The request can only > be performed by a privileged connection (e.g. dom0). So it is fine to > have no limits. > > In a follow-up patch we will want to delay request

Re: [PATCH 03/10] tools/xenstore: Don't assume conn->in points to the LU request

2021-06-21 Thread Luca Fancellu
> On 16 Jun 2021, at 15:43, Julien Grall wrote: > > From: Julien Grall > > call_delayed() is currently assuming that conn->in is NULL when > handling delayed request. However, the connection is not paused. > Therefore new request can be processed and conn->in may be non-NULL > if we have onl

Re: [PATCH 02/10] tools/xenstored: Introduce lu_get_connection() and use it

2021-06-21 Thread Luca Fancellu
> On 16 Jun 2021, at 15:43, Julien Grall wrote: > > From: Julien Grall > > At the moment, dump_state_buffered_data() is taking two connections > in parameters (one is the connection to dump, the other is the > connection used to request LU). The naming doesn't help to > distinguish (c vs con

Re: [PATCH for-4.13] compiler.h: define CONFIG_GCC_VERSION

2021-06-21 Thread Jan Beulich
On 18.06.2021 18:42, Olaf Hering wrote: > Fixes commit fa5afbbc20ef3577c5338f9d0b24dad45cef59cd, > due to lack of commit 534519f0514f52007d504e0f2eeb714de7b2468d. > > Signed-off-by: Olaf Hering Reviewed-by: Jan Beulich albeit ... > --- a/xen/include/xen/compiler.h > +++ b/xen/include/xen/compi

Re: [PATCH v1] compiler.h: define CONFIG_GCC_VERSION

2021-06-21 Thread Jan Beulich
On 18.06.2021 18:55, Olaf Hering wrote: > Am Fri, 18 Jun 2021 17:46:47 +0100 > schrieb Andrew Cooper : > >> On 18/06/2021 17:42, Olaf Hering wrote: >>> Fixes commit fa5afbbc20ef3577c5338f9d0b24dad45cef59cd, >>> due to lack of commit 534519f0514f52007d504e0f2eeb714de7b2468d. > >> Presumably you're

Re: [PATCH 3/6] xsm: enabling xsm to always be included

2021-06-21 Thread Jan Beulich
On 18.06.2021 23:20, Andrew Cooper wrote: > On 18/06/2021 13:26, Jan Beulich wrote: >> On 18.06.2021 01:39, Daniel P. Smith wrote: >>> The only difference between !CONFIG_XSM and CONFIG_XSM with >>> !CONFIG_XSM_SILO and !CONFIG_XSM_FLASK >>> is whether the XSM hooks in dummy.h are called as static