# git rev-parse -q --verify 4525c8781ec0701ce824e8bd379ae1b129e26568^{commit}
4525c8781ec0701ce824e8bd379ae1b129e26568
already have revision, skipping fetch
# git checkout -q -f -B kisskb 4525c8781ec0701ce824e8bd379ae1b129e26568
# git clean -qxdf
# < git log -1
# commit 4525c8781ec0701ce824e8bd379ae1b129e26568
# Author: Linus Torvalds <torvalds@linux-foundation.org>
# Date:   Mon Oct 26 15:45:22 2020 -0700
# 
#     scsi: qla2xxx: remove incorrect sparse #ifdef
#     
#     The code to try to shut up sparse warnings about questionable locking
#     didn't shut up sparse: it made the result not parse as valid C at all,
#     since the end result now has a label with no statement.
#     
#     The proper fix is to just always lock the hardware, the same way Bart
#     did in commit 8ae178760b23 ("scsi: qla2xxx: Simplify the functions for
#     dumping firmware").  That avoids the whole problem with having locking
#     that is not statically obvious.
#     
#     But in the meantime, just remove the incorrect attempt at trying to
#     avoid a sparse warning that just made things worse.
#     
#     This was exposed by commit 3e6efab865ac ("scsi: qla2xxx: Fix reset of
#     MPI firmware"), very similarly to how commit cbb01c2f2f63 ("scsi:
#     qla2xxx: Fix MPI failure AEN (8200) handling") exposed the same problem
#     in another place, and caused that commit 8ae178760b23.
#     
#     Please don't add code to just shut up sparse without actually fixing
#     what sparse complains about.
#     
#     Reported-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com>
#     Cc: Bart Van Assche <bvanassche@acm.org>
#     Cc: Arun Easi <aeasi@marvell.com>
#     Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
# < /opt/cross/kisskb/korg/gcc-8.1.0-nolibc/s390-linux/bin/s390-linux-gcc --version
# < /opt/cross/kisskb/korg/gcc-8.1.0-nolibc/s390-linux/bin/s390-linux-ld --version
# < git log --format=%s --max-count=1 4525c8781ec0701ce824e8bd379ae1b129e26568
# < make -s -j 8 ARCH=s390 O=/kisskb/build/linus_s390-allyesconfig_s390x-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/s390-linux/bin/s390-linux-  allyesconfig
# Added to kconfig CONFIG_BUILD_DOCSRC=n
# Added to kconfig CONFIG_MODULE_SIG=n
# < make -s -j 8 ARCH=s390 O=/kisskb/build/linus_s390-allyesconfig_s390x-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/s390-linux/bin/s390-linux-  help
# make -s -j 8 ARCH=s390 O=/kisskb/build/linus_s390-allyesconfig_s390x-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/s390-linux/bin/s390-linux-  olddefconfig
# make -s -j 8 ARCH=s390 O=/kisskb/build/linus_s390-allyesconfig_s390x-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/s390-linux/bin/s390-linux-  
/kisskb/src/drivers/gpu/drm/rockchip/cdn-dp-core.c:1124:12: warning: 'cdn_dp_resume' defined but not used [-Wunused-function]
 static int cdn_dp_resume(struct device *dev)
            ^~~~~~~~~~~~~
/kisskb/src/drivers/input/joystick/analog.c:160:2: warning: #warning Precise timer not defined for this architecture. [-Wcpp]
 #warning Precise timer not defined for this architecture.
  ^~~~~~~
/kisskb/src/drivers/media/platform/marvell-ccic/mmp-driver.c:324:12: warning: 'mmpcam_runtime_suspend' defined but not used [-Wunused-function]
 static int mmpcam_runtime_suspend(struct device *dev)
            ^~~~~~~~~~~~~~~~~~~~~~
/kisskb/src/drivers/media/platform/marvell-ccic/mmp-driver.c:310:12: warning: 'mmpcam_runtime_resume' defined but not used [-Wunused-function]
 static int mmpcam_runtime_resume(struct device *dev)
            ^~~~~~~~~~~~~~~~~~~~~
In file included from /kisskb/src/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c:42:
/kisskb/src/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c: In function 'mvpp2_setup_bm_pool':
/kisskb/src/drivers/net/ethernet/marvell/mvpp2/mvpp2.h:760:2: warning: overflow in conversion from 'long unsigned int' to 'int' changes value from '18446744073709551584' to '-32' [-Woverflow]
  ((total_size) - MVPP2_SKB_HEADROOM - MVPP2_SKB_SHINFO_SIZE)
  ^
/kisskb/src/drivers/net/ethernet/marvell/mvpp2/mvpp2.h:864:33: note: in expansion of macro 'MVPP2_RX_MAX_PKT_SIZE'
 #define MVPP2_BM_SHORT_PKT_SIZE MVPP2_RX_MAX_PKT_SIZE(MVPP2_BM_SHORT_FRAME_SIZE)
                                 ^~~~~~~~~~~~~~~~~~~~~
/kisskb/src/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c:647:41: note: in expansion of macro 'MVPP2_BM_SHORT_PKT_SIZE'
  mvpp2_pools[MVPP2_BM_SHORT].pkt_size = MVPP2_BM_SHORT_PKT_SIZE;
                                         ^~~~~~~~~~~~~~~~~~~~~~~
Completed OK
# rm -rf /kisskb/build/linus_s390-allyesconfig_s390x-gcc8
# Build took: 0:30:36.151592