On Fri, 2017-09-15 at 13:21 +0800, shuwang@xxxxxxxxxx wrote: > @@ -4548,9 +4556,11 @@ megasas_free_fusion_context(struct megasas_instance *instance) > > if (is_vmalloc_addr(fusion)) > vfree(fusion); > - else > + else { > + kmemleak_free(fusion); > free_pages((ulong)fusion, > instance->ctrl_context_pages); > + } > } Braces are still not balanced in the above code. This is something checkpatch should have told you. Anyway: Reviewed-by: Bart Van Assche <bart.vanassche@xxxxxxx>