Hi,
On 14/08/17 15:24, Julien Grall wrote:
> The parameter 'ai' is used either for attribute index or for
> permissions. Follow-up patch will rework that parameters to carry more
> information. So rename the parameter to 'flags'.
>
> Signed-off-by: Julien Grall <julien.gr...@arm.com>
Reviewed-by: Andre Przywara <andre.przyw...@arm.com>
Cheers,
Andre.
> ---
> xen/arch/arm/mm.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/xen/arch/arm/mm.c b/xen/arch/arm/mm.c
> index c0d5fda269..411fe02842 100644
> --- a/xen/arch/arm/mm.c
> +++ b/xen/arch/arm/mm.c
> @@ -986,7 +986,7 @@ static int create_xen_entries(enum xenmap_operation op,
> unsigned long virt,
> mfn_t mfn,
> unsigned long nr_mfns,
> - unsigned int ai)
> + unsigned int flags)
> {
> int rc;
> unsigned long addr = virt, addr_end = addr + nr_mfns * PAGE_SIZE;
> @@ -1021,7 +1021,7 @@ static int create_xen_entries(enum xenmap_operation op,
> }
> if ( op == RESERVE )
> break;
> - pte = mfn_to_xen_entry(mfn, ai);
> + pte = mfn_to_xen_entry(mfn, flags);
> pte.pt.table = 1;
> write_pte(entry, pte);
> break;
> @@ -1038,8 +1038,8 @@ static int create_xen_entries(enum xenmap_operation op,
> else
> {
> pte = *entry;
> - pte.pt.ro = PTE_RO_MASK(ai);
> - pte.pt.xn = PTE_NX_MASK(ai);
> + pte.pt.ro = PTE_RO_MASK(flags);
> + pte.pt.xn = PTE_NX_MASK(flags);
> if ( !pte.pt.ro && !pte.pt.xn )
> {
> printk("%s: Incorrect combination for addr=%lx\n",
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
