Re: [Xen-devel] [PATCH] x86/HAP: don't open code clear_domain_page()

[George Dunlap <george.dunlap@xxxxxxxxxx>]

On 01/06/17 13:27, Jan Beulich wrote:
> Also drop a stray initializer.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Acked-by: George Dunlap <george.dunlap@xxxxxxxxxx>

> 
> --- a/xen/arch/x86/mm/hap/hap.c
> +++ b/xen/arch/x86/mm/hap/hap.c
> @@ -248,8 +248,7 @@ static void hap_clean_dirty_bitmap(struc
>  /************************************************/
>  static struct page_info *hap_alloc(struct domain *d)
>  {
> -    struct page_info *pg = NULL;
> -    void *p;
> +    struct page_info *pg;
>  
>      ASSERT(paging_locked_by_me(d));
>  
> @@ -259,9 +258,7 @@ static struct page_info *hap_alloc(struc
>  
>      d->arch.paging.hap.free_pages--;
>  
> -    p = __map_domain_page(pg);
> -    clear_page(p);
> -    unmap_domain_page(p);
> +    clear_domain_page(page_to_mfn(pg));
>  
>      return pg;
>  }
> 
> 
> 


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