From:   "Tian, Kevin" <kevin.tian@intel.com>
To:     Alex Williamson <alex.williamson@redhat.com>,
        Grzegorz Jaszczyk <jaz@semihalf.com>
CC:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "dmy@semihalf.com" <dmy@semihalf.com>,
        "tn@semihalf.com" <tn@semihalf.com>,
        "dbehr@google.com" <dbehr@google.com>,
        "upstream@semihalf.com" <upstream@semihalf.com>,
        "dtor@google.com" <dtor@google.com>, "jgg@ziepe.ca" <jgg@ziepe.ca>,
        "cohuck@redhat.com" <cohuck@redhat.com>,
        "abhsahu@nvidia.com" <abhsahu@nvidia.com>,
        "yishaih@nvidia.com" <yishaih@nvidia.com>,
        "Liu, Yi L" <yi.l.liu@intel.com>,
        "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
        "Dominik Behr" <dbehr@chromium.org>
Subject: RE: [PATCH] vfio/pci: Propagate ACPI notifications to the user-space
Thread-Topic: [PATCH] vfio/pci: Propagate ACPI notifications to the user-space
Thread-Index: AQHZUUEbYUNRuSCltkm4EZaoKYZWuK7v+k0AgACK2yA=
Date:   Wed, 8 Mar 2023 08:11:05 +0000
Message-ID: <BN9PR11MB527645F3DFD7E278D1C652A48CB49@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <20230307220553.631069-1-jaz@semihalf.com>
 <20230307164158.4b41e32f.alex.williamson@redhat.com>
In-Reply-To: <20230307164158.4b41e32f.alex.williamson@redhat.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?QL/T4orWsXeDFJ1HH2BaAIm5y00LalofnVYwusFuAL5iNyrwOGpYS5BMzAOB?=
 =?us-ascii?Q?oNeH56OgV+y5tBoL473zfXiqwsL5SPepurOm10pe6JQGHUOClaQMs4qUBOMO?=
 =?us-ascii?Q?dOW4SXZXR+1WTXbcCAnY2DP7t5JcK6l8ovtP0j0xKF76rPg8tREKmRzGu4oJ?=
 =?us-ascii?Q?pl+Q/wlBEimKUOaoF4uO5ZpH7AG2Xb9+UZIyyy9nauGUfP1KCDHn+b5pz4OW?=
 =?us-ascii?Q?dc2TKOxFnGDPgO8sPoCxvgxtvtbgeE676vycx9fuU3ap7nG8eKOp29mzolKk?=
 =?us-ascii?Q?nOcaMnWfBi2jDthfKkXFtnYMBXn6/p0SDanjW9h/av3ULvi/Y7W1065q610s?=
 =?us-ascii?Q?G4O+WU7wovfekcJarfEjVzo5UU2T9diRhIgrNnMBnK1Q5BJo+Wk8DL3ZHMoY?=
 =?us-ascii?Q?ogWIIbtS+93MtoY8JKgnpLH2N9r3cSvGYNG24HG7PG4ZUkJ14SWjTwRYlEag?=
 =?us-ascii?Q?r88TQWllWSDxJsoa0ZV/nF5eANtfGjH2FrSOkbA4A5xaSEibpzHtBdpEy7pF?=
 =?us-ascii?Q?xT32KiIvLDIlC15m0QDobLy/C2pgSJQ721dYUBdJY3TC+i6lyGRdm2Z9WGd3?=
 =?us-ascii?Q?nMpGJwHA0160Mp0y/BqxmgU4MSLvs/XOUhDwiV2zECtxdqKoMrKWhugL7yJv?=
 =?us-ascii?Q?fYEHhrhf0eiu7qF7o11kZoMgyIpig1rD0VZTDqdqQN8ePnpAAB2ZgfJNnNbB?=
 =?us-ascii?Q?WWUKWDpf5WN6JqaAnLU6j02wwSAUreqpYJWpHSMFpiAdGlvxq5UhVzW0l7Yz?=
 =?us-ascii?Q?i/jiBtSc+xxaCUFn/pDvM6zs+Zyv3dLrxpFPNeKHLU80kfcESAuED3HBHDVU?=
 =?us-ascii?Q?0N7TRvLopVmji5MlsG6dJr3UQP7lYgKaR6egv/ALYiL1laBuguTwl/lACN4K?=
 =?us-ascii?Q?OZXxLsYTGZn0gzNYqRJKQ8iZuNkkJetX+NU+t5QP5V4lauwfMRXFYcKfH60u?=
 =?us-ascii?Q?YPVjKCqqneH09lCk0sR4yx2H4glVlP0E2/lal8WNfi8ijC21Uspeul4n1UGD?=
 =?us-ascii?Q?PcFBb/8A7VgpyVuruuQfJCj295jZoplHBEjCgnsCe46iuLpIz/yPp1g2N1VL?=
 =?us-ascii?Q?ck54yapQz1ThrR9qiaV+qU/biP/AHjjBDOwJsv0b3R+YpgbynhyDTuk/XEqe?=
 =?us-ascii?Q?D6EwYs2nVDX3K6sslucZGjnU45MYwYVisyyGW4euEUgR3IdSvCupLM/x9z9h?=
 =?us-ascii?Q?gBapD3dI73qp08zydtrkljMGRBSffFVcxPqzVPVmXCoIt+mz47yP5RPu3AUF?=
 =?us-ascii?Q?8E4Aa7HkFMNr5Enc0pfv4aK3Huw+KaF+8P12C4FgnidbEc3OMNVdxkRyZpKU?=
 =?us-ascii?Q?/43lUqrqOm6cGCaB9o/YjkZsoxR2GUR4CPRAHQB0bXKMqtT4m/uSGN/YCidM?=
 =?us-ascii?Q?3FYAI6/jje6w1FuI57mQPi7bHDXDNq1NSCcG68yY8z522BljC+nkwXVGKij+?=
 =?us-ascii?Q?O2Iv0tD0uiq7xxN/WKGirXbPFpoFXAI+xFWaLccmRtSqt4p4MbwtLlCrH47q?=
 =?us-ascii?Q?7vSt9BBEhyxbm2bU5FSnweY3xJaC6HHDmkByc4Ue8wISvHWBBTx7+fh3ZX3c?=
 =?us-ascii?Q?Wgs/nv9G0pUgAEGwqdue1F68gZcr97WskdytUHts?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BN9PR11MB5276.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ffc366ab-2340-4f55-6a8a-08db1facaa62
X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Mar 2023 08:11:05.6313
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: n/Th7zQnhDvPslpgcT+0kOVY59uKcxGCGljrHT7V5lUtOe1+JXRsuJbZwrltW6yWc2RkuevZLM2J/HE9KNzbSw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5926
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> From: Alex Williamson <alex.williamson@redhat.com>
> Sent: Wednesday, March 8, 2023 7:42 AM
>=20
> On Tue,  7 Mar 2023 22:05:53 +0000
> Grzegorz Jaszczyk <jaz@semihalf.com> wrote:
>=20
> > From: Dominik Behr <dbehr@chromium.org>
> >
> > Hitherto there was no support for propagating ACPI notifications to the
> > guest drivers. In order to provide such support, install a handler for
> > notifications on an ACPI device during vfio-pci device registration. Th=
e
> > handler role is to propagate such ACPI notifications to the user-space
> > via acpi netlink events, which allows VMM to receive and propagate them
> > further to the VMs.
> >
> > Thanks to the above, the actual driver for the pass-through device,
> > which belongs to the guest, can receive and react to device specific
> > notifications.
>=20
> What consumes these events?  Has this been proposed to any VM
> management tools like libvirt?  What sort of ACPI events are we
> expecting to see here and what does userspace do with them?
>=20

and the VM sees a virtual platform and virtual ACPI. Usually an ACPI
event triggers parsing-executing certain ACPI function which needs to
further access platform resource. Presumably someone should copy
the related ACPI function into virtual ACPI table and then in concept
we should allow an ACPI event routed to userspace only if the related
platform resource has been assigned to the user.=20

What would the mechanism to audit it? and if we have a way to do it
probably the ACPI event would be translated into a more generic
event mechanism in the vfio-platform driver covering related resource.