Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755763Ab3EIBZj (ORCPT ); Wed, 8 May 2013 21:25:39 -0400 Received: from youngberry.canonical.com ([91.189.89.112]:55302 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755623Ab3EIBZi (ORCPT ); Wed, 8 May 2013 21:25:38 -0400 MIME-Version: 1.0 In-Reply-To: References: <1367996197-32748-1-git-send-email-tiwai@suse.de> <1367996197-32748-2-git-send-email-tiwai@suse.de> Date: Thu, 9 May 2013 09:25:35 +0800 Message-ID: Subject: Re: [PATCH 1/3] firmware: Avoid superfluous usermodehelper lock From: Ming Lei To: Takashi Iwai Cc: linux-kernel@vger.kernel.org, Greg Kroah-Hartman 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: 797 Lines: 22 On Thu, May 9, 2013 at 1:51 AM, Takashi Iwai wrote: >> In other words, the first patch is no essential part of the fix. >> I can revisit the second patch without this one and resend if >> preferred. > > FWIW, below is the revised patch. > It's alone without the patch 1 in the previous series. The root cause is that your user space loader doesn't follow the current firmware loader interface. IMO, the patch is unnecessary since we already have the timeout abort(just need one patch to enable it for nowait api) Thanks, -- Ming Lei -- 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/