[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 04/13] vpci: restrict unhandled read/write operations for guests
- To: Oleksandr Andrushchenko <Oleksandr_Andrushchenko@xxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Tue, 8 Feb 2022 10:04:59 +0100
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; 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=k43vZkAdwGt8pNENvjLXehfRnoVM3lI5xkK+S7qsJVg=; b=mgMVT3+ems75nnxjDU7IKvpMbAjm1xDwUJzi0uq4yAgkvqCQ0dK7eMUu7izhfzIk4ZHvhiMwr416JMnM1Gebde1QXvER3Myp02c4eQ+fRV0JDh1zYBgTCppW8mniKVN+11XT97nnrc4eeXkGuNAF0ksz5aAj7pPq6U1RGdKB4Db0K/8TbSR0aKP28TqQAQqkgaRfU4XJo4llAjB7ZFTD3r6BYdmVswsZ40QevdFE1VDPuyaPvcNkNv+7FolKkWCA8YGfX7PGN4Bsv/e3ScLq9ZZwkR8crJR61r35T8UidGmfE1MmKTSahczkuxJD16098gO4gjjw9/L8G9zAJxpWHQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Q204pzltFXIRmeF9/XI+KmN6JQvN26yyLP7O82+D35c6XdXCdeXC3kwvLbGksWJHI0ayGYxgIILWcjSNmcG1Alg0/DLgrI8TUiGwfZh65a+c4Lqz9kJIfZhX1e5HiBAdOribdx0pxh1LtcnH3HNprD7fdu7Ak61BHVhSoAPRnBJZxO8vzQp1LZJ5Qhx6Vv17tOK6sBxld7xCfKwHSB27YAstmn6tUjwmkenPziFLyf3+gmiC+GsdCahT5/e0/rkZD03C4f9ofQF9Jsr+A7WDujNmrYXjdMlZkGOJ3K+sqXpxNXpo7AddAhD/u0htNn2J/7X68aaF5ytNUXwDGBI7nw==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: "julien@xxxxxxx" <julien@xxxxxxx>, "sstabellini@xxxxxxxxxx" <sstabellini@xxxxxxxxxx>, Oleksandr Tyshchenko <Oleksandr_Tyshchenko@xxxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Artem Mygaiev <Artem_Mygaiev@xxxxxxxx>, "andrew.cooper3@xxxxxxxxxx" <andrew.cooper3@xxxxxxxxxx>, "george.dunlap@xxxxxxxxxx" <george.dunlap@xxxxxxxxxx>, "paul@xxxxxxx" <paul@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Rahul Singh <rahul.singh@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "roger.pau@xxxxxxxxxx" <roger.pau@xxxxxxxxxx>
- Delivery-date: Tue, 08 Feb 2022 09:05:34 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 08.02.2022 09:00, Oleksandr Andrushchenko wrote:
> On 04.02.22 16:24, Oleksandr Andrushchenko wrote:
>> On 04.02.22 16:11, Jan Beulich wrote:
>>> On 04.02.2022 07:34, Oleksandr Andrushchenko wrote:
>>>> A guest can read and write those registers which are not emulated and
>>>> have no respective vPCI handlers, so it can access the HW directly.
>>> I don't think this describes the present situation. Or did I miss where
>>> devices can actually be exposed to guests already, despite much of the
>>> support logic still missing?
>> No, they are not exposed yet and you know that.
>> I will update the commit message
> BTW, all this work is about adding vpci for guests and of course this
> is not going to be enabled right away.
> I would like to hear the common acceptable way of documenting such
> things: either we just say something like "A guest can read and write"
> elsewhere or we need to invent something neutral not directly mentioning
> what the change does. With the later it all seems a bit confusing IMO
> as we do know what we are doing and for what reason: enable vpci for guests
What's the problem with describing things as they are? Code is hwdom-
only right now, and you're trying to enable DomU support. Hence it's
all about "would be able to", not "can".
Jan
|