git: e6dbc4dfd144 - stable/13 - efirt: Move comment about fpu_kern_enter to where it is called

From: John Baldwin <jhb_at_FreeBSD.org>
Date: Wed, 11 Oct 2023 19:06:26 UTC
The branch stable/13 has been updated by jhb:

URL: https://cgit.FreeBSD.org/src/commit/?id=e6dbc4dfd1444a38647fb55e9238112291b9e4cd

commit e6dbc4dfd1444a38647fb55e9238112291b9e4cd
Author:     John Baldwin <jhb@FreeBSD.org>
AuthorDate: 2023-08-25 19:33:00 +0000
Commit:     John Baldwin <jhb@FreeBSD.org>
CommitDate: 2023-10-11 15:09:52 +0000

    efirt: Move comment about fpu_kern_enter to where it is called
    
    Reviewed by:    imp, kib, andrew, markj
    Differential Revision:  https://reviews.freebsd.org/D41576
    
    (cherry picked from commit 8173fa60ddb7e9a805dec9fef7bf07e74ae4144d)
---
 sys/amd64/amd64/efirt_machdep.c | 5 -----
 sys/dev/efidev/efirt.c          | 5 +++++
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/sys/amd64/amd64/efirt_machdep.c b/sys/amd64/amd64/efirt_machdep.c
index 4c6ee4ee88a2..5734e0300df8 100644
--- a/sys/amd64/amd64/efirt_machdep.c
+++ b/sys/amd64/amd64/efirt_machdep.c
@@ -280,11 +280,6 @@ fail:
  * reason to bother with the virtual map, and no need to add a
  * complexity into loader.
  *
- * The fpu_kern_enter() call allows firmware to use FPU, as mandated
- * by the specification.  In particular, CR0.TS bit is cleared.  Also
- * it enters critical section, giving us neccessary protection against
- * context switch.
- *
  * There is no need to disable interrupts around the change of %cr3,
  * the kernel mappings are correct, while we only grabbed the
  * userspace portion of VA.  Interrupts handlers must not access
diff --git a/sys/dev/efidev/efirt.c b/sys/dev/efidev/efirt.c
index 851385eeb8d6..29f51b78eb25 100644
--- a/sys/dev/efidev/efirt.c
+++ b/sys/dev/efidev/efirt.c
@@ -269,6 +269,11 @@ efi_rt_ok(void)
 	return (0);
 }
 
+/*
+ * The fpu_kern_enter() call in allows firmware to use FPU, as
+ * mandated by the specification.  It also enters a critical section,
+ * giving us neccessary protection against context switches.
+ */
 static int
 efi_enter(void)
 {