Have `pyperformance run` run `pyperf system tune` about pyperformance

[mdboom]

I think this a good idea, with the caveat that there is a way to opt-out (compile has a --no-tune flag). There are places where it won't work (e.g. when the user doesn't have sudo, certain kinds of containers).

My other pet peeve about it (that maybe we could fix while we are in there), is that it doesn't reverse the effects afterward (pyperf system reset).

[ericsnowcurrently]

+1 to making pyperformance run match pyperformance compile as closely as possible, including calling pyperf system tune.

My other pet peeve about it (that maybe we could fix while we are in there), is that it doesn't reverse the effects afterward (pyperf system reset).

+1 for fixing that, whether here or in a separate issue/PR.

[methane]

pyperf system tune needs root but pyperformance run doesn't.
So I like current separation.

[kmod]

What about making pyperformance run throw an error if it's run on an untuned system? Maybe with an argument to disable that behavior. It kind of feels like this case is failure mode with no visible indication, you just have bad results at the end.

[abbriggs]

FWIW, pyperf system tune does not support Windows (understandably so - automatic system tuning is logistically difficult on Windows), so an opt-out of some sort for tuning on pyperformance run would be necessary.