Re: [PATCH 2/2] [RESEND] tracing: Add a new workqueue tracer

[Steven Rostedt]

On Mon, 12 Jan 2009, Frederic Weisbecker wrote:
> > WARNING: usage of NR_CPUS is often wrong - consider using cpu_possible(), 
> > num_possible_cpus(), for_each_possible_cpu(), etc
> > #953: FILE: kernel/trace/trace_workqueue.c:153:
> > +	for (i = 0; i < NR_CPUS; i++) {
> > 
> > We should probably change them to for_each_possible_cpu, or online cpu.
> > 
> > -- Steve
> 
> 
> I found it very convenient because I didn't have to allocate
> my cpu list array dynamically with it :-)
> 
> I'm preparing a v2...
>  

Thanks!

-- Steve

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/