On Tue, 2019-08-06 at 21:05 +0530, Potnuri Bharat Teja wrote:
> On Tuesday, August 08/06/19, 2019 at 20:46:13 +0530, Doug Ledford
> wrote:
> > On Tue, 2019-08-06 at 14:13 +0300, Leon Romanovsky wrote:
> > > On Tue, Aug 06, 2019 at 04:38:13PM +0530, Potnuri Bharat Teja
> > > wrote:
> > > > On Tuesday, August 08/06/19, 2019 at 15:18:49 +0530, Leon
> > > > Romanovsky
> > > > wrote:
> > > > > On Tue, Aug 06, 2019 at 11:09:02AM +0300, Leon Romanovsky
> > > > > wrote:
> > > > > > On Mon, Aug 05, 2019 at 04:36:53PM +0530, Potnuri Bharat
> > > > > > Teja
> > > > > > wrote:
> > > > > > > On Monday, July 07/29/19, 2019 at 13:16:20 +0530, Potnuri
> > > > > > > Bharat Teja wrote:
> > > > > > > > On Sunday, July 07/28/19, 2019 at 14:07:49 +0530, Leon
> > > > > > > > Romanovsky wrote:
> > > > > > > > > On Thu, Jul 25, 2019 at 03:14:24PM -0300, Jason
> > > > > > > > > Gunthorpe
> > > > > > > > > wrote:
> > > > > > > > > > On Thu, Jul 25, 2019 at 11:19:28PM +0530, Potnuri
> > > > > > > > > > Bharat
> > > > > > > > > > Teja wrote:
> > > > > > > > > > > match_device handler is no longer needed after
> > > > > > > > > > > latest
> > > > > > > > > > > device binding changes.
> > > > > > > > > > >
> > > > > > > > > > > Signed-off-by: Potnuri Bharat Teja <
> > > > > > > > > > > bharat@xxxxxxxxxxx
> > > > > > > > > > > ---
> > > > > > > > > > > providers/cxgb4/dev.c | 41 ----------------------
> > > > > > > > > > > ----
> > > > > > > > > > > ---------------
> > > > > > > > > > > 1 file changed, 41 deletions(-)
> > > > > > > > > >
> > > > > > > > > > Do you know if we can also drop the same code in
> > > > > > > > > > cxgb3?
> > > > > > > > >
> > > > > > > > > Can we simply remove cxgb3?
> > > > > > > > >
> > > > > > > >
> > > > > > > > I am in talks with the people here. I'll confirm it
> > > > > > > > soon.
> > > > > > >
> > > > > > > Hi Jason/Doug/Leon,
> > > > > > > Chelsio is fine with removing cxgb3.
> > > > > >
> > > > > > Thanks a lot.
> > > > >
> > > > > Which parts of cxgb3 can we remove? RDMA, scsi, net or
> > > > > everything?
> > > >
> > > > I can only say RDMA. For net and scsi parts of cxgb3, the
> > > > corresponding
> > > > maintainers might request for their removal.
> > > > Should I send a patch removing RDMA cxgb3?
> > >
> > > It will be the best variant.
> > >
> > > Thanks
> > >
> > > > Thanks.
> >
> > I'm not entirely sure that I want it removed yet. The cxgb3 isn't
> > the
> > most stellar device, but it will do 40GBit/s. That's still a very
> > respectable speed (unlike say mthca that was mostly 10GBit/s with
> > only a
> > short run of 20GBit/s devices before it switched over to mlx4). So
> > a
> > cxgb3 based home system is still something very usable. Are we sure
> > we
> > want to remove this?
>
> Hi Doug,
> Thanks for the suggestion. I took this discussion forward assuming
> that we
> need to remove drivers which are not/least maintained. T3 adapters
> have
> reached end of life long ago. Most of the times I see some
> features/changes
> regressing due to cxgb3's limited support. based on all these I have
> put this
> proposal of removing cxgb3 in front of the team at Chelsio. They are
> okay with
> removing it.
> I now doubt if I missed something. Do you think otherwise?
> BTW cxgb3 has supported speed is 1/10 Gbps.
[ Adding the list back in because I don't want to have to repeat this ]
I could have sworn we have 40Gig cxgb3 hardware in our lab, but it
appears I was wrong. If we had it, it's been pulled and what's left is
all 10GigE stuff.
I did a quick search on EBay (which is where I know a lot of people go
to get dirt cheap prices on stuff like old cards to build their home
networks with) and it didn't really turn up much in the way of cxgb3
gear. There was plenty of gear that would use the cxgb4 driver instead.
So, I'm more on board with removing this driver now.
--
Doug Ledford <dledford@xxxxxxxxxx>
GPG KeyID: B826A3330E572FDD
Fingerprint = AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 2FDD
Attachment:
signature.asc
Description: This is a digitally signed message part
