[PATCH] selftests/net: openvswitch: Fix spelling error in print message

Madhur Kumar posted 1 patch 1 month, 2 weeks ago
tools/testing/selftests/net/openvswitch/ovs-dpctl.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] selftests/net: openvswitch: Fix spelling error in print message
Posted by Madhur Kumar 1 month, 2 weeks ago
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

Re: [PATCH] selftests/net: openvswitch: Fix spelling error in print message
Posted by Simon Horman 1 month, 1 week ago
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
Re: [ovs-dev] [PATCH] selftests/net: openvswitch: Fix spelling error in print message
Posted by Simon Horman 1 month ago
+ 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.