Skip to content

Conversation

@gurcei
Copy link
Collaborator

@gurcei gurcei commented Feb 22, 2025

An initial attempt to update the 'r' command to show the currently selected MB.

This will be a draft PR just to allow for @gardners to assess how I went about implementing it, to see if he has any concerns with the approach.

More discussions on this topic were in the discord thread here:

@gurcei gurcei requested a review from gardners February 22, 2025 22:48
@gurcei
Copy link
Collaborator Author

gurcei commented Feb 23, 2025

Got some additional thoughts from gardners on discord:

gardners — Today at 10:34 AM
In principle should work. Note that a BRAM is 4KB, so if you have increased the size, it will need more BRAM, and will break all A100T targets. So keep the total size no larger than it was.
Also, note that we have CPU tracing via ethernet as well. It limits CPU to ~5MHz when running, but logs continuously.

Okies, as I reduced the history depth from 1024 to 512, overall, the total ram size has dropped from 24 KB to 16 KB.

I've never tried cpu tracing via ethernet before, maybe I'd better learn about that first and assure it still works ok before venturing further with this.

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