Sorry Darrick that we had sent quite a few similar reports against same commit, which is a problem in our side. The issue is valid, but we will reduce the number of similar reports. > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/djwong/xfs-linux.git btree- > private-unions > head: 30d91fbcef11ed07350f943fd3f695e0fb0635dd > commit: 35137236b91e958ca683fd4a045a29c0b5310938 [52/72] xfs: don't crash > with assfail > config: m68k-defconfig (attached as .config) > compiler: m68k-linux-gcc (GCC) 9.2.0 > reproduce: > wget https://raw.githubusercontent.com/intel/lkp- > tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > git checkout 35137236b91e958ca683fd4a045a29c0b5310938 > # save the attached .config to linux build tree > GCC_VERSION=9.2.0 make.cross ARCH=m68k > > If you fix the issue, kindly add following tag > Reported-by: kbuild test robot <lkp(a)intel.com&gt; > > All warnings (new ones prefixed by >>): > > In file included from arch/m68k/include/asm/io_mm.h:25, > from arch/m68k/include/asm/io.h:8, > from include/linux/kexec.h:19, > from include/linux/ima.h:12, > from fs///attr.c:19: > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsb': > >> arch/m68k/include/asm/raw_io.h:83:7: warning: variable '__w' set but not used > [-Wunused-but-set-variable] > 83 | ({u8 __w, __v = (b); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:430:3: note: in expansion of macro > 'rom_out_8' > 430 | rom_out_8(port, *buf++); > | ^~~~~~~~~ > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw': > arch/m68k/include/asm/raw_io.h:86:8: warning: variable '__w' set but not used [- > Wunused-but-set-variable] > 86 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:448:3: note: in expansion of macro > 'rom_out_be16' > 448 | rom_out_be16(port, *buf++); > | ^~~~~~~~~~~~ > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw_swapw': > arch/m68k/include/asm/raw_io.h:90:8: warning: variable '__w' set but not used [- > Wunused-but-set-variable] > 90 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:466:3: note: in expansion of macro > 'rom_out_le16' > 466 | rom_out_le16(port, *buf++); > | ^~~~~~~~~~~~ > -- > In file included from arch/m68k/include/asm/io_mm.h:25, > from arch/m68k/include/asm/io.h:8, > from include/linux/scatterlist.h:9, > from include/linux/blkdev.h:28, > from fs///xfs/xfs_linux.h:31, > from fs///xfs/xfs.h:22, > from fs///xfs/libxfs/xfs_bmap.c:6: > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsb': > >> arch/m68k/include/asm/raw_io.h:83:7: warning: variable '__w' set but not used > [-Wunused-but-set-variable] > 83 | ({u8 __w, __v = (b); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:430:3: note: in expansion of macro > 'rom_out_8' > 430 | rom_out_8(port, *buf++); > | ^~~~~~~~~ > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw': > arch/m68k/include/asm/raw_io.h:86:8: warning: variable '__w' set but not used [- > Wunused-but-set-variable] > 86 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:448:3: note: in expansion of macro > 'rom_out_be16' > 448 | rom_out_be16(port, *buf++); > | ^~~~~~~~~~~~ > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw_swapw': > arch/m68k/include/asm/raw_io.h:90:8: warning: variable '__w' set but not used [- > Wunused-but-set-variable] > 90 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:466:3: note: in expansion of macro > 'rom_out_le16' > 466 | rom_out_le16(port, *buf++); > | ^~~~~~~~~~~~ > fs///xfs/libxfs/xfs_bmap.c: In function 'xfs_bmap_del_extent_real': > fs///xfs/libxfs/xfs_bmap.c:5091:17: warning: variable 'bno' set but not used [- > Wunused-but-set-variable] > 5091 | xfs_fsblock_t bno; > | ^~~ > fs///xfs/libxfs/xfs_bmap.c:5279:58: warning: suggest braces around empty body > in an 'if' statement [-Wempty-body] > 5279 | xfs_trans_mod_dquot_byino(tp, ip, qfield, (long)-nblks); > | ^ > -- > In file included from arch/m68k/include/asm/io_mm.h:25, > from arch/m68k/include/asm/io.h:8, > from include/linux/scatterlist.h:9, > from include/linux/blkdev.h:28, > from fs///xfs/xfs_linux.h:31, > from fs///xfs/xfs.h:22, > from fs///xfs/xfs_ioctl.c:6: > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsb': > >> arch/m68k/include/asm/raw_io.h:83:7: warning: variable '__w' set but not used > [-Wunused-but-set-variable] > 83 | ({u8 __w, __v = (b); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:430:3: note: in expansion of macro > 'rom_out_8' > 430 | rom_out_8(port, *buf++); > | ^~~~~~~~~ > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw': > arch/m68k/include/asm/raw_io.h:86:8: warning: variable '__w' set but not used [- > Wunused-but-set-variable] > 86 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:448:3: note: in expansion of macro > 'rom_out_be16' > 448 | rom_out_be16(port, *buf++); > | ^~~~~~~~~~~~ > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw_swapw': > arch/m68k/include/asm/raw_io.h:90:8: warning: variable '__w' set but not used [- > Wunused-but-set-variable] > 90 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:466:3: note: in expansion of macro > 'rom_out_le16' > 466 | rom_out_le16(port, *buf++); > | ^~~~~~~~~~~~ > fs///xfs/xfs_ioctl.c: In function 'xfs_ioctl_setattr': > fs///xfs/xfs_ioctl.c:1519:20: warning: variable 'olddquot' set but not used [- > Wunused-but-set-variable] > 1519 | struct xfs_dquot *olddquot = NULL; > | ^~~~~~~~ > -- > In file included from arch/m68k/include/asm/io_mm.h:25, > from arch/m68k/include/asm/io.h:8, > from include/linux/scatterlist.h:9, > from include/linux/blkdev.h:28, > from fs///xfs/xfs_linux.h:31, > from fs///xfs/xfs.h:22, > from fs///xfs/xfs_iops.c:6: > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsb': > >> arch/m68k/include/asm/raw_io.h:83:7: warning: variable '__w' set but not used > [-Wunused-but-set-variable] > 83 | ({u8 __w, __v = (b); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:430:3: note: in expansion of macro > 'rom_out_8' > 430 | rom_out_8(port, *buf++); > | ^~~~~~~~~ > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw': > arch/m68k/include/asm/raw_io.h:86:8: warning: variable '__w' set but not used [- > Wunused-but-set-variable] > 86 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:448:3: note: in expansion of macro > 'rom_out_be16' > 448 | rom_out_be16(port, *buf++); > | ^~~~~~~~~~~~ > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw_swapw': > arch/m68k/include/asm/raw_io.h:90:8: warning: variable '__w' set but not used [- > Wunused-but-set-variable] > 90 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:466:3: note: in expansion of macro > 'rom_out_le16' > 466 | rom_out_le16(port, *buf++); > | ^~~~~~~~~~~~ > fs///xfs/xfs_iops.c: In function 'xfs_setattr_nonsize': > fs///xfs/xfs_iops.c:664:39: warning: variable 'olddquot2' set but not used [- > Wunused-but-set-variable] > 664 | struct xfs_dquot *olddquot1 = NULL, *olddquot2 = NULL; > | ^~~~~~~~~ > fs///xfs/xfs_iops.c:664:20: warning: variable 'olddquot1' set but not used [- > Wunused-but-set-variable] > 664 | struct xfs_dquot *olddquot1 = NULL, *olddquot2 = NULL; > | ^~~~~~~~~ > -- > In file included from arch/m68k/include/asm/io_mm.h:25, > from arch/m68k/include/asm/io.h:8, > from include/linux/scatterlist.h:9, > from include/linux/blkdev.h:28, > from fs///xfs/xfs_linux.h:31, > from fs///xfs/xfs.h:22, > from fs///xfs/xfs_super.c:7: > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsb': > >> arch/m68k/include/asm/raw_io.h:83:7: warning: variable '__w' set but not used > [-Wunused-but-set-variable] > 83 | ({u8 __w, __v = (b); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:430:3: note: in expansion of macro > 'rom_out_8' > 430 | rom_out_8(port, *buf++); > | ^~~~~~~~~ > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw': > arch/m68k/include/asm/raw_io.h:86:8: warning: variable '__w' set but not used [- > Wunused-but-set-variable] > 86 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:448:3: note: in expansion of macro > 'rom_out_be16' > 448 | rom_out_be16(port, *buf++); > | ^~~~~~~~~~~~ > arch/m68k/include/asm/raw_io.h: In function 'raw_rom_outsw_swapw': > arch/m68k/include/asm/raw_io.h:90:8: warning: variable '__w' set but not used [- > Wunused-but-set-variable] > 90 | ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > | ^~~ > >> arch/m68k/include/asm/raw_io.h:466:3: note: in expansion of macro > 'rom_out_le16' > 466 | rom_out_le16(port, *buf++); > | ^~~~~~~~~~~~ > fs///xfs/xfs_super.c: In function 'xfs_fs_statfs': > fs///xfs/xfs_super.c:781:28: warning: suggest braces around empty body in an > 'if' statement [-Wempty-body] > 781 | xfs_qm_statvfs(ip, statp); > | ^ > > vim +/__w +83 arch/m68k/include/asm/raw_io.h > > ^1da177e4c3f41 include/asm-m68k/raw_io.h Linus Torvalds 2005-04-16 49 > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 50 /* > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 51 * Atari ROM port (cartridge port) ISA adapter, used for the EtherNEC NE2000 > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 52 * network card driver. > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 53 * The ISA adapter connects address lines A9-A13 to ISA address lines A0-A4, > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 54 * and hardwires the rest of the ISA addresses for a base address of 0x300. > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 55 * > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 56 * Data lines D8-D15 are connected to ISA data lines D0-D7 for reading. > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 57 * For writes, address lines A1-A8 are latched to ISA data lines D0-D7 > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 58 * (meaning the bit pattern on A1-A8 can be read back as byte). > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 59 * > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 60 * Read and write operations are distinguished by the base address used: > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 61 * reads are from the ROM A side range, writes are through the B side range > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 62 * addresses (A side base + 0x10000). > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 63 * > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 64 * Reads and writes are byte only. > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 65 * > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 66 * 16 bit reads and writes are necessary for the NetUSBee adapter's USB > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 67 * chipset - 16 bit words are read straight off the ROM port while 16 bit > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 68 * reads are split into two byte writes. The low byte is latched to the > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 69 * NetUSBee buffer by a read from the _read_ window (with the data pattern > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 70 * asserted as A1-A8 address pattern). The high byte is then written to the > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 71 * write range as usual, completing the write cycle. > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 72 */ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 73 > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 74 #if defined(CONFIG_ATARI_ROM_ISA) > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 75 #define rom_in_8(addr) \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 76 ({ u16 __v = (*(__force volatile u16 *) (addr)); __v >>= 8; __v; }) > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 77 #define rom_in_be16(addr) \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 78 ({ u16 __v = (*(__force volatile u16 *) (addr)); __v; }) > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 79 #define rom_in_le16(addr) \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 80 ({ u16 __v = le16_to_cpu(*(__force volatile u16 *) (addr)); __v; }) > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 81 > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 82 #define rom_out_8(addr, b) \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > @83 ({u8 __w, __v = (b); u32 _addr = ((u32) (addr)); \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 84 __w = ((*(__force volatile u8 *) ((_addr | 0x10000) + (__v<<1)))); }) > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 85 #define rom_out_be16(addr, w) \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 86 ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 87 __w = ((*(__force volatile u16 *) ((_addr & 0xFFFF0000UL) + ((__v & > 0xFF)<<1)))); \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 88 __w = ((*(__force volatile u16 *) ((_addr | 0x10000) + ((__v >> 8)<<1)))); }) > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 89 #define rom_out_le16(addr, w) \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 90 ({u16 __w, __v = (w); u32 _addr = ((u32) (addr)); \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 91 __w = ((*(__force volatile u16 *) ((_addr & 0xFFFF0000UL) + ((__v >> > 8)<<1)))); \ > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 92 __w = ((*(__force volatile u16 *) ((_addr | 0x10000) + ((__v & > 0xFF)<<1)))); }) > 84b16b7b0d5c81 arch/m68k/include/asm/raw_io.h Michael Schmitz 2013-04-06 > 93 > > :::::: The code at line 83 was first introduced by commit > :::::: 84b16b7b0d5c818fadc731a69965dc76dce0c91e m68k/atari: ROM port ISA > adapter support > > :::::: TO: Michael Schmitz <schmitz(a)opal.biophys.uni-duesseldorf.de&gt; > :::::: CC: Geert Uytterhoeven <geert(a)linux-m68k.org&gt; > > --- > 0-DAY CI Kernel Test Service, Intel Corporation > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org