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
| ||
|
Date: Fri, 15 Oct 2021 16:40:36 +0200 From: Borislav Petkov <bp@...en8.de> To: Noah Goldstein <goldstein.w.n@...il.com> Cc: tglx@...utronix.de, mingo@...hat.com, X86 ML <x86@...nel.org>, hpa@...or.com, Andy Lutomirski <luto@...nel.org>, open list <linux-kernel@...r.kernel.org> Subject: Re: [PATCH v1] x86/fpu: Remove opmask state from avx512_timestamp check On Thu, Oct 14, 2021 at 10:49:25AM -0500, Noah Goldstein wrote: > Is there a cost to the change I am not seeing? The cost is having yet another macro in our crazy FPU code because apparently everything but the kitchen sink is being put into xstates. So I'd suggest you carve out that timestamp update into a separate function, i.e., something like fpu_update_avx_timestamp() or so - and add the macro in it along with the comment explaining what and why is being tracked. So that that functionality is separated out of the main flow, save_fpregs_to_fpstate() simply calls it and it doesn't get in the way of the next rewrite of the FPU code to accomodate new crap^W features. Thx. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette
Powered by blists - more mailing lists