Thread (8 messages) 8 messages, 8 authors, 2015-11-19

Re: [PATCH] tree wide: Use kvfree() than conditional kfree()/vfree()

From: Dilger, Andreas <hidden>
Date: 2015-11-09 20:41:04
Also in: dri-devel, linux-acpi, linux-fsdevel, linux-mm, lkml

On 2015/11/09, 04:56, "Tetsuo Handa" [off-list ref]
wrote:
There are many locations that do

 if (memory_was_allocated_by_vmalloc)
   vfree(ptr);
 else
   kfree(ptr);

but kvfree() can handle both kmalloc()ed memory and vmalloc()ed memory
using is_vmalloc_addr(). Unless callers have special reasons, we can
replace this branch with kvfree(). Please check and reply if you found
problems.

Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Acked-by: Michal Hocko <mhocko@suse.com>
For Lustre part:
Reviewed-by: Andreas Dilger <redacted>

Cheers, Andreas
-- 
Andreas Dilger

Lustre Principal Engineer
Intel High Performance Data Division

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help