# git rev-parse -q --verify aee101d7b95a03078945681dd7f7ea5e4a1e7686^{commit} aee101d7b95a03078945681dd7f7ea5e4a1e7686 already have revision, skipping fetch # git checkout -q -f -B kisskb aee101d7b95a03078945681dd7f7ea5e4a1e7686 # git clean -qxdf # < git log -1 # commit aee101d7b95a03078945681dd7f7ea5e4a1e7686 # Author: Nicholas Piggin # Date: Mon Jan 17 23:44:03 2022 +1000 # # powerpc/64s: Mask SRR0 before checking against the masked NIP # # Commit 314f6c23dd8d ("powerpc/64s: Mask NIP before checking against # SRR0") masked off the low 2 bits of the NIP value in the interrupt # stack frame in case they are non-zero and mis-compare against a SRR0 # register value of a CPU which always reads back 0 from the 2 low bits # which are reserved. # # This now causes the opposite problem that an implementation which does # implement those bits in SRR0 will mis-compare against the masked NIP # value in which they have been cleared. QEMU is one such implementation, # and this is allowed by the architecture. # # This can be triggered by sigfuz by setting low bits of PT_NIP in the # signal context. # # Fix this for now by masking the SRR0 bits as well. Cleaner is probably # to sanitise these values before putting them in registers or stack, but # this is the quick and backportable fix. # # Fixes: 314f6c23dd8d ("powerpc/64s: Mask NIP before checking against SRR0") # Signed-off-by: Nicholas Piggin # Signed-off-by: Michael Ellerman # Link: https://lore.kernel.org/r/20220117134403.2995059-1-npiggin@gmail.com # < /opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux-ld --version # < git log --format=%s --max-count=1 aee101d7b95a03078945681dd7f7ea5e4a1e7686 # < make -s -j 32 ARCH=powerpc O=/kisskb/build/powerpc-fixes_holly_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- holly_defconfig # < make -s -j 32 ARCH=powerpc O=/kisskb/build/powerpc-fixes_holly_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- help # make -s -j 32 ARCH=powerpc O=/kisskb/build/powerpc-fixes_holly_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- olddefconfig # make -s -j 32 ARCH=powerpc O=/kisskb/build/powerpc-fixes_holly_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c: In function 'tsi108_get_mac': /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1099:20: error: assignment of read-only location '*dev->dev_addr' dev->dev_addr[0] = 0x00; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1100:20: error: assignment of read-only location '*(dev->dev_addr + 1u)' dev->dev_addr[1] = 0x06; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1101:20: error: assignment of read-only location '*(dev->dev_addr + 2u)' dev->dev_addr[2] = 0xd2; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1102:20: error: assignment of read-only location '*(dev->dev_addr + 3u)' dev->dev_addr[3] = 0x00; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1103:20: error: assignment of read-only location '*(dev->dev_addr + 4u)' dev->dev_addr[4] = 0x00; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1105:21: error: assignment of read-only location '*(dev->dev_addr + 5u)' dev->dev_addr[5] = 0x01; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1107:21: error: assignment of read-only location '*(dev->dev_addr + 5u)' dev->dev_addr[5] = 0x02; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1117:20: error: assignment of read-only location '*dev->dev_addr' dev->dev_addr[0] = (word2 >> 16) & 0xff; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1118:20: error: assignment of read-only location '*(dev->dev_addr + 1u)' dev->dev_addr[1] = (word2 >> 24) & 0xff; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1119:20: error: assignment of read-only location '*(dev->dev_addr + 2u)' dev->dev_addr[2] = (word1 >> 0) & 0xff; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1120:20: error: assignment of read-only location '*(dev->dev_addr + 3u)' dev->dev_addr[3] = (word1 >> 8) & 0xff; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1121:20: error: assignment of read-only location '*(dev->dev_addr + 4u)' dev->dev_addr[4] = (word1 >> 16) & 0xff; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1122:20: error: assignment of read-only location '*(dev->dev_addr + 5u)' dev->dev_addr[5] = (word1 >> 24) & 0xff; ^ /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c: In function 'tsi108_set_mac': /kisskb/src/drivers/net/ethernet/tundra/tsi108_eth.c:1146:20: error: assignment of read-only location '*(dev->dev_addr + (sizetype)((unsigned int)i * 1u))' dev->dev_addr[i] = ((unsigned char *)addr)[i + 2]; ^ make[5]: *** [/kisskb/src/scripts/Makefile.build:288: drivers/net/ethernet/tundra/tsi108_eth.o] Error 1 make[4]: *** [/kisskb/src/scripts/Makefile.build:550: drivers/net/ethernet/tundra] Error 2 make[3]: *** [/kisskb/src/scripts/Makefile.build:550: drivers/net/ethernet] Error 2 make[2]: *** [/kisskb/src/scripts/Makefile.build:550: drivers/net] Error 2 make[1]: *** [/kisskb/src/Makefile:1840: drivers] Error 2 make[1]: *** Waiting for unfinished jobs.... make: *** [Makefile:219: __sub-make] Error 2 Command 'make -s -j 32 ARCH=powerpc O=/kisskb/build/powerpc-fixes_holly_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- ' returned non-zero exit status 2 # rm -rf /kisskb/build/powerpc-fixes_holly_defconfig_powerpc-gcc5 # Build took: 0:00:27.231604