Re: [PATCH v3 7/7] firmware: coreboot: Request table region for exclusive access

From: Julius Werner
Date: Thu Aug 09 2018 - 17:07:49 EST


On Thu, Aug 9, 2018 at 10:17 AM Stephen Boyd <swboyd@xxxxxxxxxxxx> wrote:
>
> Call request_mem_region() on the entire coreboot table to make sure
> other devices don't attempt to map the coreboot table in their drivers.
> If drivers need that support, it would be better to provide bus APIs
> they can use to do that through the mapping created in this file.
>

Does this prevent userspace from mapping this region via /dev/mem? If
so, let's please not do it to not break compatibility with existing
tools. (I guess an alternative would be to rewrite 'cbmem' to use
/sys/bus/coreboot/devices if available to get its coreboot table
information. But we'd still need to maintain the old path for
backwards compatibility anyway, so that would really just make it more
complicated.)