intel-gfx.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: Tomeu Vizoso <tomeu@tomeuvizoso.net>
To: Lyude Paul <lyude@redhat.com>
Cc: Intel Graphics Development <intel-gfx@lists.freedesktop.org>
Subject: Re: [RFC i-g-t 0/4] intel-gpu-tools: Add support for the Chamelium
Date: Wed, 16 Nov 2016 12:52:53 +0100	[thread overview]
Message-ID: <CAAObsKAK20LM32_7rTY1KCWyngkM4FC0kPmCykZtnVMfK3AF7A@mail.gmail.com> (raw)
In-Reply-To: <1479246283.18739.9.camel@redhat.com>

On 15 November 2016 at 22:44, Lyude Paul <lyude@redhat.com> wrote:
> I'm fine with libsoup as well, I'll check it out and probably move all
> of the code over to using that instead.

Cool.

> On Tue, 2016-11-15 at 12:44 +0100, Tomeu Vizoso wrote:
>> On 11 November 2016 at 18:53, Lyude Paul <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?
>> >
>> > I'm pretty sure it's on the chameleond side of things. Using the
>> > test
>> > server application in chameleond's source shows the same issue.
>>
>> And what's the problem? You always get CRCs with a value of zero? I
>> only tried with HDMI, but IIRC I got to a point where
>> kms_universal_plane passed.
> The issue is I don't get CRCs, period :(. It looks like somewhere down
> the line chameleond is throwing a big fit whenever I try to get them:

Have no good ideas right now, and it will pass some time before I can
look at it.

I would say to just go ahead with what matters to you right now, and I
will look at it when I rebase my branch on top of yours, which will be
once IGT gains support for the generic frame CRC ABI.

Regards,

Tomeu

> Type "help", "copyright", "credits" or "license" for more information.
> (InteractiveConsole)
>>>> p.Plug(1)
>>>> p.ComputePixelChecksum(1, 0, 0, 1920, 1080)
> Traceback (most recent call last):
>   File "<console>", line 1, in <module>
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 1243, in __call__
>     return self.__send(self.__name, args)
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 1602, in __request
>     verbose=self.__verbose
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 1283, in request
>     return self.single_request(host, handler, request_body, verbose)
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 1316, in
> single_request
>     return self.parse_response(response)
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 1493, in
> parse_response
>     return u.close()
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 800, in close
>     raise Fault(**self._stack[0])
> Fault: <Fault 1: "<type 'exceptions.OSError'>:[Errno 8] Exec format
> error">
>>>> p.ComputePixelChecksum(1, 0, 0, 1920, 1080)
> Traceback (most recent call last):
>   File "<console>", line 1, in <module>
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 1243, in __call__
>     return self.__send(self.__name, args)
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 1602, in __request
>     verbose=self.__verbose
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 1283, in request
>     return self.single_request(host, handler, request_body, verbose)
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 1316, in
> single_request
>     return self.parse_response(response)
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 1493, in
> parse_response
>     return u.close()
>   File "/usr/lib64/python2.7/xmlrpclib.py", line 800, in close
>     raise Fault(**self._stack[0])
> Fault: <Fault 1: "<type 'exceptions.OSError'>:[Errno 8] Exec format
> error">
>
> Haven't had much luck with any of the other ports either
>
>>
>> Regards,
>>
>> Tomeu
>>
>> >
>> > >
>> > > >
>> > > >
>> > > > 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.
>> > I wasn't aware of that, thanks for letting me know
>> >
>> > >
>> > >
>> > > 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-
>> > > > fixe
>> > > > s
>> > > >
>> > > >    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
>> > --
>> > Cheers,
>> >         Lyude
> --
> Cheers,
>         Lyude
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2016-11-16 11:53 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-08  0:05 [RFC i-g-t 0/4] intel-gpu-tools: Add support for the Chamelium Lyude
2016-11-08  0:05 ` [RFC i-g-t 1/4] igt_aux: Add igt_skip_without_suspend_support() Lyude
2016-11-08  0:05 ` [RFC i-g-t 2/4] igt_aux: Add igt_set_autoresume_delay() Lyude
2016-11-08  9:39   ` Chris Wilson
2016-11-08  0:05 ` [RFC i-g-t 3/4] igt_aux: Add some list helpers from wayland Lyude
2016-11-08  9:37   ` Chris Wilson
2016-11-08  0:05 ` [RFC i-g-t 4/4] Add support for hotplug testing with the Chamelium Lyude
2016-11-09 15:18   ` Tomeu Vizoso
2016-11-14  7:05   ` Daniel Vetter
2016-11-14 16:46     ` Lyude Paul
2016-11-09 15:09 ` [RFC i-g-t 0/4] intel-gpu-tools: Add support for " Tomeu Vizoso
2016-11-11 17:53   ` Lyude Paul
2016-11-15 11:44     ` Tomeu Vizoso
2016-11-15 21:44       ` Lyude Paul
2016-11-16 11:52         ` Tomeu Vizoso [this message]
2016-11-28 20:10           ` Lyude Paul

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAAObsKAK20LM32_7rTY1KCWyngkM4FC0kPmCykZtnVMfK3AF7A@mail.gmail.com \
    --to=tomeu@tomeuvizoso.net \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=lyude@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).