Re: [PATCH i-g-t 1/4] lib: Add some syncobj helpers (v2)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 28, 2017 at 02:10:12PM -0700, Jason Ekstrand wrote:
> Signed-off-by: Jason Ekstrand <jason@xxxxxxxxxxxxxx>

gtkdocs would be lovely for this, if I can motivate you ...

Thanks, Daniel

> ---
>  lib/Makefile.sources |   2 +
>  lib/igt_syncobj.c    | 207 +++++++++++++++++++++++++++++++++++++++++++++++++++
>  lib/igt_syncobj.h    |  71 ++++++++++++++++++
>  3 files changed, 280 insertions(+)
>  create mode 100644 lib/igt_syncobj.c
>  create mode 100644 lib/igt_syncobj.h
> 
> diff --git a/lib/Makefile.sources b/lib/Makefile.sources
> index 53fdb54..692fe30 100644
> --- a/lib/Makefile.sources
> +++ b/lib/Makefile.sources
> @@ -83,6 +83,8 @@ lib_source_list =	 	\
>  	uwildmat/uwildmat.c	\
>  	igt_kmod.c		\
>  	igt_kmod.h		\
> +	igt_syncobj.c		\
> +	igt_syncobj.h		\
>  	$(NULL)
>  
>  .PHONY: version.h.tmp
> diff --git a/lib/igt_syncobj.c b/lib/igt_syncobj.c
> new file mode 100644
> index 0000000..5caef2a
> --- /dev/null
> +++ b/lib/igt_syncobj.c
> @@ -0,0 +1,207 @@
> +/*
> + * Copyright © 2017 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> + * IN THE SOFTWARE.
> + */
> +
> +#include <errno.h>
> +#include <xf86drm.h>
> +
> +#include "igt.h"
> +#include "igt_syncobj.h"
> +
> +static int
> +__syncobj_create(int fd, uint32_t *handle, uint32_t flags)
> +{
> +	struct drm_syncobj_create create = { 0 };
> +	int err = 0;
> +
> +	create.flags = flags;
> +	if (drmIoctl(fd, DRM_IOCTL_SYNCOBJ_CREATE, &create))
> +		err = -errno;
> +	*handle = create.handle;
> +	return err;
> +}
> +
> +uint32_t
> +syncobj_create(int fd, uint32_t flags)
> +{
> +	uint32_t handle;
> +	igt_assert_eq(__syncobj_create(fd, &handle, flags), 0);
> +	igt_assert(handle);
> +	return handle;
> +}
> +
> +static int
> +__syncobj_destroy(int fd, uint32_t handle)
> +{
> +	struct drm_syncobj_destroy destroy = { 0 };
> +	int err = 0;
> +
> +	destroy.handle = handle;
> +	if (drmIoctl(fd, DRM_IOCTL_SYNCOBJ_DESTROY, &destroy))
> +		err = -errno;
> +	return err;
> +}
> +
> +void
> +syncobj_destroy(int fd, uint32_t handle)
> +{
> +	igt_assert_eq(__syncobj_destroy(fd, handle), 0);
> +}
> +
> +int
> +__syncobj_handle_to_fd(int fd, struct drm_syncobj_handle *args)
> +{
> +	int err = 0;
> +	if (drmIoctl(fd, DRM_IOCTL_SYNCOBJ_HANDLE_TO_FD, args))
> +		err = -errno;
> +	return err;
> +}
> +
> +int
> +syncobj_handle_to_fd(int fd, uint32_t handle, uint32_t flags)
> +{
> +	struct drm_syncobj_handle args = { 0 };
> +	args.handle = handle;
> +	args.flags = flags;
> +	igt_assert_eq(__syncobj_handle_to_fd(fd, &args), 0);
> +	igt_assert(args.fd >= 0);
> +	return args.fd;
> +}
> +
> +int
> +__syncobj_fd_to_handle(int fd, struct drm_syncobj_handle *args)
> +{
> +	int err = 0;
> +	if (drmIoctl(fd, DRM_IOCTL_SYNCOBJ_FD_TO_HANDLE, args))
> +		err = -errno;
> +	return err;
> +}
> +
> +uint32_t
> +syncobj_fd_to_handle(int fd, int syncobj_fd, uint32_t flags)
> +{
> +	struct drm_syncobj_handle args = { 0 };
> +	args.fd = syncobj_fd;
> +	args.flags = flags;
> +	igt_assert_eq(__syncobj_fd_to_handle(fd, &args), 0);
> +	igt_assert(args.handle > 0);
> +	return args.handle;
> +}
> +
> +void
> +syncobj_import_sync_file(int fd, uint32_t handle, int sync_file)
> +{
> +	struct drm_syncobj_handle args = { 0 };
> +	args.handle = handle;
> +	args.fd = sync_file;
> +	args.flags = DRM_SYNCOBJ_FD_TO_HANDLE_FLAGS_IMPORT_SYNC_FILE;
> +	igt_assert_eq(__syncobj_fd_to_handle(fd, &args), 0);
> +}
> +
> +int
> +__syncobj_wait(int fd, struct local_syncobj_wait *args)
> +{
> +	int err = 0;
> +	if (drmIoctl(fd, LOCAL_IOCTL_SYNCOBJ_WAIT, args))
> +		err = -errno;
> +	return err;
> +}
> +
> +int
> +syncobj_wait_err(int fd, uint32_t *handles, uint32_t count,
> +		 uint64_t abs_timeout_nsec, uint32_t flags)
> +{
> +	struct local_syncobj_wait wait;
> +
> +	wait.handles = to_user_pointer(handles);
> +	wait.timeout_nsec = abs_timeout_nsec;
> +	wait.count_handles = count;
> +	wait.flags = flags;
> +	wait.first_signaled = 0;
> +	wait.pad = 0;
> +
> +	return __syncobj_wait(fd, &wait);
> +}
> +
> +bool
> +syncobj_wait(int fd, uint32_t *handles, uint32_t count,
> +	     uint64_t abs_timeout_nsec, uint32_t flags,
> +	     uint32_t *first_signaled)
> +{
> +	struct local_syncobj_wait wait;
> +	int ret;
> +
> +	wait.handles = to_user_pointer(handles);
> +	wait.timeout_nsec = abs_timeout_nsec;
> +	wait.count_handles = count;
> +	wait.flags = flags;
> +	wait.first_signaled = 0;
> +	wait.pad = 0;
> +
> +	ret = __syncobj_wait(fd, &wait);
> +	if (ret == ETIME)
> +		return false;
> +
> +	igt_assert_eq(ret, 0);
> +	if (first_signaled)
> +		*first_signaled = wait.first_signaled;
> +
> +	return true;
> +}
> +
> +static int
> +__syncobj_reset(int fd, uint32_t *handles, uint32_t count)
> +{
> +	struct local_syncobj_array array = { 0 };
> +	int err = 0;
> +
> +	array.handles = to_user_pointer(handles);
> +	array.count_handles = count;
> +	if (drmIoctl(fd, LOCAL_IOCTL_SYNCOBJ_RESET, &array))
> +		err = -errno;
> +	return err;
> +}
> +
> +void
> +syncobj_reset(int fd, uint32_t *handles, uint32_t count)
> +{
> +	igt_assert_eq(__syncobj_reset(fd, handles, count), 0);
> +}
> +
> +static int
> +__syncobj_signal(int fd, uint32_t *handles, uint32_t count)
> +{
> +	struct local_syncobj_array array = { 0 };
> +	int err = 0;
> +
> +	array.handles = to_user_pointer(handles);
> +	array.count_handles = count;
> +	if (drmIoctl(fd, LOCAL_IOCTL_SYNCOBJ_SIGNAL, &array))
> +		err = -errno;
> +	return err;
> +}
> +
> +void
> +syncobj_signal(int fd, uint32_t *handles, uint32_t count)
> +{
> +	igt_assert_eq(__syncobj_signal(fd, handles, count), 0);
> +}
> diff --git a/lib/igt_syncobj.h b/lib/igt_syncobj.h
> new file mode 100644
> index 0000000..44d1378
> --- /dev/null
> +++ b/lib/igt_syncobj.h
> @@ -0,0 +1,71 @@
> +/*
> + * Copyright © 2017 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> + * IN THE SOFTWARE.
> + */
> +
> +#ifndef IGT_SYNCOBJ_H
> +#define IGT_SYNCOBJ_H
> +
> +#include <stdint.h>
> +#include <stdbool.h>
> +#include <drm.h>
> +
> +#define LOCAL_SYNCOBJ_CREATE_SIGNALED (1 << 0)
> +
> +#define LOCAL_SYNCOBJ_WAIT_FLAGS_WAIT_ALL (1 << 0)
> +#define LOCAL_SYNCOBJ_WAIT_FLAGS_WAIT_FOR_SUBMIT (1 << 1)
> +struct local_syncobj_wait {
> +       __u64 handles;
> +       /* absolute timeout */
> +       __s64 timeout_nsec;
> +       __u32 count_handles;
> +       __u32 flags;
> +       __u32 first_signaled; /* only valid when not waiting all */
> +       __u32 pad;
> +};
> +
> +struct local_syncobj_array {
> +       __u64 handles;
> +       __u32 count_handles;
> +       __u32 pad;
> +};
> +
> +#define LOCAL_IOCTL_SYNCOBJ_WAIT	DRM_IOWR(0xC3, struct local_syncobj_wait)
> +#define LOCAL_IOCTL_SYNCOBJ_RESET	DRM_IOWR(0xC4, struct local_syncobj_array)
> +#define LOCAL_IOCTL_SYNCOBJ_SIGNAL	DRM_IOWR(0xC5, struct local_syncobj_array)
> +
> +uint32_t syncobj_create(int fd, uint32_t flags);
> +void syncobj_destroy(int fd, uint32_t handle);
> +int __syncobj_handle_to_fd(int fd, struct drm_syncobj_handle *args);
> +int __syncobj_fd_to_handle(int fd, struct drm_syncobj_handle *args);
> +int syncobj_handle_to_fd(int fd, uint32_t handle, uint32_t flags);
> +uint32_t syncobj_fd_to_handle(int fd, int syncobj_fd, uint32_t flags);
> +void syncobj_import_sync_file(int fd, uint32_t handle, int sync_file);
> +int __syncobj_wait(int fd, struct local_syncobj_wait *args);
> +int syncobj_wait_err(int fd, uint32_t *handles, uint32_t count,
> +		     uint64_t abs_timeout_nsec, uint32_t flags);
> +bool syncobj_wait(int fd, uint32_t *handles, uint32_t count,
> +		  uint64_t abs_timeout_nsec, uint32_t flags,
> +		  uint32_t *first_signaled);
> +void syncobj_reset(int fd, uint32_t *handles, uint32_t count);
> +void syncobj_signal(int fd, uint32_t *handles, uint32_t count);
> +
> +#endif /* IGT_SYNCOBJ_H */
> -- 
> 2.5.0.400.gff86faf
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux