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

[Lyude Paul <lyude@redhat.com>]

As a note: I finally managed to figure out what the problem was with
the CRC readback. Turns out when I compiled the chamelium source code,
I completely forgot to point it to an arm compiler so the chamelium
kept trying to run x86_64 binaries…

On Wed, 2016-11-16 at 12:52 +0100, Tomeu Vizoso wrote:
> 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/chamel
> > > > > > eon-
> > > > > > 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