Re: [PATCH v2 2/2] x86/xstate: Make AVX512 status tracking more accurate

From: Borislav Petkov
Date: Wed Oct 27 2021 - 07:07:50 EST


On Fri, Oct 15, 2021 at 03:47:17PM -0500, Noah Goldstein wrote:
> This patch splits the timestamps for tracking the AVX512 status into

For future submissions, please avoid having "This patch" or "This
commit" in the commit message. It is tautologically useless.

...

> @@ -1282,8 +1292,27 @@ static void avx512_status(struct seq_file *m, struct task_struct *task)
> delta = LONG_MAX;
> delta = jiffies_to_msecs(delta);
> }
> + return delta;
> +}
>
> - seq_put_decimal_ll(m, "AVX512_elapsed_ms:\t", delta);

Regardless of whether I think this exposing of AVX512 usage is silly
or not, we do not break userspace by changing those strings which some
script or tool already probably relies upon.

--
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette