From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <512FD486.6030804@suse.cz> Date: Thu, 28 Feb 2013 23:04:54 +0100 From: Jiri Slaby MIME-Version: 1.0 To: Greg Kroah-Hartman , linux-kernel@vger.kernel.org CC: stable@vger.kernel.org, Jiang Liu , Gaohuai Han , Maciej Sosnowski , Dan Williams Subject: Re: [ 47/53] dca: check against empty dca_domains list before unregister provider References: <20130226235619.844721947@linuxfoundation.org> <20130226235624.751198592@linuxfoundation.org> In-Reply-To: <20130226235624.751198592@linuxfoundation.org> Content-Type: text/plain; charset=ISO-8859-2 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: On 02/27/2013 12:58 AM, Greg Kroah-Hartman wrote: > 3.0-stable review patch. If anyone has any objections, please let me know. > > ------------------ > > From: Maciej Sosnowski > > commit c419fcfd071cf34ba00f9f65282583772d2655e7 upstream. > > When providers get blocked unregister_dca_providers() is called ending up > with dca_providers and dca_domain lists emptied. Dca should be prevented from > trying to unregister any provider if dca_domain list is found empty. > > Reported-by: Jiang Liu > Tested-by: Gaohuai Han > Signed-off-by: Maciej Sosnowski > Signed-off-by: Dan Williams > Signed-off-by: Greg Kroah-Hartman > > --- > drivers/dca/dca-core.c | 5 +++++ > 1 file changed, 5 insertions(+) > > --- a/drivers/dca/dca-core.c > +++ b/drivers/dca/dca-core.c > @@ -409,6 +409,11 @@ void unregister_dca_provider(struct dca_ > > spin_lock_irqsave(&dca_lock, flags); > > + if (list_empty(&dca_domains)) { > + raw_spin_unlock_irqrestore(&dca_lock, flags); Hi, this is an incorrect backport. It should be spin_unlock_irqrestore obviously... -- js suse labs