<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 24, 2017 at 3:32 PM, Peter Krempa <span dir="ltr"><<a href="mailto:pkrempa@redhat.com" target="_blank">pkrempa@redhat.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 Fri, Feb 24, 2017 at 15:22:51 +0530, Nitesh Konkar wrote:<br>
> Signed-off-by: Nitesh Konkar <<a href="mailto:nitkon12@linux.vnet.ibm.com">nitkon12@linux.vnet.ibm.com</a>><br>
> ---<br>
>  connect.go      | 54 ++++++++++++++++++++++++++++++<wbr>++++++++++++++++++++++++<br>
>  domain.go       | 54 ++++++++++++++++++++++++++++++<wbr>++++++++++++++++++++++++<br>
>  domain_compat.h | 36 ++++++++++++++++++++++++++++++<wbr>++++++<br>
>  3 files changed, 144 insertions(+)<br>
<br>
</span>The files are improperly indented. The go files use tabs for indentation<br>
while your code uses spaces. I've already pointed this out in v1.<br></blockquote><div>I had read the review on v1 but could not understand exactly whats wrong<br></div><div>with the indentation as everything looked aligned on applying the patch.<br> Now its clear, shall send v3.  Thanks. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
><br>
> diff --git a/connect.go b/connect.go<br>
> index 3c53485..e24cf66 100644<br>
> --- a/connect.go<br>
> +++ b/connect.go<br>
> @@ -2264,6 +2264,24 @@ type DomainStatsPerf struct {<br>
>       StalledCyclesBackend     uint64<br>
>       RefCpuCyclesSet          bool<br>
>       RefCpuCycles             uint64<br>
> +        CpuClockSet              bool<br>
> +        CpuClock                 uint64<br>
> +        TaskClockSet             bool<br>
> +        TaskClock                uint64<br>
</div></div></blockquote></div><br></div></div>