Hi Stefano, On 24/10/2020 01:16, Stefano Stabellini wrote:
On Fri, 23 Oct 2020, Julien Grall wrote:bool __acpi_unmap_table(const void *ptr, unsigned long size) { vaddr_t vaddr = (vaddr_t)ptr; + unsigned int idx; + + /* We are only handling fixmap address in the arch code */ + if ( (vaddr < FIXMAP_ADDR(FIXMAP_ACPI_BEGIN)) || + (vaddr >= FIXMAP_ADDR(FIXMAP_ACPI_END)) )Is it missing "+ PAGE_SIZE"? if ( (vaddr < FIXMAP_ADDR(FIXMAP_ACPI_BEGIN)) || (vaddr >= FIXMAP_ADDR(FIXMAP_ACPI_END) + PAGE_SIZE) )
Yes it should be + PAGE_SIZE. Do you mind if I fix it on commit? Cheers, -- Julien Grall
