Re: [PATCH v2 00/14] sysctl: Add a size argument to register functions in sysctl

From: Joel Granados <j.granados@samsung.com>
To: Chris Maness <christopher.maness@gmail.com>
Cc: Luis Chamberlain <mcgrof@kernel.org>,
	Alexander Aring <alex.aring@gmail.com>,
	Alexander Gordeev <agordeev@linux.ibm.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Christian Borntraeger <borntraeger@linux.ibm.com>,
	"D. Wythe" <alibuda@linux.alibaba.com>,
	David Ahern <dsahern@kernel.org>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Florian Westphal <fw@strlen.de>,
	Heiko Carstens <hca@linux.ibm.com>,
	Herbert Xu <herbert@gondor.apana.org.au>,
	Iurii Zaikin <yzaikin@google.com>,
	Jakub Kicinski <kuba@kernel.org>,
	Jan Karcher <jaka@linux.ibm.com>, Joerg Reuter <jreuter@yaina.de>,
	Jozsef Kadlecsik <kadlec@netfilter.org>,
	Julian Anastasov <ja@ssi.bg>,
	Karsten Graul <kgraul@linux.ibm.com>,
	Kees Cook <keescook@chromium.org>,
	Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>,
	Mat Martineau <martineau@kernel.org>,
	Matthieu Baerts <matthieu.baerts@tessares.net>,
	Miquel Raynal <miquel.raynal@bootlin.com>,
	Nikolay Aleksandrov <razor@blackwall.org>,
	Pablo Neira Ayuso <pablo@netfilter.org>,
	Paolo Abeni <pabeni@redhat.com>,
	Ralf Baechle <ralf@linux-mips.org>,
	Roopa Prabhu <roopa@nvidia.com>,
	Santosh Shilimkar <santosh.shilimkar@oracle.com>,
	Simon Horman <horms@verge.net.au>,
	Stefan Schmidt <stefan@datenfreihafen.org>,
	Steffen Klassert <steffen.klassert@secunet.com>,
	Sven Schnelle <svens@linux.ibm.com>,
	Tony Lu <tonylu@linux.alibaba.com>,
	Vasily Gorbik <gor@linux.ibm.com>,
	Wen Gu <guwen@linux.alibaba.com>,
	Wenjia Zhang <wenjia@linux.ibm.com>,
	Will Deacon <will@kernel.org>, Xin Long <lucien.xin@gmail.com>,
	<bridge@lists.linux-foundation.org>, <coreteam@netfilter.org>,
	<josh@joshtriplett.org>, <linux-arm-kernel@lists.infradead.org>,
	<linux-fsdevel@vger.kernel.org>, <linux-hams@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, <linux-rdma@vger.kernel.org>,
	<linux-s390@vger.kernel.org>, <linux-sctp@vger.kernel.org>,
	<linux-wpan@vger.kernel.org>, <lvs-devel@vger.kernel.org>,
	<mptcp@lists.linux.dev>, <netdev@vger.kernel.org>,
	<netfilter-devel@vger.kernel.org>, <rds-devel@oss.oracle.com>,
	<willy@infradead.org>
Subject: Re: [PATCH v2 00/14] sysctl: Add a size argument to register functions in sysctl
Date: Tue, 8 Aug 2023 15:58:47 +0200	[thread overview]
Message-ID: <20230808135847.46ln434mbpiiklvz@localhost> (raw)
In-Reply-To: <CANnsUMGRBnatKB4-3eYjb5aG7YnXDiZG6cjuwSgtjvVF6ErJNg@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1922 bytes --]

Hey Chris

My questions inline
On Mon, Aug 07, 2023 at 07:50:44PM -0700, Chris Maness wrote:
> I tried running the current mainline kernel (current Arch Linux) with
What kernel version are we talking about exactly here? Please notice the
base that I used in the cover letter (rc2 or rc3). They should also
rebase cleanly on top of v6.5-rc5.

I'm not sure what the patches will do if you apply them to an older
kernel.

> simple single MUX socket (ax0) using LinFBB.  I was a happy camper as
> it seemed to work fine at first, then the system just slowed to a
> crawl.  I am wondering if any of these patches are addressing this
> behavior.  No kernel panic like before, but not what I was hoping for.
So you have been experiencing kernel panics from before? What does
before mean? And can you bisect to pinpoint what commit fixed it?

> I have also tried sixpack, and that explodes instantly the last time I
> have checked.   That goes all the way back to the v4 kernels.  v2 is
> fine there.
You mean that 6pack is broken in all the major versions up until v2? If
this is the case, then it might be an issue that is not related to this
patchset.I see that Andreas Koensgen <ajk@comnets.uni-bremen.de> is the
maintainer of 6pack, maybe he can chime in on the issue?

> 
> 73 de Chris KQ6UP
> 
> On Mon, Aug 7, 2023 at 4:43 PM Luis Chamberlain <mcgrof@kernel.org> wrote:
> >
> > On Mon, Aug 07, 2023 at 04:00:49PM -0700, Chris Maness wrote:
> > > When are these likely to hit the mainline release code?
> >
> > linux-next tomorrow. The first 7 patches are scheduled for mainline
> > as they were merged + tested without any hiccups. These last few patches
> > I'll wait and see. If nothing blows up on linux-next perhaps I'll
> > include them to Linux for mainline during the next merge window.
> >
> >   Luis
> 
> 
> 
> -- 
> Thanks,
> Chris Maness

-- 

Joel Granados

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]