Re: [PATCH v4 1/2] xen-pciback: prepare for the split for stub and PV

On 28.09.21 08:56, Oleksandr Andrushchenko wrote: On 28.09.21 09:42, Jan Beulich wrote: On 28.09.2021 06:18, Stefano Stabellini wrote: On Mon, 27 Sep 2021, Juergen Gross wrote: On 27.09.21 09:35, Oleksandr Andrushchenko wrote: On 27.09.21 10:26, Jan Beulich wrote: On 27.09.2021 08:58, Oleks

Re: [PATCH v4 1/2] xen-pciback: prepare for the split for stub and PV

On 28.09.21 09:42, Jan Beulich wrote: > On 28.09.2021 06:18, Stefano Stabellini wrote: >> On Mon, 27 Sep 2021, Juergen Gross wrote: >>> On 27.09.21 09:35, Oleksandr Andrushchenko wrote: On 27.09.21 10:26, Jan Beulich wrote: > On 27.09.2021 08:58, Oleksandr Andrushchenko wrote: >> From

Re: [PATCH v4 1/2] xen-pciback: prepare for the split for stub and PV

On 28.09.2021 06:18, Stefano Stabellini wrote: > On Mon, 27 Sep 2021, Juergen Gross wrote: >> On 27.09.21 09:35, Oleksandr Andrushchenko wrote: >>> >>> On 27.09.21 10:26, Jan Beulich wrote: On 27.09.2021 08:58, Oleksandr Andrushchenko wrote: > From: Oleksandr Andrushchenko > > Cur

Re: [Stratos-dev] Xen Rust VirtIO demos work breakdown for Project Stratos

On Tue, Sep 28, 2021 at 7:55 AM Christopher Clark wrote: > > On Mon, Sep 27, 2021 at 3:06 AM Alex Bennée via Stratos-dev > wrote: >> >> >> Marek Marczykowski-Górecki writes: >> >> > [[PGP Signed Part:Undecided]] >> > On Fri, Sep 24, 2021 at 05:02:46PM +0100, Alex Bennée wrote: >> > That is a si

Re: [PATCH V3 1/3] xen: Introduce "gpaddr_bits" field to XEN_SYSCTL_physinfo

Hi Oleksandr, On 24.09.2021 00:48, Oleksandr Tyshchenko wrote: > From: Oleksandr Tyshchenko > > We need to pass info about maximum supported guest address > space size to the toolstack on Arm in order to properly > calculate the base and size of the extended region (safe range) > for the guest.

Re: [Stratos-dev] Xen Rust VirtIO demos work breakdown for Project Stratos

On Mon, 27 Sep 2021, Christopher Clark wrote: > On Mon, Sep 27, 2021 at 3:06 AM Alex Bennée via Stratos-dev > wrote: > > Marek Marczykowski-Górecki writes: > > > [[PGP Signed Part:Undecided]] > > On Fri, Sep 24, 2021 at 05:02:46PM +0100, Alex Bennée wrote: > >> Hi, >

[PATCH AUTOSEL 4.4 1/5] xen-netback: correct success/error reporting for the SKB-with-fraglist case

From: Jan Beulich [ Upstream commit 3ede7f84c7c21f93c5eac611d60eba3f2c765e0f ] When re-entering the main loop of xenvif_tx_check_gop() a 2nd time, the special considerations for the head of the SKB no longer apply. Don't mistakenly report ERROR to the frontend for the first entry in the list, ev

[PATCH AUTOSEL 4.9 2/6] xen-netback: correct success/error reporting for the SKB-with-fraglist case

From: Jan Beulich [ Upstream commit 3ede7f84c7c21f93c5eac611d60eba3f2c765e0f ] When re-entering the main loop of xenvif_tx_check_gop() a 2nd time, the special considerations for the head of the SKB no longer apply. Don't mistakenly report ERROR to the frontend for the first entry in the list, ev

[PATCH AUTOSEL 4.14 2/8] xen-netback: correct success/error reporting for the SKB-with-fraglist case

From: Jan Beulich [ Upstream commit 3ede7f84c7c21f93c5eac611d60eba3f2c765e0f ] When re-entering the main loop of xenvif_tx_check_gop() a 2nd time, the special considerations for the head of the SKB no longer apply. Don't mistakenly report ERROR to the frontend for the first entry in the list, ev

[PATCH AUTOSEL 4.19 02/10] xen-netback: correct success/error reporting for the SKB-with-fraglist case

From: Jan Beulich [ Upstream commit 3ede7f84c7c21f93c5eac611d60eba3f2c765e0f ] When re-entering the main loop of xenvif_tx_check_gop() a 2nd time, the special considerations for the head of the SKB no longer apply. Don't mistakenly report ERROR to the frontend for the first entry in the list, ev

[PATCH AUTOSEL 5.4 02/11] xen-netback: correct success/error reporting for the SKB-with-fraglist case

From: Jan Beulich [ Upstream commit 3ede7f84c7c21f93c5eac611d60eba3f2c765e0f ] When re-entering the main loop of xenvif_tx_check_gop() a 2nd time, the special considerations for the head of the SKB no longer apply. Don't mistakenly report ERROR to the frontend for the first entry in the list, ev

[PATCH AUTOSEL 5.10 08/23] xen-netback: correct success/error reporting for the SKB-with-fraglist case

From: Jan Beulich [ Upstream commit 3ede7f84c7c21f93c5eac611d60eba3f2c765e0f ] When re-entering the main loop of xenvif_tx_check_gop() a 2nd time, the special considerations for the head of the SKB no longer apply. Don't mistakenly report ERROR to the frontend for the first entry in the list, ev

[PATCH AUTOSEL 5.14 29/40] swiotlb-xen: ensure to issue well-formed XENMEM_exchange requests

From: Jan Beulich [ Upstream commit 9074c79b62b6e0d91d7f716c6e4e9968eaf9e043 ] While the hypervisor hasn't been enforcing this, we would still better avoid issuing requests with GFNs not aligned to the requested order. Instead of altering the value also in the call to panic(), drop it there for

[PATCH AUTOSEL 5.14 28/40] Xen/gntdev: don't ignore kernel unmapping error

From: Jan Beulich [ Upstream commit f28347cc66395e96712f5c2db0a302ee75bafce6 ] While working on XSA-361 and its follow-ups, I failed to spot another place where the kernel mapping part of an operation was not treated the same as the user space part. Detect and propagate errors and add a 2nd pr_d

Re: [Stratos-dev] Xen Rust VirtIO demos work breakdown for Project Stratos

On Mon, Sep 27, 2021 at 3:06 AM Alex Bennée via Stratos-dev < stratos-...@op-lists.linaro.org> wrote: > > Marek Marczykowski-Górecki writes: > > > [[PGP Signed Part:Undecided]] > > On Fri, Sep 24, 2021 at 05:02:46PM +0100, Alex Bennée wrote: > >> Hi, > > > > Hi, > > > >> 2.1 Stable ABI for foreig

[PATCH AUTOSEL 5.14 09/40] xen-netback: correct success/error reporting for the SKB-with-fraglist case

From: Jan Beulich [ Upstream commit 3ede7f84c7c21f93c5eac611d60eba3f2c765e0f ] When re-entering the main loop of xenvif_tx_check_gop() a 2nd time, the special considerations for the head of the SKB no longer apply. Don't mistakenly report ERROR to the frontend for the first entry in the list, ev

Re: [PATCH v2 03/10] nvme-multipath: add error handling support for add_disk()

On 9/28/21 12:00 AM, Luis Chamberlain wrote: We never checked for errors on add_disk() as this function returned void. Now that this is fixed, use the shiny new error handling. Since we now can tell for sure when a disk was added, move setting the bit NVME_NSHEAD_DISK_LIVE only when we did add t

Re: [PATCH V5 12/12] net: netvsc: Add Isolation VM support for netvsc driver

On Mon, Sep 27, 2021 at 10:26:43PM +0800, Tianyu Lan wrote: > Hi Christoph: > Gentile ping. The swiotlb and shared memory mapping changes in this > patchset needs your reivew. Could you have a look? I'm a little too busy for a review of such a huge patchset right now. That being said here are

[linux-linus test] 165225: regressions - FAIL

flight 165225 linux-linus real [real] flight 165229 linux-linus real-retest [real] http://logs.test-lab.xenproject.org/osstest/logs/165225/ http://logs.test-lab.xenproject.org/osstest/logs/165229/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run

[PATCH v4 7/8] PCI: Replace pci_dev::driver usage by pci_dev::dev.driver

From: Uwe Kleine-König struct pci_dev::driver contains (apart from a constant offset) the same data as struct pci_dev::dev->driver. Replace all remaining users of the former pointer by the latter to allow removing the former. Reviewed-by: Boris Ostrovsky Signed-off-by: Uwe Kleine-König --- ar

[PATCH v4 0/8] PCI: Drop duplicated tracking of a pci_dev's bound driver

From: Uwe Kleine-König Hello, this is v4 of the quest to drop the "driver" member from struct pci_dev which tracks the same data (apart from a constant offset) as dev.driver. Changes since v3: - Add some Reviewed-by and Acked-by tags - Rebase to v5.15-rc3 (no conflicts) - Changes in patch #4

[PATCH v4 3/8] xen/pci: Drop some checks that are always true

From: Uwe Kleine-König pcifront_common_process() has a check at the start that exits early if pcidev or pdidev->driver are NULL. So simplify the following code by not checking these two again. Reviewed-by: Boris Ostrovsky Reviewed-by: Christoph Hellwig Signed-off-by: Uwe Kleine-König --- dri

RE: [PATCH 20/37] xen: introduce CONFIG_EFI to stub API for non-EFI architecture

On Tue, 28 Sep 2021, Wei Chen wrote: > > -Original Message- > > From: Stefano Stabellini > > Sent: 2021年9月28日 9:00 > > To: Wei Chen > > Cc: Jan Beulich ; xen-devel@lists.xenproject.org; > > jul...@xen.org; Bertrand Marquis ; Stefano > > Stabellini > > Subject: RE: [PATCH 20/37] xen: intr

RE: [PATCH 08/37] xen/x86: add detection of discontinous node memory range

On Tue, 28 Sep 2021, Wei Chen wrote: > Hi Stefano, > > > -Original Message- > > From: Stefano Stabellini > > Sent: 2021年9月28日 1:19 > > To: Wei Chen > > Cc: Stefano Stabellini ; xen- > > de...@lists.xenproject.org; jul...@xen.org; Bertrand Marquis > > ; jbeul...@suse.com; andrew.coop...@c

Re: [PATCH v4 1/2] xen-pciback: prepare for the split for stub and PV

On 28.09.21 07:18, Stefano Stabellini wrote: > On Mon, 27 Sep 2021, Juergen Gross wrote: >> On 27.09.21 09:35, Oleksandr Andrushchenko wrote: >>> On 27.09.21 10:26, Jan Beulich wrote: On 27.09.2021 08:58, Oleksandr Andrushchenko wrote: > From: Oleksandr Andrushchenko > > Currentl

Re: [PATCH] xen/arm: optee: Fix arm_smccc_smc's a0 for OPTEE_SMC_DISABLE_SHM_CACHE

On Mon, 27 Sep 2021, Oleksandr Tyshchenko wrote: > From: Oleksandr Tyshchenko > > Fix a possible copy-paste error in arm_smccc_smc's first argument (a0) > for OPTEE_SMC_DISABLE_SHM_CACHE case. > > This error causes Linux > v5.14-rc5 (b5c10dd04b7418793517e3286cde5c04759a86de > optee: Clear stale

Re: [PATCH v2 10/11] xen/arm: Do not map PCI ECAM and MMIO space to Domain-0's p2m

On 28.09.21 07:00, Stefano Stabellini wrote: > On Mon, 27 Sep 2021, Oleksandr Andrushchenko wrote: >> On 25.09.21 03:44, Stefano Stabellini wrote: >>> On Thu, 23 Sep 2021, Oleksandr Andrushchenko wrote: From: Oleksandr Andrushchenko PCI host bridges are special devices in terms of

RE: [PATCH 08/37] xen/x86: add detection of discontinous node memory range

Hi Stefano, > -Original Message- > From: Stefano Stabellini > Sent: 2021年9月28日 1:19 > To: Wei Chen > Cc: Stefano Stabellini ; xen- > de...@lists.xenproject.org; jul...@xen.org; Bertrand Marquis > ; jbeul...@suse.com; andrew.coop...@citrix.com; > roger@citrix.com; w...@xen.org > Subje

Re: [PATCH v4 1/2] xen-pciback: prepare for the split for stub and PV

On Mon, 27 Sep 2021, Juergen Gross wrote: > On 27.09.21 09:35, Oleksandr Andrushchenko wrote: > > > > On 27.09.21 10:26, Jan Beulich wrote: > > > On 27.09.2021 08:58, Oleksandr Andrushchenko wrote: > > > > From: Oleksandr Andrushchenko > > > > > > > > Currently PCI backend implements multiple fu

RE: [PATCH 20/37] xen: introduce CONFIG_EFI to stub API for non-EFI architecture

Hi Stefano, > -Original Message- > From: Stefano Stabellini > Sent: 2021年9月28日 9:00 > To: Wei Chen > Cc: Jan Beulich ; xen-devel@lists.xenproject.org; > jul...@xen.org; Bertrand Marquis ; Stefano > Stabellini > Subject: RE: [PATCH 20/37] xen: introduce CONFIG_EFI to stub API for non- >

Re: [PATCH v2 10/11] xen/arm: Do not map PCI ECAM and MMIO space to Domain-0's p2m

On Mon, 27 Sep 2021, Oleksandr Andrushchenko wrote: > On 25.09.21 03:44, Stefano Stabellini wrote: > > On Thu, 23 Sep 2021, Oleksandr Andrushchenko wrote: > >> From: Oleksandr Andrushchenko > >> > >> PCI host bridges are special devices in terms of implementing PCI > >> passthrough. According to [

RE: [PATCH 36/37] xen/arm: Provide Kconfig options for Arm to enable NUMA

On Tue, 28 Sep 2021, Wei Chen wrote: > > -Original Message- > > From: Stefano Stabellini > > Sent: 2021年9月28日 1:17 > > To: Jan Beulich > > Cc: Julien Grall ; Stefano Stabellini > > ; Wei Chen ; xen-devel > de...@lists.xenproject.org>; Bertrand Marquis > > Subject: Re: [PATCH 36/37] xen/

RE: [PATCH 36/37] xen/arm: Provide Kconfig options for Arm to enable NUMA

Hi Stefano, > -Original Message- > From: Stefano Stabellini > Sent: 2021年9月28日 1:17 > To: Jan Beulich > Cc: Julien Grall ; Stefano Stabellini > ; Wei Chen ; xen-devel de...@lists.xenproject.org>; Bertrand Marquis > Subject: Re: [PATCH 36/37] xen/arm: Provide Kconfig options for Arm to

RE: [PATCH 22/37] xen/arm: use NR_MEM_BANKS to override default NR_NODE_MEMBLKS

Hi Stefano, Julien, > -Original Message- > From: Stefano Stabellini > Sent: 2021年9月28日 0:58 > To: Julien Grall > Cc: Wei Chen ; Julien Grall ; > Stefano Stabellini ; xen-devel de...@lists.xenproject.org>; Bertrand Marquis ; > Jan Beulich ; Roger Pau Monné ; > Andrew Cooper > Subject: R

[xen-unstable test] 165218: regressions - FAIL

flight 165218 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/165218/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-arm64-arm64-libvirt-raw 17 guest-start/debian.repeat fail REGR. vs. 164945 Tests which are f

RE: [PATCH 20/37] xen: introduce CONFIG_EFI to stub API for non-EFI architecture

On Mon, 27 Sep 2021, Wei Chen wrote: > > -Original Message- > > From: Xen-devel On Behalf Of Wei > > Chen > > Sent: 2021年9月26日 18:25 > > To: Jan Beulich > > Cc: xen-devel@lists.xenproject.org; jul...@xen.org; Bertrand Marquis > > ; Stefano Stabellini > > Subject: RE: [PATCH 20/37] xen: i

Re: [PATCH] xen/arm: optee: Fix arm_smccc_smc's a0 for OPTEE_SMC_DISABLE_SHM_CACHE

Hi Oleksandr, Oleksandr Tyshchenko writes: > From: Oleksandr Tyshchenko > > Fix a possible copy-paste error in arm_smccc_smc's first argument (a0) > for OPTEE_SMC_DISABLE_SHM_CACHE case. > > This error causes Linux > v5.14-rc5 (b5c10dd04b7418793517e3286cde5c04759a86de > optee: Clear stale cac

[qemu-mainline test] 165217: regressions - FAIL

flight 165217 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/165217/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-arm64-arm64-libvirt-raw 17 guest-start/debian.repeat fail REGR. vs. 164950 Tests which are

Re: [PATCH v2 00/10] block: second batch of add_disk() error handling conversions

On 9/27/21 4:00 PM, Luis Chamberlain wrote: > This is the second series of driver conversions for add_disk() > error handling. You can find this set and the rest of the 7th set of > driver conversions on my 20210927-for-axboe-add-disk-error-handling > branch [0]. Applied 1, thank

Re: [PATCH v2 03/10] nvme-multipath: add error handling support for add_disk()

On Mon, Sep 27, 2021 at 03:00:32PM -0700, Luis Chamberlain wrote: > + /* > + * test_and_set_bit() is used because it is protecting against two nvme > + * paths simultaneously calling device_add_disk() on the same namespace > + * head. > + */ > if (!test_and_set_bit(NVM

[PATCH v2 01/10] block/brd: add error handling support for add_disk()

We never checked for errors on add_disk() as this function returned void. Now that this is fixed, use the shiny new error handling. Signed-off-by: Luis Chamberlain --- drivers/block/brd.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/block/brd.c b/drivers

[linux-linus test] 165215: regressions - FAIL

flight 165215 linux-linus real [real] flight 165224 linux-linus real-retest [real] http://logs.test-lab.xenproject.org/osstest/logs/165215/ http://logs.test-lab.xenproject.org/osstest/logs/165224/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run

[PATCH v2 06/10] nvdimm/btt: add error handling support for add_disk()

We never checked for errors on add_disk() as this function returned void. Now that this is fixed, use the shiny new error handling. Signed-off-by: Luis Chamberlain --- drivers/nvdimm/btt.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/nvdimm/btt.c b/drivers/nvdim

[PATCH v2 10/10] zram: add error handling support for add_disk()

We never checked for errors on add_disk() as this function returned void. Now that this is fixed, use the shiny new error handling. Signed-off-by: Luis Chamberlain --- drivers/block/zram/zram_drv.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/block/zram/zram_d

[PATCH v2 00/10] block: second batch of add_disk() error handling conversions

This is the second series of driver conversions for add_disk() error handling. You can find this set and the rest of the 7th set of driver conversions on my 20210927-for-axboe-add-disk-error-handling branch [0]. Changes on this v2 since the last first version of this patch series: - rebased

[PATCH v2 09/10] xen-blkfront: add error handling support for add_disk()

We never checked for errors on device_add_disk() as this function returned void. Now that this is fixed, use the shiny new error handling. The function xlvbd_alloc_gendisk() typically does the unwinding on error on allocating the disk and creating the tag, but since all that error handling was stuf

[PATCH v2 03/10] nvme-multipath: add error handling support for add_disk()

We never checked for errors on add_disk() as this function returned void. Now that this is fixed, use the shiny new error handling. Since we now can tell for sure when a disk was added, move setting the bit NVME_NSHEAD_DISK_LIVE only when we did add the disk successfully. Nothing to do here as th

[PATCH v2 04/10] nvdimm/btt: do not call del_gendisk() if not needed

We know we don't need del_gendisk() if we haven't added the disk, so just skip it. This should fix a bug on older kernels, as del_gendisk() became able to deal with disks not added only recently, after the patch titled "block: add flag for add_disk() completion notation". Signed-off-by: Luis Chamb

[PATCH v2 07/10] nvdimm/blk: avoid calling del_gendisk() on early failures

If nd_integrity_init() fails we'd get del_gendisk() called, but that's not correct as we should only call that if we're done with device_add_disk(). Fix this by providing unwinding prior to the devm call being registered and moving the devm registration to the very end. This should fix calling del

[PATCH v2 08/10] nvdimm/blk: add error handling support for add_disk()

We never checked for errors on add_disk() as this function returned void. Now that this is fixed, use the shiny new error handling. Since nvdimm/blk uses devm we just need to move the devm registration towards the end. And in hindsight, that seems to also provide a fix given del_gendisk() should n

[PATCH v2 02/10] bcache: add error handling support for add_disk()

We never checked for errors on add_disk() as this function returned void. Now that this is fixed, use the shiny new error handling. This driver doesn't do any unwinding with blk_cleanup_disk() even on errors after add_disk() and so we follow that tradition. Acked-by: Coly Li Signed-off-by: Luis

[PATCH v2 05/10] nvdimm/btt: use goto error labels on btt_blk_init()

This will make it easier to share common error paths. Signed-off-by: Luis Chamberlain --- drivers/nvdimm/btt.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/drivers/nvdimm/btt.c b/drivers/nvdimm/btt.c index 29cc7325e890..23ee8c005db5 100644 --- a/driver

Semantics of XEN_DOMCTL_SHADOW_OP_SET_ALLOCATION

Hello, A recent ABI change in Xen caused a total breakage under the Xapi toolstack, and the investigation had lead to this. First of all, the memory pool really needs renaming, because (not naming names) multiple developers were fooled into thinking that the bug was caused by a VM being unexpecte

Re: [PATCH v4 0/8] PCI: Drop duplicated tracking of a pci_dev's bound driver

Hello, On Mon, Sep 27, 2021 at 10:43:18PM +0200, Uwe Kleine-König wrote: > From: Uwe Kleine-König I sent the series from the wrong email address :-\ I should have used the above address as sender. Also I failed to add Christoph Hellwig to Cc: (fixed for this mail). I guess I'll have to send a v5

Re: [PATCH 0/2] grant table and add-to-physmap adjustments on top of XSAs 379 and 384

On 13.09.21 09:39, Jan Beulich wrote: Hi Jan I'm prepared for the "how" aspect of the 1st patch here to end up controversial. Since the observed quirk will imo want dealing with, I'd appreciate any objection to the proposed change to be accompanied by an alternative suggestion. An intention o

Re: sh_unshadow_for_p2m_change() vs p2m_set_entry()

Hi, At 13:31 +0200 on 24 Sep (1632490304), Jan Beulich wrote: > I'm afraid you're still my best guess to hopefully get an insight > on issues like this one. I'm now very rusty on all this but I'll do my best! I suspect I'll just be following you through the code. > While doing IOMMU superpage w

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

flight 165223 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/165223/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 15 migrate-support-checkfail never pass test-arm64-arm64-xl-xsm 1

Re: Xen Rust VirtIO demos work breakdown for Project Stratos

On 24.09.21 19:02, Alex Bennée wrote: Hi Alex [snip] [STR-56] 2.1 Stable ABI for foreignmemory mapping to non-dom0 ([STR-57]) ─── Currently the foreign memory mapping support only w

RE: [PATCH 08/37] xen/x86: add detection of discontinous node memory range

On Mon, 27 Sep 2021, Wei Chen wrote: > > -Original Message- > > From: Stefano Stabellini > > Sent: 2021年9月27日 13:05 > > To: Stefano Stabellini > > Cc: Wei Chen ; xen-devel@lists.xenproject.org; > > jul...@xen.org; Bertrand Marquis ; > > jbeul...@suse.com; andrew.coop...@citrix.com; roger.

Re: [PATCH 36/37] xen/arm: Provide Kconfig options for Arm to enable NUMA

On Mon, 27 Sep 2021, Jan Beulich wrote: > On 27.09.2021 10:45, Julien Grall wrote: > > On Mon, 27 Sep 2021, 10:33 Jan Beulich, wrote: > > > >> On 24.09.2021 21:39, Stefano Stabellini wrote: > >>> On Fri, 24 Sep 2021, Wei Chen wrote: > On 2021/9/24 11:31, Stefano Stabellini wrote: > > On

Re: [PATCH v2 11/17] xen/arm: PCI host bridge discovery within XEN on ARM

On Fri, 24 Sep 2021, 23:42 Stefano Stabellini, wrote: > On Fri, 24 Sep 2021, Rahul Singh wrote: > > Hi Stefano, > > > > > On 23 Sep 2021, at 8:12 pm, Stefano Stabellini > wrote: > > > > > > On Thu, 23 Sep 2021, Rahul Singh wrote: > > +goto err_exit; > > +} > > >>> > > >

Re: [PATCH 22/37] xen/arm: use NR_MEM_BANKS to override default NR_NODE_MEMBLKS

On Mon, 27 Sep 2021, Julien Grall wrote: > On Mon, 27 Sep 2021, 12:22 Wei Chen, wrote: > Hi Julien, > > From: Julien Grall > Sent: 2021年9月27日 15:36 > To: Wei Chen > Cc: Stefano Stabellini ; xen-devel > ; Bertrand Marquis > ; Jan Beulich ; Roger Pau > Monné

Re: [PATCH v2 11/17] xen/arm: PCI host bridge discovery within XEN on ARM

Hi Stefano, > On 25 Sep 2021, at 12:26 am, Stefano Stabellini > wrote: > > On Fri, 24 Sep 2021, Stefano Stabellini wrote: >> On Fri, 24 Sep 2021, Rahul Singh wrote: >>> Hi Stefano, >>> On 23 Sep 2021, at 8:12 pm, Stefano Stabellini wrote: On Thu, 23 Sep 2021, Rahul Singh

Re: [PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored

Juergen Gross writes ("Re: [PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored"): > Should I send out another version, or can this be done when committing? In any case I think you should send another version. If you don't agree with my suggestion to check /proc/sys/fs/nr_

Re: [PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored

Juergen Gross writes ("Re: [PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored"): > On 27.09.21 16:13, Andrew Cooper wrote: > > both work fine, and strace confirms they issue correct system calls. > > Not on my test system: > > # prlimit --pid 734 --nofile=unlimited > prl

Re: [PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored

On 27.09.21 16:54, Andrew Cooper wrote: On 27/09/2021 15:52, Juergen Gross wrote: On 27.09.21 16:40, Andrew Cooper wrote: On 27/09/2021 15:24, Juergen Gross wrote: On 27.09.21 16:13, Andrew Cooper wrote: On 27/09/2021 11:48, Juergen Gross wrote: Add a configuration item for the maximum numbe

Re: [PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored

On 27/09/2021 15:52, Juergen Gross wrote: > On 27.09.21 16:40, Andrew Cooper wrote: >> On 27/09/2021 15:24, Juergen Gross wrote: >>> On 27.09.21 16:13, Andrew Cooper wrote: On 27/09/2021 11:48, Juergen Gross wrote: > Add a configuration item for the maximum number of open file > descri

Re: [PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored

On 27.09.21 16:40, Andrew Cooper wrote: On 27/09/2021 15:24, Juergen Gross wrote: On 27.09.21 16:13, Andrew Cooper wrote: On 27/09/2021 11:48, Juergen Gross wrote: Add a configuration item for the maximum number of open file descriptors xenstored should be allowed to have. The default should

[PATCH v4 1/2] tools/xenstore: set oom score for xenstore daemon on Linux

Juergen Gross writes ("[PATCH v4 1/2] tools/xenstore: set oom score for xenstore daemon on Linux"): > Xenstored is absolutely mandatory for a Xen host and it can't be > restarted, so being killed by OOM-killer in case of memory shortage is > to be avoided. Reviewed-by: Ian Jackson

Re: [PATCH v3 0/3] disable building of pv-grub and qemu-trad per default

Juergen Gross writes ("Re: [PATCH v3 0/3] disable building of pv-grub and qemu-trad per default"): > On 10.09.21 17:34, Ian Jackson wrote: > > Juergen Gross writes ("[PATCH v3 0/3] disable building of pv-grub and > > qemu-trad per default"): > >> This is a first step of deprecating pv-grub and qe

Re: [PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored

On 27/09/2021 15:24, Juergen Gross wrote: > On 27.09.21 16:13, Andrew Cooper wrote: >> On 27/09/2021 11:48, Juergen Gross wrote: >>> Add a configuration item for the maximum number of open file >>> descriptors xenstored should be allowed to have. >>> >>> The default should be "unlimited" in order n

Re: [PATCH] xen/arm: optee: Fix arm_smccc_smc's a0 for OPTEE_SMC_DISABLE_SHM_CACHE

Hi Oleksandr, > On 27 Sep 2021, at 14:54, Oleksandr Tyshchenko wrote: > > From: Oleksandr Tyshchenko > > Fix a possible copy-paste error in arm_smccc_smc's first argument (a0) > for OPTEE_SMC_DISABLE_SHM_CACHE case. > > This error causes Linux > v5.14-rc5 (b5c10dd04b7418793517e3286cde5c04759a

Re: [PATCH] tools/libs: fix build of stubdoms

Juergen Gross writes ("Re: [PATCH] tools/libs: fix build of stubdoms"): > On 08.09.21 14:53, Andrew Cooper wrote: > > On 08/09/2021 13:43, Juergen Gross wrote: > >> In case abi-dumper is available the stubdom builds will fail due to a > >> false dependency on dynamic loadable libraries. Fix that. >

Re: [PATCH v2] Config: use Mini-OS commit 9f09744aa3e5982 for xen-unstable

Juergen Gross writes ("Re: [PATCH v2] Config: use Mini-OS commit 9f09744aa3e5982 for xen-unstable"): > Ping? > > On 08.09.21 14:52, Juergen Gross wrote: > > Switch the used Mini-OS commit to 9f09744aa3e5982 in xen-unstable. > > > > Signed-off-by: Juergen Gross > > --- > > V2: > > - use a specif

Re: [PATCH V5 12/12] net: netvsc: Add Isolation VM support for netvsc driver

Hi Christoph: Gentile ping. The swiotlb and shared memory mapping changes in this patchset needs your reivew. Could you have a look? Thanks. On 9/22/2021 6:34 PM, Tianyu Lan wrote: Hi Christoph:     This patch follows your purposal in the previous discussion. Could you have a look?     "u

Xen 4.16 development update; request for regression bug reports

The mainline development branch of Xen is has taken the first step towards stabilisation for 4.16: new features not yet submitted, at least in draft form, will now be deferred to the following release. We aim to make each Xen release as good as possible, and that includes high quality and particul

Re: [PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored

On 27.09.21 16:13, Andrew Cooper wrote: On 27/09/2021 11:48, Juergen Gross wrote: Add a configuration item for the maximum number of open file descriptors xenstored should be allowed to have. The default should be "unlimited" in order not to restrict xenstored in the number of domains it can su

Re: [PATCH v2 11/11] xen/arm: Translate virtual PCI bus topology for guests

On 27.09.21 17:16, Jan Beulich wrote: > On 27.09.2021 16:04, Oleksandr Andrushchenko wrote: >> On 27.09.21 16:51, Jan Beulich wrote: >>> On 27.09.2021 15:43, Oleksandr Andrushchenko wrote: On 27.09.21 16:34, Jan Beulich wrote: > On 27.09.2021 14:08, Oleksandr Andrushchenko wrote: >> O

Re: [PATCH v2 11/11] xen/arm: Translate virtual PCI bus topology for guests

On 27.09.2021 16:04, Oleksandr Andrushchenko wrote: > > On 27.09.21 16:51, Jan Beulich wrote: >> On 27.09.2021 15:43, Oleksandr Andrushchenko wrote: >>> On 27.09.21 16:34, Jan Beulich wrote: On 27.09.2021 14:08, Oleksandr Andrushchenko wrote: > On 27.09.21 14:31, Jan Beulich wrote: >>

Re: [PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored

On 27/09/2021 11:48, Juergen Gross wrote: > Add a configuration item for the maximum number of open file > descriptors xenstored should be allowed to have. > > The default should be "unlimited" in order not to restrict xenstored > in the number of domains it can support, but unfortunately the prlim

Re: [PATCH v2 11/11] xen/arm: Translate virtual PCI bus topology for guests

On 27.09.21 16:51, Jan Beulich wrote: > On 27.09.2021 15:43, Oleksandr Andrushchenko wrote: >> On 27.09.21 16:34, Jan Beulich wrote: >>> On 27.09.2021 14:08, Oleksandr Andrushchenko wrote: On 27.09.21 14:31, Jan Beulich wrote: > On 23.09.2021 14:55, Oleksandr Andrushchenko wrote: >> -

[libvirt test] 165214: regressions - FAIL

flight 165214 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/165214/ 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

[PATCH] xen/arm: optee: Fix arm_smccc_smc's a0 for OPTEE_SMC_DISABLE_SHM_CACHE

From: Oleksandr Tyshchenko Fix a possible copy-paste error in arm_smccc_smc's first argument (a0) for OPTEE_SMC_DISABLE_SHM_CACHE case. This error causes Linux > v5.14-rc5 (b5c10dd04b7418793517e3286cde5c04759a86de optee: Clear stale cache entries during initialization) to stuck repeatedly issuin

Re: [PATCH v2 11/11] xen/arm: Translate virtual PCI bus topology for guests

On 27.09.2021 15:43, Oleksandr Andrushchenko wrote: > > On 27.09.21 16:34, Jan Beulich wrote: >> On 27.09.2021 14:08, Oleksandr Andrushchenko wrote: >>> On 27.09.21 14:31, Jan Beulich wrote: On 23.09.2021 14:55, Oleksandr Andrushchenko wrote: > --- a/xen/drivers/passthrough/pci.c > ++

Re: [PATCH v2 11/11] xen/arm: Translate virtual PCI bus topology for guests

On 27.09.21 16:34, Jan Beulich wrote: > On 27.09.2021 14:08, Oleksandr Andrushchenko wrote: >> On 27.09.21 14:31, Jan Beulich wrote: >>> On 23.09.2021 14:55, Oleksandr Andrushchenko wrote: --- a/xen/drivers/passthrough/pci.c +++ b/xen/drivers/passthrough/pci.c @@ -890,6 +890,31 @@ i

Re: [PATCH v2 11/11] xen/arm: Translate virtual PCI bus topology for guests

On 27.09.2021 14:08, Oleksandr Andrushchenko wrote: > On 27.09.21 14:31, Jan Beulich wrote: >> On 23.09.2021 14:55, Oleksandr Andrushchenko wrote: >>> --- a/xen/drivers/passthrough/pci.c >>> +++ b/xen/drivers/passthrough/pci.c >>> @@ -890,6 +890,31 @@ int pci_remove_virtual_device(struct domain *d,

Re: [PATCH v2 08/11] libxl: Only map legacy PCI IRQs if they are supported

On 25.09.21 03:06, Stefano Stabellini wrote: > On Thu, 23 Sep 2021, Oleksandr Andrushchenko wrote: >> From: Oleksandr Andrushchenko >> >> Arm's PCI passthrough implementation doesn't support legacy interrupts, >> but MSI/MSI-X. This can be the case for other platforms too. >> For that reason intr

Re: [PATCH v2 10/11] xen/arm: Do not map PCI ECAM and MMIO space to Domain-0's p2m

On 25.09.21 03:44, Stefano Stabellini wrote: > On Thu, 23 Sep 2021, Oleksandr Andrushchenko wrote: >> From: Oleksandr Andrushchenko >> >> PCI host bridges are special devices in terms of implementing PCI >> passthrough. According to [1] the current implementation depends on >> Domain-0 to perform

Re: [XEN PATCH v2] build: add --full to version.sh to guess $(XEN_FULLVERSION)

Apologies for any formatting issues, responding via web mail client. On Thu, 09 Sep 2021 10:33:06 -0400 Anthony PERARD wrote > Running $(MAKE) like that in a $(shell ) while parsing the Makefile > doesn't work reliably. In some case, make will complain with > "jobserver unavailab

[xen-unstable test] 165211: regressions - FAIL

flight 165211 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/165211/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-arm64-arm64-libvirt-raw 17 guest-start/debian.repeat fail REGR. vs. 164945 Tests which are f

Ping: [XEN PATCH v2] build: add --full to version.sh to guess $(XEN_FULLVERSION)

Anyone to review this? Thanks. On Thu, Sep 09, 2021 at 03:33:06PM +0100, Anthony PERARD wrote: > Running $(MAKE) like that in a $(shell ) while parsing the Makefile > doesn't work reliably. In some case, make will complain with > "jobserver unavailable: using -j1. Add '+' to parent make rule.".

Re: [PATCH v2 11/11] xen/arm: Translate virtual PCI bus topology for guests

On 27.09.21 14:31, Jan Beulich wrote: > On 23.09.2021 14:55, Oleksandr Andrushchenko wrote: >> From: Oleksandr Andrushchenko >> >> There are three originators for the PCI configuration space access: >> 1. The domain that owns physical host bridge: MMIO handlers are >> there so we can update vPCI

Re: [PATCH v2 11/11] xen/arm: Translate virtual PCI bus topology for guests

On 23.09.2021 14:55, Oleksandr Andrushchenko wrote: > From: Oleksandr Andrushchenko > > There are three originators for the PCI configuration space access: > 1. The domain that owns physical host bridge: MMIO handlers are > there so we can update vPCI register handlers with the values > written

[PATCH v4 1/2] tools/xenstore: set oom score for xenstore daemon on Linux

Xenstored is absolutely mandatory for a Xen host and it can't be restarted, so being killed by OOM-killer in case of memory shortage is to be avoided. Set /proc/$pid/oom_score_adj (if available) per default to -500 (this translates to 50% of dom0 memory size) in order to allow xenstored to use lar

[PATCH v4 2/2] tools/xenstore: set open file descriptor limit for xenstored

Add a configuration item for the maximum number of open file descriptors xenstored should be allowed to have. The default should be "unlimited" in order not to restrict xenstored in the number of domains it can support, but unfortunately the prlimit command requires specification of a real value f

Re: [PATCH V3 3/3] libxl/arm: Add handling of extended regions for DomU

Stefano Stabellini writes ("Re: [PATCH V3 3/3] libxl/arm: Add handling of extended regions for DomU"): > On Fri, 24 Sep 2021, Oleksandr Tyshchenko wrote: > > From: Oleksandr Tyshchenko ... > > Suggested-by: Julien Grall > > Signed-off-by: Oleksandr Tyshchenko > > Reviewed-by: Stefano Stabellin

[PATCH v4 0/2] tools/xenstore: set resource limits of xenstored

Set some limits for xenstored in order to avoid it being killed by OOM killer, or to run out of file descriptors. Changes in V4: - add comments - switch to configure open file descriptors directly Changes in V3: - make oom score configurable Changes in V2: - split into 2 patches - set limits fro

Re: [PATCH 22/37] xen/arm: use NR_MEM_BANKS to override default NR_NODE_MEMBLKS

On Mon, 27 Sep 2021, 12:22 Wei Chen, wrote: > Hi Julien, > > From: Julien Grall > Sent: 2021年9月27日 15:36 > To: Wei Chen > Cc: Stefano Stabellini ; xen-devel < > xen-devel@lists.xenproject.org>; Bertrand Marquis < > bertrand.marq...@arm.com>; Jan Beulich ; Roger Pau > Monné ; Andrew Cooper > Su

RE: [PATCH 20/37] xen: introduce CONFIG_EFI to stub API for non-EFI architecture

Hi Julien, Stefano, > -Original Message- > From: Xen-devel On Behalf Of Wei > Chen > Sent: 2021年9月26日 18:25 > To: Jan Beulich > Cc: xen-devel@lists.xenproject.org; jul...@xen.org; Bertrand Marquis > ; Stefano Stabellini > Subject: RE: [PATCH 20/37] xen: introduce CONFIG_EFI to stub API

Re: [PATCH v2 05/11] xen/arm: Mark device as PCI while creating one

On 27.09.2021 12:04, Oleksandr Andrushchenko wrote: > > On 27.09.21 13:00, Jan Beulich wrote: >> On 27.09.2021 11:35, Oleksandr Andrushchenko wrote: >>> On 27.09.21 12:19, Jan Beulich wrote: On 27.09.2021 10:45, Oleksandr Andrushchenko wrote: > On 27.09.21 10:45, Jan Beulich wrote: >>

  1   2   >