On 04/10/2017 19:15, Julien Grall wrote: > @@ -68,16 +72,16 @@ static inline void *cli_get_page(xen_pfn_t cmfn, unsigned > long *pcli_mfn, > > *pcli_mfn = page_to_mfn(page); > *pcli_pfp = page;
Newline. > - return map_domain_page(_mfn(*pcli_mfn)); > + return map_domain_page(*pcli_mfn); > } > > static inline void cli_put_page(void *cli_va, struct page_info *cli_pfp, > - unsigned long cli_mfn, bool mark_dirty) > + mfn_t cli_mfn, bool mark_dirty) > { > if ( mark_dirty ) > { > put_page_and_type(cli_pfp); > - paging_mark_dirty(current->domain, _mfn(cli_mfn)); > + paging_mark_dirty(current->domain, cli_mfn); > } > else > put_page(cli_pfp); > @@ -165,7 +169,7 @@ int tmem_copy_to_client(xen_pfn_t cmfn, struct page_info > *pfp, > return -EFAULT; > } > tmem_mfn = page_to_mfn(pfp); > - tmem_va = map_domain_page(_mfn(tmem_mfn)); > + tmem_va = map_domain_page(tmem_mfn); Newline. Otherwise, Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> > if ( cli_va ) > { > memcpy(cli_va, tmem_va, PAGE_SIZE); > _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel