Re: [PATCH] show_trace() module_end = 0?

[Andrew Morton]

Hugh Dickins wrote:
> 
> > The other, minor problem is that we should walk the module_list under
> > lock_kernel() only, but I wasn't brave enough to add this to the
> > show_trace() code path.
> 
> I think it's just modlist_lock you'd need.  Not sure whether better
> to try for it or ignore it.  CC'ed Andrew "spinlock-buster" Morton
> for his opinion.

Debugging things like show_trace() may be called from any context
at all, and hence really cannot take locks.  We just live with the
race possibilities.

One could possibly play games with spin_trylock, but I don't expect
it'd help much - if you failed to get the lock what can you do?

-