Re: [RFC PATCH v2 3/3] sched: introduce synchronized idle injection

[Jacob Pan]

On Tue, 10 Nov 2015 14:23:24 +0100
Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:

> > +/* protect injection parameters from runtime changes */
> > +static DEFINE_SPINLOCK(idle_inject_lock);  
> 
> A global lock, yay :-), I think you want this to be a RAW_SPINLOCK
> though. As on -RT this would want to actually run from IRQ context
> too.
> 
I am not using this lock in per cpu timer handler, I guess it would not
be in IRQ context?
> > +	if (bitmap_weight(idle_inject_cpumask,
> > num_possible_cpus())) {  
> 
> I don't get the point of this bitmap; with the cpu notifier you
> basically ensure this is equal to online_mask.
> 
> Also, this weight test is pointless, if the bitmap is empty the
> for_each_set_bit() should be of equal cost -- and afaict nothing
> calling this is performance critical in the first place.
Agreed, I will remove that. I was planning on a subset of CPUs but not
for now, e.g. a socket or domain.
--
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/