In message: <[EMAIL PROTECTED]>
"Sam Leffler" <[EMAIL PROTECTED]> writes:
: > * 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.
This is part of my commit. The KASSERT is totally bogus.
Warner
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
