[PATCH 1/2 v2] efi: add a function to convert the status value to string
joeyli
jlee at suse.com
Fri May 3 06:15:18 UTC 2019
On Thu, May 02, 2019 at 10:53:31AM +0200, Ard Biesheuvel wrote:
> On Thu, 2 May 2019 at 06:04, Lee, Chun-Yi <joeyli.kernel at gmail.com> 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.
Thanks a lot!
Joey Lee
More information about the Linux-security-module-archive
mailing list