From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: linux-kernel@vger.kernel.org,
Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Subject: [RFC PATCH 00/13] Fix parentheses around macro parameter use in headers
Date: Thu, 4 May 2023 16:05:14 -0400 [thread overview]
Message-ID: <20230504200527.1935944-1-mathieu.desnoyers@efficios.com> (raw)
I tried to look at various list and list user headers in the kernel tree
to figure out what the proper parentheses pattern should be around macro
parameter use for new code, and it turns out that the current code base
lacks consistency.
This is not an exhaustive change of all public kernel headers, but at
least it is a start, updating those which are implementing or using
kernel lists.
The basic rules followed here are:
- Use parentheses around arguments which are rvalues, except when those
are expressions between commas "," used as arguments to functions or
other macros, or surrounded by brackets "[]".
- Do not use parentheses around arguments which are lvalues.
For consistency, when a macro argument is used both as an lvalue and as
an rvalue within the macro, use the parentheses rules applying to each
of the context: with parentheses for rvalue, without parentheses for
lvalue.
Mathieu Desnoyers (13):
rcu: rcupdate.h: Fix parentheses around macro parameter use
rculist.h: Fix parentheses around macro pointer parameter use
rculist_nulls.h: Add parentheses around macro pointer parameter use
rculist_bl.h: Fix parentheses around macro pointer parameter use
list.h: Fix parentheses around macro pointer parameter use
list_nulls.h: Fix parentheses around macro pointer parameter use
list_bl.h: Fix parentheses around macro pointer parameter use
llist.h: Fix parentheses around macro pointer parameter use
klist.h: Fix parentheses around macro parameter use
resource_ext.h: Remove useless parentheses around macro parameters
netdevice.h: Fix parentheses around macro parameter use
blk-mq.h: Fix parentheses around macro parameter use
bio.h: Fix parentheses around macro parameter use
include/linux/bio.h | 22 +++++++-------
include/linux/blk-mq.h | 38 ++++++++++++------------
include/linux/klist.h | 8 +++---
include/linux/list.h | 54 +++++++++++++++++------------------
include/linux/list_bl.h | 12 ++++----
include/linux/list_nulls.h | 8 +++---
include/linux/llist.h | 14 ++++-----
include/linux/netdevice.h | 12 ++++----
include/linux/rculist.h | 28 +++++++++---------
include/linux/rculist_bl.h | 6 ++--
include/linux/rculist_nulls.h | 4 +--
include/linux/rcupdate.h | 46 ++++++++++++++---------------
include/linux/resource_ext.h | 6 ++--
13 files changed, 129 insertions(+), 129 deletions(-)
--
2.25.1
next reply other threads:[~2023-05-04 21:01 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-04 20:05 Mathieu Desnoyers [this message]
2023-05-04 20:05 ` [RFC PATCH 01/13] rcu: rcupdate.h: Fix parentheses around macro parameter use Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 02/13] rculist.h: Fix parentheses around macro pointer " Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 03/13] rculist_nulls.h: Add " Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 04/13] rculist_bl.h: Fix " Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 05/13] list.h: " Mathieu Desnoyers
2023-05-08 12:16 ` Andy Shevchenko
2023-05-08 13:46 ` Mathieu Desnoyers
2023-05-12 11:02 ` Andy Shevchenko
2023-05-12 14:32 ` Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 06/13] list_nulls.h: " Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 07/13] list_bl.h: " Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 08/13] llist.h: " Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 09/13] klist.h: Fix parentheses around macro " Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 10/13] resource_ext.h: Remove useless parentheses around macro parameters Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 11/13] netdevice.h: Fix parentheses around macro parameter use Mathieu Desnoyers
2023-05-05 18:44 ` Jakub Kicinski
2023-05-05 18:54 ` Mathieu Desnoyers
2023-05-04 20:05 ` [RFC PATCH 12/13] blk-mq.h: " Mathieu Desnoyers
2023-05-05 13:56 ` Mathieu Desnoyers
2023-05-05 18:40 ` Linus Torvalds
2023-05-05 18:49 ` Mathieu Desnoyers
2023-05-05 19:54 ` Linus Torvalds
2023-05-05 20:08 ` Mathieu Desnoyers
2023-05-05 20:22 ` Linus Torvalds
2023-05-05 20:28 ` Mathieu Desnoyers
2023-05-08 14:28 ` Mathieu Desnoyers
2023-05-06 15:45 ` David Laight
2023-05-04 20:05 ` [RFC PATCH 13/13] bio.h: " Mathieu Desnoyers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230504200527.1935944-1-mathieu.desnoyers@efficios.com \
--to=mathieu.desnoyers@efficios.com \
--cc=akpm@linux-foundation.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®