[PATCH] efi: Explain OVMF acronym in OVMF_DEBUG_LOG help text

Geert Uytterhoeven posted 1 patch 1 month, 3 weeks ago
drivers/firmware/efi/Kconfig | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
[PATCH] efi: Explain OVMF acronym in OVMF_DEBUG_LOG help text
Posted by Geert Uytterhoeven 1 month, 3 weeks ago
People not very intimate with EFI may not know the meaning of the OVMF
acronym.  Write it in full, to help users with making good decisions
when configuring their kernels.

Fixes: f393a761763c5427 ("efi: add ovmf debug log driver")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/firmware/efi/Kconfig | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/firmware/efi/Kconfig b/drivers/firmware/efi/Kconfig
index d528c94c5859b5fa..29e0729299f5bd92 100644
--- a/drivers/firmware/efi/Kconfig
+++ b/drivers/firmware/efi/Kconfig
@@ -267,9 +267,10 @@ config OVMF_DEBUG_LOG
 	bool "Expose OVMF firmware debug log via sysfs"
 	depends on EFI
 	help
-	  Recent OVMF versions (edk2-stable202508 + newer) can write
-	  their debug log to a memory buffer.  This driver exposes the
-	  log content via sysfs (/sys/firmware/efi/ovmf_debug_log).
+	  Recent versions of the Open Virtual Machine Firmware
+	  (edk2-stable202508 + newer) can write their debug log to a memory
+	  buffer.  This driver exposes the log content via sysfs
+	  (/sys/firmware/efi/ovmf_debug_log).
 
 config UNACCEPTED_MEMORY
 	bool
-- 
2.43.0
Re: [PATCH] efi: Explain OVMF acronym in OVMF_DEBUG_LOG help text
Posted by Gerd Hoffmann 1 month, 3 weeks ago
On Tue, Aug 12, 2025 at 03:54:29PM +0200, Geert Uytterhoeven wrote:
> People not very intimate with EFI may not know the meaning of the OVMF
> acronym.  Write it in full, to help users with making good decisions
> when configuring their kernels.
> 
> Fixes: f393a761763c5427 ("efi: add ovmf debug log driver")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

> +	  Recent versions of the Open Virtual Machine Firmware
> +	  (edk2-stable202508 + newer) can write their debug log to a memory
> +	  buffer.  This driver exposes the log content via sysfs
> +	  (/sys/firmware/efi/ovmf_debug_log).

Acked-by: Gerd Hoffmann <kraxel@redhat.com>
Re: [PATCH] efi: Explain OVMF acronym in OVMF_DEBUG_LOG help text
Posted by Richard Lyu 1 month, 2 weeks ago
On 2025/08/13 10:03, Gerd Hoffmann wrote:
> On Tue, Aug 12, 2025 at 03:54:29PM +0200, Geert Uytterhoeven wrote:
> > People not very intimate with EFI may not know the meaning of the OVMF
> > acronym.  Write it in full, to help users with making good decisions
> > when configuring their kernels.
> > 
> > Fixes: f393a761763c5427 ("efi: add ovmf debug log driver")
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> > +	  Recent versions of the Open Virtual Machine Firmware
> > +	  (edk2-stable202508 + newer) can write their debug log to a memory
> > +	  buffer.  This driver exposes the log content via sysfs
> > +	  (/sys/firmware/efi/ovmf_debug_log).
> 
> Acked-by: Gerd Hoffmann <kraxel@redhat.com>

Reviewed-by: Richard Lyu <richard.lyu@suse.com>
Re: [PATCH] efi: Explain OVMF acronym in OVMF_DEBUG_LOG help text
Posted by Ard Biesheuvel 1 month ago
On Mon, 18 Aug 2025 at 08:52, Richard Lyu <richard.lyu@suse.com> wrote:
>
> On 2025/08/13 10:03, Gerd Hoffmann wrote:
> > On Tue, Aug 12, 2025 at 03:54:29PM +0200, Geert Uytterhoeven wrote:
> > > People not very intimate with EFI may not know the meaning of the OVMF
> > > acronym.  Write it in full, to help users with making good decisions
> > > when configuring their kernels.
> > >
> > > Fixes: f393a761763c5427 ("efi: add ovmf debug log driver")
> > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> > > +     Recent versions of the Open Virtual Machine Firmware
> > > +     (edk2-stable202508 + newer) can write their debug log to a memory
> > > +     buffer.  This driver exposes the log content via sysfs
> > > +     (/sys/firmware/efi/ovmf_debug_log).
> >
> > Acked-by: Gerd Hoffmann <kraxel@redhat.com>
>
> Reviewed-by: Richard Lyu <richard.lyu@suse.com>
>

Thanks, I've queued this up now.