Date: Mon, 26 Nov 2007 20:50:22 +0100 From: "Attilio Rao" <attilio@freebsd.org> To: "Jack F Vogel" <jfv@freebsd.org> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/dev/em if_em.c Message-ID: <3bbf2fe10711261150i5a69657aj3dba41b12a4b485b@mail.gmail.com> In-Reply-To: <200711261947.lAQJl4HD033502@repoman.freebsd.org> References: <200711261947.lAQJl4HD033502@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
2007/11/26, Jack F Vogel <jfv@freebsd.org>: > jfv 2007-11-26 19:47:03 UTC > > FreeBSD src repository > > Modified files: > sys/dev/em if_em.c > Log: > Fix for a reported panic in certain circumstances. When > calling em_stop() now make sure the TX lock is held as > well as CORE. Thanks! To make it more explicit, callout_stop() needs to run always with referred lock held in order to not have a race for when running the callout handler. Now both callout_stop() are covert. Attilio -- Peace can only be achieved by understanding - A. Einstein
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3bbf2fe10711261150i5a69657aj3dba41b12a4b485b>