linux-kernel - Re: InfiniBand/RDMA merge plans for 2.6.33

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Wed, 16 Dec 2009 09:20:01 -0800
From:	Roland Dreier <rdreier@...co.com>
To:	Ralph Campbell <ralph.campbell@...gic.com>
Cc:	"linux-kernel\@vger.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-rdma\@vger.kernel.org" <linux-rdma@...r.kernel.org>
Subject: Re: InfiniBand/RDMA merge plans for 2.6.33

 > I understand your frustration in having to deal with a large amount
 > of code. If you included all the Mellanox firmware in the mlx4
 > driver, it would be huge too. I'm limited in what I can do given
 > the complexity of the IBTA spec.

Sure, I understand that your driver is going to be pretty big.  However,
ipath was ~38 KLoC, which qib is ~54 KLoC (after the latest cleanup --
it started over 60K if I recall correctly).  That's 40% bigger, 16 KLoC
in absolute terms -- and you dropped support for HT HCAs!

 > QLogic is not "abandoning the ipath driver as unmaintainable".
 > The thought was that trying to incrementally patch in all the changes
 > needed to support dual ports, QDR, chip register addresses, etc.
 > would result in larger patches than renaming the driver. It was a
 > chicken-and-egg problem because until the new code was fully
 > written and debugged, we couldn't post it and we couldn't patch
 > ipath until we knew all the places that needed to be changed.

We can quibble about the reason, but the end effect is that ipath is
abandonded, right?  Maybe a better approach would have been to write a
new driver for the new chip and not try to move the old devices out of
ipath.  But of course it's way too late to start over.

Anyway, we'll get qib in eventually -- but it may take some work.

 - R.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/