-
Notifications
You must be signed in to change notification settings - Fork 23
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
[release] add docker image and related document #506
Conversation
@sequencer Please check if anything is missing or useless in the image. Currently the docker environment contains: ip-emulator with trace functionality, three intrinsic performance cases, riscv32 C gnu toolchains. |
|
||
buildPhase = '' | ||
cp -v ${./doc.typ} ./doc.typ | ||
jq '.name = "${configName}"' ${elaborateConfigJson} > config.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Personally I want to block this PR until we have a standard object model reader flow.
OM is used to generate all metadata from Chisel, it can encode address ranges, uarch configurations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But if omreader takes too much time I think merging this firstly is OK.
nix/t1/release/default.nix
Outdated
''; | ||
|
||
docker-layers = scope.callPackage ./docker-layers.nix { | ||
rv32-gcc = pkgsCross.riscv32-embedded.buildPackages.gcc; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need musl or libc as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would love to use newlib, and modify the sbrk function to make malloc/free work on our new memory bank.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok wait for next PR. We may need some codegen for it
[*Range*], [*Usage*], [*Address Range*], | ||
[0-1G], [Scalar Bank], [0x20000000], | ||
[1-3G], [DDR Bank (512M/bank)], [0x40000000], | ||
[3G-3G+2M], [SRAM Bank (256K/bank 8Banks)], [0xc0000000] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be generated from OM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But Just add TODO for now.
d320149
to
f26a5b7
Compare
c9aa208
to
1e82060
Compare
Signed-off-by: Avimitin <[email protected]>
Signed-off-by: Avimitin <[email protected]>
Signed-off-by: Avimitin <[email protected]>
Signed-off-by: Avimitin <[email protected]>
1847c32
to
9f0611d
Compare
Signed-off-by: Avimitin <[email protected]>
TODO: add workflow file to automatically upload docker image to ghcr.io