linux-kernel - Re: [PATCH 14/14] alpha: drop pre-EV56 support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <e0492052-46ff-40e4-96e0-aecb88b68f0f@kernel.org>
Date: Mon, 3 Jun 2024 08:02:03 +0200
From: Jiri Slaby <jirislaby@...nel.org>
To: Arnd Bergmann <arnd@...nel.org>, linux-alpha@...r.kernel.org
Cc: Arnd Bergmann <arnd@...db.de>,
 Richard Henderson <richard.henderson@...aro.org>,
 Ivan Kokshaysky <ink@...assic.park.msu.ru>, Matt Turner
 <mattst88@...il.com>, Alexander Viro <viro@...iv.linux.org.uk>,
 Marc Zyngier <maz@...nel.org>, Linus Torvalds
 <torvalds@...ux-foundation.org>, "Paul E. McKenney" <paulmck@...nel.org>,
 linux-kernel@...r.kernel.org, Greg KH <gregkh@...uxfoundation.org>
Subject: Re: [PATCH 14/14] alpha: drop pre-EV56 support

Cc Greg for awareness too.

On 03. 05. 24, 10:11, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@...db.de>
> 
> All EV4 machines are already gone, and the remaining EV5 based machines
> all support the slightly more modern EV56 generation as well.
> Debian only supports EV56 and later.
> 
> Drop both of these and build kernels optimized for EV56 and higher
> when the "generic" options is selected, tuning for an out-of-order
> EV6 pipeline, same as Debian userspace.
> 
> Since this was the only supported architecture without 8-bit and
> 16-bit stores, common kernel code no longer has to worry about
> aligning struct members, and existing workarounds from the block
> and tty layers can be removed.

Yes!

For TTY pieces below:
Reviewed-by: Jiri Slaby <jirislaby@...nel.org>

> diff --git a/include/linux/tty.h b/include/linux/tty.h
> index 2b2e6f0a54d6..2372f9357240 100644
> --- a/include/linux/tty.h
> +++ b/include/linux/tty.h
> @@ -145,15 +145,12 @@ struct tty_operations;
>    * @count: count of open processes, reaching zero cancels all the work for
>    *	   this tty and drops a @kref too (but does not free this tty)
>    * @winsize: size of the terminal "window" (cf. @winsize_mutex)
> - * @flow: flow settings grouped together, see also @flow.unused
> + * @flow: flow settings grouped together
>    * @flow.lock: lock for @flow members
>    * @flow.stopped: tty stopped/started by stop_tty()/start_tty()
>    * @flow.tco_stopped: tty stopped/started by %TCOOFF/%TCOON ioctls (it has
>    *		      precedence over @flow.stopped)
> - * @flow.unused: alignment for Alpha, so that no members other than @flow.* are
> - *		 modified by the same 64b word store. The @flow's __aligned is
> - *		 there for the very same reason.
> - * @ctrl: control settings grouped together, see also @ctrl.unused
> + * @ctrl: control settings grouped together
>    * @ctrl.lock: lock for @ctrl members
>    * @ctrl.pgrp: process group of this tty (setpgrp(2))
>    * @ctrl.session: session of this tty (setsid(2)). Writes are protected by both
> @@ -161,7 +158,6 @@ struct tty_operations;
>    *		  them.
>    * @ctrl.pktstatus: packet mode status (bitwise OR of %TIOCPKT_ constants)
>    * @ctrl.packet: packet mode enabled
> - * @ctrl.unused: alignment for Alpha, see @flow.unused for explanation
>    * @hw_stopped: not controlled by the tty layer, under @driver's control for CTS
>    *		handling
>    * @receive_room: bytes permitted to feed to @ldisc without any being lost
> @@ -216,8 +212,7 @@ struct tty_struct {
>   		spinlock_t lock;
>   		bool stopped;
>   		bool tco_stopped;
> -		unsigned long unused[0];
> -	} __aligned(sizeof(unsigned long)) flow;
> +	} flow;
>   
>   	struct {
>   		struct pid *pgrp;
> @@ -225,8 +220,7 @@ struct tty_struct {
>   		spinlock_t lock;
>   		unsigned char pktstatus;
>   		bool packet;
> -		unsigned long unused[0];
> -	} __aligned(sizeof(unsigned long)) ctrl;
> +	} ctrl;
>   
>   	bool hw_stopped;
>   	bool closing;

-- 
js
suse labs