Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751711AbdFGRPM (ORCPT ); Wed, 7 Jun 2017 13:15:12 -0400 Received: from mx2.suse.de ([195.135.220.15]:44649 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751393AbdFGRPK (ORCPT ); Wed, 7 Jun 2017 13:15:10 -0400 Date: Wed, 7 Jun 2017 19:15:07 +0200 From: "Luis R. Rodriguez" To: Alan Cox Cc: Dmitry Torokhov , Andy Lutomirski , "Luis R. Rodriguez" , "Theodore Ts'o" , Linux FS Devel , Stephen Boyd , "Li, Yi" , Peter Zijlstra , Jonathan Corbet , "Eric W. Biederman" , "Michael Kerrisk (man-pages)" , Greg KH , "Fuzzey, Martin" , Linux API , Daniel Wagner , David Woodhouse , jewalt@lgsinnovations.com, rafal@milecki.pl, Arend Van Spriel , "Rafael J. Wysocki" , Moritz Fischer , Petr Mladek , Johannes Berg , Emmanuel Grumbach , Luca Coelho , Kalle Valo , Linus Torvalds , Kees Cook , AKASHI Takahiro , David Howells , Peter Jones , Hans de G oede , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v2] firmware: fix sending -ERESTARTSYS due to signal on fallback Message-ID: <20170607171507.GL27288@wotan.suse.de> References: <20170526215518.GB40877@dtor-ws> <20170605202410.GQ8951@wotan.suse.de> <1496760796.5682.48.camel@linux.intel.com> <20170606164734.GB27288@wotan.suse.de> <20170606221151.ygoxqkwhhjsqw632@thunk.org> <20170607002237.GJ27288@wotan.suse.de> <20170607062515.GA23434@dtor-ws> <1496838351.5682.58.camel@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1496838351.5682.58.camel@linux.intel.com> User-Agent: Mutt/1.6.0 (2016-04-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 997 Lines: 20 On Wed, Jun 07, 2017 at 01:25:51PM +0100, Alan Cox wrote: > > What's wrong with saying that the only way to interrupt firmware > > loading is to kill the process? So ctrl-c will no longer interrupt > > it, but I do not think that ease of aborting firmware update is > > primary goal here. I consider simple is good here. > > Agreed 100%. The user process did not ask for firmware load, it asked > for an I/O operation. Semantically it should appear as if someone else > did the firmware load and it just had to wait for it to happen. Fine by me ! Will wrap up the patch for the new killable swait then. I suppose noting it as a stable fix is worth it given the known issues with for example Android killing loaders unexpectedly. Unless I hear otherwise I'll also provide a follow up to return -EINTR instead of -EAGAIN if swait returned -ERESTARTSYS, this way at least userspace could tell a signal was definitely received. I *don't* think that follow up is required for stable though. Luis