Manual cherry-pick to earlgrey_1.0.0: [pentest] Prepare the framework for Verilator experiments #28982
+577
−864
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When executing the Verilator model of EG, setting up the GPIOs in the `pentest_init_gpio` crashes. Hence, provide a Verilator specific GPIO setup. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org> (cherry picked from commit fc9ad7a)
As the Verilator EG model does not have any analog sensors connected, disable the config for them. Moreover, disable configuring the reset manager for the crash dumps as we are not reading them anyways in the Verilator FI simulation. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org> (cherry picked from commit a1a578c)
As GPIO9 is available in the Verilator simulation of EG, use this pin as the trigger for the pentests. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org> (cherry picked from commit 181900e)
This is necessary as we are using different GPIOs for the pentest triggers in the Verilator model. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org> (cherry picked from commit ac7bf01)
For fast fault injections in Verilator, the tool needs to know when a test has completed. This commit adds a new function that raises a GPIO pin once the test has completed. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org> (cherry picked from commit d76ce01)
When conducting FI experiments in Verilator, use a default configuration for the device. This helps to reduce the com. overhead allowing us to increase the performance of Verilator FI experiments. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org> (cherry picked from commit fabef16)
As we are now also return the loc_alerts, add them to the expected response. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org> (cherry picked from commit 8edfda0)
nasahlpa
force-pushed
the
verilator_fi_backport1
branch
from
68ee3f7 to
d92b81b
Compare
As the UART and its timeout in the eg100 branch is differently configured as in master, we actually need to finish the init function with an UART response. This commit adds back the sending of the SKU config over UART to the host. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org> (commit is original to earlgrey_1.0.0)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Manual backport of #28929.