* [PATCH igt] igt/gem_ppgtt: Flush the driver to idle before counting leaks
@ 2018-04-19 5:48 ` Chris Wilson
0 siblings, 0 replies; 6+ messages in thread
From: Chris Wilson @ 2018-04-19 5:48 UTC (permalink / raw)
To: intel-gfx; +Cc: igt-dev
I have a cunning plan to make the vma open/close lazy to cache frequent
reallocations (as buffers are passed between applications, e.g. DRI).
However, this will mean that we will not be immediately closing vma and
so need to tell the kernel to process the idle handlers before checking
for leaks.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
---
tests/gem_ppgtt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/gem_ppgtt.c b/tests/gem_ppgtt.c
index bed95db83..575b0e9d3 100644
--- a/tests/gem_ppgtt.c
+++ b/tests/gem_ppgtt.c
@@ -236,7 +236,7 @@ static void flink_and_close(void)
gem_sync(fd2, flinked_bo);
gem_close(fd2, flinked_bo);
- igt_drop_caches_set(fd, DROP_RETIRE);
+ igt_drop_caches_set(fd, DROP_RETIRE | DROP_IDLE);
/* the flinked bo VMA should have been cleared now, so a new bo of the
* same size should get the same offset
@@ -286,7 +286,7 @@ static void flink_and_exit(void)
exec_and_get_offset(fd3, gem_create(fd3, 4096));
close(fd3);
- igt_drop_caches_set(fd, DROP_ACTIVE | DROP_RETIRE);
+ igt_drop_caches_set(fd, DROP_ACTIVE | DROP_RETIRE | DROP_IDLE);
igt_assert(!igt_debugfs_search(fd, "i915_gem_gtt", match));
close(fd);
--
2.17.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [igt-dev] [PATCH igt] igt/gem_ppgtt: Flush the driver to idle before counting leaks
@ 2018-04-19 5:48 ` Chris Wilson
0 siblings, 0 replies; 6+ messages in thread
From: Chris Wilson @ 2018-04-19 5:48 UTC (permalink / raw)
To: intel-gfx; +Cc: igt-dev, Tvrtko Ursulin
I have a cunning plan to make the vma open/close lazy to cache frequent
reallocations (as buffers are passed between applications, e.g. DRI).
However, this will mean that we will not be immediately closing vma and
so need to tell the kernel to process the idle handlers before checking
for leaks.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
---
tests/gem_ppgtt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/gem_ppgtt.c b/tests/gem_ppgtt.c
index bed95db83..575b0e9d3 100644
--- a/tests/gem_ppgtt.c
+++ b/tests/gem_ppgtt.c
@@ -236,7 +236,7 @@ static void flink_and_close(void)
gem_sync(fd2, flinked_bo);
gem_close(fd2, flinked_bo);
- igt_drop_caches_set(fd, DROP_RETIRE);
+ igt_drop_caches_set(fd, DROP_RETIRE | DROP_IDLE);
/* the flinked bo VMA should have been cleared now, so a new bo of the
* same size should get the same offset
@@ -286,7 +286,7 @@ static void flink_and_exit(void)
exec_and_get_offset(fd3, gem_create(fd3, 4096));
close(fd3);
- igt_drop_caches_set(fd, DROP_ACTIVE | DROP_RETIRE);
+ igt_drop_caches_set(fd, DROP_ACTIVE | DROP_RETIRE | DROP_IDLE);
igt_assert(!igt_debugfs_search(fd, "i915_gem_gtt", match));
close(fd);
--
2.17.0
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for igt/gem_ppgtt: Flush the driver to idle before counting leaks
2018-04-19 5:48 ` [igt-dev] " Chris Wilson
(?)
@ 2018-04-19 13:51 ` Patchwork
-1 siblings, 0 replies; 6+ messages in thread
From: Patchwork @ 2018-04-19 13:51 UTC (permalink / raw)
To: Chris Wilson; +Cc: igt-dev
== Series Details ==
Series: igt/gem_ppgtt: Flush the driver to idle before counting leaks
URL : https://patchwork.freedesktop.org/series/41936/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4069 -> IGTPW_1283 =
== Summary - SUCCESS ==
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/41936/revisions/1/mbox/
== Known issues ==
Here are the changes found in IGTPW_1283 that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@gem_exec_suspend@basic-s3:
fi-ivb-3520m: PASS -> DMESG-WARN (fdo#106084)
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
fi-glk-1: PASS -> INCOMPLETE (fdo#103359, k.org#198133)
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
fi-bxt-dsi: PASS -> INCOMPLETE (fdo#103927)
==== Possible fixes ====
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
fi-cnl-y3: DMESG-WARN (fdo#104951) -> PASS
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
fi-ivb-3520m: DMESG-WARN (fdo#106084) -> PASS
fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
fdo#104951 https://bugs.freedesktop.org/show_bug.cgi?id=104951
fdo#106084 https://bugs.freedesktop.org/show_bug.cgi?id=106084
k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133
== Participating hosts (35 -> 32) ==
Missing (3): fi-ctg-p8600 fi-ilk-m540 fi-skl-6700hq
== Build changes ==
* IGT: IGT_4441 -> IGTPW_1283
CI_DRM_4069: 8136363fe770a1a51688172d5ba46a5017f76677 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_1283: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1283/
IGT_4441: 83ba5b7d3bde48b383df41792fc9c955a5a23bdb @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
piglit_4441: e60d247eb359f044caf0c09904da14e39d7adca1 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1283/issues.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [igt-dev] [PATCH igt] igt/gem_ppgtt: Flush the driver to idle before counting leaks
2018-04-19 5:48 ` [igt-dev] " Chris Wilson
@ 2018-04-19 14:57 ` Tvrtko Ursulin
-1 siblings, 0 replies; 6+ messages in thread
From: Tvrtko Ursulin @ 2018-04-19 14:57 UTC (permalink / raw)
To: Chris Wilson, intel-gfx; +Cc: igt-dev
On 19/04/2018 06:48, Chris Wilson wrote:
> I have a cunning plan to make the vma open/close lazy to cache frequent
> reallocations (as buffers are passed between applications, e.g. DRI).
> However, this will mean that we will not be immediately closing vma and
> so need to tell the kernel to process the idle handlers before checking
> for leaks.
>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> ---
> tests/gem_ppgtt.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/gem_ppgtt.c b/tests/gem_ppgtt.c
> index bed95db83..575b0e9d3 100644
> --- a/tests/gem_ppgtt.c
> +++ b/tests/gem_ppgtt.c
> @@ -236,7 +236,7 @@ static void flink_and_close(void)
> gem_sync(fd2, flinked_bo);
> gem_close(fd2, flinked_bo);
>
> - igt_drop_caches_set(fd, DROP_RETIRE);
> + igt_drop_caches_set(fd, DROP_RETIRE | DROP_IDLE);
>
> /* the flinked bo VMA should have been cleared now, so a new bo of the
> * same size should get the same offset
> @@ -286,7 +286,7 @@ static void flink_and_exit(void)
> exec_and_get_offset(fd3, gem_create(fd3, 4096));
> close(fd3);
>
> - igt_drop_caches_set(fd, DROP_ACTIVE | DROP_RETIRE);
> + igt_drop_caches_set(fd, DROP_ACTIVE | DROP_RETIRE | DROP_IDLE);
> igt_assert(!igt_debugfs_search(fd, "i915_gem_gtt", match));
>
> close(fd);
>
It's not interfering with test intentions so:
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [igt-dev] [PATCH igt] igt/gem_ppgtt: Flush the driver to idle before counting leaks
@ 2018-04-19 14:57 ` Tvrtko Ursulin
0 siblings, 0 replies; 6+ messages in thread
From: Tvrtko Ursulin @ 2018-04-19 14:57 UTC (permalink / raw)
To: Chris Wilson, intel-gfx; +Cc: igt-dev, Tvrtko Ursulin
On 19/04/2018 06:48, Chris Wilson wrote:
> I have a cunning plan to make the vma open/close lazy to cache frequent
> reallocations (as buffers are passed between applications, e.g. DRI).
> However, this will mean that we will not be immediately closing vma and
> so need to tell the kernel to process the idle handlers before checking
> for leaks.
>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> ---
> tests/gem_ppgtt.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/gem_ppgtt.c b/tests/gem_ppgtt.c
> index bed95db83..575b0e9d3 100644
> --- a/tests/gem_ppgtt.c
> +++ b/tests/gem_ppgtt.c
> @@ -236,7 +236,7 @@ static void flink_and_close(void)
> gem_sync(fd2, flinked_bo);
> gem_close(fd2, flinked_bo);
>
> - igt_drop_caches_set(fd, DROP_RETIRE);
> + igt_drop_caches_set(fd, DROP_RETIRE | DROP_IDLE);
>
> /* the flinked bo VMA should have been cleared now, so a new bo of the
> * same size should get the same offset
> @@ -286,7 +286,7 @@ static void flink_and_exit(void)
> exec_and_get_offset(fd3, gem_create(fd3, 4096));
> close(fd3);
>
> - igt_drop_caches_set(fd, DROP_ACTIVE | DROP_RETIRE);
> + igt_drop_caches_set(fd, DROP_ACTIVE | DROP_RETIRE | DROP_IDLE);
> igt_assert(!igt_debugfs_search(fd, "i915_gem_gtt", match));
>
> close(fd);
>
It's not interfering with test intentions so:
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Regards,
Tvrtko
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for igt/gem_ppgtt: Flush the driver to idle before counting leaks
2018-04-19 5:48 ` [igt-dev] " Chris Wilson
` (2 preceding siblings ...)
(?)
@ 2018-04-19 19:20 ` Patchwork
-1 siblings, 0 replies; 6+ messages in thread
From: Patchwork @ 2018-04-19 19:20 UTC (permalink / raw)
To: Chris Wilson; +Cc: igt-dev
== Series Details ==
Series: igt/gem_ppgtt: Flush the driver to idle before counting leaks
URL : https://patchwork.freedesktop.org/series/41936/
State : success
== Summary ==
= CI Bug Log - changes from IGT_4441_full -> IGTPW_1283_full =
== Summary - WARNING ==
Minor unknown changes coming with IGTPW_1283_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_1283_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/41936/revisions/1/mbox/
== Possible new issues ==
Here are the unknown changes that may have been introduced in IGTPW_1283_full:
=== IGT changes ===
==== Warnings ====
igt@gem_exec_schedule@deep-bsd2:
shard-kbl: PASS -> SKIP
igt@kms_fbcon_fbt@fbc-suspend:
shard-snb: PASS -> SKIP +1
igt@perf_pmu@rc6:
shard-kbl: SKIP -> PASS
== Known issues ==
Here are the changes found in IGTPW_1283_full that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@kms_frontbuffer_tracking@fbc-suspend:
shard-kbl: PASS -> INCOMPLETE (fdo#103665)
igt@kms_rotation_crc@primary-rotation-180:
shard-snb: PASS -> FAIL (fdo#103925)
igt@kms_setmode@basic:
shard-kbl: PASS -> FAIL (fdo#99912)
igt@perf@blocking:
shard-hsw: PASS -> FAIL (fdo#102252)
==== Possible fixes ====
igt@drv_suspend@forcewake:
shard-kbl: INCOMPLETE (fdo#103665) -> PASS
igt@kms_flip@2x-dpms-vs-vblank-race:
shard-hsw: FAIL (fdo#103060) -> PASS
igt@kms_flip@2x-flip-vs-expired-vblank:
shard-hsw: FAIL (fdo#102887) -> PASS
igt@kms_rotation_crc@primary-rotation-180:
shard-hsw: FAIL (fdo#103925) -> PASS
igt@perf_pmu@interrupts-sync:
shard-apl: FAIL (fdo#104485) -> PASS
fdo#102252 https://bugs.freedesktop.org/show_bug.cgi?id=102252
fdo#102887 https://bugs.freedesktop.org/show_bug.cgi?id=102887
fdo#103060 https://bugs.freedesktop.org/show_bug.cgi?id=103060
fdo#103665 https://bugs.freedesktop.org/show_bug.cgi?id=103665
fdo#103925 https://bugs.freedesktop.org/show_bug.cgi?id=103925
fdo#104485 https://bugs.freedesktop.org/show_bug.cgi?id=104485
fdo#99912 https://bugs.freedesktop.org/show_bug.cgi?id=99912
== Participating hosts (5 -> 4) ==
Missing (1): shard-glkb
== Build changes ==
* IGT: IGT_4441 -> IGTPW_1283
* Linux: CI_DRM_4059 -> CI_DRM_4069
CI_DRM_4059: c1645edc253f2b52a8c94565a75b479a6782e75f @ git://anongit.freedesktop.org/gfx-ci/linux
CI_DRM_4069: 8136363fe770a1a51688172d5ba46a5017f76677 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_1283: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1283/
IGT_4441: 83ba5b7d3bde48b383df41792fc9c955a5a23bdb @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
piglit_4441: e60d247eb359f044caf0c09904da14e39d7adca1 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1283/shards.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-04-19 19:20 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-19 5:48 [PATCH igt] igt/gem_ppgtt: Flush the driver to idle before counting leaks Chris Wilson
2018-04-19 5:48 ` [igt-dev] " Chris Wilson
2018-04-19 13:51 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2018-04-19 14:57 ` [igt-dev] [PATCH igt] " Tvrtko Ursulin
2018-04-19 14:57 ` Tvrtko Ursulin
2018-04-19 19:20 ` [igt-dev] ✓ Fi.CI.IGT: success for " Patchwork
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.