[igt-dev] [PATCH i-g-t] tests/xe: Add xe_module_error test
Kumar, Janga Rahul
janga.rahul.kumar at intel.com
Thu Jun 1 11:03:23 UTC 2023
> -----Original Message-----
> From: igt-dev <igt-dev-bounces at lists.freedesktop.org> On Behalf Of Ch, Sai
> Gowtham
> Sent: 01 June 2023 13:34
> To: Karas, Anna <anna.karas at intel.com>; Manszewski, Christoph
> <christoph.manszewski at intel.com>; igt-dev at lists.freedesktop.org
> Cc: Chris Wilson <chris.p.wilson at linux.intel.com>
> Subject: Re: [igt-dev] [PATCH i-g-t] tests/xe: Add xe_module_error test
>
> Hi Anna
>
> > -----Original Message-----
> > From: Karas, Anna <anna.karas at intel.com>
> > Sent: Thursday, June 1, 2023 1:24 PM
> > To: Manszewski, Christoph <christoph.manszewski at intel.com>; igt-
> > dev at lists.freedesktop.org
> > Cc: Ch, Sai Gowtham <sai.gowtham.ch at intel.com>; Chris Wilson
> > <chris.p.wilson at linux.intel.com>
> > Subject: Re: [igt-dev] [PATCH i-g-t] tests/xe: Add xe_module_error
> > test
> >
> > Hi Christoph,
> >
> > On 31.05.2023 16:25, Manszewski, Christoph wrote:
> > > Hi Anna,
> > >
> > > On 31.05.2023 15:49, Anna Karas wrote:
> > >> Port i915_module_error test to XE. This test checks if we can
> > >> access debugfs/sysfs after a module load error, in this case a guc
> > >> load error, for capturing the requisite debug info.
> > >>
> > >> Signed-off-by: Anna Karas <anna.karas at intel.com>
> > >
> > > Since this code is almost entirely copied, I think it would be good
> > > practice to credit the original author by adding a 'Authored-by' or
> > > 'Signed-off-by' line.
> > >
> > > Christoph
> > >
> >
> > I will add it to v2. Thanks!
> > >
> > >> ---
> > >> tests/meson.build | 1 +
> > >> tests/xe/xe_module_error.c | 145
> > >> +++++++++++++++++++++++++++++++++++++
> > >> 2 files changed, 146 insertions(+)
> > >> create mode 100644 tests/xe/xe_module_error.c
> > >>
> > >> diff --git a/tests/meson.build b/tests/meson.build index
> > >> f71be1db..533c96e8 100644
> > >> --- a/tests/meson.build
> > >> +++ b/tests/meson.build
> > >> @@ -261,6 +261,7 @@ xe_progs = [
> > >> 'xe_intel_bb',
> > >> 'xe_mmap',
> > >> 'xe_mmio',
> > >> + 'xe_module_error',
> Can you check if the spacing is correct here.
>
> Rest of the test looks good to me, However just to check have you verified
> this test ?
>
> ----
> Gowtham
> > >> 'xe_module_load',
> > >> 'xe_noexec_ping_pong',
> > >> 'xe_pm',
> > >> diff --git a/tests/xe/xe_module_error.c
> > >> b/tests/xe/xe_module_error.c new file mode 100644 index
> > >> 00000000..a17d85ca
> > >> --- /dev/null
> > >> +++ b/tests/xe/xe_module_error.c
> > >> @@ -0,0 +1,145 @@
> > >> +// SPDX-License-Identifier: MIT
> > >> +/*
> > >> + * Copyright © 2023 Intel Corporation */
> > >> +
> > >> +#include <dirent.h>
> > >> +
> > >> +#include "igt.h"
> > >> +#include "igt_kmod.h"
> > >> +#include "igt_debugfs.h"
> > >> +#include "igt_sysfs.h"
> > >> +#include "xe_drm.h"
> > >> +
> > >> +#include "xe/xe_spin.h"
> > >> +
> > >> +/**
> > >> + * TEST: Check if we can access sysfs/debugfs after a module load
> > >> +error
> > >> + * Category: Software building block
> > >> + * Sub-category: debugfs
> > >> + * Test category: functionality test
> > >> + * Run type: BAT
> > >> + */
> > >> +
> > >> +/**
> > >> + * SUBTEST: sysfs
> > >> + * Description: Read all entries from sysfs path
> > >> + * Run type: FULL
> > >> + */
> > >> +
> > >> +/**
> > >> + * SUBTEST: debugfs
> > >> + * Description: Read all entries from debugfs path
> > >> + * Run type: FULL
> > >> + */
> > >> +static void read_sysfs_entries(int path, int indent) {
> > >> + char tabs[] = "\t\t\t\t\t\t\t\t";
> > >> + struct dirent *dirent;
> > >> + DIR *dir;
> > >> +
> > >> + igt_assert(indent < sizeof(tabs) - 1);
> > >> + tabs[indent] = '\0';
> > >> +
> > >> + dir = fdopendir(path);
> > >> + if (!dir)
> > >> + return;
> > >> +
> > >> + while ((dirent = readdir(dir))) {
> > >> + if (!strcmp(dirent->d_name, ".") ||
> > >> + !strcmp(dirent->d_name, ".."))
> > >> + continue;
> > >> +
> > >> + if (dirent->d_type == DT_DIR) {
> > >> + int sub;
> > >> +
> > >> + sub = openat(path, dirent->d_name, O_RDONLY |
> > >> +O_DIRECTORY);
> > >> + if (sub < 0)
> > >> + continue;
> > >> +
> > >> + igt_kmsg(KMSG_DEBUG "Reading directory \"%s\"\n",
> > >> dirent->d_name);
> > >> + igt_debug("%sEntering subdir %s\n", tabs,
> > >> +dirent->d_name);
> > >> + read_sysfs_entries(sub, indent + 1);
> > >> + close(sub);
> > >> + } else if (dirent->d_type == DT_REG) {
> > >> + char buf[512];
> > >> + int sub;
> > >> + ssize_t ret;
> > >> +
> > >> + igt_kmsg(KMSG_DEBUG "Reading file \"%s\"\n",
> > >> dirent->d_name);
> > >> + igt_debug("%sReading file \"%s\"\n", tabs,
> > >> +dirent->d_name);
> > >> + igt_set_timeout(igt_run_in_simulation() ? 30 : 5,
> > >> + "reading sysfs entry");
> > >> +
> > >> + sub = openat(path, dirent->d_name, O_RDONLY |
> > >> +O_NONBLOCK);
> > >> + if (sub == -1) {
> > >> + igt_debug("%sCould not open file \"%s\" with error:
> > >> %m\n",
> > >> + tabs, dirent->d_name);
> > >> + continue;
> > >> + }
> > >> +
> > >> + do {
> > >> + ret = read(sub, buf, sizeof(buf));
> > >> + } while (ret == sizeof(buf));
> > >> +
> > >> + if (ret == -1)
> > >> + igt_debug("%sCould not read file \"%s\" with error:
> > >> %m\n",
> > >> + tabs, dirent->d_name);
> > >> +
> > >> + igt_reset_timeout();
> > >> + close(sub);
> > >> + }
> > >> + }
> > >> +
> > >> + closedir(dir);
> > >> +}
> > >> +
> > >> +static void abort_if_loaded(const char *module_name) {
> > >> + int err;
> > >> +
> > >> + err = igt_kmod_unload(module_name, 0);
> > >> + if (err == 0 || err == -ENOENT) /* -ENOENT == unloaded already
> > >> +*/
> > >> + return;
> > >> +
> > >> + igt_abort_on_f(err,
> > >> + "Failed to unload '%s' err:%d, leaving dangerous
> > >> modparams intact!\n",
> > >> + module_name, err);
> > >> +}
> > >> +
> > >> +static void unload(int sig)
> > >> +{
> > >> + igt_xe_driver_unload();
> > >> + abort_if_loaded("xe");
> > >> +}
> > >> +
> > >> +igt_main
> > >> +{
> > >> + int fd;
> > >> +
> > >> + igt_fixture {
> > >> + igt_xe_driver_unload();
> > >> + igt_xe_driver_load("guc_firmware_path=/dev/null
> > >> inject_probe_failure=-1 force_probe=*");
@Karas, Anna,
This test will not work as guc_firmware_path and inject_probe_failure modparams are not valid for XE.
You may have to check any alternate options to cause load error.
Thanks,
Rahul
> > >> + igt_install_exit_handler(unload);
> > >> +
> > >> + fd = drm_open_driver(DRIVER_XE);
> > >> + xe_device_get(fd);
> > >> + }
> > >> +
> > >> + igt_subtest("sysfs") {
> > >> + igt_fork(child, 1)
> > >> + read_sysfs_entries(igt_sysfs_open(fd), 0);
> > >> + igt_waitchildren();
> > >> + }
> > >> +
> > >> + igt_subtest("debugfs") {
> > >> + igt_fork(child, 1)
> > >> + read_sysfs_entries(igt_debugfs_dir(fd), 0);
> > >> + igt_waitchildren();
> > >> + }
> > >> +
> > >> + igt_fixture {
> > >> + xe_device_put(fd);
> > >> + close(fd);
> > >> + }
> > >> +}
More information about the igt-dev
mailing list