diff options
author | Damien Le Moal <dlemoal@kernel.org> | 2023-06-23 16:30:57 +0900 |
---|---|---|
committer | Martin K. Petersen <martin.petersen@oracle.com> | 2023-06-28 21:47:09 -0400 |
commit | 71e3e85ccf2b816e612c94b7460309dc5007caef (patch) | |
tree | c77309c8637db7a6d6a2613ad2785d371bd3676c /fs/buffer.c | |
parent | 4e45236982bcc3ce8a0ea719e1159cc5c935eb94 (diff) | |
download | linux-stable-71e3e85ccf2b816e612c94b7460309dc5007caef.tar.gz linux-stable-71e3e85ccf2b816e612c94b7460309dc5007caef.tar.bz2 linux-stable-71e3e85ccf2b816e612c94b7460309dc5007caef.zip |
scsi: core: Simplify scsi_cdl_check_cmd()
Reading the 800+ pages of SPC often leads to a brain shutdown and to less
than ideal code... This resulted in the checks of the rwcdlp and cdlp
fields in scsi_cdl_check_cmd() to have identical if-else branches.
Replace this with a comment describing the cases we are interested in and
replace the if-else code block with a simple test of the cdlp field that is
used as the function return value.
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Julia Lawall <julia.lawall@inria.fr>
Closes: https://lore.kernel.org/r/202306221657.BJHEADkz-lkp@intel.com/
Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
Link: https://lore.kernel.org/r/20230623073057.816199-1-dlemoal@kernel.org
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'fs/buffer.c')
0 files changed, 0 insertions, 0 deletions