From mboxrd@z Thu Jan 1 00:00:00 1970 From: Damien Lespiau Subject: [PATCH i-g-t] lib/fb: Assert, instead of silently failing, when creating fbs Date: Thu, 21 Aug 2014 15:02:40 +0100 Message-ID: <1408629760-20255-1-git-send-email-damien.lespiau@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id 657236E197 for ; Thu, 21 Aug 2014 07:03:22 -0700 (PDT) List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org We were either returning 0, or a negative value cast to an unsigned int for errors and the clients of that API weren't exactly checking anything. We're in luck, we can take shortcuts in a testing library to just assert when an expected error occurs. Signed-off-by: Damien Lespiau --- lib/igt_fb.c | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/lib/igt_fb.c b/lib/igt_fb.c index f383970..b8448c8 100644 --- a/lib/igt_fb.c +++ b/lib/igt_fb.c @@ -396,8 +396,7 @@ void igt_paint_image(cairo_t *cr, const char *filename, * for rgb pixel formats. * * Returns: - * The kms id of the created framebuffer on success or a negative error code on - * failure. + * The kms id of the created framebuffer. */ unsigned int igt_create_fb_with_bo_size(int fd, int width, int height, @@ -416,20 +415,16 @@ igt_create_fb_with_bo_size(int fd, int width, int height, bpp = igt_drm_format_to_bpp(format); ret = create_bo_for_fb(fd, width, height, bpp, tiling, &fb->gem_handle, &fb->size, &fb->stride, bo_size); - if (ret < 0) - return ret; + igt_assert(ret == 0); memset(handles, 0, sizeof(handles)); handles[0] = fb->gem_handle; memset(pitches, 0, sizeof(pitches)); pitches[0] = fb->stride; memset(offsets, 0, sizeof(offsets)); - if (drmModeAddFB2(fd, width, height, format, handles, pitches, - offsets, &fb_id, 0) < 0) { - gem_close(fd, fb->gem_handle); - - return 0; - } + ret = drmModeAddFB2(fd, width, height, format, handles, pitches, + offsets, &fb_id, 0); + igt_assert(ret == 0); fb->width = width; fb->height = height; @@ -457,8 +452,7 @@ igt_create_fb_with_bo_size(int fd, int width, int height, * for rgb pixel formats. * * Returns: - * The kms id of the created framebuffer on success or a negative error code on - * failure. + * The kms id of the created framebuffer. */ unsigned int igt_create_fb(int fd, int width, int height, uint32_t format, unsigned int tiling, struct igt_fb *fb) -- 1.8.3.1