On Fri, Feb 13, 2015 at 10:29:01AM -0800, hiren panchasara wrote: > On 02/12/15 at 01:31P, Randall Stewart wrote: > > Author: rrs > > Date: Thu Feb 12 13:31:08 2015 > > New Revision: 278623 > > URL: https://svnweb.freebsd.org/changeset/base/278623 > > > > Log: > > This fixes a bug I in-advertantly inserted when I updated the callout > > code in my last commit. The cc_exec_next is used to track the next > > when a direct call is being made from callout. It is *never* used > > in the in-direct method. When macro-izing I made it so that it > > would separate out direct/vs/non-direct. This is incorrect and can > > cause panics as Peter Holm has found for me (Thanks so much Peter for > > all your help in this). What this change does is restore that behavior > > but also get rid of the cc_next from the array and instead make it > > be part of the base callout structure. This way no one else will get > > confused since we will never use it for non-direct. > > > > Reviewed by: Peter Holm and more importantly tested by him ;-) > > MFC after: 3 days. > > Sponsored by: Netflix Inc. > > Can I get more info/backtraces for the panic, please? >
Sure. These two panics were seen: http://people.freebsd.org/~pho/stress/log/rrs004.txt http://people.freebsd.org/~pho/stress/log/rrs005.txt - Peter _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
