Reviewed-by: Ilia Mirkin <[email protected]> But please change the commit title to mention the actual effect, like "avoid segfault with enabled but unbound vertex attrib" or something. The piglit fix should be mentioned in the body of the message -- it's a nice side-effect, but not really title-worthy.
Also: Cc: [email protected] in the commit so that it gets cherry-picked into stable releases. On Mon, Jul 6, 2015 at 5:34 PM, Samuel Pitoiset <[email protected]> wrote: > Before validating vertex arrays we need to check if a VBO is present. > Checking if vb->buffer is not NULL fixes the issue. > > Signed-off-by: Samuel Pitoiset <[email protected]> > --- > src/gallium/drivers/nouveau/nv50/nv50_vbo.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/src/gallium/drivers/nouveau/nv50/nv50_vbo.c > b/src/gallium/drivers/nouveau/nv50/nv50_vbo.c > index 1fd33b8..3d200bd 100644 > --- a/src/gallium/drivers/nouveau/nv50/nv50_vbo.c > +++ b/src/gallium/drivers/nouveau/nv50/nv50_vbo.c > @@ -382,6 +382,11 @@ nv50_vertex_arrays_validate(struct nv50_context *nv50) > if (nv50->vbo_user & (1 << b)) { > address = addrs[b] + ve->pipe.src_offset; > limit = addrs[b] + limits[b]; > + } else > + if (!vb->buffer) { > + BEGIN_NV04(push, NV50_3D(VERTEX_ARRAY_FETCH(i)), 1); > + PUSH_DATA (push, 0); > + continue; > } else { > struct nv04_resource *buf = nv04_resource(vb->buffer); > if (!(refd & (1 << b))) { > -- > 2.4.5 > > _______________________________________________ > mesa-dev mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
