Patch Set 2:

I know it's a lot of work, but I think all the changes in this
series (even individual bits) should be tested separately if they
fix anything.

It's quite difficult to do given that we have no idea what these bits mean. Or by testing you mean running it for a couple of days and making sure the system is still stable?

Basically yes, do changes step by step until your problematic system
is stable. If we merge this without detailed information which parts
actually fix something, and then something goes wrong, it would be
reverted as a whole.

Okay. But if in the end I'll find out that my crashes are fixed by one or two or three lock bits (not all four) moved to the end, and the reference code sets them all in the end (which actually makes sense to me), I hope we agree that we should move all four locks to the end?

I honestly don't care what we do as long as it is properly documented.

OTOH, if it turns out that all register writes are effective even if
we don't move the locking, I see no reason to move it (assume the
registers can be dumped).

Well, maybe there's a chance that somebody in possession of SA BIOS Spec will help us documenting this... :)

View Change

To view, visit change 42455. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I31f8cddb9408d0c23c558b51f5d1f8787f0b700f
Gerrit-Change-Number: 42455
Gerrit-PatchSet: 2
Gerrit-Owner: Evgeny Zinoviev <me@ch1p.io>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Nico Huber <nico.h@gmx.de>
Gerrit-Comment-Date: Sun, 21 Jun 2020 19:14:56 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment