Re: [Mesa-dev] [PATCH] i965/gen9: use an unreserved surface alignment value

2015-06-25 Thread Pohjolainen, Topi
On Thu, Jun 25, 2015 at 08:40:33AM -0700, Nanley Chery wrote: > On Thu, Jun 25, 2015 at 12:37 AM, Pohjolainen, Topi > wrote: > > On Wed, Jun 24, 2015 at 05:57:13PM -0700, Anuj Phogat wrote: > >> On Wed, Jun 24, 2015 at 3:51 PM, Nanley Chery > >> wrote: > >> > From: Nanley Chery > >> > > >> > Al

Re: [Mesa-dev] [PATCH] i965/gen9: use an unreserved surface alignment value

2015-06-25 Thread Anuj Phogat
On Thu, Jun 25, 2015 at 12:37 AM, Pohjolainen, Topi wrote: > On Wed, Jun 24, 2015 at 05:57:13PM -0700, Anuj Phogat wrote: >> On Wed, Jun 24, 2015 at 3:51 PM, Nanley Chery wrote: >> > From: Nanley Chery >> > >> > Although the horizontal and vertical alignment fields are ignored here, >> > 0 is a

Re: [Mesa-dev] [PATCH] i965/gen9: use an unreserved surface alignment value

2015-06-25 Thread Nanley Chery
On Thu, Jun 25, 2015 at 12:37 AM, Pohjolainen, Topi wrote: > On Wed, Jun 24, 2015 at 05:57:13PM -0700, Anuj Phogat wrote: >> On Wed, Jun 24, 2015 at 3:51 PM, Nanley Chery wrote: >> > From: Nanley Chery >> > >> > Although the horizontal and vertical alignment fields are ignored here, >> > 0 is a

Re: [Mesa-dev] [PATCH] i965/gen9: use an unreserved surface alignment value

2015-06-25 Thread Pohjolainen, Topi
On Wed, Jun 24, 2015 at 05:57:13PM -0700, Anuj Phogat wrote: > On Wed, Jun 24, 2015 at 3:51 PM, Nanley Chery wrote: > > From: Nanley Chery > > > > Although the horizontal and vertical alignment fields are ignored here, > > 0 is a reserved value for them and may cause undefined behavior. Change >

Re: [Mesa-dev] [PATCH] i965/gen9: use an unreserved surface alignment value

2015-06-24 Thread Anuj Phogat
On Wed, Jun 24, 2015 at 3:51 PM, Nanley Chery wrote: > From: Nanley Chery > > Although the horizontal and vertical alignment fields are ignored here, > 0 is a reserved value for them and may cause undefined behavior. Change > the default value to an abitrary valid one. > > Signed-off-by: Nanley C

[Mesa-dev] [PATCH] i965/gen9: use an unreserved surface alignment value

2015-06-24 Thread Nanley Chery
From: Nanley Chery Although the horizontal and vertical alignment fields are ignored here, 0 is a reserved value for them and may cause undefined behavior. Change the default value to an abitrary valid one. Signed-off-by: Nanley Chery --- src/mesa/drivers/dri/i965/gen8_surface_state.c | 4 ++--