Driver patch to look at...

[Randall Stewart]

Actually, no it is used.

If you look in if_var.h int he drbr_putback() function, it does
a buf_ring_swap when the old mbuf pointer does not equal the
new mbuf pointer. This *does* happen, I crashed at least once
yesterday when the igb driver did something to free the original
mbuf and return a new mbuf with the data (prepend or some such).

I also have found several issues that I have fixed this morning.. its been
crash city on my test beds..

Here is the latest patch with all fixes and suggested changes from emaste (thanks Ed)

R


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: driver_patch.txt
URL: <http://lists.freebsd.org/pipermail/freebsd-net/attachments/20130205/b39367c2/attachment.txt>
-------------- next part --------------

On Feb 5, 2013, at 12:13 PM, John Baldwin wrote:

> On Tuesday, February 05, 2013 10:24:24 am Randall Stewart wrote:
>> Here is an updated patch? sigh.. I foobar'd the ALTQ stuff.. lots of crashes 
> ;-D
> 
> Heh, I like this better, thanks.  I think you can remove buf_ring_swap() as it 
> is no longer used?
> 
> -- 
> John Baldwin
> _______________________________________________
> freebsd-net at freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-net
> To unsubscribe, send any mail to "freebsd-net-unsubscribe at freebsd.org"
> 

------------------------------
Randall Stewart
803-317-4952 (cell)