[Mesa-dev] [PATCH V2 1/2] i965/gen10: Don't set Smooth Point Enable in 3DSTATE_SF if num_samples > 1
Anuj Phogat
anuj.phogat at gmail.com
Fri Oct 27 23:08:37 UTC 2017
On Fri, Oct 27, 2017 at 4:00 PM, Kenneth Graunke <kenneth at whitecape.org> wrote:
> On Friday, October 27, 2017 3:17:20 PM PDT Anuj Phogat wrote:
>> V2: Use _NEW_BUFFERS in place of _NEW_MULTISAMPLE
>>
>> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
>> Cc: Kenneth Graunke <kenneth at whitecape.org>
>> ---
>> src/mesa/drivers/dri/i965/genX_state_upload.c | 13 ++++++++++++-
>> 1 file changed, 12 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c
>> index 4ccfd48919..25240f01d9 100644
>> --- a/src/mesa/drivers/dri/i965/genX_state_upload.c
>> +++ b/src/mesa/drivers/dri/i965/genX_state_upload.c
>> @@ -1626,6 +1626,16 @@ genX(upload_sf)(struct brw_context *brw)
>> sf.SmoothPointEnable = true;
>> #endif
>>
>> +#if GEN_GEN == 10
>> + /* _NEW_BUFFERS
>> + * Smooth Point Enable bit MUST not be set when NUM_MULTISAMPLES > 1.
>> + */
>> + const bool multisampled_fbo =
>> + _mesa_geometric_samples(ctx->DrawBuffer) > 1;
>> + if (multisampled_fbo)
>> + sf.SmoothPointEnable = false;
>> +#endif
>> +
>> #if GEN_IS_G4X || GEN_GEN >= 5
>> sf.AALineDistanceMode = AALINEDISTANCE_TRUE;
>> #endif
>> @@ -1681,7 +1691,8 @@ static const struct brw_tracked_state genX(sf_state) = {
>> _NEW_POINT |
>> _NEW_PROGRAM |
>> (GEN_GEN >= 6 ? _NEW_MULTISAMPLE : 0) |
>> - (GEN_GEN <= 7 ? _NEW_BUFFERS | _NEW_POLYGON : 0),
>> + (GEN_GEN <= 7 ? _NEW_POLYGON : 0) |
>> + (GEN_GEN <= 7 || GEN_GEN == 10 ? _NEW_BUFFERS : 0),
>
> Let's leave the GEN <= 7 bit as is and just add
>
> (GEN_GEN == 10 ? _NEW_BUFFERS : 0)
Done. Thanks.
>
> With that, both are
> Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
More information about the mesa-dev
mailing list