> * Alfred Perlstein <[EMAIL PROTECTED]> [030316 21:19] wrote:
> > um..
> >
> ...
> 840 _FLAGS_OUTRANGE) {
> 841 WI_UNLOCK(sc);
> 842 return;
> 843 }
> 844 KASSERT((ifp->if_flags & IFF_OACTIVE) == 0,
> 845 ("wi_start: if_flags %x\n", ifp->if_flags));
> 846
> 847 memset(&frmhdr, 0, sizeof(frmhdr));
>
> >
> > What's up here?
>
> It's a race, we shouldn't be inspecting the ifp without a lock.
>
> I think this kassert should be removed.
>
> Do you guys concurr?
Warner has this pending with some other fixes; perhaps he can accelerate
doing the commit? The assert is actually just bogus (if_start can be called
under certain conditions with IFF_OACTIVE set.
Sam
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
