Hi Emil,
This patch and the nine part of the second patch look fine.
Reviewed-by: Axel Davy <[email protected]>
for them.
Yours,
Axel
On 29/08/2018 19:13, Emil Velikov wrote:
From: Emil Velikov <[email protected]>
As the newly introduced comment says:
The pipe loader takes ownership of the fd
Thus, there's no need to close it again.
Cc: Patrick Rudolph <[email protected]>
Cc: Axel Davy <[email protected]>
Cc: [email protected]>
Signed-off-by: Emil Velikov <[email protected]>
---
src/gallium/targets/d3dadapter9/drm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/targets/d3dadapter9/drm.c
b/src/gallium/targets/d3dadapter9/drm.c
index 85b3e10633e..a2a36dbbda9 100644
--- a/src/gallium/targets/d3dadapter9/drm.c
+++ b/src/gallium/targets/d3dadapter9/drm.c
@@ -107,7 +107,7 @@ drm_destroy( struct d3dadapter9_context *ctx )
if (drm->dev)
pipe_loader_release(&drm->dev, 1);
- close(drm->fd);
+ /* The pipe loader takes ownership of the fd */
FREE(ctx);
}
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
