Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753146Ab2KHEbP (ORCPT ); Wed, 7 Nov 2012 23:31:15 -0500 Received: from mail-qa0-f46.google.com ([209.85.216.46]:40800 "EHLO mail-qa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751461Ab2KHEbN (ORCPT ); Wed, 7 Nov 2012 23:31:13 -0500 MIME-Version: 1.0 In-Reply-To: <509B0B7F.3030308@cn.fujitsu.com> References: <509B0B7F.3030308@cn.fujitsu.com> Date: Thu, 8 Nov 2012 12:31:12 +0800 Message-ID: Subject: Re: [PATCH -next] ACPI: fix missing unlock on error in acpi_memory_remove_memory() From: Wei Yongjun To: wency@cn.fujitsu.com Cc: lenb@kernel.org, rjw@sisk.pl, yongjun_wei@trendmicro.com.cn, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, dan.carpenter@oracle.com Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1248 Lines: 37 Hi Wen, On 11/08/2012 09:31 AM, Wen Congyang wrote: > Hi, wei > > At 11/07/2012 08:38 PM, Wei Yongjun Wrote: >> From: Wei Yongjun >> >> Add the missing mutex_unlock() before return from function >> acpi_memory_remove_memory() in the error handling case. >> >> Introduce by commit 85fcb3758c10e063a2a30dfad75017097999deed >> 'ACPI / memory-hotplug: introduce a mutex lock to protect the list in acpi_memory_device' >> >> dpatch engine is used to auto generate this patch. >> (https://github.com/weiyj/dpatch) >> >> Signed-off-by: Wei Yongjun > Thanks for your patch. > > Dan Carpenter has told me this problem. I fix it, but don't > resend it because it is in test now. I saw your patchset has already applied to linux-pm.git tree: http://git.kernel.org/?p=linux/kernel/git/rafael/linux-pm.git;a=shortlog;h=refs/heads/linux-next So I think what need is to send a new patch to fix it, not resend the full patchset. Regards, Yongjun Wei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/