main: Use a derived value for the default sample count
authorNeil Roberts <neil@linux.intel.com>
Thu, 4 Feb 2016 16:12:07 +0000 (16:12 +0000)
committerNeil Roberts <neil@linux.intel.com>
Fri, 5 Feb 2016 11:05:10 +0000 (11:05 +0000)
commiteb9cf3cfc99db0cff0a5a584941e8be43ac9b961
tree3109de36e560b9a90ea9f1d12a7d1a448e4497fe
parent5fd848f6c9ee86b3d0e024cfa6255cd2d721462f
main: Use a derived value for the default sample count

Previously the framebuffer default sample count was taken directly
from the value given by the application. On the i965 driver on HSW if
the value wasn't one that is supported by the hardware it would hit an
assert when it tried to program the state for it. This patch fixes it
by adding a derived sample count to the state for the default
framebuffer. The driver can then quantize this to one of the valid
values in its UpdateState handler when the _NEW_BUFFERS state changes.
_mesa_geometric_samples is changed to use the new derived value.

Fixes the piglit test arb_framebuffer_no_attachments-query

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93957
Cc: Ilia Mirkin <imirkin@alum.mit.edu>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
src/mesa/drivers/dri/i965/brw_context.c
src/mesa/main/framebuffer.h
src/mesa/main/mtypes.h