linux-kernel - Re: + cpufreq-revert-remove-rwsem-lock-from-cpufreq_gov_stop-call-second-c all-site.patch added to -mm tree

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>] [day] [month] [year] [list]

Message-ID: <20100514224227.GD14234@Krystal>
Date:	Fri, 14 May 2010 18:42:27 -0400
From:	Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca>
To:	akpm@...ux-foundation.org
Cc:	mm-commits@...r.kernel.org, andrej.gelenberg@....edu,
	davej@...emonkey.org.uk, venkatesh.pallipadi@...el.com,
	linux-kernel@...r.kernel.org
Subject: Re: +
	cpufreq-revert-remove-rwsem-lock-from-cpufreq_gov_stop-call-second-c
	all-site.patch added to -mm tree

* akpm@...ux-foundation.org (akpm@...ux-foundation.org) wrote:
> 
> The patch titled
>      cpufreq: revert "[CPUFREQ] remove rwsem lock from CPUFREQ_GOV_STOP call (second call site)"
> has been added to the -mm tree.  Its filename is
>      cpufreq-revert-remove-rwsem-lock-from-cpufreq_gov_stop-call-second-call-site.patch
> 
> Before you just go and hit "reply", please:
>    a) Consider who else should be cc'ed
>    b) Prefer to cc a suitable mailing list as well
>    c) Ideally: find the original patch on the mailing list and do a
>       reply-to-all to that, adding suitable additional cc's
> 
> *** Remember to use Documentation/SubmitChecklist when testing your code ***
> 
> See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find
> out what to do about this
> 
> The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/
> 
> ------------------------------------------------------
> Subject: cpufreq: revert "[CPUFREQ] remove rwsem lock from CPUFREQ_GOV_STOP call (second call site)"
> From: Andrej Gelenberg <andrej.gelenberg@....edu>
> 
> 395913d0b1db37092ea3d9d69b832183b1dd84c5 ("[CPUFREQ] remove rwsem lock
> from CPUFREQ_GOV_STOP call (second call site)") is not needed, because
> there is no rwsem lock in cpufreq_ondemand and cpufreq_conservativ

nit: missing "e"

> anymore.  Lock should not be released until the work done.

Yep, locking has been restructured in the workqueue handler so the
deadlock scenario cannot occur anymore, so this fix seems appropriate.
It is needed to synchronize the governor start/stop. Without this fix, a
race between stop/start could corrupt the ondemand/conservative timer.

Acked-by: Mathieu Desnoyers <mathieu.desnoyers@...icios.com>

> 
> Addresses https://bugzilla.kernel.org/show_bug.cgi?id=1594
> 
> Signed-off-by: Andrej Gelenberg <andrej.gelenberg@....edu>
> Cc: Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca>
> Cc: Venkatesh Pallipadi <venkatesh.pallipadi@...el.com>
> Cc: Dave Jones <davej@...emonkey.org.uk>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> ---
> 
>  drivers/cpufreq/cpufreq.c |   11 +----------
>  1 file changed, 1 insertion(+), 10 deletions(-)
> 
> diff -puN drivers/cpufreq/cpufreq.c~cpufreq-revert-remove-rwsem-lock-from-cpufreq_gov_stop-call-second-call-site drivers/cpufreq/cpufreq.c
> --- a/drivers/cpufreq/cpufreq.c~cpufreq-revert-remove-rwsem-lock-from-cpufreq_gov_stop-call-second-call-site
> +++ a/drivers/cpufreq/cpufreq.c
> @@ -1758,17 +1758,8 @@ static int __cpufreq_set_policy(struct c
>  			dprintk("governor switch\n");
>  
>  			/* end old governor */
> -			if (data->governor) {
> -				/*
> -				 * Need to release the rwsem around governor
> -				 * stop due to lock dependency between
> -				 * cancel_delayed_work_sync and the read lock
> -				 * taken in the delayed work handler.
> -				 */
> -				unlock_policy_rwsem_write(data->cpu);
> +			if (data->governor)
>  				__cpufreq_governor(data, CPUFREQ_GOV_STOP);
> -				lock_policy_rwsem_write(data->cpu);
> -			}
>  
>  			/* start new governor */
>  			data->governor = policy->governor;
> _
> 
> Patches currently in -mm which might be from andrej.gelenberg@....edu are
> 
> origin.patch
> cpufreq-revert-remove-rwsem-lock-from-cpufreq_gov_stop-call-second-call-site.patch
> 

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com
--
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/