[PATCH 00/13] coverity
Hi On Sat, Apr 5, 2014 at 11:44 AM, Daniel Vetter wrote: > Hi all, > > Rainy w/e here and I got a bit bored, so looked at coverity again. I've closed > all outstanding issues in drivers/gpu now either as false positives or fixed > in > this series expect for vmwgfx/ttm stuff (not enough clue) and one insane > savage > init issue (no desire to wake dragons today). All patches besides "drm/ast: Remove dead code from cbr_scan2" are: Reviewed-by: David Herrmann However, a few notes: - ttm_bo_unref() sets "bo = NULL;", so the following condition is always true. Your commit-message argues with an unconditional kref_unref(). I think that's misleading (but still true) as that does not change the fact that bo is always NULL afterwards. - I am a big fan of "if (!obj) return;" in destructors. It simplifies error-paths considerably and we can call them despite objects being already cleared. But that's just personal style, and I don't deal much with TTM anyway. But generally, I think we shouldn't remove these checks blindly. - The AST patch should be reviewed by someone who knows that hw. The code is obviously wrong, but that doesn't mean we cannot fix it properly. Thanks for the cleanups, the coverity reports have been pending here for months.. David
[PATCH 00/13] coverity
Hi all, Rainy w/e here and I got a bit bored, so looked at coverity again. I've closed all outstanding issues in drivers/gpu now either as false positives or fixed in this series expect for vmwgfx/ttm stuff (not enough clue) and one insane savage init issue (no desire to wake dragons today). Cheers, Daniel Daniel Vetter (13): drm/mgag200: Remove unnecessary NULL check in bo_unref drm/mgag200: Remove unecessary NULL check in gem_free drm/cirrus: Remove unnecessary NULL check in bo_unref drm/cirrus: Remove unecessary NULL check in gem_free drm/ast: Remove unnecessary NULL check in bo_unref drm/mgag200: Remove unecessary NULL check in gem_free drm/via: Remove unecessary NULL check drm/ast: Remove dead code from cbr_scan2 drm/udl: Initialize ret in udl_driver_load drm/bochs: Remove unnecessary NULL check in bo_unref drm/bochs: Remove unecessary NULL check in gem_free drm/i2c/tda998x: Fix signed overflow issue drm: Fix error handling in drm_master_create drivers/gpu/drm/ast/ast_main.c | 7 ++- drivers/gpu/drm/ast/ast_post.c | 2 -- drivers/gpu/drm/bochs/bochs_mm.c | 6 +- drivers/gpu/drm/cirrus/cirrus_main.c | 6 +- drivers/gpu/drm/drm_stub.c | 5 - drivers/gpu/drm/i2c/tda998x_drv.c | 2 +- drivers/gpu/drm/mgag200/mgag200_main.c | 6 +- drivers/gpu/drm/udl/udl_main.c | 1 + drivers/gpu/drm/via/via_mm.c | 2 +- 9 files changed, 12 insertions(+), 25 deletions(-) -- 1.8.5.2
[PATCH 00/13] coverity
Hi all, Rainy w/e here and I got a bit bored, so looked at coverity again. I've closed all outstanding issues in drivers/gpu now either as false positives or fixed in this series expect for vmwgfx/ttm stuff (not enough clue) and one insane savage init issue (no desire to wake dragons today). Cheers, Daniel Daniel Vetter (13): drm/mgag200: Remove unnecessary NULL check in bo_unref drm/mgag200: Remove unecessary NULL check in gem_free drm/cirrus: Remove unnecessary NULL check in bo_unref drm/cirrus: Remove unecessary NULL check in gem_free drm/ast: Remove unnecessary NULL check in bo_unref drm/mgag200: Remove unecessary NULL check in gem_free drm/via: Remove unecessary NULL check drm/ast: Remove dead code from cbr_scan2 drm/udl: Initialize ret in udl_driver_load drm/bochs: Remove unnecessary NULL check in bo_unref drm/bochs: Remove unecessary NULL check in gem_free drm/i2c/tda998x: Fix signed overflow issue drm: Fix error handling in drm_master_create drivers/gpu/drm/ast/ast_main.c | 7 ++- drivers/gpu/drm/ast/ast_post.c | 2 -- drivers/gpu/drm/bochs/bochs_mm.c | 6 +- drivers/gpu/drm/cirrus/cirrus_main.c | 6 +- drivers/gpu/drm/drm_stub.c | 5 - drivers/gpu/drm/i2c/tda998x_drv.c | 2 +- drivers/gpu/drm/mgag200/mgag200_main.c | 6 +- drivers/gpu/drm/udl/udl_main.c | 1 + drivers/gpu/drm/via/via_mm.c | 2 +- 9 files changed, 12 insertions(+), 25 deletions(-) -- 1.8.5.2