From: kernel test robot <lkp@intel.com>
To: Eric Biggers <ebiggers@kernel.org>, linux-kernel@vger.kernel.org
Cc: oe-kbuild-all@lists.linux.dev, Ard Biesheuvel <ardb@kernel.org>,
linux-crypto@vger.kernel.org
Subject: Re: [PATCH 2/3] lib/crc/crc32: change crc32() from macro to inline function and remove cast
Date: Fri, 20 Jun 2025 15:01:04 +0800 [thread overview]
Message-ID: <202506201401.OhkkTlo0-lkp@intel.com> (raw)
In-Reply-To: <20250619183414.100082-3-ebiggers@kernel.org>
Hi Eric,
kernel test robot noticed the following build errors:
[auto build test ERROR on ee925097a5a76eaf9c4954cdd7288a070d57a8d4]
url: https://github.com/intel-lab-lkp/linux/commits/Eric-Biggers/lib-crc-crc32-document-crc32_le-crc32_be-and-crc32c/20250620-023634
base: ee925097a5a76eaf9c4954cdd7288a070d57a8d4
patch link: https://lore.kernel.org/r/20250619183414.100082-3-ebiggers%40kernel.org
patch subject: [PATCH 2/3] lib/crc/crc32: change crc32() from macro to inline function and remove cast
config: x86_64-buildonly-randconfig-004-20250620 (https://download.01.org/0day-ci/archive/20250620/202506201401.OhkkTlo0-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/20250620/202506201401.OhkkTlo0-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/oe-kbuild-all/202506201401.OhkkTlo0-lkp@intel.com/
All errors (new ones prefixed by >>):
drivers/nvmem/layouts/u-boot-env.c: In function 'u_boot_env_parse':
>> drivers/nvmem/layouts/u-boot-env.c:151:16: error: called object 'crc32' is not a function or function pointer
151 | calc = crc32(~0, buf + crc32_data_offset, crc32_data_len) ^ ~0L;
| ^~~~~
drivers/nvmem/layouts/u-boot-env.c:99:18: note: declared here
99 | uint32_t crc32;
| ^~~~~
vim +/crc32 +151 drivers/nvmem/layouts/u-boot-env.c
5f15811286aff4 Rafał Miłecki 2024-09-02 88
5f15811286aff4 Rafał Miłecki 2024-09-02 89 int u_boot_env_parse(struct device *dev, struct nvmem_device *nvmem,
5f15811286aff4 Rafał Miłecki 2024-09-02 90 enum u_boot_env_format format)
5f15811286aff4 Rafał Miłecki 2024-09-02 91 {
5f15811286aff4 Rafał Miłecki 2024-09-02 92 size_t crc32_data_offset;
5f15811286aff4 Rafał Miłecki 2024-09-02 93 size_t crc32_data_len;
5f15811286aff4 Rafał Miłecki 2024-09-02 94 size_t crc32_offset;
5f15811286aff4 Rafał Miłecki 2024-09-02 95 __le32 *crc32_addr;
5f15811286aff4 Rafał Miłecki 2024-09-02 96 size_t data_offset;
5f15811286aff4 Rafał Miłecki 2024-09-02 97 size_t data_len;
5f15811286aff4 Rafał Miłecki 2024-09-02 98 size_t dev_size;
5f15811286aff4 Rafał Miłecki 2024-09-02 99 uint32_t crc32;
5f15811286aff4 Rafał Miłecki 2024-09-02 100 uint32_t calc;
5f15811286aff4 Rafał Miłecki 2024-09-02 101 uint8_t *buf;
5f15811286aff4 Rafał Miłecki 2024-09-02 102 int bytes;
5f15811286aff4 Rafał Miłecki 2024-09-02 103 int err;
5f15811286aff4 Rafał Miłecki 2024-09-02 104
5f15811286aff4 Rafał Miłecki 2024-09-02 105 dev_size = nvmem_dev_size(nvmem);
5f15811286aff4 Rafał Miłecki 2024-09-02 106
5f15811286aff4 Rafał Miłecki 2024-09-02 107 buf = kzalloc(dev_size, GFP_KERNEL);
5f15811286aff4 Rafał Miłecki 2024-09-02 108 if (!buf) {
5f15811286aff4 Rafał Miłecki 2024-09-02 109 err = -ENOMEM;
5f15811286aff4 Rafał Miłecki 2024-09-02 110 goto err_out;
5f15811286aff4 Rafał Miłecki 2024-09-02 111 }
5f15811286aff4 Rafał Miłecki 2024-09-02 112
5f15811286aff4 Rafał Miłecki 2024-09-02 113 bytes = nvmem_device_read(nvmem, 0, dev_size, buf);
5f15811286aff4 Rafał Miłecki 2024-09-02 114 if (bytes < 0) {
5f15811286aff4 Rafał Miłecki 2024-09-02 115 err = bytes;
5f15811286aff4 Rafał Miłecki 2024-09-02 116 goto err_kfree;
5f15811286aff4 Rafał Miłecki 2024-09-02 117 } else if (bytes != dev_size) {
5f15811286aff4 Rafał Miłecki 2024-09-02 118 err = -EIO;
5f15811286aff4 Rafał Miłecki 2024-09-02 119 goto err_kfree;
5f15811286aff4 Rafał Miłecki 2024-09-02 120 }
5f15811286aff4 Rafał Miłecki 2024-09-02 121
5f15811286aff4 Rafał Miłecki 2024-09-02 122 switch (format) {
5f15811286aff4 Rafał Miłecki 2024-09-02 123 case U_BOOT_FORMAT_SINGLE:
5f15811286aff4 Rafał Miłecki 2024-09-02 124 crc32_offset = offsetof(struct u_boot_env_image_single, crc32);
5f15811286aff4 Rafał Miłecki 2024-09-02 125 crc32_data_offset = offsetof(struct u_boot_env_image_single, data);
5f15811286aff4 Rafał Miłecki 2024-09-02 126 data_offset = offsetof(struct u_boot_env_image_single, data);
5f15811286aff4 Rafał Miłecki 2024-09-02 127 break;
5f15811286aff4 Rafał Miłecki 2024-09-02 128 case U_BOOT_FORMAT_REDUNDANT:
5f15811286aff4 Rafał Miłecki 2024-09-02 129 crc32_offset = offsetof(struct u_boot_env_image_redundant, crc32);
5f15811286aff4 Rafał Miłecki 2024-09-02 130 crc32_data_offset = offsetof(struct u_boot_env_image_redundant, data);
5f15811286aff4 Rafał Miłecki 2024-09-02 131 data_offset = offsetof(struct u_boot_env_image_redundant, data);
5f15811286aff4 Rafał Miłecki 2024-09-02 132 break;
5f15811286aff4 Rafał Miłecki 2024-09-02 133 case U_BOOT_FORMAT_BROADCOM:
5f15811286aff4 Rafał Miłecki 2024-09-02 134 crc32_offset = offsetof(struct u_boot_env_image_broadcom, crc32);
5f15811286aff4 Rafał Miłecki 2024-09-02 135 crc32_data_offset = offsetof(struct u_boot_env_image_broadcom, data);
5f15811286aff4 Rafał Miłecki 2024-09-02 136 data_offset = offsetof(struct u_boot_env_image_broadcom, data);
5f15811286aff4 Rafał Miłecki 2024-09-02 137 break;
5f15811286aff4 Rafał Miłecki 2024-09-02 138 }
5f15811286aff4 Rafał Miłecki 2024-09-02 139
5f15811286aff4 Rafał Miłecki 2024-09-02 140 if (dev_size < data_offset) {
5f15811286aff4 Rafał Miłecki 2024-09-02 141 dev_err(dev, "Device too small for u-boot-env\n");
5f15811286aff4 Rafał Miłecki 2024-09-02 142 err = -EIO;
5f15811286aff4 Rafał Miłecki 2024-09-02 143 goto err_kfree;
5f15811286aff4 Rafał Miłecki 2024-09-02 144 }
5f15811286aff4 Rafał Miłecki 2024-09-02 145
5f15811286aff4 Rafał Miłecki 2024-09-02 146 crc32_addr = (__le32 *)(buf + crc32_offset);
5f15811286aff4 Rafał Miłecki 2024-09-02 147 crc32 = le32_to_cpu(*crc32_addr);
5f15811286aff4 Rafał Miłecki 2024-09-02 148 crc32_data_len = dev_size - crc32_data_offset;
5f15811286aff4 Rafał Miłecki 2024-09-02 149 data_len = dev_size - data_offset;
5f15811286aff4 Rafał Miłecki 2024-09-02 150
5f15811286aff4 Rafał Miłecki 2024-09-02 @151 calc = crc32(~0, buf + crc32_data_offset, crc32_data_len) ^ ~0L;
5f15811286aff4 Rafał Miłecki 2024-09-02 152 if (calc != crc32) {
5f15811286aff4 Rafał Miłecki 2024-09-02 153 dev_err(dev, "Invalid calculated CRC32: 0x%08x (expected: 0x%08x)\n", calc, crc32);
5f15811286aff4 Rafał Miłecki 2024-09-02 154 err = -EINVAL;
5f15811286aff4 Rafał Miłecki 2024-09-02 155 goto err_kfree;
5f15811286aff4 Rafał Miłecki 2024-09-02 156 }
5f15811286aff4 Rafał Miłecki 2024-09-02 157
5f15811286aff4 Rafał Miłecki 2024-09-02 158 buf[dev_size - 1] = '\0';
5f15811286aff4 Rafał Miłecki 2024-09-02 159 err = u_boot_env_parse_cells(dev, nvmem, buf, data_offset, data_len);
5f15811286aff4 Rafał Miłecki 2024-09-02 160
5f15811286aff4 Rafał Miłecki 2024-09-02 161 err_kfree:
5f15811286aff4 Rafał Miłecki 2024-09-02 162 kfree(buf);
5f15811286aff4 Rafał Miłecki 2024-09-02 163 err_out:
5f15811286aff4 Rafał Miłecki 2024-09-02 164 return err;
5f15811286aff4 Rafał Miłecki 2024-09-02 165 }
5f15811286aff4 Rafał Miłecki 2024-09-02 166 EXPORT_SYMBOL_GPL(u_boot_env_parse);
5f15811286aff4 Rafał Miłecki 2024-09-02 167
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-06-20 7:02 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-19 18:34 [PATCH 0/3] lib/crc: improve docs and change crc32() to inline function Eric Biggers
2025-06-19 18:34 ` [PATCH 1/3] lib/crc/crc32: document crc32_le(), crc32_be(), and crc32c() Eric Biggers
2025-06-19 18:34 ` [PATCH 2/3] lib/crc/crc32: change crc32() from macro to inline function and remove cast Eric Biggers
2025-06-20 7:01 ` kernel test robot [this message]
2025-06-19 18:34 ` [PATCH 3/3] lib/crc/crc64: add include/linux/crc64.h to kernel-api.rst Eric Biggers
2025-06-20 21:40 ` [PATCH 0/3] lib/crc: improve docs and change crc32() to inline function Ard Biesheuvel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202506201401.OhkkTlo0-lkp@intel.com \
--to=lkp@intel.com \
--cc=ardb@kernel.org \
--cc=ebiggers@kernel.org \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®