[v1] objtool/klp: Add the debian-based package name of xxhash to the hint

[PATCH] objtool/klp: Add the debian-based package name of xxhash to the hint

Posted by Borislav Petkov 3 months, 3 weeks ago

From: "Borislav Petkov (AMD)" <bp@alien8.de>

Add the package name for the devel version of the xxHash package
"libxxhash-dev" as used in debian-based repositories, which objtool
now uses.

Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
---
 tools/objtool/builtin-check.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/objtool/builtin-check.c b/tools/objtool/builtin-check.c
index 7b6fd60b022b..1e1ea8396eb3 100644
--- a/tools/objtool/builtin-check.c
+++ b/tools/objtool/builtin-check.c
@@ -164,7 +164,7 @@ static bool opts_valid(void)
 
 #ifndef BUILD_KLP
 	if (opts.checksum) {
-		ERROR("--checksum not supported; install xxhash-devel and recompile");
+		ERROR("--checksum not supported; install xxhash-devel/libxxhash-dev and recompile");
 		return false;
 	}
 #endif
-- 
2.51.0

Re: [PATCH] objtool/klp: Add the debian-based package name of xxhash to the hint

Posted by kernel test robot 3 months, 3 weeks ago

Hi Borislav,

kernel test robot noticed the following build errors:

[auto build test ERROR on next-20251017]
[cannot apply to linus/master v6.18-rc1 v6.17 v6.17-rc7 v6.18-rc1]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Borislav-Petkov/objtool-klp-Add-the-debian-based-package-name-of-xxhash-to-the-hint/20251018-091553
base:   next-20251017
patch link:    https://lore.kernel.org/r/20251017194732.7713-1-bp%40kernel.org
patch subject: [PATCH] objtool/klp: Add the debian-based package name of xxhash to the hint
:::::: branch date: 25 hours ago
:::::: commit date: 25 hours ago
config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20251019/202510190931.1SovbysA-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251019/202510190931.1SovbysA-lkp@intel.com/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 <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202510190931.1SovbysA-lkp@intel.com/

All errors (new ones prefixed by >>):

>> error: objtool: --checksum not supported; install xxhash-devel/libxxhash-dev and recompile
--
   drivers/message/fusion/mptsas.c: In function 'mptsas_issue_tm':
   drivers/message/fusion/mptsas.c:4791:26: warning: variable 'timeleft' set but not used [-Wunused-but-set-variable]
    4791 |         unsigned long    timeleft;
         |                          ^~~~~~~~
>> error: objtool: --checksum not supported; install xxhash-devel/libxxhash-dev and recompile
--
   mptsas.c: In function 'mptsas_issue_tm':
   mptsas.c:4791:26: warning: variable 'timeleft' set but not used [-Wunused-but-set-variable]
    4791 |         unsigned long    timeleft;
         |                          ^~~~~~~~
>> error: objtool: --checksum not supported; install xxhash-devel/libxxhash-dev and recompile

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

Re: [PATCH] objtool/klp: Add the debian-based package name of xxhash to the hint

Posted by Philip Li 3 months, 3 weeks ago

On Sun, Oct 19, 2025 at 01:07:22PM +0800, kernel test robot wrote:
> Hi Borislav,
> 
> kernel test robot noticed the following build errors:
> 
> [auto build test ERROR on next-20251017]
> [cannot apply to linus/master v6.18-rc1 v6.17 v6.17-rc7 v6.18-rc1]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch#_base_tree_information]
> 
> url:    https://github.com/intel-lab-lkp/linux/commits/Borislav-Petkov/objtool-klp-Add-the-debian-based-package-name-of-xxhash-to-the-hint/20251018-091553
> base:   next-20251017
> patch link:    https://lore.kernel.org/r/20251017194732.7713-1-bp%40kernel.org
> patch subject: [PATCH] objtool/klp: Add the debian-based package name of xxhash to the hint
> :::::: branch date: 25 hours ago
> :::::: commit date: 25 hours ago
> config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20251019/202510190931.1SovbysA-lkp@intel.com/config)
> compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251019/202510190931.1SovbysA-lkp@intel.com/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 <lkp@intel.com>
> | Closes: https://lore.kernel.org/r/202510190931.1SovbysA-lkp@intel.com/
> 
> All errors (new ones prefixed by >>):

Sorry, kindly ignore this report. which is a false positve.

> 
> >> error: objtool: --checksum not supported; install xxhash-devel/libxxhash-dev and recompile
> --
>    drivers/message/fusion/mptsas.c: In function 'mptsas_issue_tm':
>    drivers/message/fusion/mptsas.c:4791:26: warning: variable 'timeleft' set but not used [-Wunused-but-set-variable]
>     4791 |         unsigned long    timeleft;
>          |                          ^~~~~~~~
> >> error: objtool: --checksum not supported; install xxhash-devel/libxxhash-dev and recompile
> --
>    mptsas.c: In function 'mptsas_issue_tm':
>    mptsas.c:4791:26: warning: variable 'timeleft' set but not used [-Wunused-but-set-variable]
>     4791 |         unsigned long    timeleft;
>          |                          ^~~~~~~~
> >> error: objtool: --checksum not supported; install xxhash-devel/libxxhash-dev and recompile
> 
> -- 
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
> 
>

[tip: objtool/core] objtool/klp: Add the debian-based package name of xxhash to the hint

Posted by tip-bot2 for Borislav Petkov (AMD) 3 months, 2 weeks ago

The following commit has been merged into the objtool/core branch of tip:

Commit-ID:     da247eff96dd32380dfb0cb089be8671ac4bdcd0
Gitweb:        https://git.kernel.org/tip/da247eff96dd32380dfb0cb089be8671ac4bdcd0
Author:        Borislav Petkov (AMD) <bp@alien8.de>
AuthorDate:    Fri, 17 Oct 2025 21:47:32 +02:00
Committer:     Borislav Petkov (AMD) <bp@alien8.de>
CommitterDate: Wed, 22 Oct 2025 13:51:11 +02:00

objtool/klp: Add the debian-based package name of xxhash to the hint

Add the debian package name for the devel version of the xxHash package
"libxxhash-dev".

Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
Link: https://patch.msgid.link/20251017194732.7713-1-bp@kernel.org
---
 tools/objtool/builtin-check.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/objtool/builtin-check.c b/tools/objtool/builtin-check.c
index 7b6fd60..1e1ea83 100644
--- a/tools/objtool/builtin-check.c
+++ b/tools/objtool/builtin-check.c
@@ -164,7 +164,7 @@ static bool opts_valid(void)
 
 #ifndef BUILD_KLP
 	if (opts.checksum) {
-		ERROR("--checksum not supported; install xxhash-devel and recompile");
+		ERROR("--checksum not supported; install xxhash-devel/libxxhash-dev and recompile");
 		return false;
 	}
 #endif