[PATCH igt v7 3/4] tests/core_prop_blob: Add multiple blobs per connection

[Daniel Stone]

This should hit the bug fixed in:
XXX FIXME INSERT SEANPAUL COMMIT CITE

which was introduced with the initial blob support in:
    commit e2f5d2ea479b9b2619965d43db70939589afe43a
    Author: Daniel Stone <daniels at collabora.com>
    Date:   Fri May 22 13:34:51 2015 +0100

        drm/mode: Add user blob-creation ioctl

        Add an ioctl which allows users to create blob properties from supplied
        data. Currently this only supports modes, creating a drm_display_mode 
from
        the userspace drm_mode_modeinfo.

        v2: Removed size/type checks.
            Rebased on new patches to allow error propagation from create_blob,
            as well as avoiding double-allocation.

        Signed-off-by: Daniel Stone <daniels at collabora.com>
        Reviewed-by: Maarten Lankhorst <maarten.lankhorst at intel.com>
        Tested-by: Sean Paul <seanpaul at chromium.org>
        Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>

Signed-off-by: Daniel Stone <daniels at collabora.com>
---
 tests/core_prop_blob.c | 40 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/tests/core_prop_blob.c b/tests/core_prop_blob.c
index df4f3ad..365d728 100644
--- a/tests/core_prop_blob.c
+++ b/tests/core_prop_blob.c
@@ -208,6 +208,43 @@ test_lifetime(int fd)
 }

 static void
+test_multiple(int fd)
+{
+       uint32_t prop_ids[5];
+       int fd2;
+       int i;
+
+       fd2 = drm_open_driver(DRIVER_ANY);
+       igt_assert_fd(fd2);
+
+       /* Ensure destroying multiple properties explicitly works as needed. */
+       for (i = 0; i < ARRAY_SIZE(prop_ids); i++) {
+               prop_ids[i] = create_prop(fd2);
+               igt_assert_eq(validate_prop(fd, prop_ids[i]), 0);
+               igt_assert_eq(validate_prop(fd2, prop_ids[i]), 0);
+       }
+       for (i = 0; i < ARRAY_SIZE(prop_ids); i++) {
+               igt_assert_eq(destroy_prop(fd2, prop_ids[i]), 0);
+               igt_assert_eq(validate_prop(fd2, prop_ids[i]), ENOENT);
+       }
+       igt_assert_eq(close(fd2), 0);
+
+       fd2 = drm_open_driver(DRIVER_ANY);
+       igt_assert_fd(fd2);
+
+       /* Ensure that multiple properties get cleaned up on fd close. */
+       for (i = 0; i < ARRAY_SIZE(prop_ids); i++) {
+               prop_ids[i] = create_prop(fd2);
+               igt_assert_eq(validate_prop(fd, prop_ids[i]), 0);
+               igt_assert_eq(validate_prop(fd2, prop_ids[i]), 0);
+       }
+       igt_assert_eq(close(fd2), 0);
+
+       for (i = 0; i < ARRAY_SIZE(prop_ids); i++)
+               igt_assert_eq(validate_prop(fd, prop_ids[i]), ENOENT);
+}
+
+static void
 test_core(int fd)
 {
        uint32_t prop_id;
@@ -256,6 +293,9 @@ igt_main
        igt_subtest("blob-prop-lifetime")
                test_lifetime(fd);

+       igt_subtest("blob-multiple")
+               test_multiple(fd);
+
        igt_fixture
                close(fd);
 }
-- 
2.5.0