Re: [PATCH] firmware: export x86_64 platform flash bios region via sysfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Thu, Nov 11, 2021 at 11:42:52AM -0300, Mauro Lima wrote:
> > > > Having said that the hardware sequencer used in the recent CPUs should
> > > > be much safer in that sense.
> > >
> > > FWIW, I'd be fine if we had RO access for HWSEQ flash access only. If
> > > I understood correctly that's what Mauro proposed (with a patch) and
> > > instead was told that it was being rewritten as a mtd driver
> > > completion time unknown.
> >
> > I think Mauro proposed something different, basically exposing RO parts
> > of the driver only.
> 
> My patch was intended to move the read functionality of the spi chip
> to be able to compile the driver with just that and then remove the
> dangerous tag. So we can use that functionality to read the flash, I'm
> missing what is different from the things being discussed here sorry.

I'm hinting that we could make this "non-DANGEROUS" for hardware
sequencer parts of the driver. Basically moving only the software
sequencer bits as DANGEROUS or something like that. The hardware
sequencer is much more safer because it does not allow to run random
opcodes.

In case someone is unfamiliar with this, the Intel SPI hardware
exposes two interfaces through the same controller. One that is called
software sequencer and there is a register of "allowed" opcodes that
software can use as it wishes. This register can be locked down but is
not always. The second interface is the hardware sequencer that only
exposes higher level commands like read, write and so on and internally
then executes whatever opcode the controller got from the chip
"supported opcodes table" (SFDP).  The recent Intel hardware, all
big-cores, only provide hardware sequencer and the software one is not
even available.

Regardless of all this the driver needs to be converted from MTD to SPI
(SPI MEM) before we can add any features. I'm planning to send v4 of
that series next week.



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux