[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
- Date: Tue, 12 Oct 2021 10:41:23 +0000
- Accept-language: en-GB, en-US
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.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=Yu7nT04e8SwimeoWa4uhZ2M5IJUjFclNEp5sAd9mDcU=; b=M2HXLF8KrsGLfQffjNbJBjlXilQ8tr6CwvO4gUehBU4C5UCrT7yz8TSDhak7ZpkYS6uOWFUsh83koB1hiXM1wvNmg6ByDBui4SmAt3thQFlfb5NeVwYyWOgcPnAuZhVDbT0toTPlP8gOW0T9qEA6aXBrYq6raG2Ks9/ptdRQFhT+rFmw2o8TzWgXFDnUz+xym/xGSXEI4iPmtJ9GVLPLQEeu68qkWDGCAAaJfNh3pU3F4bj+eoDTzDbH325/2TdqsdEoNsIzTm6rQvK1NreV1lC5yu5wTXysnKHd5zVOEOhk1C3/0kxyu2CYyoTbN85dpcHROO/Q+qdYvdL0WHP7OQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=m/Xh3t6EPbUDcgwHDidQqGzYBEPBaxPQ2jrazC4ZEDromrbAJ4uQnlAjpwnmV/L/MROwRVyNOCTMR+J7xiLOUv0TJTMZf4do54wMBYStDv9wQkyCTwxPhS2lhl8ZcrrE6lUoY5wC+jFKeXrmisZs3J+H394kyjOhqCXI2gPnf0Scp5bMzXk3KiVAA97Lln2E5k04ilkH2wU+yDEbSlXD2eah115Snaw5CDrXG8T+ieOUWaqyulbnV1P6GaBK8kUb3Eoh6Xl3SSZi69lNUn8O+m7Dflf2G2rBQ7w4aTQyFVXDiw3uz+fVp8mlshLTKt8bZR7v5z41IPwpwQfbjJ9lAw==
- Authentication-results-original: suse.com; dkim=none (message not signed) header.d=none;suse.com; dmarc=none action=none header.from=arm.com;
- Cc: Oleksandr Andrushchenko <Oleksandr_Andrushchenko@xxxxxxxx>, Rahul Singh <Rahul.Singh@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andre Przywara <Andre.Przywara@xxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
- Delivery-date: Tue, 12 Oct 2021 10:41:53 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Nodisclaimer: true
- Original-authentication-results: suse.com; dkim=none (message not signed) header.d=none;suse.com; dmarc=none action=none header.from=arm.com;
- Thread-index: AQHXutl5d8zkP9bsD0GXT4+5QPGclKvNtfkAgAAGc4CAABOEAIAABXoAgAAEuYCAAAVHAIAAJiWAgAAK0ICAAQCPgIAAA0qAgAAOJYCAAAHCAIAABnQAgAABQgCAAAQNAIAABdAA
- Thread-topic: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.
Hi Jan,
> On 12 Oct 2021, at 11:20, Jan Beulich <jbeulich@xxxxxxxx> wrote:
>
> On 12.10.2021 12:06, Oleksandr Andrushchenko wrote:
>> On 12.10.21 13:01, Jan Beulich wrote:
>>> On 12.10.2021 11:38, Oleksandr Andrushchenko wrote:
>>>> On 12.10.21 12:32, Jan Beulich wrote:
>>>>> The minimal thing I'd suggest (or maybe you're doing this already)
>>>>> would be to expose such BARs to the guest as r/o zero, rather than
>>>>> letting their port nature "shine through".
>>>> If we have the same, but baremetal then which entity disallows
>>>> those BARs to shine?
>>> I'm sorry, but I don't understand what you're trying to say.
>>>
>>>> I mean that if guest wants to crash... why
>>>> should we stop it and try emulating something special for it?
>>> This isn't about a guest "wanting to crash", but a driver potentially
>>> getting mislead into thinking that it can driver a device a certain
>>> way.
>> Well, for the guest, as we do not advertise IO in the emulated host
>> bridge, the driver won't be able to allocate any IO at all. Thus, even
>> if we have a BAR with PCI_BASE_ADDRESS_SPACE_IO bit set, the
>> driver won't get anything. So, I think this is equivalent to a baremetal
>> use-case where we have no IO supported by the host bridge and
>> a device with IO BAR.
>
> Oh, now I follow. Fair enough.
So there is no comment remaining on this patch ?
Would it possible to get an ack on it ?
Thanks
Bertrand
>
> Jan
|
