[Mesa-dev] [PATCH] u_vbuf: fix src_offset alignment in u_vbuf_create_vertex_elements()

[Brian Paul]

On 06/24/2015 06:49 AM, Emil Velikov wrote:
> Hello gents,
>
> On 19 June 2015 at 15:39, Brian Paul <brianp at vmware.com> wrote:
>> If the driver says PIPE_CAP_VERTEX_ELEMENT_SRC_OFFSET_4BYTE_ALIGNED_ONLY=1,
>> the driver should never receive a pipe_vertex_element::src_offset value
>> that's not a multiple of four.  But the vbuf code wasn't actually adjusting
>> the src_offset value when creating the vertex element state object.
>>
>> We just need to align the src_offset values put in the driver_attribs[]
>> array.
>>
>> See the piglit gl-1.5-vertex-buffer-offsets test.
> Is this something we want in the 10.6 stable branch ? According to the
> comment in the piglit test, the spec does not explicitly state what
> should happen in the particular case, but I'd assume that we want to
> provide some safe behaviour ?

I think the patch fixes a pretty rare situation so I wasn't too 
concerned about porting to the 10.6 branch.  But I don't think there'd 
be any harm in cherry-picking it to 10.6.

-Brian