Re: [RFC rdma-next 0/3] Move mlx5 vendor specific to UAPI directory

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

 



On Wed, Sep 14, 2016 at 03:46:07PM -0400, Laurence Oberman wrote:
>
>
> ----- Original Message -----
> > From: "Jason Gunthorpe" <jgunthorpe@xxxxxxxxxxxxxxxxxxxx>
> > To: "Leon Romanovsky" <leon@xxxxxxxxxx>
> > Cc: dledford@xxxxxxxxxx, linux-rdma@xxxxxxxxxxxxxxx
> > Sent: Wednesday, September 14, 2016 1:21:45 PM
> > Subject: Re: [RFC rdma-next 0/3] Move mlx5 vendor specific to UAPI directory
> >
> > On Wed, Sep 14, 2016 at 11:27:26AM +0300, Leon Romanovsky wrote:
> > > The consolidate library revealed the fact that vendor specific
> > > structures and defines are copy/pasted between kernel and user-space,
> > > instead of sharing.
> > >
> > > This RFC was tested by compilation only and not-ready for merge yet.
> >
> > Yah, this is exactly what I had in mind. We need to give this
> > treatment to all the drivers :|
> >
> > > Leon Romanovsky (3):
> > >   IB/mlx5: Move and decouple user vendor structures
> > >   IB/core: Export mlx5 vendor specific strucutres
> > >   MAINTAINERS: Add mlx5 uapi file
> >
> > One patch is probably enough, 3 patches per driver is going to be
> > alot.
> >
> > Jason
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> >
> Leon, I have not kept up with testing all the mlx5 changes and latest patches.
> Going to wait until this settles then will take snapshot and start testing.

Thanks Laurence,

I will set the topic branch when I'll create patches for all drivers. I don't
expect any additional testing to compilations and header installations.

At the end, It is simple file moves from one place to another without
any functional changes.

>
> Thanks
> Laurence

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux