Change in flashrom[master]: ichspi.c: Check SPI Cycle In-Progress prior start HW Seq

[Subrata Banik (Code Review)]

Attention is currently required from: Nico Huber, Caveh Jalali, Tim Wawrzynczak, Rizwan Qureshi, Edward O'Callaghan, Angel Pons, Nick Vaccaro, Alex Levin.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/61854 )

Change subject: ichspi.c: Check SPI Cycle In-Progress prior start HW Seq
......................................................................

Patch Set 2:

(1 comment)

Patchset:

PS2:
> > > While you are flashing the SPI using flashrom on DUT at S0, there are cases when we are seeing some flashrom failure issue and debug data that we have so far is not very conclusive to say that CSE is doing anything wrong.
> >
> > Was upstream flashrom used?
>
> Yup.
> > Why do you assume that the CSE is involved?
>
> Because Chrome OS needs few features that involve CSE to start it's services post booting to OS. Those services might further need to fetch the CSE active partition residing in SPI at regular intervals. We suspect flashrom operation is conflicting with such access as flashrom doesn't sync based on the SCIP bit.
> > I would expect the hardware to coordinate between the different masters.
>
> Unfortunately, it seems doesn't support multimaster protocol.
> > At least,
> > the SCIP bit can never be enough to synchronize masters. Even with the added
> > waiting loop, there is still much room for race conditions (e.g. second master
> > reading SCIP = 0 before the first posted its write to start a cycle).
>
> I believe this sync might work like this.
> When first master is creating the command and haven't set the FGO bit. There is no operation that is taking place so, 2nd master can read SCIP = 0 and if 2nd master wish to start a new command, it has still room for that. But immediately after FGO is set, the HW logic will flip the SCIP bit so, 2nd master if about to start the new command, it has to in wait loop till 1st master finishes the operation. And I have created the blocking logic. At based on the initial code analysis, CSE FW will ensure not to intrude into host cpu operation but host CPU (using flashrom) might run into issue because CSE is using SPI bus for read operation hence flashrom operation might fail.
> I guess a simple experiment could give a hint if it does: The hardware is also
> able to start cycles on its own for memory-mapped flash access. You could test
> if this toggles the SCIP bit in the host master interface. For instance, poll
> SCIP and then read from the memory-mapped BIOS area. If polling too fast is an
> issue, I'd measure the time of a short cycle, then take half of that as the > polling

delay.

I believe this scenario is valid while we are booting from SPI memory mapped memory (till we have resources are memory mapped) and not in the case of while we are running the flashrom from OS. There should not be any code which is getting fetched from XIP/SPI mapped memory while BIOS is done.