From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A1FC7C004C9 for ; Fri, 3 May 2019 08:01:56 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 7196420675 for ; Fri, 3 May 2019 08:01:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726483AbfECIBz (ORCPT ); Fri, 3 May 2019 04:01:55 -0400 Received: from smtp.nue.novell.com ([195.135.221.5]:41580 "EHLO smtp.nue.novell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725775AbfECIBz (ORCPT ); Fri, 3 May 2019 04:01:55 -0400 Received: from emea4-mta.ukb.novell.com ([10.120.13.87]) by smtp.nue.novell.com with ESMTP (TLS encrypted); Fri, 03 May 2019 10:01:53 +0200 Received: from linux-l9pv.suse (nwb-a10-snat.microfocus.com [10.120.13.201]) by emea4-mta.ukb.novell.com with ESMTP (TLS encrypted); Fri, 03 May 2019 09:01:22 +0100 Date: Fri, 3 May 2019 16:01:16 +0800 From: joeyli To: Ard Biesheuvel Cc: "Lee, Chun-Yi" , James Morris , "Serge E . Hallyn" , David Howells , Josh Boyer , Nayna Jain , Mimi Zohar , linux-efi , linux-security-module , Linux Kernel Mailing List , Kees Cook , Anton Vorontsov , Colin Cross , Tony Luck Subject: Re: [PATCH 1/2 v2] efi: add a function to convert the status value to string Message-ID: <20190503080116.GO11486@linux-l9pv.suse> References: <20190502040441.30372-1-jlee@suse.com> <20190503061518.GM11486@linux-l9pv.suse> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 03, 2019 at 08:16:42AM +0200, Ard Biesheuvel wrote: > On Fri, 3 May 2019 at 08:15, joeyli wrote: > > > > On Thu, May 02, 2019 at 10:53:31AM +0200, Ard Biesheuvel wrote: > > > On Thu, 2 May 2019 at 06:04, Lee, Chun-Yi wrote: > > > > > > > > This function can be used to convert EFI status value to string > > > > for printing out debug message. Using this function can improve > > > > the readability of log. > > > > > > > > v2. > > > > > > Please move the changelog out of the commit log (move it below the --- > > > further down) > > > > > > > OK! I will moved the changelog out of the commit log. > > > > > > - Changed the wording in subject and description. > > > > - Moved the marco immediately after the status value definitions. > > > > - Turned into a proper function instead of inline. > > > > > > > > > > You missed my point here. A proper function means the function in a .c > > > file, and only the declaration in a .h file. This way, you are still > > > duplicating the literal strings into every object file that references > > > this function. > > > > > > > Sorry for I missunderstood. I will move the function to load_uefi.c. > > > > No, please move it to a file that is shared between all EFI code. I see! I will move the function to the position that it is just behind efi_status_to_str() in drivers/firmware/efi/efi.c. Please let me know if the position is not suitable. Thanks a lot! Joey Lee