svn commit: r249800 - head/sys/dev/bwn

John Baldwin jhb at freebsd.org
Tue Apr 23 15:46:42 UTC 2013


On Tuesday, April 23, 2013 9:02:58 am Eitan Adler wrote:
> Author: eadler
> Date: Tue Apr 23 13:02:57 2013
> New Revision: 249800
> URL: http://svnweb.freebsd.org/changeset/base/249800
> 
> Log:
>   Avoid warning about uninitalized variable
>   
>   PR:		kern/176712
>   Submitted by:	Hiren Panchasara <hiren.panchasara at gmail.com> (earlier vesion)
>   Approved by:	cperciva (mentor)

I don't think this is correct.  In the 'back' case you want to reuse the previous
mbuf.  It doesn't have a physical address of 0.  Instead, the dr->setdesc() at the
bottom of this routine should use the previous physical address.  I think you can
fix this by reverting this and changing dr->setdesc() to use meta->mt_paddr instead
of paddr.

-- 
John Baldwin


More information about the svn-src-head mailing list