<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, May 16, 2017 at 3:41 PM, Anuj Phogat <span dir="ltr"><<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, May 12, 2017 at 4:38 PM, Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> wrote:<br>
> As sRGB now supports lossless compression, don't we also need to stop<br>
> resolving single sampled color render buffers for sRGB formats in Gen 10.<br>
><br>
</span>s/don't we/we. Fixed locally.<br>
<div class="HOEnZb"><div class="h5"><br>
> Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>><br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>context.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<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 1247d03..9e19617 100644<br>
> --- a/src/mesa/drivers/dri/i965/<wbr>brw_context.c<br>
> +++ b/src/mesa/drivers/dri/i965/<wbr>brw_context.c<br>
> @@ -326,7 +326,7 @@ intel_update_state(struct gl_context * ctx, GLuint new_state)<br>
>      * enabled because otherwise the surface state will be programmed with the<br>
>      * linear equivalent format anyway.<br>
>      */<br>
> -   if (brw->gen >= 9 && ctx->Color.sRGBEnabled) {<br>
> +   if (brw->gen == 9 && ctx->Color.sRGBEnabled) {<br></div></div></blockquote><div><br></div><div>I think I'd mildly prefer that we make this based on isl_format_supports_ccs_e at some point but that doesn't have to happen today.<br><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
>        struct gl_framebuffer *fb = ctx->DrawBuffer;<br>
>        for (int i = 0; i < fb->_NumColorDrawBuffers; i++) {<br>
>           struct gl_renderbuffer *rb = fb->_ColorDrawBuffers[i];<br>
> --<br>
> 2.9.3<br>
><br>
______________________________<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></div>