[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <alpine.LFD.2.00.0910122053540.9428@localhost.localdomain>
Date: Mon, 12 Oct 2009 20:56:25 +0200 (CEST)
From: Thomas Gleixner <tglx@...utronix.de>
To: Ingo Molnar <mingo@...e.hu>
cc: Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca>,
akpm@...ux-foundation.org, linux-kernel@...r.kernel.org,
"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
Subject: Re: [patch 0/4] DEBUG_RCU_HEAD: Debug and fix racy call_rcu()
users
On Mon, 12 Oct 2009, Ingo Molnar wrote:
> * Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca> wrote:
>
> > * Ingo Molnar (mingo@...e.hu) wrote:
> > >
> > > * Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca> wrote:
> > >
> > > > Here is a patchset, done on 2.6.30.9, which permits to detect and fix
> > > > racy call_rcu() users.
> > >
> > > Nice idea - but any reason why this isnt using the debugojects
> > > framework? That debugging framework is upstream already and allows this
> > > kind of object lifetime debugging - in a much broader way.
> > >
> >
> > Other than: it did not occur to me, no, there is no reason for not using
> > debugobjects there. :-)
> >
> > As long as we can tag the object as
> > - initialized
> > - active
> > - inactive
> >
> > And dump a nice fat warning when the object is activated twice, that
> > should suffice. Now time is a bit short on my side, but I'll keep in
> > mind to respin a version on top of debugobjects soon.
It does, and it also provides you a mechanism to prevent
damage. i.e. you can provide a callback function which can deactivate
an active object before activating it again. That turned out to be
useful for timer wreckage which happened usually way after the
offending code corrupted lists etc.
Thanks,
tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists