linux-kselftest.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] selftests/net: openvswitch: Fix spelling error in print message
@ 2025-08-14 18:00 Madhur Kumar
  2025-08-26 17:34 ` Simon Horman
  0 siblings, 1 reply; 3+ messages in thread
From: Madhur Kumar @ 2025-08-14 18:00 UTC (permalink / raw)
  To: aconole, echaudro, i.maximets, davem, edumazet, kuba, pabeni,
	shuah
  Cc: horms, netdev, dev, linux-kselftest, linux-kernel, Madhur Kumar

Fix a typo in a print statement in ovs-dpctl.py:
"Unkonwn" → "Unknown".
 
---
 tools/testing/selftests/net/openvswitch/ovs-dpctl.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/net/openvswitch/ovs-dpctl.py b/tools/testing/selftests/net/openvswitch/ovs-dpctl.py
index 8a0396bfaf99..b521e0dea506 100644
--- a/tools/testing/selftests/net/openvswitch/ovs-dpctl.py
+++ b/tools/testing/selftests/net/openvswitch/ovs-dpctl.py
@@ -1877,7 +1877,7 @@ class OvsPacket(GenericNetlinkSocket):
                     elif msg["cmd"] == OvsPacket.OVS_PACKET_CMD_EXECUTE:
                         up.execute(msg)
                     else:
-                        print("Unkonwn cmd: %d" % msg["cmd"])
+                        print("Unknown cmd: %d" % msg["cmd"])
             except NetlinkError as ne:
                 raise ne
 
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] selftests/net: openvswitch: Fix spelling error in print message
  2025-08-14 18:00 [PATCH] selftests/net: openvswitch: Fix spelling error in print message Madhur Kumar
@ 2025-08-26 17:34 ` Simon Horman
  2025-08-29 13:14   ` [ovs-dev] " Simon Horman
  0 siblings, 1 reply; 3+ messages in thread
From: Simon Horman @ 2025-08-26 17:34 UTC (permalink / raw)
  To: Madhur Kumar
  Cc: aconole, echaudro, i.maximets, davem, edumazet, kuba, pabeni,
	shuah, netdev, dev, linux-kselftest, linux-kernel

On Thu, Aug 14, 2025 at 11:30:01PM +0530, Madhur Kumar wrote:
> Fix a typo in a print statement in ovs-dpctl.py:
> "Unkonwn" → "Unknown".

Hi Madhur,

You need to include a Signed-off-by line here.

Please see [1] and [2].

[1] https://docs.kernel.org/process/submitting-patches.html#developer-s-certificate-of-origin-1-1
[2] https://docs.kernel.org/process/maintainer-netdev.html#changes-requested

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [ovs-dev] [PATCH] selftests/net: openvswitch: Fix spelling error in print message
  2025-08-26 17:34 ` Simon Horman
@ 2025-08-29 13:14   ` Simon Horman
  0 siblings, 0 replies; 3+ messages in thread
From: Simon Horman @ 2025-08-29 13:14 UTC (permalink / raw)
  To: Madhur Kumar, dev, netdev, linux-kernel, i.maximets, edumazet,
	linux-kselftest, kuba, pabeni, shuah, davem
  Cc: Aaron Conole, Praveen Balakrishnan

+ Aaron Conole, Praveen Balakrishnan

On Tue, Aug 26, 2025 at 06:34:15PM +0100, Simon Horman via dev wrote:
> On Thu, Aug 14, 2025 at 11:30:01PM +0530, Madhur Kumar wrote:
> > Fix a typo in a print statement in ovs-dpctl.py:
> > "Unkonwn" → "Unknown".
> 
> Hi Madhur,
> 
> You need to include a Signed-off-by line here.
> 
> Please see [1] and [2].
> 
> [1] https://docs.kernel.org/process/submitting-patches.html#developer-s-certificate-of-origin-1-1
> [2] https://docs.kernel.org/process/maintainer-netdev.html#changes-requested

Hi again,

Aaron (CCed) brought to my attention that a superset of this patch has been
posted by Praveen Balakrishnan.

- [PATCH] selftests: net: fix spelling and grammar mistakes
  https://lore.kernel.org/netdev/20250828211100.51019-1-praveen.balakrishnan@magd.ox.ac.uk/

I'd like to suggest that we keep things simple and go with that patch.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-08-29 13:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-14 18:00 [PATCH] selftests/net: openvswitch: Fix spelling error in print message Madhur Kumar
2025-08-26 17:34 ` Simon Horman
2025-08-29 13:14   ` [ovs-dev] " Simon Horman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).