From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Fleming Subject: Re: [PATCH] efivarfs: Never return ENOENT from firmware again Date: Mon, 13 May 2013 20:21:47 +0100 Message-ID: <51913D4B.2000606@intel.com> References: <5b6476bf31dc699a0e6f3983e78447eeac34d831.1368180585.git.lxiang@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5b6476bf31dc699a0e6f3983e78447eeac34d831.1368180585.git.lxiang-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Lingzhu Xiang Cc: Jeremy Kerr , Matthew Garrett , linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Josh Boyer , "Lee, Chun-Yi" , Andy Whitcroft List-Id: linux-efi@vger.kernel.org On 05/10/2013 11:29 AM, Lingzhu Xiang wrote: > Previously in 1fa7e69 efi_status_to_err() translated firmware status > EFI_NOT_FOUND to -EIO instead of -ENOENT for efivarfs operations to > avoid confusion. After refactoring in e14ab23, it is also used in other > places where the translation may be unnecessary. > > So move the translation to efivarfs specific code. Also return EOF > for reading zero-length files, which is what users would expect. > > Cc: Josh Boyer > Cc: Jeremy Kerr > Cc: Lee, Chun-Yi > Cc: Andy Whitcroft > Signed-off-by: Lingzhu Xiang > --- > This is to be applied against mainline or Matt Fleming's chainsaw branch. Applied, thanks. FYI, I rebased my 'urgent' branch on v3.10-rc1, so your patch now applies there too.