From c5e515b7026872ccf8cd72f163e7929dd0936722 Mon Sep 17 00:00:00 2001 From: Bart Van Assche Date: Thu, 29 Apr 2021 03:15:01 +0000 Subject: [PATCH] Address checkpatch complaints about duplicated words git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@9383 d57e44dd-8a1f-0410-8b47-8ef2f437770f --- iscsi-scst/README | 6 +++--- iscsi-scst/README_in-tree | 6 +++--- scripts/kernel-functions | 2 +- scst/include/scst.h | 2 +- scst/src/scst_lib.c | 4 ++-- 5 files changed, 10 insertions(+), 10 deletions(-) diff --git a/iscsi-scst/README b/iscsi-scst/README index 08d5b5c21..330743243 100644 --- a/iscsi-scst/README +++ b/iscsi-scst/README @@ -183,7 +183,7 @@ is /sys/kernel/scst_tgt/targets/iscsi. It has the following entries: - allowed_portal[num] - optional attribute, which specifies, on which portals (target's IP addresses) this target will be available. If not - specified (default) the target will be available on all all portals. + specified (default) the target will be available on all portals. As soon as at least one allowed_portal specified, the target will be accessible for initiators only on the specified portals. There might be any number of the allowed_portal attributes. The portals @@ -738,7 +738,7 @@ echo 'add_target_attribute iqn.2006-10.net.vlnb:tgt allowed_portal !*' >/sys/ker will disable all portals. -2. If you want to want to allow only only specific set of initiators be +2. If you want to want to allow only specific set of initiators be able to connect to your target, you should don't add any default LUNs for the target and create for allowed initiators a security group to which they will be assigned. @@ -852,7 +852,7 @@ simultaneously on both ports, i.e. they transfer data via both ports on the same speed as via any single port. Thus, using such adapters in MPIO configuration can't improve performance. To allow MPIO to have double performance you should either use separate network adapters, or find a -dual-port adapter capable to to transfer data simultaneously on both +dual-port adapter capable to transfer data simultaneously on both ports. You can check it by running 2 iperf's through both ports in parallel. diff --git a/iscsi-scst/README_in-tree b/iscsi-scst/README_in-tree index 50a6bb160..0413184fd 100644 --- a/iscsi-scst/README_in-tree +++ b/iscsi-scst/README_in-tree @@ -66,7 +66,7 @@ is /sys/kernel/scst_tgt/targets/iscsi. It has the following entries: - allowed_portal[num] - optional attribute, which specifies, on which portals (target's IP addresses) this target will be available. If not - specified (default) the target will be available on all all portals. + specified (default) the target will be available on all portals. As soon as at least one allowed_portal specified, the target will be accessible for initiators only on the specified portals. There might be any number of the allowed_portal attributes. The portals @@ -582,7 +582,7 @@ echo 'add_target_attribute iqn.2006-10.net.vlnb:tgt allowed_portal !*' >/sys/ker will disable all portals. -2. If you want to want to allow only only specific set of initiators be +2. If you want to want to allow only specific set of initiators be able to connect to your target, you should don't add any default LUNs for the target and create for allowed initiators a security group to which they will be assigned. @@ -694,7 +694,7 @@ simultaneously on both ports, i.e. they transfer data via both ports on the same speed as via any single port. Thus, using such adapters in MPIO configuration can't improve performance. To allow MPIO to have double performance you should either use separate network adapters, or find a -dual-port adapter capable to to transfer data simultaneously on both +dual-port adapter capable to transfer data simultaneously on both ports. You can check it by running 2 iperf's through both ports in parallel. diff --git a/scripts/kernel-functions b/scripts/kernel-functions index 81dbd7e19..3a6a0d552 100644 --- a/scripts/kernel-functions +++ b/scripts/kernel-functions @@ -690,7 +690,7 @@ time: This is because both dtc-lexer as well as dtc-parser define the same global symbol yyloc. Before with -fcommon those were merged into one -defintion. The proper solution would be to to mark this as "extern", +defintion. The proper solution would be to mark this as "extern", however that leads to: dtc-lexer.l:26:16: error: redundant redeclaration of 'yylloc' [-Werror=redundant-decls] diff --git a/scst/include/scst.h b/scst/include/scst.h index 019a2eff2..5cffea355 100644 --- a/scst/include/scst.h +++ b/scst/include/scst.h @@ -2341,7 +2341,7 @@ struct scst_cmd { uint8_t lba_off; /* LBA offset in cdb */ uint8_t lba_len; /* LBA length in cdb */ uint8_t len_off; /* length offset in cdb */ - uint8_t len_len; /* length length in cdb */ + uint8_t len_len; /* length of length in cdb */ /* If not zero, logarithm base 2 of the maximum data buffer length. */ uint8_t log2_max_buf_len; uint32_t op_flags; /* various flags of this opcode */ diff --git a/scst/src/scst_lib.c b/scst/src/scst_lib.c index 2a0fdfb78..9c8d60c75 100644 --- a/scst/src/scst_lib.c +++ b/scst/src/scst_lib.c @@ -542,7 +542,7 @@ struct scst_sdbops { uint8_t info_lba_off; /* LBA offset in cdb */ uint8_t info_lba_len; /* LBA length in cdb */ uint8_t info_len_off; /* length offset in cdb */ - uint8_t info_len_len; /* length length in cdb */ + uint8_t info_len_len; /* length of length in cdb */ uint8_t info_data_direction; /* * init --> target: SCST_DATA_WRITE @@ -12171,7 +12171,7 @@ int scst_get_cdb_info(struct scst_cmd *cmd) ptr->devkey[9], /* commdev */ ptr->info_op_name); TRACE_DBG("data direction %d, op flags 0x%x, lba off %d, " - "lba len %d, len off %d, len len %d", + "lba len %d, len off %d, len of len %d", ptr->info_data_direction, ptr->info_op_flags, ptr->info_lba_off, ptr->info_lba_len, ptr->info_len_off, ptr->info_len_len);