Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review LRM #267

Open
oladahl opened this issue Sep 7, 2021 · 1 comment
Open

Review LRM #267

oladahl opened this issue Sep 7, 2021 · 1 comment
Assignees
Milestone

Comments

@oladahl
Copy link
Contributor

oladahl commented Sep 7, 2021

Review current version of LRM

Review code comments and code documentation

In addition, review the pdf quality - we have seen problems when searching in the pdf version of the LRM

Consider also updates based on comments from WG meetings e.g. CCI parameters should not be allowed to interfere with the SystemC kernel (as pointed out by Jakob Engblom)

DoD: a change proposal for the LRM, to be implemented by the CCI WG

@oladahl oladahl mentioned this issue Sep 7, 2021
@jakob-engblom-i
Copy link

To clarify the point about CCI parameters and the kernel. This is an actual question raised during our implementation of CCI for SystemC inside of Simics.

Should it be allowed for a callback to a CCI parameter change to affect the kernel state - for example by posting events? I.e., what is the limit for side-effects from CCI parameters? Can they be used as a way to make "anything" happen in the simulation, or is there a stricter limitation? Can you use CCI parameters to do things like start a test operation in the simulated system, instead of using an explicit transaction interface for that?

I am fine with either, but we need to explicitly document it so that implementers know what to expect.

@oladahl oladahl added this to the CCI 1.0.1 milestone Oct 12, 2021
@oladahl oladahl self-assigned this Nov 23, 2021
@oladahl oladahl modified the milestones: CCI 1.0.1, CCI 1.1 Jan 25, 2022
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

No branches or pull requests

2 participants