Message ID | 20230419-const-partition-v1-1-2d66f2d83873@weissschuh.net (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | block: constify some structures of partitions/core.c | expand |
Hi Thomas, kernel test robot noticed the following build warnings: [auto build test WARNING on af67688dca57999fd848f051eeea1d375ba546b2] url: https://github.com/intel-lab-lkp/linux/commits/Thomas-Wei-schuh/block-constify-partition-prober-array/20230419-155356 base: af67688dca57999fd848f051eeea1d375ba546b2 patch link: https://lore.kernel.org/r/20230419-const-partition-v1-1-2d66f2d83873%40weissschuh.net patch subject: [PATCH 1/4] block: constify partition prober array config: m68k-allyesconfig (https://download.01.org/0day-ci/archive/20230419/202304191640.SkNk7kVN-lkp@intel.com/config) compiler: m68k-linux-gcc (GCC) 12.1.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/intel-lab-lkp/linux/commit/9bc1f4308c10322c327c9e86ede6bb9e862440aa git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Thomas-Wei-schuh/block-constify-partition-prober-array/20230419-155356 git checkout 9bc1f4308c10322c327c9e86ede6bb9e862440aa # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=m68k olddefconfig COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=m68k SHELL=/bin/bash block/partitions/ If you fix the issue, kindly add following tag where applicable | Reported-by: kernel test robot <lkp@intel.com> | Link: https://lore.kernel.org/oe-kbuild-all/202304191640.SkNk7kVN-lkp@intel.com/ All warnings (new ones prefixed by >>): >> block/partitions/core.c:15:8: warning: type qualifiers ignored on function return type [-Wignored-qualifiers] 15 | static const int (*check_part[])(struct parsed_partitions *) = { | ^~~~~ vim +15 block/partitions/core.c 14 > 15 static const int (*check_part[])(struct parsed_partitions *) = { 16 /* 17 * Probe partition formats with tables at disk address 0 18 * that also have an ADFS boot block at 0xdc0. 19 */ 20 #ifdef CONFIG_ACORN_PARTITION_ICS 21 adfspart_check_ICS, 22 #endif 23 #ifdef CONFIG_ACORN_PARTITION_POWERTEC 24 adfspart_check_POWERTEC, 25 #endif 26 #ifdef CONFIG_ACORN_PARTITION_EESOX 27 adfspart_check_EESOX, 28 #endif 29 30 /* 31 * Now move on to formats that only have partition info at 32 * disk address 0xdc0. Since these may also have stale 33 * PC/BIOS partition tables, they need to come before 34 * the msdos entry. 35 */ 36 #ifdef CONFIG_ACORN_PARTITION_CUMANA 37 adfspart_check_CUMANA, 38 #endif 39 #ifdef CONFIG_ACORN_PARTITION_ADFS 40 adfspart_check_ADFS, 41 #endif 42
Hi Thomas, kernel test robot noticed the following build warnings: [auto build test WARNING on af67688dca57999fd848f051eeea1d375ba546b2] url: https://github.com/intel-lab-lkp/linux/commits/Thomas-Wei-schuh/block-constify-partition-prober-array/20230419-155356 base: af67688dca57999fd848f051eeea1d375ba546b2 patch link: https://lore.kernel.org/r/20230419-const-partition-v1-1-2d66f2d83873%40weissschuh.net patch subject: [PATCH 1/4] block: constify partition prober array config: arm64-randconfig-r001-20230416 (https://download.01.org/0day-ci/archive/20230419/202304192132.a6MR0cFR-lkp@intel.com/config) compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project 437b7602e4a998220871de78afcb020b9c14a661) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install arm64 cross compiling tool for clang build # apt-get install binutils-aarch64-linux-gnu # https://github.com/intel-lab-lkp/linux/commit/9bc1f4308c10322c327c9e86ede6bb9e862440aa git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Thomas-Wei-schuh/block-constify-partition-prober-array/20230419-155356 git checkout 9bc1f4308c10322c327c9e86ede6bb9e862440aa # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 olddefconfig COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash block/partitions/ If you fix the issue, kindly add following tag where applicable | Reported-by: kernel test robot <lkp@intel.com> | Link: https://lore.kernel.org/oe-kbuild-all/202304192132.a6MR0cFR-lkp@intel.com/ All warnings (new ones prefixed by >>): >> block/partitions/core.c:15:8: warning: 'const' type qualifier on return type has no effect [-Wignored-qualifiers] static const int (*check_part[])(struct parsed_partitions *) = { ^~~~~~ block/partitions/core.c:47:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types] efi_partition, /* this must come before msdos */ ^~~~~~~~~~~~~ block/partitions/core.c:50:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types] sgi_partition, ^~~~~~~~~~~~~ block/partitions/core.c:56:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types] msdos_partition, ^~~~~~~~~~~~~~~ block/partitions/core.c:59:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types] osf_partition, ^~~~~~~~~~~~~ block/partitions/core.c:62:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types] sun_partition, ^~~~~~~~~~~~~ block/partitions/core.c:65:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types] amiga_partition, ^~~~~~~~~~~~~~~ block/partitions/core.c:71:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types] mac_partition, ^~~~~~~~~~~~~ 1 warning and 7 errors generated. vim +/const +15 block/partitions/core.c 14 > 15 static const int (*check_part[])(struct parsed_partitions *) = { 16 /* 17 * Probe partition formats with tables at disk address 0 18 * that also have an ADFS boot block at 0xdc0. 19 */ 20 #ifdef CONFIG_ACORN_PARTITION_ICS 21 adfspart_check_ICS, 22 #endif 23 #ifdef CONFIG_ACORN_PARTITION_POWERTEC 24 adfspart_check_POWERTEC, 25 #endif 26 #ifdef CONFIG_ACORN_PARTITION_EESOX 27 adfspart_check_EESOX, 28 #endif 29 30 /* 31 * Now move on to formats that only have partition info at 32 * disk address 0xdc0. Since these may also have stale 33 * PC/BIOS partition tables, they need to come before 34 * the msdos entry. 35 */ 36 #ifdef CONFIG_ACORN_PARTITION_CUMANA 37 adfspart_check_CUMANA, 38 #endif 39 #ifdef CONFIG_ACORN_PARTITION_ADFS 40 adfspart_check_ADFS, 41 #endif 42
Hi Thomas,
kernel test robot noticed the following build errors:
[auto build test ERROR on af67688dca57999fd848f051eeea1d375ba546b2]
url: https://github.com/intel-lab-lkp/linux/commits/Thomas-Wei-schuh/block-constify-partition-prober-array/20230419-155356
base: af67688dca57999fd848f051eeea1d375ba546b2
patch link: https://lore.kernel.org/r/20230419-const-partition-v1-1-2d66f2d83873%40weissschuh.net
patch subject: [PATCH 1/4] block: constify partition prober array
config: arm64-randconfig-r001-20230416 (https://download.01.org/0day-ci/archive/20230420/202304200014.y79VnmJo-lkp@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project 437b7602e4a998220871de78afcb020b9c14a661)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://github.com/intel-lab-lkp/linux/commit/9bc1f4308c10322c327c9e86ede6bb9e862440aa
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Thomas-Wei-schuh/block-constify-partition-prober-array/20230419-155356
git checkout 9bc1f4308c10322c327c9e86ede6bb9e862440aa
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash block/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202304200014.y79VnmJo-lkp@intel.com/
All errors (new ones prefixed by >>):
block/partitions/core.c:15:8: warning: 'const' type qualifier on return type has no effect [-Wignored-qualifiers]
static const int (*check_part[])(struct parsed_partitions *) = {
^~~~~~
>> block/partitions/core.c:47:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types]
efi_partition, /* this must come before msdos */
^~~~~~~~~~~~~
block/partitions/core.c:50:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types]
sgi_partition,
^~~~~~~~~~~~~
block/partitions/core.c:56:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types]
msdos_partition,
^~~~~~~~~~~~~~~
block/partitions/core.c:59:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types]
osf_partition,
^~~~~~~~~~~~~
block/partitions/core.c:62:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types]
sun_partition,
^~~~~~~~~~~~~
block/partitions/core.c:65:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types]
amiga_partition,
^~~~~~~~~~~~~~~
block/partitions/core.c:71:2: error: incompatible function pointer types initializing 'const int (*)(struct parsed_partitions *)' with an expression of type 'int (struct parsed_partitions *)' [-Wincompatible-function-pointer-types]
mac_partition,
^~~~~~~~~~~~~
1 warning and 7 errors generated.
vim +47 block/partitions/core.c
387048bf67eeff Christoph Hellwig 2020-03-24 42
387048bf67eeff Christoph Hellwig 2020-03-24 43 #ifdef CONFIG_CMDLINE_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 44 cmdline_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 45 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 46 #ifdef CONFIG_EFI_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 @47 efi_partition, /* this must come before msdos */
387048bf67eeff Christoph Hellwig 2020-03-24 48 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 49 #ifdef CONFIG_SGI_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 50 sgi_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 51 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 52 #ifdef CONFIG_LDM_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 53 ldm_partition, /* this must come before msdos */
387048bf67eeff Christoph Hellwig 2020-03-24 54 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 55 #ifdef CONFIG_MSDOS_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 56 msdos_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 57 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 58 #ifdef CONFIG_OSF_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 59 osf_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 60 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 61 #ifdef CONFIG_SUN_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 62 sun_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 63 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 64 #ifdef CONFIG_AMIGA_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 65 amiga_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 66 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 67 #ifdef CONFIG_ATARI_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 68 atari_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 69 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 70 #ifdef CONFIG_MAC_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 71 mac_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 72 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 73 #ifdef CONFIG_ULTRIX_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 74 ultrix_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 75 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 76 #ifdef CONFIG_IBM_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 77 ibm_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 78 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 79 #ifdef CONFIG_KARMA_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 80 karma_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 81 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 82 #ifdef CONFIG_SYSV68_PARTITION
387048bf67eeff Christoph Hellwig 2020-03-24 83 sysv68_partition,
387048bf67eeff Christoph Hellwig 2020-03-24 84 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 85 NULL
387048bf67eeff Christoph Hellwig 2020-03-24 86 };
387048bf67eeff Christoph Hellwig 2020-03-24 87
Hi Thomas,
kernel test robot noticed the following build warnings:
[auto build test WARNING on af67688dca57999fd848f051eeea1d375ba546b2]
url: https://github.com/intel-lab-lkp/linux/commits/Thomas-Wei-schuh/block-constify-partition-prober-array/20230419-155356
base: af67688dca57999fd848f051eeea1d375ba546b2
patch link: https://lore.kernel.org/r/20230419-const-partition-v1-1-2d66f2d83873%40weissschuh.net
patch subject: [PATCH 1/4] block: constify partition prober array
config: x86_64-randconfig-s023 (https://download.01.org/0day-ci/archive/20230420/202304200454.r3d73iaw-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.4-39-gce1a6720-dirty
# https://github.com/intel-lab-lkp/linux/commit/9bc1f4308c10322c327c9e86ede6bb9e862440aa
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Thomas-Wei-schuh/block-constify-partition-prober-array/20230419-155356
git checkout 9bc1f4308c10322c327c9e86ede6bb9e862440aa
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=x86_64 olddefconfig
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=x86_64 SHELL=/bin/bash
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202304200454.r3d73iaw-lkp@intel.com/
sparse warnings: (new ones prefixed by >>)
>> block/partitions/core.c:24:9: sparse: sparse: incorrect type in initializer (different modifiers) @@ expected int ( const * )( ... ) @@ got int ( * )( ... ) @@
block/partitions/core.c:24:9: sparse: expected int ( const * )( ... )
block/partitions/core.c:24:9: sparse: got int ( * )( ... )
block/partitions/core.c:47:9: sparse: sparse: incorrect type in initializer (different modifiers) @@ expected int ( const * )( ... ) @@ got int ( * )( ... ) @@
block/partitions/core.c:47:9: sparse: expected int ( const * )( ... )
block/partitions/core.c:47:9: sparse: got int ( * )( ... )
block/partitions/core.c:53:9: sparse: sparse: incorrect type in initializer (different modifiers) @@ expected int ( const * )( ... ) @@ got int ( * )( ... ) @@
block/partitions/core.c:53:9: sparse: expected int ( const * )( ... )
block/partitions/core.c:53:9: sparse: got int ( * )( ... )
block/partitions/core.c:56:9: sparse: sparse: incorrect type in initializer (different modifiers) @@ expected int ( const * )( ... ) @@ got int ( * )( ... ) @@
block/partitions/core.c:56:9: sparse: expected int ( const * )( ... )
block/partitions/core.c:56:9: sparse: got int ( * )( ... )
block/partitions/core.c:68:9: sparse: sparse: incorrect type in initializer (different modifiers) @@ expected int ( const * )( ... ) @@ got int ( * )( ... ) @@
block/partitions/core.c:68:9: sparse: expected int ( const * )( ... )
block/partitions/core.c:68:9: sparse: got int ( * )( ... )
block/partitions/core.c:71:9: sparse: sparse: incorrect type in initializer (different modifiers) @@ expected int ( const * )( ... ) @@ got int ( * )( ... ) @@
block/partitions/core.c:71:9: sparse: expected int ( const * )( ... )
block/partitions/core.c:71:9: sparse: got int ( * )( ... )
vim +24 block/partitions/core.c
387048bf67eeff Christoph Hellwig 2020-03-24 14
9bc1f4308c1032 Thomas Weißschuh 2023-04-19 15 static const int (*check_part[])(struct parsed_partitions *) = {
387048bf67eeff Christoph Hellwig 2020-03-24 16 /*
387048bf67eeff Christoph Hellwig 2020-03-24 17 * Probe partition formats with tables at disk address 0
387048bf67eeff Christoph Hellwig 2020-03-24 18 * that also have an ADFS boot block at 0xdc0.
387048bf67eeff Christoph Hellwig 2020-03-24 19 */
387048bf67eeff Christoph Hellwig 2020-03-24 20 #ifdef CONFIG_ACORN_PARTITION_ICS
387048bf67eeff Christoph Hellwig 2020-03-24 21 adfspart_check_ICS,
387048bf67eeff Christoph Hellwig 2020-03-24 22 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 23 #ifdef CONFIG_ACORN_PARTITION_POWERTEC
387048bf67eeff Christoph Hellwig 2020-03-24 @24 adfspart_check_POWERTEC,
387048bf67eeff Christoph Hellwig 2020-03-24 25 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 26 #ifdef CONFIG_ACORN_PARTITION_EESOX
387048bf67eeff Christoph Hellwig 2020-03-24 27 adfspart_check_EESOX,
387048bf67eeff Christoph Hellwig 2020-03-24 28 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 29
387048bf67eeff Christoph Hellwig 2020-03-24 30 /*
387048bf67eeff Christoph Hellwig 2020-03-24 31 * Now move on to formats that only have partition info at
387048bf67eeff Christoph Hellwig 2020-03-24 32 * disk address 0xdc0. Since these may also have stale
387048bf67eeff Christoph Hellwig 2020-03-24 33 * PC/BIOS partition tables, they need to come before
387048bf67eeff Christoph Hellwig 2020-03-24 34 * the msdos entry.
387048bf67eeff Christoph Hellwig 2020-03-24 35 */
387048bf67eeff Christoph Hellwig 2020-03-24 36 #ifdef CONFIG_ACORN_PARTITION_CUMANA
387048bf67eeff Christoph Hellwig 2020-03-24 37 adfspart_check_CUMANA,
387048bf67eeff Christoph Hellwig 2020-03-24 38 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 39 #ifdef CONFIG_ACORN_PARTITION_ADFS
387048bf67eeff Christoph Hellwig 2020-03-24 40 adfspart_check_ADFS,
387048bf67eeff Christoph Hellwig 2020-03-24 41 #endif
387048bf67eeff Christoph Hellwig 2020-03-24 42
diff --git a/block/partitions/core.c b/block/partitions/core.c index 7b8ef6296abd..667f3dcebd59 100644 --- a/block/partitions/core.c +++ b/block/partitions/core.c @@ -12,7 +12,7 @@ #include <linux/raid/detect.h> #include "check.h" -static int (*check_part[])(struct parsed_partitions *) = { +static const int (*check_part[])(struct parsed_partitions *) = { /* * Probe partition formats with tables at disk address 0 * that also have an ADFS boot block at 0xdc0.
The array is never modified so it can be const. Signed-off-by: Thomas Weißschuh <linux@weissschuh.net> --- block/partitions/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)