Messages in this thread | | | From | Bart Van Assche <> | Subject | Re: [PATCH V2] megaraid: kmemleak: Track page allocation for fusion | Date | Fri, 15 Sep 2017 16:16:57 +0000 |
| |
On Fri, 2017-09-15 at 13:21 +0800, shuwang@redhat.com 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@wdc.com>
| |