On 4/25/19 10:54 AM, Pavel Tatashin wrote: > rc = add_memory(numa_node, new_res->start, resource_size(new_res)); > - if (rc) > + if (rc) { > + release_resource(new_res); > + kfree(new_res); > return rc; > + } Looks good to me: Reviewed-by: Dave Hansen <dave.hansen@xxxxxxxxx>