From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 86433D2E9D3 for ; Mon, 11 Nov 2024 10:26:16 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id D12886B0085; Mon, 11 Nov 2024 05:26:15 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id CC3F16B0088; Mon, 11 Nov 2024 05:26:15 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id B8A546B0089; Mon, 11 Nov 2024 05:26:15 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 95FC36B0085 for ; Mon, 11 Nov 2024 05:26:15 -0500 (EST) Received: from smtpin09.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 2B1CC1218AC for ; Mon, 11 Nov 2024 10:26:15 +0000 (UTC) X-FDA: 82773432084.09.93E5A38 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.21]) by imf19.hostedemail.com (Postfix) with ESMTP id AAE601A0014 for ; Mon, 11 Nov 2024 10:25:21 +0000 (UTC) Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b="NJSH/J/D"; dmarc=pass (policy=none) header.from=intel.com; spf=pass (imf19.hostedemail.com: domain of lkp@intel.com designates 198.175.65.21 as permitted sender) smtp.mailfrom=lkp@intel.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1731320711; a=rsa-sha256; cv=none; b=BviRnEzK7kumnlJ7FcM0eA4HDlPuz2hPQEKyl9aH2fChhy8ysA+jhylwaP6RJOwMi5lVnz i0/5m3RxlI4SQ4k84CZJahmwVvdOFZky834a6GFhSz6qz39GZ+OE0ni9WVNsOh8P7kmwM9 XE7+gyMvPJsrbDK/h4aJ+c+EJRXNtA4= ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b="NJSH/J/D"; dmarc=pass (policy=none) header.from=intel.com; spf=pass (imf19.hostedemail.com: domain of lkp@intel.com designates 198.175.65.21 as permitted sender) smtp.mailfrom=lkp@intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1731320711; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=HVavvc8Aq4I63HvP8KcntJyDk9RJoypUZgTVUNXt1xk=; b=YGr0/YNcc/FggNSNi+gChSGtZejk+K27SuN7u/TCAeud4kvhf89FDHQjn/ixxJfaVV/tEV Yr0QbXVUHH7Wbu0LXWxmAff8PbFhYmgLne/H8NHGAFyhp1Y9z++nwvAyCZPKMyW2ItbJju sgYLSHuE8Nxev8ZoXeJOQaav5+tjWM8= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1731320772; x=1762856772; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=APD2TjIxAvb8PHQKlfqdSkDm3jLyE2wTGiDrAbkSdZs=; b=NJSH/J/DnhUY+IQEO7CNGsS0+g9IccFmyK3+sIzjN2UySA3QpLAkFmyg 2QMKjB1ZwceX5S5vC9kI3RfUoAGjEMCywv3obbBrp7TGVFXEe4b3Gv0DP i+eNy06X4InRSTSIfPcINXlDLKVyV96OA/TTw9W7vkeFeDdu9d8L6GWn2 JmbEXQ4HoHYrsrURVTFZCZTElzv5zZJxhkVhQ7boQ9xhY3TGIPDr1JgxP dlDPOuQ2HIHYABttaCE+ggE8CzJ0/FMCu6bCiYXx0AV+FSZvnI56hA3QU TpbHlTWpmK8C58UA2dUMUzjlTdGitSiPKjfbZcqnSuwFZbzy6G1h7WbPa w==; X-CSE-ConnectionGUID: fg1nI5JoRcq0vEN2arxyWw== X-CSE-MsgGUID: ZykdMc9/SYG/4qkynqEyJw== X-IronPort-AV: E=McAfee;i="6700,10204,11222"; a="31080892" X-IronPort-AV: E=Sophos;i="6.11,199,1725346800"; d="scan'208";a="31080892" Received: from fmviesa005.fm.intel.com ([10.60.135.145]) by orvoesa113.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Nov 2024 02:26:10 -0800 X-CSE-ConnectionGUID: r9yjkMffQou478zgqlPq1w== X-CSE-MsgGUID: DDTqgKjKTZK25QJi2u1k/g== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.12,144,1728975600"; d="scan'208";a="91369316" Received: from lkp-server01.sh.intel.com (HELO dc8184e5aea1) ([10.239.97.150]) by fmviesa005.fm.intel.com with ESMTP; 11 Nov 2024 02:26:07 -0800 Received: from kbuild by dc8184e5aea1 with local (Exim 4.96) (envelope-from ) id 1tARcS-0000An-0A; Mon, 11 Nov 2024 10:26:04 +0000 Date: Mon, 11 Nov 2024 18:25:59 +0800 From: kernel test robot To: Jacob Keller , Vladimir Oltean , Andrew Morton , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Tony Nguyen , Przemek Kitszel , Masahiro Yamada , netdev Cc: oe-kbuild-all@lists.linux.dev, Linux Memory Management List , linux-kbuild@vger.kernel.org, Jacob Keller Subject: Re: [PATCH net-next v5 3/9] lib: packing: add pack_fields() and unpack_fields() Message-ID: <202411111823.Y2NHM1AE-lkp@intel.com> References: <20241111-packing-pack-fields-and-ice-implementation-v5-3-80c07349e6b7@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241111-packing-pack-fields-and-ice-implementation-v5-3-80c07349e6b7@intel.com> X-Rspam-User: X-Rspamd-Queue-Id: AAE601A0014 X-Rspamd-Server: rspam01 X-Stat-Signature: 3o5hc6y6ehcw11ywyjkkq1nani5kicp1 X-HE-Tag: 1731320721-34101 X-HE-Meta: U2FsdGVkX1+RJwdz2v5QYJRBk8wDv8kWTl0a9dG2mgbrCFaMZjNALZagBHLFJZ0WDufoQ5iW92Ibrk5I7KiajJ82S4w0XI2QBEo4PT+NCofUQwVxI7FOQIEe4Ia2DxGuOAceEg9nUPRXaB1zVV1gfNlFgzt0R/wTDBqlG7xol1HupPpmfQ4Bj5QnSXUMvlNvAuizNyf8kPNT69rjk1qUnv7ytQh+oQ2fgNTMC86amVxMxap5SgdMTdoNqErS0His2Oa0VEpTPq1vNd9RcGqhVNNiOUdYwk3Z7w91OgaX3MrAM/JtSrrqo6Qolj39GzybhhW2UxoDNqh3KL3ONGyDCZeTCdi6PTARpskLsL1tmmt8tG/PEU2mlhl7EftXkOtfQHNI0wqrdP/PBV5bi3IIXJo3taydZ0tiQgFLzX4ic33KFvPPiZ4L7sinfxlWtZ8uBK/4pWFu5aNdMlUD2ukKIZzs3kFXKMCmOHm4dQOo96rkqw4Oaso/J5NuGKbe+bhZzgYk1fgjogQ+zaShjZz1TU7oMCg9spQmnVIDsHT4+0UuhObo7gKMXtls/atW/P4Q2bkeuEYAks0o9pgqfAqBrhyCkyC6BMLMSBIvQs3wm8MuERvRP09o6hEAevHDP6t3T05XubzTO0I8cDRA/FvlaHC76MvtuhhPBL8fgvAXqjOWIpIcOqRoWeZC1s0rEWROwtqj9APJdbA1utwZI+kB8podmTO9Uz4/Ml+0g4p5h8Gx87qMP88Rlch6HgGaZHU/QALD9vRTNuVsqf3a9mtjQFL8OcaaBvvbxjq4rA0yHelmhMb3T4ogGfWzUEvxK2H1wC4Cs5VGXNXHoMvh7/X21Gv+j5zH9b/KubnYC/qTVzkR71wqZKlFEv1UcLyB9g+A9pZRsn2x0SCyPaOWiSVPPGDRvc62koHxVlpIgtnZnuD3w83s+DLYPxFMMorQx81oFJ+dzpEfmXygB3Hi8DF 6GOn/5yw y3adiJ8ehmuPhsUEdFNYSqJPiI+D78k5xA2LdSCEmYStw61fpdaKiPOJXFd8bHmgY4Pwxpl2CX9/UGgEmNF5Er5LtWXjqQPvMuyBZPpgGBIZkPNiIb2zXii2GdBKXmILZuVuqU0SDYY0iLiGm+qteLbhR5dDjLtMl+vVnUzxQyusBv8wPD/1S+SmQjDkQb9HyNhe7tOaWupfIUfHnczKYsQUelyt2ER4Ii4eJBe61V01bdAVRBNuf3yilbz4qWhRTmzp6q+FyUasYuUeuJMvlIyVJRWPdqxqqXOaqK6dlQbeZFgvDL0MFzlbBnDmEVJZC+e3amqq+ZKL8aZ463PQfaqwjY1J6CUxdOKI56dHkJbTffrCb9/PTuZM1AO0sHbZPMbvE X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Hi Jacob, kernel test robot noticed the following build warnings: [auto build test WARNING on 774ca6d3bf24287ff60b7d6dd4171ebb6e47760a] url: https://github.com/intel-lab-lkp/linux/commits/Jacob-Keller/lib-packing-create-__pack-and-__unpack-variants-without-error-checking/20241111-161131 base: 774ca6d3bf24287ff60b7d6dd4171ebb6e47760a patch link: https://lore.kernel.org/r/20241111-packing-pack-fields-and-ice-implementation-v5-3-80c07349e6b7%40intel.com patch subject: [PATCH net-next v5 3/9] lib: packing: add pack_fields() and unpack_fields() config: x86_64-rhel-8.3 (https://download.01.org/0day-ci/archive/20241111/202411111823.Y2NHM1AE-lkp@intel.com/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/20241111/202411111823.Y2NHM1AE-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 | Closes: https://lore.kernel.org/oe-kbuild-all/202411111823.Y2NHM1AE-lkp@intel.com/ All warnings (new ones prefixed by >>): In file included from scripts/mod/packed_fields.c:15: scripts/mod/packed_fields.c: In function 'handle_packed_field_symbol': >> scripts/mod/packed_fields.c:134:23: warning: format '%u' expects argument of type 'unsigned int', but argument 5 has type 'Elf64_Xword' {aka 'long unsigned int'} [-Wformat=] 134 | error("[%s.ko] \"%s\" has size %u which is not a multiple of the field size (%zu)\n", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 135 | mod->name, symname, sym->st_size, field_size); | ~~~~~~~~~~~~ | | | Elf64_Xword {aka long unsigned int} scripts/mod/modpost.h:207:51: note: in definition of macro 'error' 207 | #define error(fmt, args...) modpost_log(true, fmt, ##args) | ^~~ scripts/mod/packed_fields.c:134:49: note: format string is defined here 134 | error("[%s.ko] \"%s\" has size %u which is not a multiple of the field size (%zu)\n", | ~^ | | | unsigned int | %lu -- In file included from scripts/mod/packed_fields.c:15: scripts/mod/packed_fields.c: In function 'handle_packed_field_symbol': >> scripts/mod/packed_fields.c:134:23: warning: format '%u' expects argument of type 'unsigned int', but argument 5 has type 'Elf64_Xword' {aka 'long unsigned int'} [-Wformat=] 134 | error("[%s.ko] \"%s\" has size %u which is not a multiple of the field size (%zu)\n", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 135 | mod->name, symname, sym->st_size, field_size); | ~~~~~~~~~~~~ | | | Elf64_Xword {aka long unsigned int} scripts/mod/modpost.h:207:51: note: in definition of macro 'error' 207 | #define error(fmt, args...) modpost_log(true, fmt, ##args) | ^~~ scripts/mod/packed_fields.c:134:49: note: format string is defined here 134 | error("[%s.ko] \"%s\" has size %u which is not a multiple of the field size (%zu)\n", | ~^ | | | unsigned int | %lu vim +134 scripts/mod/packed_fields.c 99 100 void handle_packed_field_symbol(struct module *mod, struct elf_info *info, 101 Elf_Sym *sym, const char *symname) 102 { 103 unsigned int secindex = get_secindex(info, sym); 104 struct packed_field_elem elem = {}, prev = {}; 105 enum element_order order = FIRST_ELEMENT; 106 enum field_type type = UNKNOWN_SECTION; 107 size_t field_size, count; 108 const void *data, *ptr; 109 const char *section; 110 111 /* Skip symbols without a name */ 112 if (*symname == '\0') 113 return; 114 115 /* Skip symbols with invalid sections */ 116 if (secindex >= info->num_sections) 117 return; 118 119 section = sec_name(info, secindex); 120 121 if (strcmp(section, ".rodata.packed_fields_s") == 0) 122 type = PACKED_FIELD_S; 123 else if (strcmp(section, ".rodata.packed_fields_m") == 0) 124 type = PACKED_FIELD_M; 125 126 /* Other sections don't relate to packed fields */ 127 if (type == UNKNOWN_SECTION) 128 return; 129 130 field_size = field_type_to_size(type); 131 132 /* check that the data is a multiple of the size */ 133 if (sym->st_size % field_size != 0) { > 134 error("[%s.ko] \"%s\" has size %u which is not a multiple of the field size (%zu)\n", -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki