[PATCH i-g-t v7 00/16] Test coverage for GPU debug support

[Zbigniew Kempczyński]

On Fri, Sep 20, 2024 at 09:08:26AM -0700, Dixit, Ashutosh wrote:
> On Fri, 20 Sep 2024 05:57:14 -0700, Manszewski, Christoph wrote:
> >
> 
> Hi Christoph,
> 
> > Hi Ashutosh,
> >
> > On 19.09.2024 20:02, Dixit, Ashutosh wrote:
> > > On Wed, 18 Sep 2024 04:30:01 -0700, Christoph Manszewski wrote:
> > >>
> > >> Hi,
> > >>
> > >> In this series the eudebug kernel and validation team would like to
> > >> add test coverage for GPU debug support recently proposed as an RFC.
> > >> (https://patchwork.freedesktop.org/series/136572/)
> > >>
> > >
> > > Something got merged from this series but seems there are a bunch of
> > > warnings. Please fix.
> > >
> > > $ ninja -C build
> > > ninja: Entering directory `build'
> > > [427/1905] Generating lib/iga64_generated_codes.c with a custom command
> > > iga64 assemblies not changed, reusing pre-compiled file ../lib/iga64_generated_codes.c.
> > > [1899/1905] Generating docs/testplan/xe_tests.rst with a custom command
> > > xe_exec_sip_eudebug.testlist: Testlist file not found.
> > > xe_eudebug_online.testlist: Testlist file not found.
> > > xe_eudebug.testlist: Testlist file not found.
> >
> > These were removed by 'scripts/test_list: Don't log missing testlist' (for
> > not compiled tests).
> 
> OK, these seem to be gone now.
> 
> >
> > > INFO: Found documentation for 'xe_eudebug_online' but no binary
> > > INFO: Found documentation for 'xe_eudebug' but no binary
> > > INFO: Found documentation for 'xe_exec_sip_eudebug' but no binary
> >
> > This is an info log, that was added by Kamil's patch (which I slightly
> > modified).
> 
> I am still seeing these because I don't have build_xe_eudebug enabled it
> seems. I think if someone doesn't enable a component, they shouldn't be
> treated to a bunch of warnings. Maybe we'll bring it up with Kamil.

Right. I'm going to rid of those infos. We're mostly interested of
missing documentation, not the opposite.

--
Zbigniew

> 
> Thanks.
> --
> Ashutosh