Re: [Xen-devel] [PATCH 2/4] x86/hap: Remove bogus assertion in hap_free_p2m_page()

[Tim Deegan <tim@xxxxxxx>]

At 13:10 +0100 on 24 Sep (1380028236), Andrew Cooper wrote:
> --- a/xen/arch/x86/mm/hap/hap.c
> +++ b/xen/arch/x86/mm/hap/hap.c
> @@ -289,7 +289,6 @@ static void hap_free_p2m_page(struct domain *d, struct 
> page_info *pg)
>      d->arch.paging.hap.p2m_pages--;
>      d->arch.paging.hap.total_pages++;
>      hap_free(d, page_to_mfn(pg));
> -    ASSERT(d->arch.paging.hap.p2m_pages >= 0);

Huh, I'm surprised clang didn't complain about that one.

Acked-by: Tim Deegan <tim@xxxxxxx>.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel