# git rev-parse -q --verify e3d773ddb5a1140780a15703b3e0e2618274cce9^{commit} e3d773ddb5a1140780a15703b3e0e2618274cce9 already have revision, skipping fetch # git checkout -q -f -B kisskb e3d773ddb5a1140780a15703b3e0e2618274cce9 # git clean -qxdf # < git log -1 # commit e3d773ddb5a1140780a15703b3e0e2618274cce9 # Author: Jordan Niethe # Date: Thu Feb 25 14:19:46 2021 +1100 # # powerpc/sstep: Fix VSX instruction emulation # # Commit af99da74333b ("powerpc/sstep: Support VSX vector paired storage # access instructions") added loading and storing 32 word long data into # adjacent VSRs. However the calculation used to determine if two VSRs # needed to be loaded/stored inadvertently prevented the load/storing # taking place for instructions with a data length less than 16 words. # # This causes the emulation to not function correctly, which can be seen # by the alignment_handler selftest: # # $ ./alignment_handler # [snip] # test: test_alignment_handler_vsx_207 # tags: git_version:powerpc-5.12-1-0-g82d2c16b350f # VSX: 2.07B # Doing lxsspx: PASSED # Doing lxsiwax: FAILED: Wrong Data # Doing lxsiwzx: PASSED # Doing stxsspx: PASSED # Doing stxsiwx: PASSED # failure: test_alignment_handler_vsx_207 # test: test_alignment_handler_vsx_300 # tags: git_version:powerpc-5.12-1-0-g82d2c16b350f # VSX: 3.00B # Doing lxsd: PASSED # Doing lxsibzx: PASSED # Doing lxsihzx: PASSED # Doing lxssp: FAILED: Wrong Data # Doing lxv: PASSED # Doing lxvb16x: PASSED # Doing lxvh8x: PASSED # Doing lxvx: PASSED # Doing lxvwsx: FAILED: Wrong Data # Doing lxvl: PASSED # Doing lxvll: PASSED # Doing stxsd: PASSED # Doing stxsibx: PASSED # Doing stxsihx: PASSED # Doing stxssp: PASSED # Doing stxv: PASSED # Doing stxvb16x: PASSED # Doing stxvh8x: PASSED # Doing stxvx: PASSED # Doing stxvl: PASSED # Doing stxvll: PASSED # failure: test_alignment_handler_vsx_300 # [snip] # # Fix this by making sure all VSX instruction emulation correctly # load/store from the VSRs. # # Fixes: af99da74333b ("powerpc/sstep: Support VSX vector paired storage access instructions") # Signed-off-by: Jordan Niethe # Reviewed-by: Ravi Bangoria # Signed-off-by: Michael Ellerman # Link: https://lore.kernel.org/r/20210225031946.1458206-1-jniethe5@gmail.com # < /opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux-ld --version # < git log --format=%s --max-count=1 e3d773ddb5a1140780a15703b3e0e2618274cce9 # < make -s -j 48 ARCH=powerpc O=/kisskb/build/powerpc-fixes_mpc5200_defconfig_powerpc-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux- mpc5200_defconfig # < make -s -j 48 ARCH=powerpc O=/kisskb/build/powerpc-fixes_mpc5200_defconfig_powerpc-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux- help # make -s -j 48 ARCH=powerpc O=/kisskb/build/powerpc-fixes_mpc5200_defconfig_powerpc-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux- olddefconfig # make -s -j 48 ARCH=powerpc O=/kisskb/build/powerpc-fixes_mpc5200_defconfig_powerpc-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux- /kisskb/src/arch/powerpc/platforms/52xx/lite5200_sleep.S: Assembler messages: /kisskb/src/arch/powerpc/platforms/52xx/lite5200_sleep.S:184: Warning: invalid register expression /kisskb/src/block/genhd.c: In function 'diskstats_show': /kisskb/src/block/genhd.c:1314:1: warning: the frame size of 1160 bytes is larger than 1024 bytes [-Wframe-larger-than=] } ^ /kisskb/src/lib/zstd/compress.c: In function 'ZSTD_compressBlock_doubleFast': /kisskb/src/lib/zstd/compress.c:1393:1: warning: the frame size of 1584 bytes is larger than 1024 bytes [-Wframe-larger-than=] } ^ INFO: Uncompressed kernel (size 0x864b78) overlaps the address of the wrapper(0x400000) INFO: Fixing the link_address of wrapper to (0x900000) INFO: Uncompressed kernel (size 0x864b78) overlaps the address of the wrapper(0x400000) INFO: Fixing the link_address of wrapper to (0x900000) INFO: Uncompressed kernel (size 0x864b78) overlaps the address of the wrapper(0x400000) INFO: Fixing the link_address of wrapper to (0x900000) INFO: Uncompressed kernel (size 0x85459c) overlaps the address of the wrapper(0x400000) INFO: Fixing the link_address of wrapper to (0x900000) Image Name: Linux-5.12.0-rc1-ge3d773ddb5a1 Created: Tue Mar 2 01:03:32 2021 Image Type: PowerPC Linux Kernel Image (gzip compressed) Data Size: 3838957 Bytes = 3748.98 KiB = 3.66 MiB Load Address: 00000000 Entry Point: 00000000 Image Name: Linux-5.12.0-rc1-ge3d773ddb5a1 Created: Tue Mar 2 01:03:32 2021 Image Type: PowerPC Linux Kernel Image (gzip compressed) Data Size: 3869907 Bytes = 3779.21 KiB = 3.69 MiB Load Address: 00900000 Entry Point: 00900274 Image Name: Linux-5.12.0-rc1-ge3d773ddb5a1 Created: Tue Mar 2 01:03:32 2021 Image Type: PowerPC Linux Kernel Image (gzip compressed) Data Size: 3870022 Bytes = 3779.32 KiB = 3.69 MiB Load Address: 00900000 Entry Point: 00900274 Image Name: Linux-5.12.0-rc1-ge3d773ddb5a1 Created: Tue Mar 2 01:03:32 2021 Image Type: PowerPC Linux Kernel Image (gzip compressed) Data Size: 3868788 Bytes = 3778.11 KiB = 3.69 MiB Load Address: 00900000 Entry Point: 00900274 Completed OK # rm -rf /kisskb/build/powerpc-fixes_mpc5200_defconfig_powerpc-gcc4.9 # Build took: 0:01:10.143363