netdev - Re: [PATCH] New driver "sfc" for Solarstorm SFC4000 controller (try #7)

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

Message-ID: <20080305142203.GQ2988@solarflare.com>
Date:	Wed, 5 Mar 2008 14:22:04 +0000
From:	Ben Hutchings <bhutchings@...arflare.com>
To:	David Miller <davem@...emloft.net>
Cc:	shemminger@...tta.com, dcbw@...hat.com, netdev@...r.kernel.org,
	linux-net-drivers@...arflare.com
Subject: Re: [PATCH] New driver "sfc" for Solarstorm SFC4000 controller (try #7)

David,

We concede that the driver is currently too large to be properly
reviewed in one go.

We propose to:

1. Remove what we can - i.e. internal HW support, extraneous comments,
unused HW register defines, and non-essential peripheral driver features
such as sensor support - leaving a minimal network driver.

2. Post the minimal driver in sub-100K patches to netdev.  This
minimal driver will still require several patches, but future patches
should represent independent pieces of functionality.

If you can see reasons why this will still not get us to a point where
a review can take place then please tell us what else is required.

We hope to complete this work within the next few days.

Ben.

-- 
Ben Hutchings, Senior Software Engineer, Solarflare Communications
Not speaking for my employer; that's the marketing department's job.
--
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