cva6 icon indicating copy to clipboard operation
cva6 copied to clipboard

Address translation request moved from icache to frontend

Open yanicasa opened this issue 1 year ago • 6 comments

In order to aligned the ways address translation is managed between FETCH and DATA the address translation request is moved from the icache to the frontend.

The change is supposed to be transparent, no functional impact, same perf, same critical path...

yanicasa avatar Apr 30 '24 08:04 yanicasa

@cfuguet can you review this PR, maybe this modif could be useful if you are planning to design an HPICache for CVA6

yanicasa avatar Apr 30 '24 08:04 yanicasa

Hi @yanicasa, I left you some comments

Best regards

cfuguet avatar Apr 30 '24 09:04 cfuguet

:x: failed run, report available here.

github-actions[bot] avatar Apr 30 '24 10:04 github-actions[bot]

:x: failed run, report available here.

github-actions[bot] avatar May 01 '24 21:05 github-actions[bot]

@yanicasa @cfuguet I would prefer to move step by step. As the first goal of the PR was to move the address translation request from icache to frontend, can we merge this modification before submitting another PR to improve even more the design ?

JeanRochCoulon avatar May 01 '24 21:05 JeanRochCoulon

IMHO, adding a third kill signal to the instruction fetch interface is adding more complexity. And it is kind of still leaving a part of the address translation mechanism in the Icache.

cfuguet avatar May 02 '24 05:05 cfuguet

👋 Hi there!

This pull request seems inactive. Need more help or have updates? Feel free to let us know. If there are no updates within the next few days, we'll go ahead and close this PR. 😊

github-actions[bot] avatar Jun 02 '24 01:06 github-actions[bot]

Superseeded by #2179

JeanRochCoulon avatar Jun 03 '24 06:06 JeanRochCoulon