cvs commit: src/sys/net bpf.c

Christian S.J. Peron csjp at FreeBSD.org
Tue May 27 15:00:19 UTC 2008


John

Sorry for the delay.  I don't think there was a reason.  I just
took a look at the change and i think it should be safe to be MFCed

On Wed, May 07, 2008 at 04:17:57PM -0400, John Baldwin wrote:
> On Thursday 14 June 2007 10:53:51 pm Christian S.J. Peron wrote:
> > csjp        2007-06-15 02:53:51 UTC
> > 
> >   FreeBSD src repository
> > 
> >   Modified files:
> >     sys/net              bpf.c 
> >   Log:
> >   - Conditionally pickup Giant around the network interface
> >     ioctl routines if we are running with !mpsafenet
> >   - Change un-conditional Giant acquisition around ifpromisc
> >     to occur only if we are running with !mpsafenet
> >   
> >   With these locking bits in place, we can now remove the Giant
> >   requirement from BPF, so drop the D_NEEDGIANT device flag.
> >   This change removes Giant acquisitions around BPF device
> >   handlers (read, write, ioctl etc).
> >   
> >   MFC after:      1 month
> >   Discussed with: rwatson
> 
> Would it be ok to MFC this to 6.x?  (Looks like it was just never MFC'd, was 
> curious if a reason came up why it couldn't be MFC'd?)
> 
> -- 
> John Baldwin


More information about the cvs-src mailing list