Ah, ignore my previous comment. Could we squash this into the patch that added the PAN_SHARED_BO_RW define?
On Thu, Sep 05, 2019 at 09:41:48PM +0200, Boris Brezillon wrote: > Should already be added in panfrost_draw_vbo() and panfrost_clear(), > no need to add it here too. > > Signed-off-by: Boris Brezillon <boris.brezil...@collabora.com> > --- > src/gallium/drivers/panfrost/pan_fragment.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/src/gallium/drivers/panfrost/pan_fragment.c > b/src/gallium/drivers/panfrost/pan_fragment.c > index cbb95b79f52a..00ff363a1bba 100644 > --- a/src/gallium/drivers/panfrost/pan_fragment.c > +++ b/src/gallium/drivers/panfrost/pan_fragment.c > @@ -42,9 +42,6 @@ panfrost_initialize_surface( > struct panfrost_resource *rsrc = pan_resource(surf->texture); > > rsrc->slices[level].initialized = true; > - > - assert(rsrc->bo); > - panfrost_batch_add_bo(batch, rsrc->bo, PAN_SHARED_BO_RW); > } > > /* Generate a fragment job. This should be called once per frame. (According > to > -- > 2.21.0 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev