Re: [Mesa-dev] [PATCH] i965: Drop index buffer re-alignment code.

2017-06-22 Thread Ian Romanick
On 06/22/2017 05:29 PM, Kenneth Graunke wrote: > On Thursday, June 22, 2017 4:10:39 PM PDT Ian Romanick wrote: >> That's weird... I wonder why this code was ever added. Regardless, >> >> Reviewed-by: Ian Romanick > > https://bugs.freedesktop.org/show_bug.cgi?id=11910 Ugh... "BTW, your test case

Re: [Mesa-dev] [PATCH] i965: Drop index buffer re-alignment code.

2017-06-22 Thread Kenneth Graunke
On Thursday, June 22, 2017 4:10:39 PM PDT Ian Romanick wrote: > That's weird... I wonder why this code was ever added. Regardless, > > Reviewed-by: Ian Romanick https://bugs.freedesktop.org/show_bug.cgi?id=11910 signature.asc Description: This is a digitally signed message part. _

Re: [Mesa-dev] [PATCH] i965: Drop index buffer re-alignment code.

2017-06-22 Thread Ian Romanick
That's weird... I wonder why this code was ever added. Regardless, Reviewed-by: Ian Romanick On 06/22/2017 11:16 AM, Kenneth Graunke wrote: > This shouldn't ever happen - GL requires it to be aligned: > >"Clients must align data elements consistent with the requirements > of the client

[Mesa-dev] [PATCH] i965: Drop index buffer re-alignment code.

2017-06-22 Thread Kenneth Graunke
This shouldn't ever happen - GL requires it to be aligned: "Clients must align data elements consistent with the requirements of the client platform, with an additional base-level requirement that an offset within a buffer to a datum comprising N basic machine units be a multiple of