On 20 July 2018 at 13:39, Gurchetan Singh <[email protected]> wrote: > Reviewed-by: Gurchetan Singh <[email protected]> > On Wed, Jul 18, 2018 at 4:06 AM Erik Faye-Lund > <[email protected]> wrote: >> >> The IOCTLs doesn't pass this along, so computing them in the first >> place is kinda pointless. >> >> Signed-off-by: Erik Faye-Lund <[email protected]> >> --- >> >> This is just a cleanup I noticed based on some discussion with Gert. >> >> A question is, what code here expects this stride to be respected? The >> call-sites in virgl_*_transfer_map and virgl_*_transfer_unmap kinda >> looks like they do... They'll get a bit of a surprise here, no?
>> >> Anyway, this is already broken, so I think this should be OK. But >> perhaps this patch shows some code-paths that need some love? I reverted this as it didn't fixup vtest, and it introduced build time warnings. I agree this should get some more investigation but make sure vtest and drm backends don't regress. Dave. _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
