Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758979Ab3CETS7 (ORCPT ); Tue, 5 Mar 2013 14:18:59 -0500 Received: from mail-ee0-f53.google.com ([74.125.83.53]:50518 "EHLO mail-ee0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752847Ab3CETS5 (ORCPT ); Tue, 5 Mar 2013 14:18:57 -0500 From: Maciej Rutecki Reply-To: maciej.rutecki@gmail.com To: Mandeep Singh Baines Subject: Re: [PATCH] thinkpad-acpi: fix potential suspend blocking issue Date: Tue, 5 Mar 2013 20:18:52 +0100 User-Agent: KMail/1.13.7 (Linux/3.9.0-rc1; KDE/4.8.4; x86_64; ; ) Cc: linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, ibm-acpi@hmh.eng.br, ibm-acpi-devel@lists.sourceforge.net, platform-driver-x86@vger.kernel.org, Aaron Lu , Henrique de Moraes Holschuh , Tejun Heo , Oleg Nesterov , Andrew Morton References: <201303042055.38040.maciej.rutecki@gmail.com> <1362504883-9180-1-git-send-email-msb@chromium.org> In-Reply-To: <1362504883-9180-1-git-send-email-msb@chromium.org> MIME-Version: 1.0 Content-Type: Text/Plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Message-Id: <201303052018.52320.maciej.rutecki@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1236 Lines: 33 On wtorek, 5 marca 2013 o 18:34:43 Mandeep Singh Baines wrote: > Fixes the following lockdep error: > > [ BUG: ktpacpi_nvramd/446 still has locks held! ] > > hotkey_kthread() calls set_freezable() after acquiring the > hotkey_kthread_mutex(). set_freezable() calls try_to_freeze(). > This could block suspend if we were to freeze at this point > and another task were to block on the mutex, potentially via > writing to one of the sysfs attrs. This race is unlikely but > can be easily fixed by moving the set_freezable() call. > > Reported-by: Maciej Rutecki > Signed-off-by: Mandeep Singh Baines > CC: Aaron Lu > CC: Henrique de Moraes Holschuh > CC: Tejun Heo > CC: Oleg Nesterov > CC: Andrew Morton Tested-by: Maciej Rutecki Patch fixes the problem. Thanks! Regards -- Maciej Rutecki http://www.mrutecki.pl -- 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/