diff options
author | Mika Kuoppala <mika.kuoppala@intel.com> | 2014-05-07 16:46:19 +0300 |
---|---|---|
committer | Mika Kuoppala <mika.kuoppala@intel.com> | 2014-05-07 17:31:40 +0300 |
commit | 8741c2289f17e9bcb740a01cad4764a71c918eea (patch) | |
tree | fd8f5f99140e3cfdae2ad1430f7b16dae7ec3721 /tests/prime_self_import.c | |
parent | 7935bbd9ba9508d6993ca019a5b8f3262676e7b5 (diff) |
tests/gem_flink_race,prime_self_import: fix object counts
We need to add one drm_open_any() before getting the object counts
as first call to drm_open_any() allocates file descriptors for
exit handlers and thus is not symmetrical.
The regression, assymmetric behaviour, was introduced in:
commit 2f2c491cf3167befe7c79e4b17afb4f6284dfc84
Author: Mika Kuoppala <mika.kuoppala@intel.com>
Date: Fri Mar 28 10:52:46 2014 +0200
lib/drmtest: don't dup quiescent fd
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=77867
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=77875
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
Diffstat (limited to 'tests/prime_self_import.c')
-rw-r--r-- | tests/prime_self_import.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/tests/prime_self_import.c b/tests/prime_self_import.c index efdd24f6..41c203f9 100644 --- a/tests/prime_self_import.c +++ b/tests/prime_self_import.c @@ -258,6 +258,11 @@ static void test_reimport_close_race(void) int obj_count; void *status; uint32_t handle; + int fake; + + /* Allocate exit handler fds in here so that we dont screw + * up the counts */ + fake = drm_open_any(); obj_count = get_object_count(); @@ -294,6 +299,9 @@ static void test_reimport_close_race(void) obj_count = get_object_count() - obj_count; printf("leaked %i objects\n", obj_count); + + close(fake); + igt_assert(obj_count == 0); } |