[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: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Mon, 11 Oct 2021 15:20:54 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.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=egF0V+tHtMeF59ZvOVaYN9PSfuOExJamhIvU720waeU=; b=oQDTXRqj4r5H3HpIbqAWwMbyma5yq2vX17vDWmSTSdSinXDzcxog/VAy2+8C6uq2C5+IHj7oRhzyrTanN1JgmlJjb55ThJlfltQxxEnNYMr1PhXjCAP/qA0DUxuIz3rNXHaF80KUUE2DEIfY0jk57aN0LBjgFYnu3Rwd89nLq2eqVrt/EpLcdIFO1acQHVNTwX13y7SFFiynZs936fOmH2bb12ueX3t4STN2dKR4Wfm+2RClvO1hfWmcY52RVxCmg1cmHyQNrt6xVG8ohBn6d1TJKnbtYsPzAPHaH2nA7hHYg7mswIqsy57av3UeR9Bz/laAY4gAtr9lAHVuiTyMAA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=O7vLOzsSWC4o2AGLa2/Ix40DnvS4ZhVMJzB5MVCBIiSzOPXmwkiD6efd/9k6oYHvtNXdLiqsPwsf+8lhuCpKiPZnVtXmMtT1SCJddjLJgctgjtz5Nwk2vqblcFjRlxdGPjru6qx+uq+QPBlLooWiWCmez4pSxoJpTZ4SI4EulLe8Q8z6WD9IvODcwE0MOR9ZG1ywB/WSgwEHNQLaVHYXgvJMxsngpJ5d+iFPhyDlJDwpkNSJ8xA7SAtHdYJdDtPBQX21WQMzjVo2oklCaaj4kBgUx9ofrn251XsJD7BY2tUeYqi1PPjzxyKN+5X4Jq1Gl+SlELcj9Yd3FVpHaVWd5w==
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: 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>
  • Delivery-date: Mon, 11 Oct 2021 13:21:45 +0000
  • Ironport-data: A9a23:25TZZKO1jqtTjYXvrR3vkcFynXyQoLVcMsEvi/4bfWQNrUorgmZWz msXXWvVOq6Kazehc41ybo+y9UoHsJSHnYNhTAto+SlhQUwRpJueD7x1DKtR0wB+jCHnZBg6h ynLQoCYdKjYdpJYz/uUGuCJQUNUjMlkfZKhTr6ZUsxNbVU8En540Us6w7dRbrNA2rBVPSvc4 bsenOWHULOV82Yc3rU8sv/rRLtH5ZweiRtA1rAMTakjUGz2zhH5OKk3N6CpR0YUd6EPdgKMq 0Qv+5nilo/R109F5tpICd8XeGVSKlLZFVDmZna7x8FOK/WNz8A/+v9TCRYSVatYo3a2tI5h9 +RAib+bClozBJzlnN0XTBYNRkmSPYUekFPGCX22sMjVxEzaaXr8hf5pCSnaP6VBpLwxWzsXs 6VFdnZdNXhvhMrvqF6/YvNrick5atHiIasUu216zCGfBvEjKXzGa/iWtYAFg2Zs7ixINa/9a cUidT9FVhKeU0BkE2lNMqsnpfj90xETdBUH8QnI9MLb+VP70whZwLXrdt3PdbSiRNpXtlaVo Hrc+Gb0CQ1cM8aQoRKe6W6ljOLLmSL9WaoRGae++/osh0ecrkQRBwcRTkCTuuSihwi1XNc3A 3Yd+jcq65Mz8kOrZtDnWluzp3vsljQRQcZKGus2riSE0LPJ4h2xD3IBCDVGbbQOnck6XzFs7 kWbjszgHzVHuaeQD3ma89+8vT60fCQYM2IGTSsFVhcepcnuppkpiRDCRcolF7S65uAZAhmpn WrM9nJnwexO04hbjM1X4GwrnRrvmbOXXjEE6zn0dSH49DlITYqcbZOnvA2zAel7EK6VSVyIv X4hkseY7fwTAZzlqBFhUNnhD5nyuK7baGy0bUpHWsB7rWz0qiHLkZV4uWkmfC9U3tA4lSgFi aM5kTha44NPJzOUZKtzbpPZ5y8CnPW4S4qNuhw5aLNzjnlNmO2voHEGia24hTmFfK0QfUcXY 8rznSGEVydyNEif5GDqL9rxKJdyrszE+UvdRIrg0zOs2qeEaXieRN8taQXVNbBotPzd+VmLq L6z0vdmLT0ECoUShQGNoOYuwa0idyBnVfgaVeQGHgJ8HuaWMD54UKKAqV/QU4dkg75Uho/1E oKVASdlJK7ErSSfc22iMyk7AJu2BMoXhS9rbEQEYAfzs1B+MNnH0UvqX8ZuFVXR3Lc4lqAco jhsU5joP8mjvRyeo25DMsKn8NA7HPlp7CrXVxeYjPEEV8cIbyTC+8P+fxup8y8LDyGtstA5r aHm3QTeKafvjSw4ZCoPQP7wnV63o1YHn+d+AxnBLtVJIR2++4l2MS3hyPQwJphUexnEwzKb0 SeQAAsZ+raR89NkroGRiPDWtZqtHst/AlFeQzvR44GpOHSI5WGk24JBDrqFJGiPSGPu9ay+T uxJ1PWgYuYflVNHvtMkQbZmxK4z/fX1oLpewlg2FXnHdQ3zWLhhPmOHzY9EsagUnu1Vvg6/W 0Su/NhGOOrWZJO5QQBJfAd8N7aNz/AZnDXW/M8ZGkSi6X8l5qeDXGVTIwKI1H5XIoxqPd532 uwmosMXtVCy00J4LtacgylI3G2QNXhcAb4/v5QXDYK32AomzlZOPc7VBiPsuczdbtxNNg8hI yOOhbqEjLNZnxKQf302HHnL/OxcmZVR50wakA5cfwyEyojfm/s6/BxN6jBmHA1awyJO3/93J mU2ZVZ+Ir+D/mswicVON4x299qt2PFNFpTN9mY0
  • Ironport-hdrordr: A9a23:BAZrFKvYgHD7lT5FwiT5iYGC7skDctV00zEX/kB9WHVpm6uj5q eTdZUgpHvJYVMqM03I9urtBEDtexzhHP1OgbX5X43NYOCOggLBRuxfBODZogHIKmnT8fNcyL clU4UWMqyUMbGit7eY3OBvKadD/OW6
  • Ironport-sdr: DGYQh72bENi4XMeM7yrPXYSvypyxLVd0o+ygSMJVoEUq57KDaBiEpTbMo5qGfjhidA0BWe1lT2 kzwmFYUWr9d925X64hkIWkXlmvQqcRICU+uQbdivvZIpQeL/639MZPXSXNuYzMPwABlOITURwl fij9AhPNkujR9vxtGeRkAxAJOGHIa9/1SdDtyrmCqXMX2JsYbvLWyNKClhjTApIKoY8T5KUMaQ 2tYKfm4pgJ9PQoE2hXtAKKjxZn4+t3eJ/EF1QtCsw4WItY4HufVHdE9Rgv5pmEc1aA+9gZkot5 OIV620q9Tx7Ocy6zdMdz8NfH
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Mon, Oct 11, 2021 at 12:11:04PM +0000, Bertrand Marquis wrote:
> Hi Roger,
> 
> > On 11 Oct 2021, at 12:47, Roger Pau Monné <roger.pau@xxxxxxxxxx> wrote:
> > 
> > On Wed, Oct 06, 2021 at 06:40:27PM +0100, Rahul Singh wrote:
> >> ARM architecture does not implement I/O ports. Ignore this call on ARM
> >> to avoid the overhead of making a hypercall just for Xen to return
> >> -ENOSYS.
> > 
> > What is the cal trace of this function actually on Arm?
> > 
> > AFAICT libxl will only call xc_domain_ioport_permission if there are
> > IO ports explicitly defined in the guest configuration, or if any of
> > the BARs of the PCI device is in the IO space, which is not possible
> > on Arm.
> 
> PCI devices BARs can be in the IO space as the PCI devices are not
> Arm specific. There is not ioports on arm so to be used those can be
> in some cases remapped and accessed as MMIOs or are not possible
> to use at all.
> 
> But the IO space does appear when BARs are listed even on Arm.

Urg, I wonder whether those devices with IO BARs will work correctly
under Arm then.

How do you know whether the BAR has been remapped from IO space into
MMIO?

IMO instead of faking a successful return value from
xc_domain_ioport_permission we should avoid the call completely in the
first place, specially if we need to instead issue a call to
xc_domain_iomem_permission.

Thanks, Roger.



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.