cvs commit: src/sys/dev/mxge if_mxge.c
    Andrew Gallatin 
    gallatin at cs.duke.edu
       
    Wed Jan 31 16:23:50 UTC 2007
    
    
  
Scott Long writes:
 > Yeah, I can't stress enough that busdma maps are opaque handles and
 > should always be handled like they are valid.  It's only an
 > implementation detail that they are often NULL on i386 and amd64.
It was overly cautious resource freeing code, influenced by lots
of programming on other OSes, where free(NULL) is a panic :(
I was able to simplify it some..
FWIW, I've got one small leak of 2 "temp" items left that I'm going to
wait for dtrace to track down.
Drew
    
    
More information about the cvs-src
mailing list