[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/6] xen_pvdev: Do not assume Dom0 when creating a directory
- To: David Woodhouse <dwmw2@xxxxxxxxxxxxx>
- From: Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
- Date: Fri, 24 Nov 2023 00:24:44 +0000
- Accept-language: en-US
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=epam.com; dmarc=pass action=none header.from=epam.com; dkim=pass header.d=epam.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=HTGjCRKPV+GhqL+983Xg/W5ncFSyldI8vndC+osq6gQ=; b=E2ehPulxSAW21M2WQRmNSuLr3s28AE0PtlZeX9ZPyTIw3SOlC9Wfh3hu/03RnymrGtwWu5sc41SjvURqh+gGdNr5bZ8O7DeXOdzw8QqNpBSm9OYu3u/jnUrzyAfITs9VFAVnnWhxtPkPHWr2gasDmklMAgJWgiTBs6ck+lpFSXTbmhGsggwfEczpCoa2WZmzRcCMKQBRlno+CC9xPYCqPOnzvlG08A5sJr5OM+hyKsMb57UYCTjQbMLuVk7TQ+aYcf5lVB736Lr0iVBiVIzI+1soTDpdM5i/WEIzZum5VcRDJ+IAsuYjMiUHoiAFEPcK1simCrKF0pgESCsSSjgJeA==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=nLglW6xJh8OjH26zqO4oUedRj7yreNlMNzWAtzXfX6AF5xHNUJ4qVbR0YspPLtw7qM0JjXIYYK/cg0Rr7kCRwaR0JLZrqzGXeY+0T3aC/swgykhXBCN/j/9YalLWA2p2/35dJBMtuVFo3cPI/FqUsQCLqAgtn/wMzqMDxj8KunIl8aRXllZPsRPVJb3acBF4TkolQj69+Wrap25jKNqkb+1WWJbbWQtfGSSIE4Ux2U6eVfpeiu5HGyBP+Mnfuk6DXINjSlIBwqt0qv03JLdCW/mbxw09gGerHW1TXLVr+AHjMYTsRVtRizLm6Zrna6vdSQc1gBh6gx3eal6jr276CQ==
- Cc: "paul@xxxxxxx" <paul@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, "qemu-devel@xxxxxxxxxx" <qemu-devel@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Oleksandr Tyshchenko <Oleksandr_Tyshchenko@xxxxxxxx>, Anthony Perard <anthony.perard@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Fri, 24 Nov 2023 00:25:29 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Thread-index: AQHaHMeIsCU+MAEGAkSnPoa9ThBNz7CGlGOAgABYsYCAAAmAgIAAAakAgAAAyoCAAAJ4gIAABwqAgAAFCoCAAJ2+AIAAJb8AgAAD5ICAAASYAIAAA+AAgADHXgA=
- Thread-topic: [PATCH v2 4/6] xen_pvdev: Do not assume Dom0 when creating a directory
Hi David,
David Woodhouse <dwmw2@xxxxxxxxxxxxx> writes:
> On 23 November 2023 12:17:57 GMT, Volodymyr Babchuk
> <Volodymyr_Babchuk@xxxxxxxx> wrote:
>>
>>Hi David,
>>
>>David Woodhouse <dwmw2@xxxxxxxxxxxxx> writes:
>>> Which PV backends do you care about? We already have net, block and console
>>> converted.
>>
>>Well, this is all what we need, actually. Even console only will be
>>sufficient, as we are using QEMU to provide virtio-pci backends, so both
>>storage and networking should be provided by virtio. Are you proposing
>>to just drop this patch at all? I believe we can live without it, yes.
>
> Yeah, I think you can drop anything that's only needed for the legacy
> backends. I'm tempted to make a separate config option to compile those out.
Yep, we need this. Because without the patch ("xen_pvdev: Do not assume
Dom0 when creating a directory") I can't run QEMU in the driver domain:
root@spider-domd:~# qemu-system-aarch64 -machine xenpv -m 128M
xen be core: xs_mkdir device-model/0/backends/vkbd: failed
xen be core: xs_mkdir device-model/0/backends/vkbd: failed
xen be core: xs_mkdir device-model/0/backends/9pfs: failed
xen be core: xs_mkdir device-model/0/backends/9pfs: failed
So yeah, we need something like CONFIG_XEN_LEGACY_BACKENDS option if we
don't want to fix xenstore_mkdir()
--
WBR, Volodymyr
|
