<div dir='auto'><div><div class="gmail_extra"><div class="gmail_quote">On Jul 23, 2018 22:05, Dave Airlie <airlied@gmail.com> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">On 20 July 2018 at 13:39, Gurchetan Singh <gurchetansingh@chromium.org> wrote:
<br>
> Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>
<br>
> On Wed, Jul 18, 2018 at 4:06 AM Erik Faye-Lund
<br>
> <erik.faye-lund@collabora.com> wrote:
<br>
>>
<br>
>> The IOCTLs doesn't pass this along, so computing them in the first
<br>
>> place is kinda pointless.
<br>
>>
<br>
>> Signed-off-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
<br>
>> ---
<br>
>>
<br>
>> This is just a cleanup I noticed based on some discussion with Gert.
<br>
>>
<br>
>> A question is, what code here expects this stride to be respected? The
<br>
>> call-sites in virgl_*_transfer_map and virgl_*_transfer_unmap kinda
<br>
>> looks like they do... They'll get a bit of a surprise here, no?
<br>

<br>
>>
<br>
>> Anyway, this is already broken, so I think this should be OK. But
<br>
>> perhaps this patch shows some code-paths that need some love?
<br>

<br>
I reverted this as it didn't fixup vtest, and it introduced build time warnings.
<br>

<br>
I agree this should get some more investigation but make sure vtest and
<br>
drm backends don't regress.
<br>

<br>
Dave.
<br>
</p>
</blockquote></div>Good call! </div></div></div>