[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAPcyv4jGrvkwjZG+qiu0VGU9ifj95t0Yi7ripJEtZr+_kVrWog@mail.gmail.com>
Date: Mon, 2 Nov 2020 11:26:01 -0800
From: Dan Williams <dan.j.williams@...el.com>
To: Jason Gunthorpe <jgg@...dia.com>
Cc: Dave Jiang <dave.jiang@...el.com>,
"Raj, Ashok" <ashok.raj@...el.com>,
Thomas Gleixner <tglx@...utronix.de>,
Vinod Koul <vkoul@...nel.org>,
"Dey, Megha" <megha.dey@...el.com>, maz@...nel.org,
Bjorn Helgaas <bhelgaas@...gle.com>,
Alex Williamson <alex.williamson@...hat.com>,
Jacob jun Pan <jacob.jun.pan@...el.com>,
Yi L Liu <yi.l.liu@...el.com>, Baolu Lu <baolu.lu@...el.com>,
"Tian, Kevin" <kevin.tian@...el.com>,
Sanjay K Kumar <sanjay.k.kumar@...el.com>,
"Luck, Tony" <tony.luck@...el.com>, Jing Lin <jing.lin@...el.com>,
kwankhede@...dia.com, eric.auger@...hat.com,
Parav Pandit <parav@...lanox.com>,
"Rafael J. Wysocki" <rafael@...nel.org>, netanelg@...lanox.com,
shahafs@...lanox.com, yan.y.zhao@...ux.intel.com,
Paolo Bonzini <pbonzini@...hat.com>,
Samuel Ortiz <samuel.ortiz@...el.com>,
Mona Hossain <mona.hossain@...el.com>,
Megha Dey <megha.dey@...ux.intel.com>,
dmaengine@...r.kernel.org,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Linux PCI <linux-pci@...r.kernel.org>,
KVM list <kvm@...r.kernel.org>
Subject: Re: [PATCH v4 00/17] Add VFIO mediated device support and DEV-MSI
support for the idxd driver
On Mon, Nov 2, 2020 at 10:52 AM Jason Gunthorpe <jgg@...dia.com> wrote:
>
> On Mon, Nov 02, 2020 at 10:38:28AM -0800, Dan Williams wrote:
>
> > > I think you will be the first to use the namespace stuff for this, it
> > > seems like a good idea and others should probably do so as well.
> >
> > I was thinking either EXPORT_SYMBOL_NS, or auxiliary bus, because you
> > should be able to export an ops structure with all the necessary
> > callbacks.
>
> 'or'?
>
> Auxiliary bus should not be used with huge arrays of function
> pointers... The module providing the device should export a normal
> linkable function interface. Putting that in a namespace makes a lot
> of sense.
True, probably needs to be a mixture of both.
Powered by blists - more mailing lists