<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 9, 2016 at 7:03 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Nayan,<br>
<br>
Just a couple of fly-by comments. As always don't read too much into them.<br>
<span class=""><br>
On 9 September 2016 at 14:17, Nayan Deshmukh <<a href="mailto:nayan26deshmukh@gmail.com">nayan26deshmukh@gmail.com</a>> wrote:<br>
> use a linear buffer in case of back buffer<br>
><br>
</span>You might want to mention a bit more about the implementation and/or<br>
why doing things like X won't work/is bad idea.<br>
<span class=""><br></span></blockquote><div>Yeah I will add more detail to the commit message. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> +   if (scrn->is_different_gpu) {<br>
> +      templ.format = PIPE_FORMAT_B8G8R8X8_UNORM;<br>
> +      templ.target = PIPE_TEXTURE_2D;<br>
> +      templ.last_level = 0;<br>
> +      templ.width0 = scrn->width;<br>
> +      templ.height0 = scrn->height;<br>
> +      templ.depth0 = 1;<br>
> +      templ.array_size = 1;<br>
</span>bind == 0 here, that doesn't seem right ?<br>
<span class=""><br></span></blockquote><div>I was trying to replicate the code used in loader_dri3_helper.c. I was not sure </div><div>which bind flags should be used, I was hoping that michel can have a look </div><div>and suggest the necessary changes. I tested it on my hardware and it was </div><div>working fine. On second thought it should have a PIPE_BIND_RENDER_TARGET </div><div>flag.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> +      buffer->texture = scrn->base.pscreen->resource_<wbr>create(scrn->base.pscreen,<br>
> +                                                            &templ);<br>
> +      if (!buffer->texture)<br>
> +         goto unmap_shm;<br>
> +<br>
</span>With the bind fixed the above hunk is common so can be kept prior to<br>
the conditional, no ?<br>
<span class=""><br></span></blockquote><div>With bind flag fixed this common code could be taken out of the conditional. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<br>
> @@ -485,6 +516,17 @@ vl_dri3_flush_frontbuffer(<wbr>struct pipe_screen *screen,<br>
>              return;<br>
>     }<br>
><br>
> +   if (scrn->is_different_gpu) {<br>
> +      u_box_origin_2d(scrn->width,<br>
> +                      scrn->height,<br>
> +                      &src_box);<br>
> +      scrn->pipe->resource_copy_<wbr>region(scrn->pipe,<br>
> +                                       back->linear_buffer,<br>
> +                                       0, 0, 0, 0,<br>
> +                                       back->texture,<br>
> +                                       0, &src_box);<br>
> +<br>
</span>Please try to use the full width of the line.<br>
<br></blockquote><div>The entire file uses this kind of wraping so I was trying to follow the style.</div><div>It would be better if entire file has the same kind of wraping. </div><div><br></div><div>Thanks for the review. I'll make the necessary changes in the next version </div><div>once michel has a look and approves it.  </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Regards,<br>
Emil<br>
</blockquote></div><br></div></div>