From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B2C05CD68F9 for ; Tue, 10 Oct 2023 06:58:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1442385AbjJJG6q convert rfc822-to-8bit (ORCPT ); Tue, 10 Oct 2023 02:58:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47426 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1442319AbjJJG6p (ORCPT ); Tue, 10 Oct 2023 02:58:45 -0400 X-Greylist: delayed 602 seconds by postgrey-1.37 at lindbergh.monkeyblade.net; Mon, 09 Oct 2023 23:58:43 PDT Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8DB0499; Mon, 9 Oct 2023 23:58:43 -0700 (PDT) Received: from omf06.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 9C649140199; Tue, 10 Oct 2023 06:48:39 +0000 (UTC) Received: from [HIDDEN] (Authenticated sender: joe@perches.com) by omf06.hostedemail.com (Postfix) with ESMTPA id 379272000F; Tue, 10 Oct 2023 06:48:11 +0000 (UTC) Message-ID: Subject: Re: [PATCH 6/7] fs/sysfs/group: make attribute_group pointers const From: Joe Perches To: Max Kellermann , Guenter Roeck Cc: Jens Axboe , Greg Kroah-Hartman , "Rafael J. Wysocki" , Borislav Petkov , Tony Luck , James Morse , Mauro Carvalho Chehab , Robert Richter , Jean Delvare , Jason Gunthorpe , Leon Romanovsky , Bart Van Assche , Joerg Roedel , Will Deacon , Robin Murphy , Dan Williams , Vishal Verma , Dave Jiang , Ira Weiny , Keith Busch , Christoph Hellwig , Sagi Grimberg , Alessandro Zummo , Alexandre Belloni , Jiri Slaby , Suzuki K Poulose , Mike Leach , James Clark , Leo Yan , Thomas Gleixner , Peter Zijlstra , Pavel Machek , Lee Jones , Arnd Bergmann , Bjorn Helgaas , Ingo Molnar , Arnaldo Carvalho de Melo , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , Ian Rogers , Adrian Hunter , Sebastian Reichel , Krzysztof Kozlowski , Wim Van Sebroeck , "James E.J. Bottomley" , "Martin K. Petersen" , linux-block@vger.kernel.org, linux-kernel@vger.kernel.org, linux-edac@vger.kernel.org, linux-hwmon@vger.kernel.org, linux-rdma@vger.kernel.org, iommu@lists.linux.dev, nvdimm@lists.linux.dev, linux-nvme@lists.infradead.org, linux-rtc@vger.kernel.org, linux-serial@vger.kernel.org, coresight@lists.linaro.org, linux-arm-kernel@lists.infradead.org, linux-leds@vger.kernel.org, linux-pci@vger.kernel.org, linux-perf-users@vger.kernel.org, linux-pm@vger.kernel.org, linux-usb@vger.kernel.org, linux-watchdog@vger.kernel.org, linux-scsi@vger.kernel.org Date: Mon, 09 Oct 2023 23:48:10 -0700 In-Reply-To: References: <20231009165741.746184-1-max.kellermann@ionos.com> <20231009165741.746184-6-max.kellermann@ionos.com> <264fa39d-aed6-4a54-a085-107997078f8d@roeck-us.net> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT User-Agent: Evolution 3.48.4 (3.48.4-1.fc38) MIME-Version: 1.0 X-Stat-Signature: x66gji9pr8zwbank5ef871m9qoojjjfr X-Rspamd-Server: rspamout04 X-Rspamd-Queue-Id: 379272000F X-Session-Marker: 6A6F6540706572636865732E636F6D X-Session-ID: U2FsdGVkX19bm9nHqj2XrX3TKwatvC/C1GajNvD5h9I= X-HE-Tag: 1696920491-749924 X-HE-Meta: U2FsdGVkX1/+029EmPc9sKVNFK2VRUjBCte/A2E+5y43hxFCP+MAKFoIxkcikfn6Jucu52vLqnvIqQDvdm6WvqlwnI8PH42O3yPwjrf6/Rk44z8SFUR40tWN1QdYZq3HPOEvKn0PLMPTqRRMr7zlpFwkMW85ySQYIyr+J8E4OP6W/LoiP3zYhR+0cmnV5ToftIwEMltweE+g/zUx+NL1fSjTbqk3Sf4w33VMSw7k9oncskJ8CUN/bQZEOiHhpNbgGpLqoQJ7ob3InQymsj8a//o/6ldtQbuoxFCk/RnYuGfC0EahOFEMbfIs5+kJ0g+hK4chvkcdMx/L2QmQgmCvPhqiadvx8BYdkeJ2IpCV5Y5g+W85x67f7ON0uqOVXWab Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org On Mon, 2023-10-09 at 22:05 +0200, Max Kellermann wrote: > On Mon, Oct 9, 2023 at 7:24 PM Guenter Roeck wrote: > > Also, I don't know why checkpatch is happy with all the > > > > const struct attribute_group *const*groups; > > > > instead of > > > > const struct attribute_group *const *groups; > > I found out that checkpatch has no check for this at all; it does > complain about such lines, but only for local variables. But that > warning is actually a bug, because this is a check for unary > operators: it thinks the asterisk is a dereference operator, not a > pointer declaration, and complains that the unary operator must be > preceded by a space. Thus warnings on local variable are only correct > by coincidence, not by design. > > Inside structs or parameters (where my coding style violations can be > found), it's a different context and thus checkpatch doesn't apply the > rules for unary operators. My opinion is that const use in the kernel should almost always have whitespace before and after it except when preceded by a open parenthesis or a newline. $ git grep -wh const -- '*.[ch]' | \ grep -oP "[ \*\(]?const[ \*]?" | \ sort | uniq -c | sort -rn 222438 const 83386 const 51667 (const 2766 *const 834 const 442 const 343 const* 88 *const 37 (const 4 *const*