RE: [PATCH v1 14/18] vfio/mdev: Add mdev available instance checking to the core

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> From: Eric Farman <farman@xxxxxxxxxxxxx>
> Sent: Tuesday, June 7, 2022 4:24 AM
> 
> On Tue, 2022-06-07 at 01:32 +0530, Kirti Wankhede wrote:
> >
> > On 6/2/2022 10:49 PM, Eric Farman wrote:
> > > From: Jason Gunthorpe <jgg@xxxxxxxxxx>
> > > @@ -106,6 +106,7 @@ structure to represent a mediated device's
> > > driver::
> > >   	     int  (*probe)  (struct mdev_device *dev);
> > >   	     void (*remove) (struct mdev_device *dev);
> > >   	     struct device_driver    driver;
> > > +	     unsigned int (*get_available)(struct mdev_type *mtype);
> > >        };
> > >
> >
> > This patch conflicts with Christoph Hellwig's patch. I see
> > 'supported_type_groups' is not is above structure, I beleive that
> > your
> > patch is applied on top of Christoph's patch series.
> >
> > but then in below part of code, 'add_mdev_supported_type' has also
> > being
> > removed in Christoph's patch. So this patch would not get applied
> > cleanly.
> 
> Apologies. This series was fit to 5.18 as the merge window progressed.
> Both this patch and the previous one have to adjust to the removal of
> mdev_parent_ops that came about from
> 
> commit 6b42f491e17ce13f5ff7f2d1f49c73a0f4c47b20
> Author: Jason Gunthorpe <jgg@xxxxxxxx>
> Date:   Mon Apr 11 16:14:01 2022 +0200
> 
>     vfio/mdev: Remove mdev_parent_ops
> 
> I have this rebased for v2.
> 

btw with those latest changes [1] we don't need .get_available() then,
as mdev type is now added by mdev driver one-by-one then the
available instance can be provided directly in that path.

[1] https://lore.kernel.org/all/20220607055653.GA8848@xxxxxx/T/#m82caa423d728bce0729b8c9e8d8aff76250ec9ac




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Kernel Development]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Info]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Linux Media]     [Device Mapper]

  Powered by Linux