svn commit: r278623 - head/sys/kern
hiren panchasara
hiren at strugglingcoder.info
Fri Feb 13 22:25:31 UTC 2015
On 02/13/15 at 08:11P, Peter Holm wrote:
> 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
Thank you, Peter.
cheers
Hiren
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 618 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-src-head/attachments/20150213/840276c4/attachment.sig>
More information about the svn-src-head
mailing list