linux-kernel - Re: [patch v2 1/2] sched: check for prev_cpu == this_cpu before calling wake

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-Id: <1271782980.2871.6.camel@sbs-t61.sc.intel.com>
Date:	Tue, 20 Apr 2010 10:03:00 -0700
From:	Suresh Siddha <suresh.b.siddha@...el.com>
To:	Peter Zijlstra <peterz@...radead.org>
Cc:	Mike Galbraith <efault@....de>, Ingo Molnar <mingo@...e.hu>,
	Arjan van de Ven <arjan@...ux.jf.intel.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Vaidyanathan Srinivasan <svaidy@...ux.vnet.ibm.com>,
	Yanmin Zhang <yanmin_zhang@...ux.jf.intel.com>,
	Gautham R Shenoy <ego@...ibm.com>
Subject: Re: [patch v2 1/2] sched: check for prev_cpu == this_cpu before
 calling wake_affine()

On Tue, 2010-04-20 at 01:55 -0700, Peter Zijlstra wrote:
> On Tue, 2010-04-20 at 10:46 +0200, Peter Zijlstra wrote:
> > So should we be using idle_cpu(i) instead?
> 
> something like the below..

Looks good to me.

Acked-by: Suresh Siddha <suresh.b.siddha@...el.com>

> 
> ---
> Index: linux-2.6/kernel/sched_fair.c
> ===================================================================
> --- linux-2.6.orig/kernel/sched_fair.c
> +++ linux-2.6/kernel/sched_fair.c
> @@ -1375,26 +1375,25 @@ find_idlest_cpu(struct sched_group *grou
>  /*
>   * Try and locate an idle CPU in the sched_domain.
>   */
> -static int
> -select_idle_sibling(struct task_struct *p, int target)
> +static int select_idle_sibling(struct task_struct *p, int target)
>  {
>  	int cpu = smp_processor_id();
>  	int prev_cpu = task_cpu(p);
> -	int i;
>  	struct sched_domain *sd;
> +	int i;
>  
>  	/*
>  	 * If the task is going to be woken-up on this cpu and if it is
>  	 * already idle, then it is the right target.
>  	 */
> -	if (target == cpu && !cpu_rq(cpu)->cfs.nr_running)
> +	if (target == cpu && idle_cpu(cpu))
>  		return cpu;
>  
>  	/*
>  	 * If the task is going to be woken-up on the cpu where it previously
>  	 * ran and if it is currently idle, then it the right target.
>  	 */
> -	if (target == prev_cpu && !cpu_rq(prev_cpu)->cfs.nr_running)
> +	if (target == prev_cpu && idle_cpu(prev_cpu))
>  		return prev_cpu;
>  
>  	/*
> @@ -1405,7 +1404,7 @@ select_idle_sibling(struct task_struct *
>  			break;
>  
>  		for_each_cpu_and(i, sched_domain_span(sd), &p->cpus_allowed) {
> -			if (!cpu_rq(i)->cfs.nr_running) {
> +			if (idle_cpu(i)) {
>  				target = i;
>  				break;
>  			}
> @@ -1479,16 +1478,14 @@ select_task_rq_fair(struct rq *rq, struc
>  				want_sd = 0;
>  		}
>  
> -		if (want_affine) {
> -			/*
> -			 * If both cpu and prev_cpu are part of this domain,
> -			 * cpu is a valid SD_WAKE_AFFINE target.
> -			 */
> -			if (cpumask_test_cpu(prev_cpu, sched_domain_span(tmp))
> -			    && (tmp->flags & SD_WAKE_AFFINE)) {
> -				affine_sd = tmp;
> -				want_affine = 0;
> -			}
> +		/*
> +		 * If both cpu and prev_cpu are part of this domain,
> +		 * cpu is a valid SD_WAKE_AFFINE target.
> +		 */
> +		if (want_affine && (tmp->flags & SD_WAKE_AFFINE) &&
> +		    cpumask_test_cpu(prev_cpu, sched_domain_span(tmp))) {
> +			affine_sd = tmp;
> +			want_affine = 0;
>  		}
>  
>  		if (!want_sd && !want_affine)
> 
> 

--
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/