Quoting Deepak Rawat (2018-09-06 01:03:47) > vmwgfx does not support GEM interface so calling gem_close on vmwgfx > results in error. Call dumb destroy IOCTL in case have dumb buffer. > > Signed-off-by: Deepak Rawat <dra...@vmware.com> > --- > lib/igt_fb.c | 5 ++++- > lib/igt_kms.c | 15 +++++++++++++++ > lib/igt_kms.h | 1 + > 3 files changed, 20 insertions(+), 1 deletion(-) > > diff --git a/lib/igt_fb.c b/lib/igt_fb.c > index ae71d967..ba995a1a 100644 > --- a/lib/igt_fb.c > +++ b/lib/igt_fb.c > @@ -1920,7 +1920,10 @@ void igt_remove_fb(int fd, struct igt_fb *fb) > > cairo_surface_destroy(fb->cairo_surface); > do_or_die(drmModeRmFB(fd, fb->fb_id)); > - gem_close(fd, fb->gem_handle); > + if (fb->is_dumb) > + kmstest_dumb_destroy(fd, fb->gem_handle); > + else > + gem_close(fd, fb->gem_handle); > fb->fb_id = 0; > } > > diff --git a/lib/igt_kms.c b/lib/igt_kms.c > index 62d84684..9e9414cf 100644 > --- a/lib/igt_kms.c > +++ b/lib/igt_kms.c > @@ -639,6 +639,21 @@ void *kmstest_dumb_map_buffer(int fd, uint32_t handle, > uint64_t size, > return ptr; > } > > +/** > + * kmstest_dumb_destroy: > + * @fd: Opened drm file descriptor > + * @handle: Offset in the file referred to by fd > + */ > +void kmstest_dumb_destroy(int fd, uint32_t handle) > +{ > + struct drm_mode_destroy_dumb arg = {}; > + > + igt_assert_neq(handle, 0);
Don't bother doing the kernel's job for it. Abusing the ioctl iface is the name of the game. > + arg.handle = handle; > + do_ioctl(fd, DRM_IOCTL_MODE_DESTROY_DUMB, &arg); struct drm_mode_destroy_dumb arg = { handle }; Try never to use do_ioctl() if you ever want to debug an error. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx