v3d: Don't leak the GPU fd for renderonly usage.

The CTS was running out of fds, because of the ro->gpu_fd never being
closed.  ro->gpu_fd should match the screen (in case the caller of
v3d_drm_screen_create_renderonly() has a scanout_for_resource() that uses
gpu_fd) and the screen is expected to close its fd at the end, fixing the
resource leak.

Fixes: e113b21cb779 ("v3d: Add renderonly support.")

diff --git a/src/gallium/winsys/v3d/drm/v3d_drm_winsys.c b/src/gallium/winsys/v3d/drm/v3d_drm_winsys.c
index a4786068bbae640d4d8cb67bc4d5e820de3dbd93..cda830f3e84390fe3a5bfcd19726fc94f50bf7f4 100644 (file)
--- a/src/gallium/winsys/v3d/drm/v3d_drm_winsys.c
+++ b/src/gallium/winsys/v3d/drm/v3d_drm_winsys.c
@@ -37,5 +37,5 @@ v3d_drm_screen_create(int fd)
 struct pipe_screen *
 v3d_drm_screen_create_renderonly(struct renderonly *ro)
 {
-       return v3d_screen_create(fcntl(ro->gpu_fd, F_DUPFD_CLOEXEC, 3), ro);
+       return v3d_screen_create(ro->gpu_fd, ro);
 }