Received-SPF: pass (google.com: domain of linux-kernel+bounces-41230-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
From: David Laight <David.Laight@ACULAB.COM>
To: 'Arnd Bergmann' <arnd@arndb.de>, Joe Damato <jdamato@fastly.com>, "Greg
 Kroah-Hartman" <gregkh@linuxfoundation.org>
CC: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Netdev
	<netdev@vger.kernel.org>, Chuck Lever <chuck.lever@oracle.com>, Jeff Layton
	<jlayton@kernel.org>, "linux-api@vger.kernel.org"
	<linux-api@vger.kernel.org>, Christian Brauner <brauner@kernel.org>, "Eric
 Dumazet" <edumazet@google.com>, "David S . Miller" <davem@davemloft.net>,
	"alexander.duyck@gmail.com" <alexander.duyck@gmail.com>, Sridhar Samudrala
	<sridhar.samudrala@intel.com>, Jakub Kicinski <kuba@kernel.org>, "Willem de
 Bruijn" <willemdebruijn.kernel@gmail.com>, "weiwan@google.com"
	<weiwan@google.com>, Jonathan Corbet <corbet@lwn.net>, Alexander Viro
	<viro@zeniv.linux.org.uk>, Jan Kara <jack@suse.cz>, Michael Ellerman
	<mpe@ellerman.id.au>, Nathan Lynch <nathanl@linux.ibm.com>, Steve French
	<stfrench@microsoft.com>, Thomas Zimmermann <tzimmermann@suse.de>, Jiri Slaby
	<jirislaby@kernel.org>, Julien Panis <jpanis@baylibre.com>, Andrew Waterman
	<waterman@eecs.berkeley.edu>, Thomas Huth <thuth@redhat.com>, Palmer Dabbelt
	<palmer@dabbelt.com>, "open list:DOCUMENTATION" <linux-doc@vger.kernel.org>,
	"open list:FILESYSTEMS (VFS and infrastructure)"
	<linux-fsdevel@vger.kernel.org>
Subject: RE: [PATCH net-next v3 3/3] eventpoll: Add epoll ioctl for
 epoll_params
Thread-Topic: [PATCH net-next v3 3/3] eventpoll: Add epoll ioctl for
 epoll_params
Thread-Index: AQHaUCR+m5BGvEW2OU6rGFqZjiahCbDtv1nw
Date: Sat, 27 Jan 2024 14:51:58 +0000
Message-ID: <f99dd6cfe7744ed8b9cfe9489aa499de@AcuMS.aculab.com>
References: <20240125225704.12781-1-jdamato@fastly.com>
 <20240125225704.12781-4-jdamato@fastly.com>
 <2024012551-anyone-demeaning-867b@gregkh> <20240126001128.GC1987@fastly.com>
 <2024012525-outdoors-district-2660@gregkh> <20240126023630.GA1235@fastly.com>
 <57b62135-2159-493d-a6bb-47d5be55154a@app.fastmail.com>
In-Reply-To: <57b62135-2159-493d-a6bb-47d5be55154a@app.fastmail.com>
Accept-Language: en-GB, en-US
Precedence: bulk
MIME-Version: 1.0
Content-Language: en-US
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

From: Arnd Bergmann
> Sent: 26 January 2024 06:16
>=20
> On Fri, Jan 26, 2024, at 03:36, Joe Damato wrote:
> > On Thu, Jan 25, 2024 at 04:23:58PM -0800, Greg Kroah-Hartman wrote:
> >> On Thu, Jan 25, 2024 at 04:11:28PM -0800, Joe Damato wrote:
> >> > On Thu, Jan 25, 2024 at 03:21:46PM -0800, Greg Kroah-Hartman wrote:
> >> > > On Thu, Jan 25, 2024 at 10:56:59PM +0000, Joe Damato wrote:
> >> > > > +struct epoll_params {
> >> > > > +=09u64 busy_poll_usecs;
> >> > > > +=09u16 busy_poll_budget;
> >> > > > +
> >> > > > +=09/* for future fields */
> >> > > > +=09u8 data[118];
> >> > > > +} EPOLL_PACKED;
> >> > >
> >
> > Sure, that makes sense to me. I'll remove it in the v4 alongside the ot=
her
> > changes you've requested.
> >
> > Thanks for your time and patience reviewing my code. I greatly apprecia=
te
> > your helpful comments and feedback.
>=20
> Note that you should still pad the structure to its normal
> alignment. On non-x86 targets this would currently mean a
> multiple of 64 bits.
>=20
> I would suggest dropping the EPOLL_PACKED here entirely and
> just using a fully aligned structure on all architectures, like
>=20
> struct epoll_params {
>       __aligned_u64 busy_poll_usecs;
>       __u16 busy_poll_budget;
>       __u8 __pad[6];
> };
>=20
> The explicit padding can help avoid leaking stack data when
> a structure is copied back from kernel to userspace, so I would
> just always use it in ioctl data structures.

Or just use 32bit types for both fields.
Both values need erroring before they get that large.
32bit of usec is about 20 seconds.

=09David

-
Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1=
PT, UK
Registration No: 1397386 (Wales)