Re: [RFC i-g-t 0/4] intel-gpu-tools: Add support for the Chamelium

[Tomeu Vizoso <tomeu@tomeuvizoso.net>]

Hi Lyude,

I think this looks very good.

On 8 November 2016 at 01:05, Lyude <lyude@redhat.com> wrote:
>
>  - While writing this patch series, I found that quite a few of the RPC calls
>    for chameleond don't work as expected. For instance, I have had absolutely
>    no luck getting CRCs from any of the display types that the chamelium
>    supports.

When I looked at this a few months ago, frame CRCs were working just
fine. I was using libsoup, so maybe there's some problem with the
unpacking of the checksum?

> This isn't a huge deal though, since we usually just use the
>    native CRC read back on the GPU anyway.

I'm not completely sure what you mean by that, but not all graphic
pipelines are able to provide frame CRCs so I think this Chamelium
work will be very useful when running tests that do check frame CRCs.

Regards,

Tomeu

>
>  - Among other things that are broken with the chameleon, video signal
>    detection for DisplayPort is one of them. After the first plug/unplug cycle,
>    the DisplayPort receiver gets stuck and gives the wrong results for
>    WaitForInputStable. Luckily I've already got a fix I'll be submitting to the
>    ChromeOS guys when I get around to setting up their homebrew git tools:
>
>         https://github.com/Lyude/chameleond/tree/wip/chameleon-fixes
>
>    For now, expect the dp-display tests to fail without those patches.
>
> Lyude (4):
>   igt_aux: Add igt_skip_without_suspend_support()
>   igt_aux: Add igt_set_autoresume_delay()
>   igt_aux: Add some list helpers from wayland
>   Add support for hotplug testing with the Chamelium
>
>  configure.ac           |  13 +
>  lib/Makefile.am        |  10 +-
>  lib/igt.h              |   1 +
>  lib/igt_aux.c          |  94 ++++++++
>  lib/igt_aux.h          |  41 ++++
>  lib/igt_chamelium.c    | 628 +++++++++++++++++++++++++++++++++++++++++++++++++
>  lib/igt_chamelium.h    |  77 ++++++
>  lib/igt_kms.c          | 107 +++++++++
>  lib/igt_kms.h          |  13 +-
>  scripts/run-tests.sh   |   4 +-
>  tests/Makefile.am      |   5 +-
>  tests/Makefile.sources |   1 +
>  tests/chamelium.c      | 549 ++++++++++++++++++++++++++++++++++++++++++
>  13 files changed, 1538 insertions(+), 5 deletions(-)
>  create mode 100644 lib/igt_chamelium.c
>  create mode 100644 lib/igt_chamelium.h
>  create mode 100644 tests/chamelium.c
>
> --
> 2.7.4
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx