[coreboot-gerrit] [S] Change in coreboot[master]: acpi: Add missing cbfs_unmap()

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Lance Zhao, Konrad Adamczyk, Jakub Czapiga, Tim Wawrzynczak, Grzegorz Bernacki, Tim Van Patten.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74715 )

Change subject: acpi: Add missing cbfs_unmap()
......................................................................

Patch Set 3:

(1 comment)

File src/acpi/acpi.c:

https://review.coreboot.org/c/coreboot/+/74715/comment/1aea8913_01be9ffd 
PS2, Line 1934: 	cbfs_unmap(dsdt_file);
> This call doesn't actually free the memory. […]

For this change, we can cbfs_unmap(dsdt_file) after cbfs_unmap(slic_file) is done.

Beyond the scope of this change, I wonder if we can make mem_pool library flexible by freeing the buffer if it is in either last_alloc or second_to_last alloc.