[Intel-gfx] [RFC i-g-t 0/4] intel-gpu-tools: Add support for the Chamelium
Lyude Paul
lyude at redhat.com
Mon Nov 28 20:10:03 UTC 2016
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 at 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 at 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 at lists.freedesktop.org
> > > > > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> > > >
> > > > --
> > > > Cheers,
> > > > Lyude
> >
> > --
> > Cheers,
> > Lyude
More information about the Intel-gfx
mailing list