Re: [Xen-devel] [PATCH] arm/acpi: Add __acpi_unmap_table function for ARM

[Wei Xu <xuwei5@xxxxxxxxxxxxx>]

Hi Alexandru,

On 2020/1/21 18:01, Alexandru Stefan ISAILA wrote:
> 
> 
> On 21.01.2020 11:49, Wei Xu wrote:
>> Add __acpi_unmap_table function for ARM and invoke it at acpi_os_unmap_memory
>> to make sure the related fixmap has been cleared before using it for a
>> different mapping.
>>
>> Signed-off-by: Wei Xu<xuwei5@xxxxxxxxxxxxx>
>> ---
>>   xen/arch/arm/acpi/lib.c | 25 +++++++++++++++++++++++++
>>   xen/drivers/acpi/osl.c  |  2 ++
>>   xen/include/xen/acpi.h  |  1 +
>>   3 files changed, 28 insertions(+)
>>
>> diff --git a/xen/arch/arm/acpi/lib.c b/xen/arch/arm/acpi/lib.c
>> index 4fc6e17..69e87ec 100644
>> --- a/xen/arch/arm/acpi/lib.c
>> +++ b/xen/arch/arm/acpi/lib.c
>> @@ -49,6 +49,31 @@ char *__acpi_map_table(paddr_t phys, unsigned long size)
>>       return ((char *) base + offset);
>>   }
>>   
>> +void __acpi_unmap_table(void __iomem * virt, unsigned long size)
> 
> You have a stray space here after "*"

Sorry, I will remove it.

> 
>> +{
>> +    unsigned long base, end;
>> +    int idx;
>> +
>> +    base = FIXMAP_ADDR(FIXMAP_ACPI_BEGIN);
>> +    end = FIXMAP_ADDR(FIXMAP_ACPI_END);
> 
> Maybe the initialization can be moved to the declaration?

OK.
I will move it.

Thanks!

Best Regards,
Wei

> 
> 
> Alex
> 


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