Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/net

In article <>,
David Holland  <> wrote:
>On Thu, Dec 05, 2013 at 10:55:35AM -0500, Christos Zoulas wrote:
> > Modified Files:
> >     src/sys/net: bpf.c
> > 
> > Log Message:
> > It is silly to kill the system when an interface failed to clear
> > promiscuous mode. Some return EINVAL when they are dying, but
> > others like USB return EIO.  Downgrade to a DIAGNOSTIC printf. Same
> > should be done for the malloc/NOWAIT, but this is rarely hit.
>Having an interface unexpectedly in promiscuous mode tends to cause
>panic()-like behavior in operators and sysadmins, so it's not clear if
>this is actually an improvement. :-)
>I think the printout should be unconditional, and rather than
>"ifpromisc failed" it should say "bpf: failed to turn off promiscuous
>mode on %s" with the interface name.

Sure, but remember:
        1. it did not warn/panic before if the driver returned EINVAL
        2. most of the time this is because the driver is a dying state
           (someone pulled a usb ethernet out).


Home | Main Index | Thread Index | Old Index