diff for duplicates of <20161209003527.GA9062@wunner.de> diff --git a/a/1.txt b/N1/1.txt index 43a849d..ad65208 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,5 +1,5 @@ On Thu, Dec 08, 2016 at 05:31:13PM +0000, David Howells wrote: -> Lukas Wunner <lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org> wrote: +> Lukas Wunner <lukas@wunner.de> wrote: > > > +out_efi_err: > > > + pr_efi_err(sys_table_arg, "Could not determine UEFI Secure Boot status.\n"); > > > + if (status == EFI_NOT_FOUND) @@ -24,7 +24,7 @@ If you feel the change is justified, fine, I won't argue against it since I don't have a dog in this fight. But obviously it's something that a reader of your patch will trip over, -so at least explain it in the commit message. It would also be good to +so@least explain it in the commit message. It would also be good to explain why you're moving the pr_efi_err() calls in the first place. ISTR it has to do with the different interpretation of an error, what I wrote in my previous e-mail: x86 defaults to considering secureboot diff --git a/a/content_digest b/N1/content_digest index de55f02..edbc8a1 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -4,22 +4,14 @@ "ref\0148111671977.23390.12452925207541146423.stgit@warthog.procyon.org.uk\0" "ref\06009.1481184981@warthog.procyon.org.uk\0" "ref\014655.1481218273@warthog.procyon.org.uk\0" - "ref\014655.1481218273-S6HVgzuS8uM4Awkfq6JHfwNdhmdF6hFW@public.gmane.org\0" - "From\0Lukas Wunner <lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>\0" - "Subject\0Re: [PATCH 5/8] efi: Get the secure boot status [ver #5]\0" + "From\0lukas@wunner.de (Lukas Wunner)\0" + "Subject\0[PATCH 5/8] efi: Get the secure boot status [ver #5]\0" "Date\0Fri, 9 Dec 2016 01:35:27 +0100\0" - "To\0David Howells <dhowells-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>\0" - "Cc\0matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org" - ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org - linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - linux-security-module-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - keyrings-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - " linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Thu, Dec 08, 2016 at 05:31:13PM +0000, David Howells wrote:\n" - "> Lukas Wunner <lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org> wrote:\n" + "> Lukas Wunner <lukas@wunner.de> wrote:\n" "> > > +out_efi_err:\n" "> > > +\tpr_efi_err(sys_table_arg, \"Could not determine UEFI Secure Boot status.\\n\");\n" "> > > +\tif (status == EFI_NOT_FOUND)\n" @@ -44,7 +36,7 @@ "since I don't have a dog in this fight.\n" "\n" "But obviously it's something that a reader of your patch will trip over,\n" - "so at least explain it in the commit message. It would also be good to\n" + "so@least explain it in the commit message. It would also be good to\n" "explain why you're moving the pr_efi_err() calls in the first place.\n" "ISTR it has to do with the different interpretation of an error,\n" "what I wrote in my previous e-mail: x86 defaults to considering secureboot\n" @@ -56,4 +48,4 @@ "\n" Lukas -2ef320d5db32c08cd402dd4252908bc357ac1d9199b048fb21d43dc39bdc03fd +9151d3464487a33bc38518ecb5c40b0ca3ff489c081d48b297e819eca3b1ddae
diff --git a/a/1.txt b/N2/1.txt index 43a849d..248a8de 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,5 +1,5 @@ On Thu, Dec 08, 2016 at 05:31:13PM +0000, David Howells wrote: -> Lukas Wunner <lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org> wrote: +> Lukas Wunner <lukas@wunner.de> wrote: > > > +out_efi_err: > > > + pr_efi_err(sys_table_arg, "Could not determine UEFI Secure Boot status.\n"); > > > + if (status == EFI_NOT_FOUND) diff --git a/a/content_digest b/N2/content_digest index de55f02..9bdead1 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -4,22 +4,21 @@ "ref\0148111671977.23390.12452925207541146423.stgit@warthog.procyon.org.uk\0" "ref\06009.1481184981@warthog.procyon.org.uk\0" "ref\014655.1481218273@warthog.procyon.org.uk\0" - "ref\014655.1481218273-S6HVgzuS8uM4Awkfq6JHfwNdhmdF6hFW@public.gmane.org\0" - "From\0Lukas Wunner <lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>\0" + "From\0Lukas Wunner <lukas@wunner.de>\0" "Subject\0Re: [PATCH 5/8] efi: Get the secure boot status [ver #5]\0" "Date\0Fri, 9 Dec 2016 01:35:27 +0100\0" - "To\0David Howells <dhowells-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>\0" - "Cc\0matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org" - ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org - linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - linux-security-module-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - keyrings-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - " linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org\0" + "To\0David Howells <dhowells@redhat.com>\0" + "Cc\0matt@codeblueprint.co.uk" + ard.biesheuvel@linaro.org + linux-efi@vger.kernel.org + linux-kernel@vger.kernel.org + linux-security-module@vger.kernel.org + keyrings@vger.kernel.org + " linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Thu, Dec 08, 2016 at 05:31:13PM +0000, David Howells wrote:\n" - "> Lukas Wunner <lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org> wrote:\n" + "> Lukas Wunner <lukas@wunner.de> wrote:\n" "> > > +out_efi_err:\n" "> > > +\tpr_efi_err(sys_table_arg, \"Could not determine UEFI Secure Boot status.\\n\");\n" "> > > +\tif (status == EFI_NOT_FOUND)\n" @@ -56,4 +55,4 @@ "\n" Lukas -2ef320d5db32c08cd402dd4252908bc357ac1d9199b048fb21d43dc39bdc03fd +dd96ea023f4d15f097615f661d0ed3814c2790ef244ca4008975626233b7b62b
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.