modetest: make use of drmModeRmFB

We should remove the framebuffer before destroying the buffer.

Signed-off-by: Joonyoung Shim <jy0922.shim@samsung.com>
[Emil Velikov: Tweak the commit message. fb_id = 0 is unused]
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>

diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 966c2b6..66d13c5 100644 (file)
--- a/tests/modetest/modetest.c
+++ b/tests/modetest/modetest.c
@@ -1061,6 +1061,7 @@ static void set_mode(struct device *dev, struct pipe_arg *pipes, unsigned int co
 
        dev->mode.width = 0;
        dev->mode.height = 0;
+       dev->mode.fb_id = 0;
 
        for (i = 0; i < count; i++) {
                struct pipe_arg *pipe = &pipes[i];
@@ -1121,6 +1122,8 @@ static void set_mode(struct device *dev, struct pipe_arg *pipes, unsigned int co
 
 static void clear_mode(struct device *dev)
 {
+       if (dev->mode.fb_id)
+               drmModeRmFB(dev->fd, dev->mode.fb_id);
        if (dev->mode.bo)
                bo_destroy(dev->mode.bo);
 }
@@ -1261,6 +1264,7 @@ static void test_page_flip(struct device *dev, struct pipe_arg *pipes, unsigned
                drmHandleEvent(dev->fd, &evctx);
        }
 
+       drmModeRmFB(dev->fd, other_fb_id);
        bo_destroy(other_bo);
 }