From: Stephen Boyd <swboyd@chromium.org> To: Julius Werner <jwerner@chromium.org> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, LKML <linux-kernel@vger.kernel.org>, Wei-Ning Huang <wnhuang@chromium.org>, Brian Norris <briannorris@chromium.org>, samuel@sholland.org Subject: Re: [PATCH v3 7/7] firmware: coreboot: Request table region for exclusive access Date: Thu, 09 Aug 2018 19:54:27 -0700 [thread overview] Message-ID: <153386966793.37448.12640092735399834661@swboyd.mtv.corp.google.com> (raw) In-Reply-To: <CAODwPW8+QSittJdNLWpWKU=JNJa5bssMHQ4ER4CfnUAWaL1JGA@mail.gmail.com> What's with the top posting? ;-) Quoting Julius Werner (2018-08-09 16:44:43) > Actually, looking at what IO_STRICT_DEVMEM really does, would it > really prevent userspace accesses to these areas? Because it seems > that it only prevents accesses to areas marked as IORESOURCE_BUSY, and > while I can't fully follow how the kernel assigns that, comments > suggest that this is only set when "Driver has marked this resource > busy". Requesting the memory region as is done in this patch marks it as busy. > > So after you make the change to the other patch where we immediately > unmap the coreboot table again at the end of the probe() function, > shouldn't it become available to userspace again even with > IO_STRICT_DEVMEM set? Yes, if we unmap the region immediately after devices are populated then this whole point is moot with the assumption that this code isn't running at the same time as the cbmem utility. I've done this already and it is working on arm. I still need to build/boot/test on an x86 platform which I should be able to do tomorrow.
next prev parent reply other threads:[~2018-08-10 2:54 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-08-09 17:17 [PATCH v3 0/7] firmware: coreboot: Fix probe and simplify code Stephen Boyd 2018-08-09 17:17 ` [PATCH v3 1/7] firmware: coreboot: Let OF core populate platform device Stephen Boyd 2018-08-09 17:31 ` Brian Norris 2018-08-09 17:17 ` [PATCH v3 2/7] firmware: coreboot: Unmap ioregion on failure Stephen Boyd 2018-08-09 17:49 ` Brian Norris 2018-08-09 19:40 ` Stephen Boyd 2018-08-09 19:52 ` Brian Norris 2018-08-09 23:25 ` Stephen Boyd 2018-08-09 17:17 ` [PATCH v3 3/7] firmware: coreboot: Make bus registration symmetric Stephen Boyd 2018-08-09 18:10 ` Julius Werner 2018-08-09 23:30 ` Stephen Boyd 2018-08-09 17:17 ` [PATCH v3 4/7] firmware: coreboot: Collapse platform drivers into bus core Stephen Boyd 2018-08-09 17:17 ` [PATCH v3 5/7] firmware: coreboot: Remap RAM with memremap() instead of ioremap() Stephen Boyd 2018-08-09 18:24 ` Julius Werner 2018-08-09 22:07 ` Stephen Boyd 2018-08-09 17:17 ` [PATCH v3 6/7] firmware: coreboot: Only populate devices in coreboot_table_init() Stephen Boyd 2018-08-09 21:02 ` Julius Werner 2018-08-09 23:43 ` Stephen Boyd 2018-08-09 17:17 ` [PATCH v3 7/7] firmware: coreboot: Request table region for exclusive access Stephen Boyd 2018-08-09 21:07 ` Julius Werner 2018-08-09 23:03 ` Stephen Boyd 2018-08-09 23:37 ` Julius Werner 2018-08-09 23:44 ` Julius Werner 2018-08-10 2:54 ` Stephen Boyd [this message] 2018-08-10 23:24 ` Stephen Boyd 2018-08-09 18:03 ` [PATCH v3 0/7] firmware: coreboot: Fix probe and simplify code Brian Norris
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=153386966793.37448.12640092735399834661@swboyd.mtv.corp.google.com \ --to=swboyd@chromium.org \ --cc=briannorris@chromium.org \ --cc=gregkh@linuxfoundation.org \ --cc=jwerner@chromium.org \ --cc=linux-kernel@vger.kernel.org \ --cc=samuel@sholland.org \ --cc=wnhuang@chromium.org \ --subject='Re: [PATCH v3 7/7] firmware: coreboot: Request table region for exclusive access' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).