On 27 June 2018 at 09:55, Emil Velikov <emil.l.veli...@gmail.com> wrote:
> Hi Drew,
>
> Just some food for thought. The patch in itself looks correct albeit partial.
>
> On 27 June 2018 at 00:00, Drew Davenport <ddavenp...@chromium.org> wrote:
>> This change lets the following test pass on virgl:
>> dEQP-GLES31.functional.state_query.framebuffer_default.framebuffer_default_samples_get_framebuffer_parameteriv
>> ---
>> src/gallium/drivers/virgl/virgl_screen.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/src/gallium/drivers/virgl/virgl_screen.c
>> b/src/gallium/drivers/virgl/virgl_screen.c
>> index 1eefbd6519f..3035d4b5e20 100644
>> --- a/src/gallium/drivers/virgl/virgl_screen.c
>> +++ b/src/gallium/drivers/virgl/virgl_screen.c
>> @@ -495,6 +495,10 @@ virgl_is_format_supported( struct pipe_screen *screen,
>> }
>>
>> if (bind & PIPE_BIND_RENDER_TARGET) {
>> + /* For ARB_framebuffer_no_attachments. */
>> + if (format == PIPE_FORMAT_NONE)
>> + return TRUE;
>> +
>
> For ARB_framebuffer_no_attachments to be advertised, one should return
> 1 for the PIPE_CAP_FRAMEBUFFER_NO_ATTACHMENT query.
> In current master says "not supported" (returns 0) and I couldn't spot
> any patch that toggles it.
>
> Is this a test which requires the functionality, without checking for
> the extension presence?
> Or perhaps the test is part of a larger series, which flips the switch?
>
Silly typo, above should read
"Or perhaps the patch is part ..."
-Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
