| Summary: | sync instruction breaks insndb traversal | ||
|---|---|---|---|
| Product: | Libre-SOC's first SoC | Reporter: | Dmitry Selyutin <ghostmansd> |
| Component: | Source Code | Assignee: | Jacob Lifshay <programmerjake> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | libre-soc-bugs, programmerjake |
| Priority: | --- | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Windows | ||
| See Also: |
https://bugs.libre-soc.org/show_bug.cgi?id=980 https://bugs.libre-soc.org/show_bug.cgi?id=1253 |
||
| NLnet milestone: | --- | total budget (EUR) for completion of task and all subtasks: | 0 |
| budget (EUR) for this task, excluding subtasks' budget: | 0 | parent task for budget allocation: | |
| child tasks for budget allocation: | The table of payments (in EUR) for this task; TOML format: | ||
Dropped blocking status, decreased priority, since I can skip it by comparing the name. (In reply to Dmitry Selyutin from comment #1) > Dropped blocking status, decreased priority, since I can skip it by > comparing the name. don't bother, I'm currently fixing this and should be done in a few min... https://git.libre-soc.org/?p=openpower-isa.git;a=commitdiff;h=95d3aa48bd080b75f7301784581a0c7541c3e7fc commit 95d3aa48bd080b75f7301784581a0c7541c3e7fc Author: Jacob Lifshay <programmerjake@gmail.com> Date: Tue Jan 16 01:18:58 2024 -0800 fixedsync.mdwn: change sync from v3.1B version to v3.0B version to use the v3.1B version requires fixing fields.text to use the new L and SC fields, which requires deciphering all the places where the current L (named L2) field is used and making sure they'll be correct still. using the old interpretation of sync should be correct, but less optimal, assuming OpenPower got their backwards compatibility correct. |
Attempting to traverse insndb instruction and all dynamic operands, I observe the following: File "/home/ghostmansd/src/openpower-isa/src/openpower/insndb/core.py", line 1007, in span return self.record.fields[self.name] KeyError: 'SC' The problem can be reproduced with $(pysvp64db operands sync). Please either rollback this commit or populate fields.text with SC operand.