Re: [PATCH] arch/x86: reset MXCSR to default in kernel_fpu_begin()

[Shuah Khan]

On 6/2/20 11:19 PM, Petteri Aimonen wrote:
> Hi,
>
> > Is it correct to assume the stuff checked differs from test to test
> > and done in user-space.
> >
> > > undo_evil_state();
> >
> > Is it correct to assume undoing evil differs from test to test
> > and done in user-space, provide it can be done from userspace.
>
> Yes, currently the test works like:
>
> do_test_setup();
> read_from_debugfs_file();
> check_results();

You will need a 4th clanup step step of undo_test_setup().

> and the middle step stays the same. But of course in general case there
> could be argument passing etc, even though the test for this issue
> doesn't need them.
>
> Myself I don't see the problem with just adding a file under debugfs and
> bind to its read.

thanks,
-- Shuah