On Wed, Sep 05, 2012 at 12:11:14AM -0500, Mike Silbersack wrote: > On 9/5/12 3:56 PM, YongHyeon PYUN wrote: > >On Tue, Sep 04, 2012 at 11:35:13PM -0500, Mike Silbersack wrote: > >>Does anyone want to review this patch before I check it in? The change > >>has been reviewed and tested by coworkers, but not yet reviewed by any > >>other FreeBSD committers. > >> > >>http://www.silby.com/patches/if_bxe.c-safestop.patch > >> > >>This resolves an issue we saw at work where IPMI would report bus errors > >>when you rebooted a system with bxe NICs if you had not UP'd all of the > >>bxe NICs before the shutdown. > >Yeah I also have a similar patch. But I checked sc->state after > >getting a BXE_CORE_LOCK as the state is protected by the lock. > > > >>Thanks, > >> > >>Mike "Silby" Silbersack > > Good catch. How does this look? > > http://www.silby.com/patches/if_bxe.c-safestop-2.patch >
Patch looks good to me. _______________________________________________ freebsd-net@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-net To unsubscribe, send any mail to "freebsd-net-unsubscr...@freebsd.org"
