2024-02-03 06:06:52

by kernel test robot

[permalink] [raw]
Subject: [tip:x86/sev 16/29] drivers/crypto/ccp/sev-dev.c:93:28: sparse: sparse: symbol 'snp_range_list' was not declared. Should it be static?

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/sev
head: 1bfca8d2800ab5ef0dfed335a2a29d1632c99411
commit: 1ca5614b84eed5904f65f143e0e7aaab0ac4c6b2 [16/29] crypto: ccp: Add support to initialize the AMD-SP for SEV-SNP
config: x86_64-randconfig-r131-20240203 (https://download.01.org/0day-ci/archive/20240203/[email protected]/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240203/[email protected]/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <[email protected]>
| Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

sparse warnings: (new ones prefixed by >>)
>> drivers/crypto/ccp/sev-dev.c:93:28: sparse: sparse: symbol 'snp_range_list' was not declared. Should it be static?

vim +/snp_range_list +93 drivers/crypto/ccp/sev-dev.c

87
88 /*
89 * SEV_DATA_RANGE_LIST:
90 * Array containing range of pages that firmware transitions to HV-fixed
91 * page state.
92 */
> 93 struct sev_data_range_list *snp_range_list;
94

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki


Subject: [tip: x86/sev] crypto: ccp: Make snp_range_list static

The following commit has been merged into the x86/sev branch of tip:

Commit-ID: f9e6f00d93d34f60f90b42c24e80194b11a72bb2
Gitweb: https://git.kernel.org/tip/f9e6f00d93d34f60f90b42c24e80194b11a72bb2
Author: Borislav Petkov (AMD) <[email protected]>
AuthorDate: Sat, 03 Feb 2024 11:40:48 +01:00
Committer: Borislav Petkov (AMD) <[email protected]>
CommitterDate: Sat, 03 Feb 2024 11:41:41 +01:00

crypto: ccp: Make snp_range_list static

Fix:

drivers/crypto/ccp/sev-dev.c:93:28: sparse: sparse: symbol 'snp_range_list' was not declared. Should it be static?

Reported-by: kernel test robot <[email protected]>
Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]
Signed-off-by: Borislav Petkov (AMD) <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
---
drivers/crypto/ccp/sev-dev.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/crypto/ccp/sev-dev.c b/drivers/crypto/ccp/sev-dev.c
index f1a5795..182427f 100644
--- a/drivers/crypto/ccp/sev-dev.c
+++ b/drivers/crypto/ccp/sev-dev.c
@@ -107,7 +107,7 @@ static void *sev_init_ex_buffer;
* Array containing range of pages that firmware transitions to HV-fixed
* page state.
*/
-struct sev_data_range_list *snp_range_list;
+static struct sev_data_range_list *snp_range_list;

static inline bool sev_version_greater_or_equal(u8 maj, u8 min)
{