On 08/03/2017 14:46, Chris Wilson wrote:
Sometimes we want to explicitly page out all available objects from igt,
i.e. call i915_gem_shrink_all() and check that subsequent operations
succeed. This adds DROP_SHRINK_ALL [0x8] to the set of flags for
Not 0x20 ?
debugfs/i915_drop_caches.
Signed-off-by: Chris Wilson <[email protected]>
---
drivers/gpu/drm/i915/i915_debugfs.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c
b/drivers/gpu/drm/i915/i915_debugfs.c
index 3a2ef08ed3a1..d301119378e5 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -4210,11 +4210,13 @@ DEFINE_SIMPLE_ATTRIBUTE(i915_ring_test_irq_fops,
#define DROP_RETIRE 0x4
#define DROP_ACTIVE 0x8
#define DROP_FREED 0x10
+#define DROP_SHRINK_ALL 0x20
#define DROP_ALL (DROP_UNBOUND | \
DROP_BOUND | \
DROP_RETIRE | \
DROP_ACTIVE | \
- DROP_FREED)
+ DROP_FREED | \
+ DROP_SHRINK_ALL)
static int
i915_drop_caches_get(void *data, u64 *val)
{
@@ -4255,6 +4257,9 @@ i915_drop_caches_set(void *data, u64 val)
if (val & DROP_UNBOUND)
i915_gem_shrink(dev_priv, LONG_MAX, I915_SHRINK_UNBOUND);
+ if (val & DROP_SHRINK_ALL)
+ i915_gem_shrink_all(dev_priv);
+
I don't see that DROP_ACTIVE is currently handled as in translated to
I915_SHRINK_ACTIVE?
You want to allow multiple passes if DROP_BOUND | DROP_UNBOUND |
DROP_SHRINK_ALL is given? It doesn't matter hugely, but if DROP_ACTIVE
was handled it would be just a matter of passing in all three flags to
the same effect. Like:
flags |= val & DROP_BOUND ? I915_SHRINK_BOUND : 0;
... for all three flags..
i915_gem_shrink(..., flags);
?
unlock:
mutex_unlock(&dev->struct_mutex);
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
