This is a substantial change to the EM driver that I would appreciate some testing and feedback on. You just replace the two files in your STABLE tree. Its big enough that this seemed easier than a patch. Whats in this: A change Mike Silbersack came up with, it makes the watchdog period twice as long, but then in the middle of it fires off a clean. Thats a small change. Secondly, I took the locking mechanism I used in ixgbe and added it here. It uses two locks: core and tx instead of one. Third, added the EEPROM dumper into the driver, if you do `sysctl dev.em.X.debug=2' it will dump 32 words worth of the prom, this will allow an easy way to tell if you need a patch to an 82573 for instance. Next, I found a serious bug for Zoar (82575) users, it had to do with how and when I decide to try and use MSI/X, the fact that no one has squawked about this must mean all users are running 7 :) cuz on STABLE right now the driver load will FAIL if you enable MSI/X on the system. What I did was to create a new function for setting up msi and msix, its much cleaner and fixes this problem. In any case, I have done some pounding on this but wider testing would be good, let me know what you find. I hope to get this all resolved and into the tree for 6.3. Jack -------------- next part -------------- A non-text attachment was scrubbed... Name: if_em.h Type: application/octet-stream Size: 15134 bytes Desc: not available Url : http://lists.freebsd.org/pipermail/freebsd-stable/attachments/20071101/063777a8/if_em.obj
> You just replace the two files in your STABLE tree. Its big > enough that this seemed easier than a patch.Did you miss a file ? I nly see a new header file in here. Am keen to try this out though... -pcf.
Eh, what I see is if_em.h and if_em.c, does the version that came thru not have both?? Jack On 11/1/07, Pete French <petefrench@ticketswitch.com> wrote:> > You just replace the two files in your STABLE tree. Its big > > enough that this seemed easier than a patch. > > Did you miss a file ? I nly see a new header file in here. Am keen to > try this out though... > > -pcf. >
Although I see it at least one person claims the message came thru with only the header file, so I am going to send if_em.c thru again. Jack
Jack Vogel wrote:> Although I see it at least one person claims the message > came thru with only the header file, so I am going > to send if_em.c thru again. > > Jack > > ------------------------------------------------------------------------ > > _______________________________________________ > freebsd-net@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-net > To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org"In this mail if_em.c is not here for me, as in the previous.
On Fri, Nov 02, 2007 at 12:10:59AM -0800, Jack Vogel wrote:> Although I see it at least one person claims the message > came thru with only the header file, so I am going > to send if_em.c thru again. >Attachments are usually stripped off by the mailing list software, so you are probably better off putting the files on a webpage somewhere and just telling us the URL. -- <Insert your favourite quote here.> Erik Trulsson ertr1013@student.uu.se