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 for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ac337470d5af8b70458c9bb7981f137b7cdce1ec.camel@surriel.com>
Date: Sat, 11 Jan 2025 21:36:54 -0500
From: Rik van Riel <riel@...riel.com>
To: Dave Hansen <dave.hansen@...el.com>, x86@...nel.org
Cc: linux-kernel@...r.kernel.org, kernel-team@...a.com, 
	dave.hansen@...ux.intel.com, luto@...nel.org, peterz@...radead.org, 
	tglx@...utronix.de, mingo@...hat.com, bp@...en8.de, hpa@...or.com, 
	akpm@...ux-foundation.org, nadav.amit@...il.com,
 zhengqi.arch@...edance.com, 	linux-mm@...ck.org
Subject: Re: [PATCH 09/12] x86/mm: enable broadcast TLB invalidation for
 multi-threaded processes

On Mon, 2025-01-06 at 10:40 -0800, Dave Hansen wrote:
> On 12/30/24 09:53, Rik van Riel wrote:
> ...
> > +#ifdef CONFIG_CPU_SUP_AMD
> > +	struct list_head broadcast_asid_list;
> > +	u16 broadcast_asid;
> > +	bool asid_transition;
> > +#endif
> 
> Could we either do:
> 
> 	config X86_TLB_FLUSH_BROADCAST_HW
> 		bool
> 		depends on CONFIG_CPU_SUP_AMD

I went with this option.

I also got rid of the list_head (thanks Nadav), and
moved the remaining two items into the same word as
the protection key bits.

I think I have everybody's comments addressed in my
code base now. I'll post a new version soon-ish :)


-- 
All Rights Reversed.

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ