<div dir="ltr"><div dir="ltr"><br><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 10, 2020 at 2:42 AM Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Quoting Anshuman Gupta (2020-12-10 05:02:48)<br>
> On 2020-12-09 at 16:47:01 +0000, Chris Wilson wrote:<br>
> > Quoting Anshuman Gupta (2020-12-09 16:25:02)<br>
> > > On 2020-12-09 at 16:25:55 +0000, Chris Wilson wrote:<br>
> > > > Quoting Anshuman Gupta (2020-12-09 16:06:38)<br>
> > > > > Create a separate igt test group and move package C<br>
> > > > > state in to this subgroup.<br>
> > > > > Run powertop --auto-tune to tune SOC power configuration<br>
> > > > > for package C state tests.<br>
> > > > > <br>
> > > > > Signed-off-by: Anshuman Gupta <<a href="mailto:anshuman.gupta@intel.com" target="_blank">anshuman.gupta@intel.com</a>><br>
> > > > > ---<br>
> > > > >  tests/i915/i915_pm_rpm.c | 35 +++++++++++++++++++++++++++++++----<br>
> > > > >  1 file changed, 31 insertions(+), 4 deletions(-)<br>
> > > > > <br>
> > > > > diff --git a/tests/i915/i915_pm_rpm.c b/tests/i915/i915_pm_rpm.c<br>
> > > > > index af55b569..42bc44d9 100644<br>
> > > > > --- a/tests/i915/i915_pm_rpm.c<br>
> > > > > +++ b/tests/i915/i915_pm_rpm.c<br>
> > > > > @@ -832,6 +832,25 @@ static void basic_subtest(void)<br>
> > > > >         /* XXX Also we can test wake up via exec nop */<br>
> > > > >  }<br>
> > > > >  <br>
> > > > > +static bool setup_powertop(void)<br>
> > > > > +{<br>
> > > > > +       FILE *fp;<br>
> > > > > +       char tmp[512];<br>
> > > > > +<br>
> > > > > +       fp = popen("powertop --auto-tune", "r");<br>
> > > > <br>
> > > > Doesn't this defeat the point of having it work out of the box?<br>
> May be misunderstood your comment, is it PC state or powertop should work<br>
> out of box ?<br>
<br>
Powermanagement should not require the user to configure it before it<br>
can work.<br>
<br>
powertop in particular may tweak the gfx, which is verboten.<br>
<br>
Manually perform any configuration you think is required, and warn if the<br>
initial configuration does not support reaching the lowest pc state the<br>
system can. File bugs.<br></blockquote><div><br></div><div>I understand why Anshuman did this. There are many many components</div><div>out there that are blocking the PC10 and this powertop --auto-tune</div><div>is trying to take care of all the corner cases to get the "best" of all the</div><div>components so we can try to garantee that we are not getting blocked by other components.</div><div><br></div><div>However this is very very dangerous. It toggles a lot of unsafe and untested cases.</div><div><br></div><div>I have lost a laptop right after a powertop --auto-tune and have never used that</div><div>in my life again. Coincidence? Maybe... but I don't want to take the risk in my</div><div>machine and I definitely cannot recommend its use.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
-Chris<br>
_______________________________________________<br>
igt-dev mailing list<br>
<a href="mailto:igt-dev@lists.freedesktop.org" target="_blank">igt-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/igt-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/igt-dev</a><br>
</blockquote></div></div>