Re: [PATCH v2 12/14] platform/x86/intel/ifs: Add current_batch sysfs entry

Luck, Tony
2022-11-12T23:33:37Z

> Btw, do you guys really want to run this in production?

Absolutely yes.

> As in, are you really that confident that all those test sequence
> executions really do not affect other CPU execution at all?

Tests may draw a lot of power, so prevent
other cores going into turbo frequency states.
But otherwise no effect.

> Because if this is going to be run during downtime, as Thiago says, then
> you can just as well use debugfs for this. And then there's no need to
> cast any API in stone and so on.

Did Thiago say “during downtime”? I think
he talked about some users opportunistic
use of scan tests. But that’s far from only
during downtime. We fully expect CSPs to
run these scans periodically on production
machines.

-Tony