Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 28 Oct 2022 17:07:02 +0300
From:   Dan Carpenter <dan.carpenter@oracle.com>
To:     Bagas Sanjaya <bagasdotme@gmail.com>
Cc:     =?utf-8?B?0L3QsNCx?= <nabijaczleweli@nabijaczleweli.xyz>,
        Jonathan Corbet <corbet@lwn.net>,
        Federico Vaga <federico.vaga@vaga.pv.it>,
        Alex Shi <alexs@kernel.org>,
        Yanteng Si <siyanteng@loongson.cn>,
        Hu Haowen <src.res@email.cn>,
        Thomas Sailer <t.sailer@alumni.ethz.ch>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub =?utf-8?B?S2ljacWEc2tp?= <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Jiri Slaby <jirislaby@kernel.org>, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        linux-doc-tw-discuss@lists.sourceforge.net,
        linux-hams@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: [PATCH 01/15] hamradio: baycom: remove BAYCOM_MAGIC
Message-ID: <Y1viBi7DAAJAn7kP@kadam>
References: <9a453437b5c3b4b1887c1bd84455b0cc3d1c40b2.1666822928.git.nabijaczleweli@nabijaczleweli.xyz>
 <47c2bffb-6bfe-7f5d-0d2d-3cbb99d31019@gmail.com>
 <Y1vccrsHSnF1QOIb@kadam>
 <61febb47-28a4-3343-081c-4c06b87ba870@gmail.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <61febb47-28a4-3343-081c-4c06b87ba870@gmail.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?IWJm1kgM0vpH9DA7ts3StuWrIA4sCuRDpOU3T58xSQ6AmXC7Jugg32bJgMd1?=
 =?us-ascii?Q?MvnRpY00Y/dG8iMAQUZImMg/jJZVEcdBvHQSUcmIMBQwzjjp68bUUBlsLGAU?=
 =?us-ascii?Q?EkYRUErmjZI7l5ddFpn93Iwv5km+OZP8oK2tAb8vMOC7C3NSwcqve+GGy2Kz?=
 =?us-ascii?Q?7rNfdmAMdlzrBoiiPkssn+wcIsDKQc7QlysarHZK5kqnHAyc50UtRL7J8Hrb?=
 =?us-ascii?Q?HLo02g7nMv8js2Va4E23RDHcKvxofcd96oDoDUAUCRwd/S6vpP0OG2QWkUon?=
 =?us-ascii?Q?F/UBxEzpPTyekZm4uSzQjM5Dq1dsEfRAYyP/K89M0rt/YXgpXgXysodF4jG7?=
 =?us-ascii?Q?l5AydknNSoYjGyR+YxlgkEVvo3+RFn+il5vTKsrniUIQ1d5oZZker8RDT5Jf?=
 =?us-ascii?Q?sK7iVAiKsSdb/tIVfuQAD2VpCHAbegXsIIFSmuy5y6LpQyeQoMB0Y0Rzt2z4?=
 =?us-ascii?Q?oky2Ahlq1f/fyu/0VF+Egg+M+qdz/vdtq2P6LcD0v0jXJk4T07NXAUPMZDE5?=
 =?us-ascii?Q?MZDG87SgYKgevTy+MSxyhTzyuVO2zF0W1EAZlnI4LgRJ2QIRHNyoFLeon0B5?=
 =?us-ascii?Q?D1/9y0JFbGFcEPwj9sC5XWDcu0DERhL4FE39+xwkJnMYxZQGw0m0diVjAKuF?=
 =?us-ascii?Q?qelkNslI/5B+d1YM079U/1/qEvzJY/MENJ5Ilk1GRMGiWOUGhiwxIGFwDrIf?=
 =?us-ascii?Q?ndC4pthCmYfE5dgXrQl/nmLd3rYvFn52kYik2Umpb7wSp40zVyq9zAFmJrZg?=
 =?us-ascii?Q?YVhsFtNxQ6BwZK7kzONX9Sm5ZrZm4oIHChP5rYQfd3Y3QL1mEOx/PxuIj8T1?=
 =?us-ascii?Q?r7JeF8Cso1d+Gp+9J3THs2H2YLpzav6HBgmupephkeRpOcjW44IaS7OOwHC2?=
 =?us-ascii?Q?JMpYkQrOzP96FBCTt9WfvfvgiLpshodfmZxU5JUNviPIz+BHVJYikVqTwL0J?=
 =?us-ascii?Q?OrQH7x6eF5JbxbU5HmEJvJFWTpA2EHklKp3/mMUy5vUGTpEGOhHZOhBzlYfV?=
 =?us-ascii?Q?3VMRqohAFhft05CqdAqFo9uuHH5i+YRXuSutqTbnpkBNBMxoyCMlkgjUfyVI?=
 =?us-ascii?Q?dxWc47Xh5oCUKZr/i7K+lQrYNpZPdw7V81z/hZsXoI9QXIbqeQXq95ebfFuo?=
 =?us-ascii?Q?GcKR82DVQhp+ldA1AOXUOgznrJlWHjjB+HZ1nQlhXaxDEOBw66494esi3yOC?=
 =?us-ascii?Q?VAzx7zREVa63se1asdWcSHKvOVGxaNnVNzQCmNbuUnJjc6nI2xnP0Bw84IHO?=
 =?us-ascii?Q?cpnVjPfrViKeoJP2UXEmGwpVetG/v0VTRJ+6uEnX1qMezHUQQOEK4FWtGMtN?=
 =?us-ascii?Q?IQ+iGUX3wufNt6LRgkNTu+eKgKNoWpzn+VvV5kyKCfu9nEEQQWRiNn0gwJ+7?=
 =?us-ascii?Q?MSK+xFQajftODOt2kyQwEQCnniCOJEyhrKI+2D+kXXtuFZlJdu6eok0WwVpc?=
 =?us-ascii?Q?L5AbofA3qA0pNKHNKhTO7vSgA6fcyDlYb+k8ogkzjWAG7ayovvAU3+5NiDon?=
 =?us-ascii?Q?mSOP/i38t/Ps26UM5o4kJhjMDiq4c1TV5SycJQGiDQHU4+ORPhOY0hFMorPe?=
 =?us-ascii?Q?XpF8sNQa2jt0HLz5C66lWXiPMBpuGvLBMZxLyC0+v3DUmcG9mnFpGgjnxX6B?=
 =?us-ascii?Q?pg=3D=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0ae65770-0a78-490b-386a-08dab8edb97a
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Oct 2022 14:07:18.9275
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: zvL3UMzYjaBZswe5GrWlDsQDDizuzyAJppKnms0yRCJsQtbNnqgEi1XT/LSntpuuVfEi2lM3R4kM9qZ3njl1QzP08/4HV3HAEZseYHJgzDA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR10MB4221
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.895,Hydra:6.0.545,FMLib:17.11.122.1
 definitions=2022-10-28_07,2022-10-27_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 adultscore=0 mlxscore=0 mlxlogscore=759
 spamscore=0 phishscore=0 bulkscore=0 malwarescore=0 suspectscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2210170000
 definitions=main-2210280088
X-Proofpoint-GUID: brr0eHMmY66ho7Gho_oV3ufpix3-UiQr
X-Proofpoint-ORIG-GUID: brr0eHMmY66ho7Gho_oV3ufpix3-UiQr
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Oct 28, 2022 at 08:50:59PM +0700, Bagas Sanjaya wrote:
> On 10/28/22 20:43, Dan Carpenter wrote:
> >>
> >> Also, s/Kill it/Remove BAYCOM_MAGIC from magic numbers table/ (your
> >> wording is kinda mature).
> >>
> > 
> > The kernel has almost 13 thousand kills...
> > 
> > $ git grep -i kill | wc -l
> > 12975
> > $
> > 
> > It's fine.
> > 
> 
> The word meaning depends on context. In this case, the author means
> removing SOME_MAGIC magic number from the table, one by one until
> the magic number documentation is removed (due to historical cruft).

Kernel devs are naturally blood thirsty people...

$ git log --after=2022-01-01 | grep -w kill | wc -l
207

When people talk about killing stuff they mostly mean deleting code.
Look at this sample of the very first kills from January.  Seven out
of ten times this is what they meant.

$ git log --after=2022-01-01 | grep -w kill | head -n 10
    useless now, kill it.
    net: ipa: kill ipa_table_valid()
    net: ipa: kill two constant symbols
      io_uring: kill hot path fixed file bitmap debug checks
                   newattrs.ia_valid = ATTR_FORCE | kill;
                newattrs.ia_valid = ATTR_FORCE | kill;
                        newattrs.ia_valid = attr_force | kill;
    kill it off before it ends up in a release. It was just part of the
    io_uring: kill hot path fixed file bitmap debug checks
      block: kill deprecated BUG_ON() in the flush handling
$

regards,
dan carpenter