Updates to address issue #36 #37
Conversation
ved-rivos
force-pushed
the
issue_36_updated
branch
from
4f52971
to
9d760a6
Compare
| @@ -414,20 +442,22 @@ the hardware unit. | |||
| {bits: 1, name: 'c'}, | |||
There was a problem hiding this comment.
I was expecting to see a set of bits associated with the error event to provide details about the specifics of the event and subsume bits like c and scrub.
There was a problem hiding this comment.
A set of bits in the status_i register are designated for future standard use and may be defined by future standard extensions for additional qualifiers.
reri_err_reporting.adoc
Outdated
| @@ -446,8 +446,7 @@ the hardware unit. | |||
| {bits: 4, name: 'ait'}, | |||
| {bits: 1, name: 'siv'}, | |||
| {bits: 1, name: 'tsv'}, | |||
| {bits: 2, name: 'WPRI'}, | |||
| {bits: 1, name: 'scrub'}, | |||
| {bits: 3, name: 'WPRI'}, | |||
There was a problem hiding this comment.
Ved, this bit has value in enabling the RAS agent to quickly determine if and what action it should take. There should be more than one bit. My desire was that it be subsumed by a set of qualifier or other readily available bits of information to support that. Since those bits are NOT part of this revision as I requested, I am concerned about removing it now.
I am fully supportive of moving and consolidating such bits to enable a later incorporation of them into a more capable qualifier rather that having them being spread across different parts of the status register, but there is still value here.
ved-rivos
force-pushed
the
issue_36_updated
branch
from
950bb5d
to
4992957
Compare
No description provided.