Subject: updated sk(4) patches - final review, please?
To: None <tech-net@NetBSD.org>
From: Jeff Rizzo <riz@NetBSD.org>
List: tech-net
Date: 02/07/2006 15:59:45
This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
--------------enigFC38DDFDF5298AE880E8059B
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

I finally got around to finishing up the interrupt moderation timer
patches for sk(4), which I hadn't touched since early December, to
address some of the remaining criticisms.  I would greatly appreciate if
someone could review these;  I'd like to get them committed this week,
if possible.  I've also included the "jumbo frames" patch, which applies
_after_ the interrupt moderation patch, and which I'd like to commit
shortly thereafter.


Changes since the previous version of the interrupt moderation patches:

- interrupt moderation timer is per skc device, instead of global.

- changes to the timer take effect on the next interrupt after the
sysctl is changed, rather than requiring a reset of the interface.  I am
particularly keen on having someone look this part over to make sure I
didn't do something really stupid - especially the piece in
sk_sysctl_verify() where I match the device name... it seems a bit
hackish, but I could not figure another way to achieve the goal.

As far as the jumbo patches, I have not changed anything since the last
version - I've gotten a couple reports that it works for people, and one
mixed report, though I'm waiting to hear back whether the problem was
somewhere else.  (in particular, the wm(4) driver, which seems likely
given the fix that appeared for jumbo frames on wm yesterday)    I did
receive a suggestion that I incorporate this patch from OpenBSD:

http://www.openbsd.org/cgi-bin/cvsweb/src/sys/dev/pci/if_sk.c.diff?r1=3D1=
=2E69&r2=3D1.70&f=3Dh

=2E..but I couldn't duplicate the memory leak that it fixes, and the patc=
h
rendered my sk(4) interfaces inoperable.  Suggestions welcome.

The patches can be retrieved here:
http://www.tastylime.net/netbsd/sk_intmod.diff
http://www.tastylime.net/netbsd/sk_jumbo.diff


Thanks!
+j


--------------enigFC38DDFDF5298AE880E8059B
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (Darwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iQCVAwUBQ+k0d7OuUtxCgar5AQO9/QP+M1ouTDCbJG5DhG+r0ZwNIDcnq4qUr9LU
iNDtxmZSnMUxxD5GfeFR5oKG+cE2S4i3uMO52f1IoNloy7f+55YRGwzzWiGC9K8y
HtG46qmwukMp796XgRRNJbdSkk7FZKXGVqqhLm0qqeCtE3hORRnw3ptzYJEd5Hm0
1wwBZ9MXiaM=
=xBBM
-----END PGP SIGNATURE-----

--------------enigFC38DDFDF5298AE880E8059B--