[Intel-gfx] [PATCH] Fixed the review issues for pm_rc6_residency IGT case
Jesse Barnes
jbarnes at virtuousgeek.org
Mon Jun 16 22:38:22 CEST 2014
On Mon, 16 Jun 2014 11:43:30 -0700
Ben Widawsky <benjamin.widawsky at intel.com> wrote:
> Hi Wendy. Daniel has reverted your original commit here:
> commit 35554a1bcaaea55c1cfa88c0176c58d2fb3b8013
> Author: Daniel Vetter <daniel.vetter at ffwll.ch>
> Date: Tue Jun 10 11:05:16 2014 +0200
>
> Revert "Add rc6_residency_counter subtest"
>
> Note that I absolutely do not agree with the decision to revert your
> patch as was stated in the commit message. I am not sure how Daniel got
> the impression that I thought this was "in order."
>
> Can you please resubmit the patch based on the latest intel-gpu-tools?
I also made that clear when Daniel and I discussed it. I simply don't
understand why a revert was necessary, especially given that we had an
incremental patch to address many of the comments. Was the test
breaking i-g-t runs (i.e. preventing tests from running)? Was it
somehow crashing and causing false reports?
--
Jesse Barnes, Intel Open Source Technology Center
More information about the Intel-gfx
mailing list