Thread (3 messages) 3 messages, 2 authors, 2025-02-18

Re: [PATCH REPOST] selftests: net: Fix minor typos in MPTCP and psock_tpacket tests

From: Suchit K <hidden>
Date: 2025-02-18 16:44:31
Also in: linux-kernel-mentees, lkml

Hi Matthieu. I'm sorry I was using Gmail and I realized from the
documentation that it shouldn't be used. I've configured git
send-email and will send the patch using that. Apologies for any
inconvenience and thanks for the feedback.

On Tue, 18 Feb 2025 at 15:48, Matthieu Baerts [off-list ref] wrote:
Hi Suchit,

On 18/02/2025 10:28, Suchit K wrote:
quoted
Fixes minor spelling errors:
- `simult_flows.sh`: "al testcases" -> "all testcases"
- `psock_tpacket.c`: "accross" -> "across"

Signed-off-by: Suchit Karunakaran <redacted>
---
 tools/testing/selftests/net/mptcp/simult_flows.sh | 2 +-
 tools/testing/selftests/net/psock_tpacket.c       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/testing/selftests/net/mptcp/simult_flows.sh
b/tools/testing/selftests/net/mptcp/simult_flows.sh
The patch is exactly the same as v1, containing the same issues: long
lines are wrapped, corrupting the patch, like here above.
quoted
index 9c2a41597..2329c2f85 100755
--- a/tools/testing/selftests/net/mptcp/simult_flows.sh
+++ b/tools/testing/selftests/net/mptcp/simult_flows.sh
@@ -28,7 +28,7 @@ size=0

 usage() {
  echo "Usage: $0 [ -b ] [ -c ] [ -d ] [ -i]"
- echo -e "\t-b: bail out after first error, otherwise runs al testcases"
+ echo -e "\t-b: bail out after first error, otherwise runs all testcases"
  echo -e "\t-c: capture packets for each test using tcpdump (default:
no capture)"
Same here
quoted
  echo -e "\t-d: debug this script"
  echo -e "\t-i: use 'ip mptcp' instead of 'pm_nl_ctl'"
diff --git a/tools/testing/selftests/net/psock_tpacket.c
b/tools/testing/selftests/net/psock_tpacket.c
Same here
quoted
index 404a2ce75..221270cee 100644
--- a/tools/testing/selftests/net/psock_tpacket.c
+++ b/tools/testing/selftests/net/psock_tpacket.c
@@ -12,7 +12,7 @@
  *
  * Datapath:
  *   Open a pair of packet sockets and send resp. receive an a priori known
- *   packet pattern accross the sockets and check if it was received resp.
+ *   packet pattern across the sockets and check if it was received resp.
  *   sent correctly. Fanout in combination with RX_RING is currently not
  *   tested here.
  *
How are you sending this patch? Using 'git send-email' following
instructions like the ones from [1]?

I do recommend using b4 to prepare and send patches, see [2].

[1] https://git-send-email.io
[2] https://b4.docs.kernel.org/en/latest/contributor/overview.html

Cheers,
Matt
--
Sponsored by the NGI0 Core fund.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help