On 5 June 2016 at 22:17, Ilia Mirkin <[email protected]> wrote: > On Sun, Jun 5, 2016 at 5:16 PM, Emil Velikov <[email protected]> wrote: >> On 5 June 2016 at 22:13, Emil Velikov <[email protected]> wrote: >>> On 5 June 2016 at 17:56, Samuel Pitoiset <[email protected]> wrote: >>>> We should not call nouveau_bufctx_reset() inside a validate function. >>> This seems to contradict the changes introduced in nvc0_compute.c. >>> Worth explaining a bit better the dos and don'ts ? >>> >> As this is already in master, can you please provide a more >> elaborate/correct summary for -stable ? > > I think it's fine as is. > > Do: reset bufctx when setting dirty bit > Don't: reset bufctx in validate logic, since it's "too late" by then. > (Not strictly wrong, but just should do it earlier.)
So nvc0_compute_*validate*_surfaces is not validate logic ? Err... what a confusing name it has ;-) Emil _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
