On Tuesday, March 20, 2012 1:45:32 pm Jason Wolfe wrote: > On Thu, Mar 15, 2012 at 11:17 AM, John Baldwin <j...@freebsd.org> wrote: > > On Sunday, March 11, 2012 3:47:07 am Hooman Fazaeli wrote: > >> On 3/11/2012 5:31 AM, Adrian Chadd wrote: > >> > Are you able to post the patch here? > >> > Maybe Jack can look at what's going on and apply it to the latest > >> > intel ethernet driver. > >> > > >> > > >> > Adrian > >> > > >> > >> Below is the patch for if_em.c (7.2.3). It simply checks driver's > >> queue status when the link state changes (inactive -> active) and > >> start transmit task if queue(s) are not empty. > >> > >> It also contains stuff I have added to compile on 7 plus some code > >> for test and diagnostics. > > > > Hmm, so I have yet to test this, but I found several bugs related to transmit > > in em(4) and igb(4) recently just reading the code. (Mostly unnecessary > > scheduling of tasks for transmit.) I've included your change of restarting > > TX when link becomes active. I've also updated it to fix resume for em > > and igb to DTRT when buf_ring is used, and to not include old-style start > > routines at all when using multiq. It is at > > http://www.freebsd.org/~jhb/patches/e1000_txeof2.patch > > > > -- > > John Baldwin > > John/Hooman, > > Thank for the patch sirs, so far it does look like it did the trick. > I'll know for certain here in a few days if I'm still in the clear. > I'm guessing after it goes through some more testing it'll be too late > to slip it into 8.3?
Yes, this is too late for 8.3, but thanks for testing! -- John Baldwin _______________________________________________ freebsd-net@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-net To unsubscribe, send any mail to "freebsd-net-unsubscr...@freebsd.org"
