<div dir="ltr">Thomas,<div>    Thanks for the review. This time I put the patch in the attachment so it should be applied cleanly.</div><div> I added Signed-off-by line too. </div><div>      Thanks</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 27, 2015 at 6:36 AM, Thomas Wood <span dir="ltr"><<a href="mailto:thomas.wood@intel.com" target="_blank">thomas.wood@intel.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 23 April 2015 at 17:54, Yunlian Jiang <<a href="mailto:yunlian@chromium.org">yunlian@chromium.org</a>> wrote:<br>
> The bug entry is at<br>
> <a href="https://code.google.com/p/chromium/issues/detail?id=476001" target="_blank">https://code.google.com/p/chromium/issues/detail?id=476001</a><br>
><br>
> The patch below makes clang happy.<br>
<br>
</span>Thanks for the patch; Ben Widawsky reviewed it and added his<br>
reviewed-by tag. Unfortunately it doesn't apply cleanly because tabs<br>
have been converted to spaces. Your commit message is also missing a<br>
signed-off-by line. It may be easier to apply if you are able to use<br>
git format-patch or git send-email to send your patch. The<br>
CONTRIBUTING file in intel-gpu-tools has more details about using git<br>
send-email/git format-patch and the developer's certificate of origin<br>
requirements.<br>
<span class=""><br>
<br>
><br>
>  debugger/eudb.c | 6 +-----<br>
>  1 file changed, 1 insertion(+), 5 deletions(-)<br>
><br>
> diff --git a/debugger/eudb.c b/debugger/eudb.c<br>
> index 0e810db..7188a4f 100644<br>
> --- a/debugger/eudb.c<br>
> +++ b/debugger/eudb.c<br>
> @@ -207,15 +207,11 @@ wait_for_attn(int timeout, int *out_bits) {<br>
>  #define eu_tid(bit) eu_info->debuggees[bit].tid<br>
>  static struct eu_state *<br>
>  find_eu_shmem(int bit, volatile uint8_t *buf) {<br>
> -       struct per_thread_data {<br>
> -               uint8_t ____[dh.per_thread_scratch];<br>
> -       }__attribute__((packed)) *data;<br>
>         struct eu_state *eu;<br>
>         int mem_tid, mem_euid, i;<br>
><br>
> -       data = (struct per_thread_data *)buf;<br>
>         for(i = 0; i < eu_info->num_threads; i++) {<br>
> -               eu = (struct eu_state *)&data[i];<br>
> +               eu = (struct eu_state *)&buff[i*dh.per_thread_scratch];<br>
>                 mem_tid = eu->sr0 & 0x7;<br>
>                 mem_euid = (eu->sr0 >> 8) & 0xf;<br>
>                 if (mem_tid == eu_tid(bit) && mem_euid == eu_id(bit))<br>
> --<br>
> 2.2.0.rc0.207.ga3a616c<br>
><br>
><br>
</span>> _______________________________________________<br>
> Intel-gfx mailing list<br>
> <a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
</blockquote></div><br></div>