Re: [PATCH v2] perf: Store active software events in a hashlist

[Frederic Weisbecker <fweisbec@gmail.com>]

On Thu, Apr 08, 2010 at 10:04:41PM +0200, Eric Dumazet wrote:
> Le jeudi 08 avril 2010 à 21:39 +0200, Frederic Weisbecker a écrit :
> 
> ...
> 
> > @@ -5401,6 +5507,16 @@ static void __cpuinit perf_event_init_cpu(int cpu)
> >  	spin_lock(&perf_resource_lock);
> >  	cpuctx->max_pertask = perf_max_events - perf_reserved_percpu;
> >  	spin_unlock(&perf_resource_lock);
> > +
> > +	mutex_lock(&cpuctx->hlist_mutex);
> > +	if (&cpuctx->hlist_refcount > 0) {
> 
> 
> 	if (cpuctx->hlist_refcount > 0)


Doh! Thanks, will fix.


> 
> > +		struct swevent_hlist *hlist;
> > +
> > +		hlist = kzalloc(sizeof(*hlist), GFP_KERNEL);
> > +		WARN_ON_ONCE(!hlist);
> > +		rcu_assign_pointer(cpuctx->swevent_hlist, hlist);
> > +	}
> > +	mutex_unlock(&cpuctx->hlist_mutex);
> >  }
> 
> 
>