[Mesa-dev] [PATCH v1 8/8] nvc0: disable MS images on GM107+

[Samuel Pitoiset]

On 07/20/2016 04:03 AM, Ilia Mirkin wrote:
> Series is
>
> Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>
>
> Note that you'll start using PBO downloads via images as a result of
> this change, please run some of the deqp pbo tests (might be in gles3,
> not gles31) to double-check that all's well.

./piglit-run.py -1 --dmesg tests/deqp_gles3.py -t pbo gm107-pbo
[504/504] pass: 504 |
Thank you for running Piglit!

I think it's not bad! :)

>
>   -ilia
>
> On Tue, Jul 19, 2016 at 4:15 PM, Samuel Pitoiset
> <samuel.pitoiset at gmail.com> wrote:
>> MS images have to be handled explicitly and I don't plan to implement
>> them for now.
>>
>> v1: - check that sample_count > 1
>>
>> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
>> ---
>>  src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 7 +++++++
>>  1 file changed, 7 insertions(+)
>>
>> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
>> index f681631..a3cd046 100644
>> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
>> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
>> @@ -90,6 +90,13 @@ nvc0_screen_is_format_supported(struct pipe_screen *pscreen,
>>                   PIPE_BIND_LINEAR |
>>                   PIPE_BIND_SHARED);
>>
>> +   if (bindings & PIPE_BIND_SHADER_IMAGE && sample_count > 1 &&
>> +       nouveau_screen(pscreen)->class_3d >= GM107_3D_CLASS) {
>> +      /* MS images are currently unsupported on Maxwell because they have to
>> +       * be handled explicitly. */
>> +      return false;
>> +   }
>> +
>>     return (( nvc0_format_table[format].usage |
>>              nvc0_vertex_format[format].usage) & bindings) == bindings;
>>  }
>> --
>> 2.9.0
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev