Re: [PATCH 2/2] firmware: coreboot: Export active CBFS partition

2019-10-18 Thread Julius Werner
> I don't know why we need to draw a line in the sand and say that if the > kernel doesn't need to know about it then it shouldn't parse it. I want > there to be a consistent userspace ABI that doesn't just move things > straight from memory to userspace in some binary format. I'd rather we > have

Re: [PATCH 2/2] firmware: coreboot: Export active CBFS partition

2019-10-16 Thread Stephen Boyd
Quoting Julius Werner (2019-10-10 11:37:58) > > > I'll expose the coreboot tables using a sysfs driver, which then can be > > > used by coreboot tools instead of accessing /dev/mem. As it holds the > > > FMAP and "boot media params" that's all I need for now. > > > > > > The downside is that the

Re: [PATCH 2/2] firmware: coreboot: Export active CBFS partition

2019-10-10 Thread Samuel Holland
On 10/9/19 4:19 PM, Julius Werner wrote: >> Somehow we've gotten /sys/firmware/log to be the coreboot log, and quite >> frankly that blows my mind that this path was accepted upstream. >> Userspace has to know it's running on coreboot firmware to know that >> /sys/firmware/log is actually the

Re: [PATCH 2/2] firmware: coreboot: Export active CBFS partition

2019-10-10 Thread Julius Werner
> > I'll expose the coreboot tables using a sysfs driver, which then can be > > used by coreboot tools instead of accessing /dev/mem. As it holds the > > FMAP and "boot media params" that's all I need for now. > > > > The downside is that the userspace tools need to be keep in sync with > > the

Re: [PATCH 2/2] firmware: coreboot: Export active CBFS partition

2019-10-10 Thread Stephen Boyd
-Filipe bounce Quoting patrick.rudo...@9elements.com (2019-10-10 02:46:53) > On Wed, 2019-10-09 at 14:19 -0700, Julius Werner wrote: > > > > But I also wonder why this is being exposed by the kernel at all? > > > > It's difficult for userspace tools to find out how to access the flash > and

Re: [PATCH 2/2] firmware: coreboot: Export active CBFS partition

2019-10-10 Thread patrick . rudolph
On Wed, 2019-10-09 at 14:19 -0700, Julius Werner wrote: > > Somehow we've gotten /sys/firmware/log to be the coreboot log, and > > quite > > frankly that blows my mind that this path was accepted upstream. > > Userspace has to know it's running on coreboot firmware to know > > that > >

Re: [PATCH 2/2] firmware: coreboot: Export active CBFS partition

2019-10-09 Thread Julius Werner
> Somehow we've gotten /sys/firmware/log to be the coreboot log, and quite > frankly that blows my mind that this path was accepted upstream. > Userspace has to know it's running on coreboot firmware to know that > /sys/firmware/log is actually the coreboot log. Not really sure I understand your

Re: [PATCH 2/2] firmware: coreboot: Export active CBFS partition

2019-10-08 Thread Stephen Boyd
Quoting patrick.rudo...@9elements.com (2019-10-08 04:53:26) > From: Patrick Rudolph > > Expose the name of the active CBFS partition under > /sys/firmware/cbfs_active_partition Somehow we've gotten /sys/firmware/log to be the coreboot log, and quite frankly that blows my mind that this path was