search for: 920d2b5ef215

...9:5-11: inconsistent IS_ERR and PTR_ERR on line 111. PTR_ERR should access the value just tested by IS_ERR Semantic patch information: There can be false positives in the patch case, where it is the call to IS_ERR that is wrong. Generated by: scripts/coccinelle/tests/odd_ptr_err.cocci Fixes: 920d2b5ef215 ("drm/nouveau/mmu: define user interfaces to mmu vmm opertaions") Signed-off-by: Fengguang Wu <fengguang.wu at intel.com> --- Please take the patch only if it's a positive warning. Thanks! uvmm.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) --- a/drivers/gpu...

...ote: 'stack' declared here This makes the buffer large enough so it should serve the purpose that the author presumably had in mind. Alternatively we could just get rid of it completely and simplify the code at the cost of always doing the kmalloc (as we do in the current version). Fixes: 920d2b5ef215 ("drm/nouveau/mmu: define user interfaces to mmu vmm opertaions") Signed-off-by: Arnd Bergmann <arnd at arndb.de> --- Cc: Martin Sebor <msebor at gcc.gnu.org> Martin: this one is interesting, I think it qualifies as a false-positive warning that gcc should not print because t...

...e 111. > > PTR_ERR should access the value just tested by IS_ERR > > Semantic patch information: > There can be false positives in the patch case, where it is the call to > IS_ERR that is wrong. > > Generated by: scripts/coccinelle/tests/odd_ptr_err.cocci > > Fixes: 920d2b5ef215 ("drm/nouveau/mmu: define user interfaces to mmu vmm opertaions") > Signed-off-by: Fengguang Wu <fengguang.wu at intel.com> > --- > > Please take the patch only if it's a positive warning. Thanks! > > uvmm.c | 5 +++-- > 1 file changed, 3 insertions(+),...