linux-kernel - Re: [patch v3 1/6] sched, nohz: introduce nohz

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20111206121401.GA12083@linux.vnet.ibm.com>
Date:	Tue, 6 Dec 2011 17:44:02 +0530
From:	Srivatsa Vaddagiri <vatsa@...ux.vnet.ibm.com>
To:	Suresh Siddha <suresh.b.siddha@...el.com>
Cc:	Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...e.hu>,
	Venki Pallipadi <venki@...gle.com>,
	Mike Galbraith <efault@....de>,
	linux-kernel <linux-kernel@...r.kernel.org>,
	Tim Chen <tim.c.chen@...ux.jf.intel.com>, alex.shi@...el.com
Subject: Re: [patch v3 1/6] sched, nohz: introduce nohz_flags in the struct rq

* Suresh Siddha <suresh.b.siddha@...el.com> [2011-12-01 17:07:32]:

> @@ -4866,18 +4866,15 @@ static void nohz_balancer_kick(int cpu)
>  			return;
>  	}
> 
> -	if (!cpu_rq(ilb_cpu)->nohz_balance_kick) {
> -		cpu_rq(ilb_cpu)->nohz_balance_kick = 1;
> -
> -		smp_mb();
> -		/*
> -		 * Use smp_send_reschedule() instead of resched_cpu().
> -		 * This way we generate a sched IPI on the target cpu which
> -		 * is idle. And the softirq performing nohz idle load balance
> -		 * will be run before returning from the IPI.
> -		 */
> -		smp_send_reschedule(ilb_cpu);
> -	}
> +	if (test_and_set_bit(NOHZ_BALANCE_KICK, nohz_flags(cpu)))

s/cpu/ilb_cpu?

Also given that 'cpu' argument to nohz_balancer_kick() is no longer used, we can
avoid passing any argument to it as well ..

> +		return;
> +	/*
> +	 * Use smp_send_reschedule() instead of resched_cpu().
> +	 * This way we generate a sched IPI on the target cpu which
> +	 * is idle. And the softirq performing nohz idle load balance
> +	 * will be run before returning from the IPI.
> +	 */
> +	smp_send_reschedule(ilb_cpu);
>  	return;
>  }

- vatsa

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