[Intel-gfx] [igt-dev] [PATCH i-g-t 3/8] lib: Restore the i915.reset modparam before cleaning up
Antonio Argenziano
antonio.argenziano at intel.com
Tue Feb 19 18:22:26 UTC 2019
On 17/02/19 06:35, Chris Wilson wrote:
> We force a reset on test exit so that we can rapidly cleanup after a
> naughty test, it is not unknown for us to leave a queue of hanging
> batches around. However, if we have also fiddled with the i915.reset
> parameter in the meantime, this can leave the kernel unable to fulfil
typo -------------------------------------------------------------^
> our request (and those naughty batches continue to disrupt testing).
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Mika Kuoppala <mika.kuoppala at intel.com>
> Cc: Petri Latvala <petri.latvala at intel.com>
Re-enabling reset sounds good.
Acked-by: Antonio Argenziano <antonio.argenziano at intel.com>
> ---
> lib/drmtest.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/lib/drmtest.c b/lib/drmtest.c
> index 1964795a6..6c0a0e381 100644
> --- a/lib/drmtest.c
> +++ b/lib/drmtest.c
> @@ -54,6 +54,7 @@
> #include "igt_device.h"
> #include "igt_gt.h"
> #include "igt_kmod.h"
> +#include "igt_sysfs.h"
> #include "version.h"
> #include "config.h"
> #include "intel_reg.h"
> @@ -345,6 +346,7 @@ static void __cancel_work_at_exit(int fd)
> {
> igt_terminate_spin_batches(); /* for older kernels */
>
> + igt_sysfs_set_parameter(fd, "reset", "%x", -1u /* any method */);
> igt_drop_caches_set(fd,
> /* cancel everything */
> DROP_RESET_ACTIVE | DROP_RESET_SEQNO |
>
More information about the Intel-gfx
mailing list