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

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

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.

Thanks.
--
Ashutosh