<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 27, 2017 at 7:16 PM, Matthew Auld <span dir="ltr"><<a href="mailto:matthew.william.auld@gmail.com" target="_blank">matthew.william.auld@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 03/23, Robert Bragg wrote:<br>
> These are auto generated from an XML description of metric sets,<br>
> currently maintained in gputop, ref:<br>
><br>
>  <a href="https://github.com/rib/gputop" rel="noreferrer" target="_blank">https://github.com/rib/gputop</a><br>
>  > gputop-data/oa-*.xml<br>
>  > scripts/i915-perf-kernelgen.py<br>
><br>
>  $ make -C gputop-data -f Makefile.xml<br>
><br>
> Signed-off-by: Robert Bragg <<a href="mailto:robert@sixbynine.org">robert@sixbynine.org</a>><br>
> ---<br>
<br>
</span><SNIP><br>
<br>
>  int i915_oa_select_metric_set_bdw(<wbr>struct drm_i915_private *dev_priv)<br>
>  {<br>
> -     dev_priv->perf.oa.mux_regs = NULL;<br>
> -     dev_priv->perf.oa.mux_regs_len = 0;<br>
> -     dev_priv->perf.oa.b_counter_<wbr>regs = NULL;<br>
> -     dev_priv->perf.oa.b_counter_<wbr>regs_len = 0;<br>
> -     dev_priv->perf.oa.flex_regs = NULL;<br>
> -     dev_priv->perf.oa.flex_regs_<wbr>len = 0;<br>
> +     dev_priv->perf.oa.mux_regs = NULL;<br>
> +     dev_priv->perf.oa.mux_regs_len = 0;<br>
> +     dev_priv->perf.oa.b_counter_<wbr>regs = NULL;<br>
> +     dev_priv->perf.oa.b_counter_<wbr>regs_len = 0;<br>
> +     dev_priv->perf.oa.flex_regs = NULL;<br>
> +     dev_priv->perf.oa.flex_regs_<wbr>len = 0;<br>
What changed? I can't tell from the diff...<br></blockquote><div><br></div><div>I don't think anything changed in those lines, it's just that the diff uses the start of the function for context and then has to delete these to add the full replacement for the function body which included substantial changes to add the cases for the additional configs.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Otherwise assuming you re-spin with the DRM_DEBUG changes:<br>
Reviewed-by: Matthew Auld <<a href="mailto:matthew.auld@intel.com">matthew.auld@intel.com</a>><br></blockquote><div><br></div><div>thanks <br></div></div><br></div></div>