[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 1/2] dmar: iommu mem leak fix



>>> On 20.08.18 at 12:42, <zhenzhong.duan@xxxxxxxxxx> wrote:
> Release memory allocated for drhd iommu in error path.
> 
> -v2: fixup wrong parameter hiden due to my removing -Werror

This line belongs ...

> Signed-off-by: Zhenzhong Duan <zhenzhong.duan@xxxxxxxxxx>
> ---

... here, so it won't end up in the commit message. Still

Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>

And as an advice - removing -Werror is pretty strongly to be
advised against.

Finally, while on v1 the Cc list was too short, it ended up too
long now: There's no reason for the TXT maintainers to be Cc-ed
on this patch, only patch 2 is in needed of this.

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.