[dpdk-dev] [PATCH 0/4] Broadcom 10G NIC Poll Mode Driver

Thomas Monjalon thomas.monjalon at 6wind.com
Fri Feb 13 09:31:53 CET 2015


2015-02-12 16:45, Stephen Hemminger:
> On Thu, 12 Feb 2015 14:56:06 +0000
> Thomas Monjalon <thomas.monjalon at 6wind.com> wrote:
> 
> > 2015-02-12 06:33, Stephen Hemminger:
> > > On Thu, 12 Feb 2015 15:00:13 +0100
> > > Thomas Monjalon <thomas.monjalon at 6wind.com> wrote:
> > > 
> > > > Hi Stephen,
> > > > 
> > > > 2015-02-07 07:15, Stephen Hemminger:
> > > > > Source was from FreeBSD.
> > > > > 
> > > > > changes were to make it work and lots of de-uglification.
> > > > 
> > > > Maintenance would be eased by showing this work with separate commits.
> > > > You should at least have a patch for importing FreeBSD source without
> > > > modifications.
> > > > 
> > > > Other comment: there could be other drivers for Broadcom devices.
> > > > Why not keeping the name bnx2x?
> > > > 
> > > > Thanks
> > > 
> > > That history is lost.
> > 
> > I don't understand your answer,
> > and you forgot one question.
> > Please reply below each question.
> > 
> > I said: "You should at least have a patch for importing FreeBSD source
> > without modfications". Is the initial FreeBSD source lost?

Without answer, I guess Brocade lost its initial import.

> > Do you agree to rename to bnx2x?
> 
> This was based of driver called bxe in FreeBsd but the
> changes are so massive that there is no point in creating such a
> patch except to keep some lawyer happy.
> 
>  37 files changed, 33668 insertions(+), 159047 deletions(-)
> 
> It might be okay to rename the visible config option and directory
> to bxe but not going to change internal things.

Yes it's a minimal change.
By the way, you named it "bcm" but it has been acquired by QLogic:
	http://www.broadcom.com/press/release.php?id=s832628
So we should refer at it as "NetXtreme II".
I have no strong opinion about the name of the pmd (bnx2x, bxe, whatever),
except it should not be bcm.

When making a v2, please don't forget to add a section in the MAINTAINERS file.

Thank you



More information about the dev mailing list