<div dir="ltr">Hi Curro,<div><br></div><div>Thanks for being thorough about this -- I think there is still one area where things might be a bit wobbly; if we end up taking a sw fallback, swrast only does 4 bits. I'm not sure that matters though.</div><div><br></div><div>- Chris</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 8, 2016 at 11:01 AM, Francisco Jerez <span dir="ltr"><<a href="mailto:currojerez@riseup.net" target="_blank">currojerez@riseup.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Chris Forbes <<a href="mailto:chrisf@ijw.co.nz">chrisf@ijw.co.nz</a>> writes:<br>
<br>
> The mesa default is 4, but we program the hardware for 8 on all<br>
> generations.<br>
><br>
<br>
</span>I happened to come across this inconsistency a couple of weeks ago -- I<br>
just double-checked that it doesn't cause any conformance regressions<br>
because some of the rasterization tests use the GL_SUBPIXEL_BITS value<br>
to determine the error tolerance so increasing the value could<br>
potentially uncover additional approximation errors.  Doesn't seem to<br>
cause any regressions though in our CI system, series is:<br>
<br>
Reviewed-by: Francisco Jerez <<a href="mailto:currojerez@riseup.net">currojerez@riseup.net</a>><br>
<span class="im HOEnZb"><br>
> Signed-off-by: Chris Forbes <<a href="mailto:chrisforbes@google.com">chrisforbes@google.com</a>><br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>context.c | 1 +<br>
>  1 file changed, 1 insertion(+)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/<wbr>brw_context.c b/src/mesa/drivers/dri/i965/<wbr>brw_context.c<br>
> index 3085a98..d8174c6 100644<br>
> --- a/src/mesa/drivers/dri/i965/<wbr>brw_context.c<br>
> +++ b/src/mesa/drivers/dri/i965/<wbr>brw_context.c<br>
> @@ -538,6 +538,7 @@ brw_initialize_context_<wbr>constants(struct brw_context *brw)<br>
>        ctx->Const.<wbr>MaxProgramTextureGatherCompone<wbr>nts = 1;<br>
><br>
>     ctx->Const.MaxUniformBlockSize = 65536;<br>
> +   ctx->Const.SubPixelBits = 8;<br>
><br>
>     for (int i = 0; i < MESA_SHADER_STAGES; i++) {<br>
>        struct gl_program_constants *prog = &ctx->Const.Program[i];<br>
> --<br>
> 2.10.2<br>
><br>
</span><div class="HOEnZb"><div class="h5">> ______________________________<wbr>_________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div>