x86/fpu: Allow multiple bits in clearcpuid= parameter
authorArvind Sankar <nivedita@alum.mit.edu>
Mon, 7 Sep 2020 21:39:19 +0000 (17:39 -0400)
committerBorislav Petkov <bp@suse.de>
Thu, 10 Sep 2020 16:32:05 +0000 (18:32 +0200)
commit0a4bb5e5507a585532cc413125b921c8546fc39f
tree3784e5e88c7033f16b2208549a738ebf4a39f8cd
parentf4d51dffc6c01a9e94650d95ce0104964f8ae822
x86/fpu: Allow multiple bits in clearcpuid= parameter

Commit

  0c2a3913d6f5 ("x86/fpu: Parse clearcpuid= as early XSAVE argument")

changed clearcpuid parsing from __setup() to cmdline_find_option().
While the __setup() function would have been called for each clearcpuid=
parameter on the command line, cmdline_find_option() will only return
the last one, so the change effectively made it impossible to disable
more than one bit.

Allow a comma-separated list of bit numbers as the argument for
clearcpuid to allow multiple bits to be disabled again. Log the bits
being disabled for informational purposes.

Also fix the check on the return value of cmdline_find_option(). It
returns -1 when the option is not found, so testing as a boolean is
incorrect.

Fixes: 0c2a3913d6f5 ("x86/fpu: Parse clearcpuid= as early XSAVE argument")
Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lkml.kernel.org/r/20200907213919.2423441-1-nivedita@alum.mit.edu
Documentation/admin-guide/kernel-parameters.txt
arch/x86/kernel/fpu/init.c