Skip to content

Conversation

@Biancaa-R
Copy link

@Biancaa-R Biancaa-R commented Nov 8, 2025

  • Solves the issue : Assertion failure when force-reading a dirty register #1309
  • Added a warning message incase of force writing a dirty register.
  • Helps to see which registers are inconsistent.
  • Can later filter by this message if you suspect cache corruption.
  • A soft indication if regs are forced before initializaton.

* Added a warning message incase of force writing a dirty register.
* Helps to see which registers are inconsistent.
* Can later filter by this message if you suspect cache corruption.

Signed-off-by: Biancaa Ramesh <87770461+Biancaa-R@users.noreply.github.com>
@Biancaa-R
Copy link
Author

So as for the issue raised by @en-sc

The reg a0 0 / reg a0 force commands expect a fully functional RISC-V GDB server or real hardware.

Spike + remote_bitbang driver in OpenOCD is very minimal — it supports halt/resume, memory access, and basic register reads, but not all “force register write” operations reliably.

Assertion is a bug in OpenOCD, triggered by doing a force write before the register cache is valid.

  • So this patch aims to send a warning in case of such an issue ,instead of abruptly stopping riscv -openocd.

Avoid uninitialized register cache warning by adding proper check

Signed-off-by: Biancaa Ramesh <biancaa2210329@ssn.edu.in>
@Biancaa-R
Copy link
Author

Fix errors pointed by CI :
image

@en-sc
Copy link
Collaborator

en-sc commented Nov 11, 2025

@Biancaa-R, I believe this patch does not fix the problem, just hides the symptom.

I should have added a detailed description of the issue when creating it.
Please, take a look #1309 (comment)

@Biancaa-R
Copy link
Author

@Biancaa-R, I believe this patch does not fix the problem, just hides the symptom.

I should have added a detailed description of the issue when creating it. Please, take a look #1309 (comment)

Sure , I'll look into it and get back .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants