[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20240430124116.vcgeggkcwtbbvibm@joelS2.panther.com>
Date: Tue, 30 Apr 2024 14:41:16 +0200
From: Joel Granados <j.granados@...sung.com>
To: Sabrina Dubroca <sd@...asysnail.net>
CC: "David S. Miller" <davem@...emloft.net>, Eric Dumazet
<edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni
<pabeni@...hat.com>, Alexander Aring <alex.aring@...il.com>, Stefan Schmidt
<stefan@...enfreihafen.org>, Miquel Raynal <miquel.raynal@...tlin.com>,
David Ahern <dsahern@...nel.org>, Steffen Klassert
<steffen.klassert@...unet.com>, Herbert Xu <herbert@...dor.apana.org.au>,
Matthieu Baerts <matttbe@...nel.org>, Mat Martineau <martineau@...nel.org>,
Geliang Tang <geliang@...nel.org>, Remi Denis-Courmont
<courmisch@...il.com>, Allison Henderson <allison.henderson@...cle.com>,
David Howells <dhowells@...hat.com>, Marc Dionne <marc.dionne@...istor.com>,
Marcelo Ricardo Leitner <marcelo.leitner@...il.com>, Xin Long
<lucien.xin@...il.com>, Wenjia Zhang <wenjia@...ux.ibm.com>, Jan Karcher
<jaka@...ux.ibm.com>, "D. Wythe" <alibuda@...ux.alibaba.com>, Tony Lu
<tonylu@...ux.alibaba.com>, Wen Gu <guwen@...ux.alibaba.com>, Trond
Myklebust <trond.myklebust@...merspace.com>, Anna Schumaker
<anna@...nel.org>, Chuck Lever <chuck.lever@...cle.com>, Jeff Layton
<jlayton@...nel.org>, Neil Brown <neilb@...e.de>, Olga Kornievskaia
<kolga@...app.com>, Dai Ngo <Dai.Ngo@...cle.com>, Tom Talpey
<tom@...pey.com>, Jon Maloy <jmaloy@...hat.com>, Ying Xue
<ying.xue@...driver.com>, Martin Schiller <ms@....tdt.de>, Pablo Neira Ayuso
<pablo@...filter.org>, Jozsef Kadlecsik <kadlec@...filter.org>, Florian
Westphal <fw@...len.de>, Roopa Prabhu <roopa@...dia.com>, Nikolay
Aleksandrov <razor@...ckwall.org>, Simon Horman <horms@...ge.net.au>, Julian
Anastasov <ja@....bg>, Joerg Reuter <jreuter@...na.de>, Luis Chamberlain
<mcgrof@...nel.org>, Kees Cook <keescook@...omium.org>,
<netdev@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<dccp@...r.kernel.org>, <linux-wpan@...r.kernel.org>,
<mptcp@...ts.linux.dev>, <linux-hams@...r.kernel.org>,
<linux-rdma@...r.kernel.org>, <rds-devel@....oracle.com>,
<linux-afs@...ts.infradead.org>, <linux-sctp@...r.kernel.org>,
<linux-s390@...r.kernel.org>, <linux-nfs@...r.kernel.org>,
<tipc-discussion@...ts.sourceforge.net>, <linux-x25@...r.kernel.org>,
<netfilter-devel@...r.kernel.org>, <coreteam@...filter.org>,
<bridge@...ts.linux.dev>, <lvs-devel@...r.kernel.org>
Subject: Re: [PATCH v5 1/8] net: Remove the now superfluous sentinel
elements from ctl_table array
On Mon, Apr 29, 2024 at 04:49:18PM +0200, Sabrina Dubroca wrote:
> 2024-04-29, 14:33:15 +0200, Joel Granados wrote:
> > On Mon, Apr 29, 2024 at 10:53:47AM +0200, Sabrina Dubroca wrote:
> > > 2024-04-26, 12:46:53 +0200, Joel Granados via B4 Relay wrote:
...
> > > > {
> > > > + size_t table_size = ARRAY_SIZE(mpls_table);
> > >
> > > This table still has a {} as its final element. It should be gone too?
> > Now, how did that get away? I'll run my coccinelle scripts once more to
> > make sure that I don't have more of these hiding in the shadows.
>
> I didn't spot any other with a dumb
>
> sed -n '<line>,^};/p' <file>
I used a coccinelle script:
* ran it with `make coccicheck MODE=patch SPFLAGS="--in-place --debug" COCCI=script.cocci`
* script:
```
virtual patch
@r1@
identifier ctl_table_name;
@@
static struct ctl_table ctl_table_name[] = {
...
-, {}
};
```
and a gawk script
* ran it with `for f in $(git grep -l "struct ctl_table") ; do script $f ; done`
* script:
```
#!/usr/bin/gawk -f
BEGINFILE {
RS=","
has_struct = 0
}
/(static )?(const )?struct ctl_table/ {
has_struct = 1
}
has_struct && /^(\n)?[\t ]*{(\n)*[\t ]*}/ {
print "Filename : " FILENAME ", Record Number : " FNR
}
```
At this point the coccinelle script gives me too many false positives
but the gawk is spot on. Thx for the sed one. Will make a note of it.
Best
--
Joel Granados
Download attachment "signature.asc" of type "application/pgp-signature" (660 bytes)
Powered by blists - more mailing lists