[02/50] x86/kernel/fpu/bugs.c: fix missing include

Message ID 20231216024834.3510073-3-kent.overstreet@linux.dev
State New
Headers
Series big header dependency cleanup targeting sched.h |

Commit Message

Kent Overstreet Dec. 16, 2023, 2:47 a.m. UTC
  Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
---
 arch/x86/kernel/fpu/bugs.c | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Sohil Mehta Dec. 18, 2023, 11:08 a.m. UTC | #1
For the x86 patches [2-5/50], should the patch subject be a bit more
generic rather than having the full file name listed.

For example, this patch could be "x86/fpu: Fix missing include".

On 12/16/2023 8:17 AM, Kent Overstreet wrote:
> Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
> ---
>  arch/x86/kernel/fpu/bugs.c | 1 +
>  1 file changed, 1 insertion(+)
>
  
Kent Overstreet Dec. 19, 2023, 2:05 a.m. UTC | #2
On Mon, Dec 18, 2023 at 04:38:33PM +0530, Sohil Mehta wrote:
> For the x86 patches [2-5/50], should the patch subject be a bit more
> generic rather than having the full file name listed.
> 
> For example, this patch could be "x86/fpu: Fix missing include".

In the latest version of the patchset I've just rolled all of the
arch/x86 missing includes into one patch.
  

Patch

diff --git a/arch/x86/kernel/fpu/bugs.c b/arch/x86/kernel/fpu/bugs.c
index 794e70151203..a06b876bbf2d 100644
--- a/arch/x86/kernel/fpu/bugs.c
+++ b/arch/x86/kernel/fpu/bugs.c
@@ -2,6 +2,7 @@ 
 /*
  * x86 FPU bug checks:
  */
+#include <asm/cpufeature.h>
 #include <asm/fpu/api.h>
 
 /*