On Jul 28, 2016 7:37 PM, "Anuj Phogat" <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> wrote: > > Fixes the vulkan cts regression in test dEQP-VK.api.info.device.properties > > Cc: Mark Janes <<a href="mailto:mark.a.janes@intel.com">mark.a.janes@intel.com</a>> > Cc: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> > Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> > --- > src/intel/vulkan/anv_device.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c > index c2c5153..9fd9683 100644 > --- a/src/intel/vulkan/anv_device.c > +++ b/src/intel/vulkan/anv_device.c > @@ -524,9 +524,9 @@ void anv_GetPhysicalDeviceProperties( > .maxTexelOffset = 7, > .minTexelGatherOffset = -8, > .maxTexelGatherOffset = 7, > - .minInterpolationOffset = 0, /* FIXME */ > - .maxInterpolationOffset = 0, /* FIXME */ > - .subPixelInterpolationOffsetBits = 0, /* FIXME */ > + .minInterpolationOffset = -0.5, > + .maxInterpolationOffset = 0.5, > + .subPixelInterpolationOffsetBits = 4, I looked and neither gen7_pipeline.c nor gen8_pipeline.c set the subpixel precision so we get a default of enum value 0 which is 8 bits. We should probably be explicitly setting it to something. I think 4 is probably fine but we should be seeing it. As a side note, I think we could probably combine 3DSTATE_SF and 3DSTATE_RASTER setup between gen7 and gen8 (and move it to genX_pipeline_until.h). We'd probably have to do some #define trick like we did for 3DSTATE_SBE and 3DSTATE_SBE_SWIZ because the two packets are combined on gen7 and split on gen8. > .maxFramebufferWidth = (1 << 14), > .maxFramebufferHeight = (1 << 14), > .maxFramebufferLayers = (1 << 10), > -- > 2.5.5 >