linux-kernel - Re: [PATCH] net: dsa: sja1105: Remove an unprofessional word from the code comments

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <-gMuTlvCAcfhIMqI1V4ylxBXKdWffUwznP_0cODJaam4l3hyfEFTLkZUZMLQV23BCeO8S6vTh1OfN0yOi7M5PhfX0rOX8SkL9JknVM2nKOA=@protonmail.com>
Date:   Wed, 09 Dec 2020 11:54:20 +0000
From:   Aswin C <realc@...tonmail.com>
To:     Vladimir Oltean <olteanv@...il.com>
Cc:     Aswin C <aswinraman2013@...il.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "David S. Miller" <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>, Andrew Lunn <andrew@...n.ch>,
        Florian Fainelli <f.fainelli@...il.com>,
        Vivien Didelot <vivien.didelot@...il.com>
Subject: Re: [PATCH] net: dsa: sja1105: Remove an unprofessional word from the code comments

Hello. Thank you very much for the reply.

Firstly, yes, it's my first contribution and apologies for making some mistakes and I'd like to thank you for being kind and pointing it out.

I came to know about this driver in a casual discussion among my peers when one of them pointed out the presence of this particular word on the driver. I just thought that I'd try to help and get rid of that, if possible. Nothing more.

As you mentioned, I ran my eyes over drivers/net/dsa/sja1105/sja1105_tas.c and the other files. Other than a very minor spelling error, I didn't come across wording conventions like the one we are discussing about.

And yes, as you mentioned, "poorly organized" sounds perfect, too.

I believe that not signing off and not sending it to all maintainers is all, in the things I did wrong. So, I hope I can fix them and send the patch as a reply to this thread.

Thank you.

‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Tuesday, December 8, 2020 6:05 PM, Vladimir Oltean <olteanv@...il.com> wrote:

&gt; Hi Aswin,
&gt;
&gt; On Tue, Dec 08, 2020 at 04:03:32PM +0530, Aswin C wrote:
&gt;
&gt; &gt; Remove the word 'retarded' from the code comments with a more professional word
&gt; &gt; 'erroneous' to make it less profane.
&gt; &gt;
&gt; &gt; ---------------------------------------------------------------------------------------------------------------------
&gt; &gt;
&gt; &gt; drivers/net/dsa/sja1105/sja1105_dynamic_config.c | 2 +-
&gt; &gt; 1 file changed, 1 insertion(+), 1 deletion(-)
&gt; &gt; diff --git a/drivers/net/dsa/sja1105/sja1105_dynamic_config.c b/drivers/net/dsa/sja1105/sja1105_dynamic_config.c
&gt; &gt; index b777d3f37..7a76180f3 100644
&gt; &gt; --- a/drivers/net/dsa/sja1105/sja1105_dynamic_config.c
&gt; &gt; +++ b/drivers/net/dsa/sja1105/sja1105_dynamic_config.c
&gt; &gt; @@ -250,7 +250,7 @@ sja1105pqrs_l2_lookup_cmd_packing(void *buf, struct sja1105_dyn_cmd *cmd,
&gt; &gt; SJA1105PQRS_SIZE_L2_LOOKUP_ENTRY, op);
&gt; &gt; }
&gt; &gt; -/* The switch is so retarded that it makes our command/entry abstraction
&gt; &gt; +/* The switch is so erroneous that it makes our command/entry abstraction
&gt; &gt;
&gt; &gt; -   crumble apart.
&gt; &gt; -
&gt; &gt; -   On P/Q/R/S, the switch tries to say whether a FDB entry
&gt; &gt;     --
&gt; &gt;     2.29.2
&gt; &gt;
&gt;
&gt; Thank you for the patch and for what looks like your first contribution
&gt; to the kernel. First of all, when you submit a patch, you should follow
&gt; the development process at:
&gt; https://www.kernel.org/doc/html/latest/process/submitting-patches.html
&gt; https://www.kernel.org/doc/Documentation/networking/netdev-FAQ.rst
&gt;
&gt; You should use ./scripts/get_maintainer.pl and send the email to all
&gt; maintainers that get listed there, plus the mailing list.
&gt;
&gt; I don't feel that "erroneous" is the right replacement. Maybe "poorly organized".
&gt;
&gt; Although I am not sure that "professional" is something that I was going
&gt; for when I wrote this driver.
&gt;
&gt; This makes me curious what exactly motivated you to make the change? Do
&gt; you feel offended by the use of profanity when used to describe hardware?
&gt; How did you even pick the sja1105 driver. I can think of two possibilities:
&gt;
&gt; -   You are working with it. But in this case, I do wonder how you feel by
&gt;     the larger amount of profanity in drivers/net/dsa/sja1105/sja1105_tas.c.
&gt;     Are you planning further patches for that?
&gt;
&gt; -   You just searched the kernel sources for profanity, or, putting it
&gt;     differently, you were actively trying to be offended.
&gt;

</olteanv@...il.com>