[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <834a009bba0d4db1b7a1c32e8f20611d@huawei.com>
Date: Mon, 5 Jul 2021 10:18:59 +0000
From: Shameerali Kolothum Thodi <shameerali.kolothum.thodi@...wei.com>
To: Max Gurtovoy <mgurtovoy@...dia.com>,
Leon Romanovsky <leon@...nel.org>
CC: "kvm@...r.kernel.org" <kvm@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-crypto@...r.kernel.org" <linux-crypto@...r.kernel.org>,
"alex.williamson@...hat.com" <alex.williamson@...hat.com>,
"jgg@...dia.com" <jgg@...dia.com>, Linuxarm <linuxarm@...wei.com>,
liulongfang <liulongfang@...wei.com>,
"Zengtao (B)" <prime.zeng@...ilicon.com>,
yuzenghui <yuzenghui@...wei.com>,
Jonathan Cameron <jonathan.cameron@...wei.com>,
"Wangzhou (B)" <wangzhou1@...ilicon.com>
Subject: RE: [RFC v2 1/4] hisi-acc-vfio-pci: add new vfio_pci driver for
HiSilicon ACC devices
> -----Original Message-----
> From: Max Gurtovoy [mailto:mgurtovoy@...dia.com]
> Sent: 05 July 2021 10:42
> To: Shameerali Kolothum Thodi <shameerali.kolothum.thodi@...wei.com>;
> Leon Romanovsky <leon@...nel.org>
> Cc: kvm@...r.kernel.org; linux-kernel@...r.kernel.org;
> linux-crypto@...r.kernel.org; alex.williamson@...hat.com; jgg@...dia.com;
> Linuxarm <linuxarm@...wei.com>; liulongfang <liulongfang@...wei.com>;
> Zengtao (B) <prime.zeng@...ilicon.com>; yuzenghui
> <yuzenghui@...wei.com>; Jonathan Cameron
> <jonathan.cameron@...wei.com>; Wangzhou (B) <wangzhou1@...ilicon.com>
> Subject: Re: [RFC v2 1/4] hisi-acc-vfio-pci: add new vfio_pci driver for HiSilicon
> ACC devices
>
>
> On 7/5/2021 11:47 AM, Shameerali Kolothum Thodi wrote:
> >
> >> -----Original Message-----
> >> From: Leon Romanovsky [mailto:leon@...nel.org]
> >> Sent: 04 July 2021 08:04
> >> To: Shameerali Kolothum Thodi <shameerali.kolothum.thodi@...wei.com>
> >> Cc: kvm@...r.kernel.org; linux-kernel@...r.kernel.org;
> >> linux-crypto@...r.kernel.org; alex.williamson@...hat.com;
> jgg@...dia.com;
> >> mgurtovoy@...dia.com; Linuxarm <linuxarm@...wei.com>; liulongfang
> >> <liulongfang@...wei.com>; Zengtao (B) <prime.zeng@...ilicon.com>;
> >> yuzenghui <yuzenghui@...wei.com>; Jonathan Cameron
> >> <jonathan.cameron@...wei.com>; Wangzhou (B)
> <wangzhou1@...ilicon.com>
> >> Subject: Re: [RFC v2 1/4] hisi-acc-vfio-pci: add new vfio_pci driver for
> HiSilicon
> >> ACC devices
> >>
> >> On Fri, Jul 02, 2021 at 10:58:46AM +0100, Shameer Kolothum wrote:
> >>> Add a vendor-specific vfio_pci driver for HiSilicon ACC devices.
> >>> This will be extended in follow-up patches to add support for
> >>> vfio live migration feature.
> >>>
> >>> Signed-off-by: Shameer Kolothum
> >> <shameerali.kolothum.thodi@...wei.com>
> >>> ---
> >>> drivers/vfio/pci/Kconfig | 9 +++
> >>> drivers/vfio/pci/Makefile | 2 +
> >>> drivers/vfio/pci/hisi_acc_vfio_pci.c | 100
> +++++++++++++++++++++++++++
> >>> 3 files changed, 111 insertions(+)
> >>> create mode 100644 drivers/vfio/pci/hisi_acc_vfio_pci.c
> >> <...>
> >>
> >>> +static const struct vfio_device_ops hisi_acc_vfio_pci_ops = {
> >>> + .name = "hisi-acc-vfio-pci",
> >>> + .open = hisi_acc_vfio_pci_open,
> >>> + .release = vfio_pci_core_release,
> >>> + .ioctl = vfio_pci_core_ioctl,
> >>> + .read = vfio_pci_core_read,
> >>> + .write = vfio_pci_core_write,
> >>> + .mmap = vfio_pci_core_mmap,
> >>> + .request = vfio_pci_core_request,
> >>> + .match = vfio_pci_core_match,
> >>> + .reflck_attach = vfio_pci_core_reflck_attach,
> >> I don't remember what was proposed in vfio-pci-core conversion patches,
> >> but would expect that default behaviour is to fallback to vfio_pci_core_*
> API
> >> if ".release/.ioctl/e.t.c" are not redefined.
> > Yes, that would be nice, but don't think it does that in latest(v4).
> >
> > Hi Max,
> > Could we please consider fall back to the core defaults, may be check and
> assign defaults
> > in vfio_pci_core_register_device() ?
>
> I don't see why we should do this.
>
> vfio_pci_core.ko is just a library driver. It shouldn't decide for the
> vendor driver ops.
>
> If a vendor driver would like to use its helper functions - great.
>
> If it wants to override it - great.
>
> If it wants to leave some op as NULL - it can do it also.
Based on the documentation of the vfio_device_ops callbacks,
It looks like we already have a precedence in the case of reflck_attach
callback where it uses the vfio core default one, if it is not implemented.
Also I would imagine that in majority use cases the vendor drivers will be
defaulting to core functions.
I think, in any case, it would be good to update the Documentation based on
which way we end up doing this.
Thanks,
Shameer
>
>
> >
> > Thanks,
> > Shameer
Powered by blists - more mailing lists