Use __anon_inode_getfd instead of opencoding the logic using
get_unused_fd_flags + anon_inode_getfile.

Signed-off-by: Christoph Hellwig <h...@lst.de>
---
 drivers/gpu/drm/drm_syncobj.c | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/drm_syncobj.c b/drivers/gpu/drm/drm_syncobj.c
index 42d46414f7679..b69a7be34e8c7 100644
--- a/drivers/gpu/drm/drm_syncobj.c
+++ b/drivers/gpu/drm/drm_syncobj.c
@@ -581,18 +581,11 @@ int drm_syncobj_get_fd(struct drm_syncobj *syncobj, int 
*p_fd)
        struct file *file;
        int fd;
 
-       fd = get_unused_fd_flags(O_CLOEXEC);
+       fd = __anon_inode_getfd("syncobj_file", &drm_syncobj_file_fops,
+                       syncobj, O_CLOEXEC, &file);
        if (fd < 0)
                return fd;
 
-       file = anon_inode_getfile("syncobj_file",
-                                 &drm_syncobj_file_fops,
-                                 syncobj, 0);
-       if (IS_ERR(file)) {
-               put_unused_fd(fd);
-               return PTR_ERR(file);
-       }
-
        drm_syncobj_get(syncobj);
        fd_install(fd, file);
 
-- 
2.26.2

Reply via email to