netdev - Re: [PATCH] [RFC] New driver "sfc" for Solarstorm SFC4000 controller

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>] [day] [month] [year] [list]

Date:	Thu, 29 Nov 2007 16:27:41 +0000
From:	Robert Stonehouse <rstonehouse@...arflare.com>
To:	Stephen Hemminger <shemminger@...ux-foundation.org>
Cc:	netdev@...r.kernel.org, linux-net-drivers@...arflare.com,
	bhutchings@...arflare.com
Subject: Re: [PATCH] [RFC] New driver "sfc" for Solarstorm SFC4000 controller

Stephen Hemminger wrote:
> On Fri, 23 Nov 2007 17:08:15 +0000
> Ben Hutchings <bhutchings@...arflare.com> wrote:
>   
>> 1. When we enable NAPI polling, we need to set __LINK_STATE_START in
>>    the net device used for NAPI.  This bit is commented as private in
>>    netdevice.h, but e1000 also does this.  Is this incorrect?
>>     
> Why are you using it directly?
>   
It seems this line is historic and we can remove it.

> The driver is pretty big (28K loc), and non trivial to get a good review.

We are aware that it appears to be a large amount of code.

The driver does support many types of PHY (10Gbase-T, XFP, CX4) on five
different 10G reference designs and one 1G NIC ref design.

There is also support for two generations of controller silicon, full
ethtool support, start of day self-tests and an mtd driver for putting
PXE images into flash.

Perhaps we could help making it more reviewable by suggesting some sets
of files that can be reviewed together that represent different parts of
the functionality. The main functionality is contained in efx.c, rx.c,
tx.c and falcon.c if that helps.

> Minor note:
> * use u8 not uint8_t (etc.)
> * gone overboard with docbook style comments, they are only needed
>   on external API's.
> * Please use dev_err() rather than reinventing own message macros.

OK - we will look at addressing these as well as checkpatch violations
and resubmit a patch shortly.

> * why are you exporting symbol's?
>   
We need a small API so that other drivers can use parts of the hardware
after the main driver has performed initialisation.

One example is the mtd driver to access the flash. Another example is an
accelerated driver for Xen that Solarflare is in the process of
submitting.

Regards

-- 
Rob Stonehouse
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html