lkml.org 
[lkml]   [2014]   [Mar]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH RT] dont calculate hweight in update_migrate_disable
On Mon, 24 Mar 2014 13:18:48 +0100
Nicholas Mc Guire <der.herr@hofr.at> wrote:

> Hi !
>
> Proposal for a minor optimization in update_migrate_disable - its only a few
> instructions saved but those are in the hot path of locks so it might be worth
> it
>
> When being scheduled out while migrate_disable > 0 and migrate_disabled_updated
> is not yet set we end up here (kernel/sched/core.c):
>
> static inline void update_migrate_disable(struct task_struct *p)
> {
> ...
>
> mask = tsk_cpus_allowed(p);
>
> if (p->sched_class->set_cpus_allowed)
> p->sched_class->set_cpus_allowed(p, mask);
> p->nr_cpus_allowed = cpumask_weight(mask);
>
>
> as we only can get here if migrate_disable > 0 there is no need to calculate
> the cpumask_weight(mask) as tsk_cpus_allowed in that case will return
> cpumask_of(task_cpu(p)) which only can have a hamming weight of 1 anyway.
> So we can simply do:
>
> p->nr_cpus_allowed = 1;
>
> without changing the behavior.
>
> Patch is on top of 3.12.13-rt21
>

Reviewed-by: Steven Rostedt <rostedt@goodmis.org>

-- Steve



\
 
 \ /
  Last update: 2014-03-24 15:41    [W:0.106 / U:0.052 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site