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.

Sign up for GitHub

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

Final changes for BlackParrot integration #7

Open
wants to merge 12 commits into
base: black-parrot
Choose a base branch
from
Open

Final changes for BlackParrot integration #7

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
d3a818c
OP local changes
Shashank-Vijay Jun 30, 2021
c4d8f32
Bumping BP submodule
Shashank-Vijay Jun 30, 2021
a509765
More changes
Shashank-Vijay Jul 9, 2021
9ded7e8
Minor changes
Shashank-Vijay Jul 9, 2021
c4c6e5f
Update sims,2.0
dpetrisko Jul 12, 2021
da77c23
Cosim module addition
Shashank-Vijay Jul 16, 2021
274a517
Working cacheable-only $readmemh memory for simulation
Jbalkind Jul 24, 2020
204f3e8
Various updates needed to run OP
dpetrisko Oct 3, 2021
5943677
Updating tcl scripts
dpetrisko Oct 4, 2021
5a2de3b
Moving blackparrot submodule
dpetrisko Oct 4, 2021
c48a5dd
Moving black-parrot submodule
dpetrisko Oct 4, 2021
85f552f
Use sdk environment variable
dpetrisko Oct 4, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion piton/design/chip/tile/blackparrot
Open in desktop
Submodule blackparrot updated 551 files
3 changes: 3 additions & 0 deletions piton/design/xilinx/genesys2/constraints.xdc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,9 @@ set_false_path -from [get_ports trst_ni]
set_max_delay -datapath_only -from [get_clocks -include_generated_clocks chipset_clk_clk_mmcm] -to [get_clocks tck_i] 15.000
set_max_delay -datapath_only -from [get_clocks tck_i] -to [get_clocks -include_generated_clocks chipset_clk_clk_mmcm] 15.000

# Retiming
set_property BLOCK_SYNTH.RETIMING 1 [get_cells -hierarchical *pipe_fma*]

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This didn't end up helping much, right? Should we keep this out and wait for the manually retimed FPU?

set_property -dict {PACKAGE_PIN Y29 IOSTANDARD LVCMOS33} [get_ports trst_ni]
set_property -dict {PACKAGE_PIN AD27 IOSTANDARD LVCMOS33} [get_ports tck_i]
set_property -dict {PACKAGE_PIN W27 IOSTANDARD LVCMOS33} [get_ports td_i]
Expand Down
1 change: 1 addition & 0 deletions piton/tools/bin/rv64_img
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
# create mem.image
#riscv64-unknown-elf-objcopy --reverse-bytes 4 -I elf32-littleriscv -O binary diag.exe diag.o
${RV64_TARGET_TRIPLE}-objcopy -I elf64-littleriscv -O binary diag.exe diag.o
cp diag.exe prog.elf
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this for?

# pad with zero to 128byte boundary
du diag.o -b | awk '{print(128 - ($1 % 128));}' | xargs -t -ISIZE truncate diag.o -s +SIZE
printf "\n@0000000080000000\t// Section '.RED_SEC', segment 'text'\n" >mem.image
Expand Down
2 changes: 1 addition & 1 deletion piton/tools/src/proto/block.list
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
# Format:
# BlockID BlockPath Supported Board,Frequency(MHz),DDRSize(Mbytes)
piton_aws ../../build/f1/piton_aws/design f1,62.5,4096
system . vc707,60,1024;genesys2,66.667,1024;nexysVideo,30,512;vcu118,100,2048;xupp3r,60,32768
system . vc707,60,1024;genesys2,25,1024;nexysVideo,30,512;vcu118,100,2048;xupp3r,60,32768
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still not excited about this - do we have a sense of the manually retimed FPU's timing?

chipset chipset genesys2,66.667,1024;piton_board,50,0
passthru passthru piton_board,100,0
passthru_loopback fpga_tests/passthru_loopback piton_board,100,0
Expand Down
1,300 changes: 846 additions & 454 deletions piton/tools/src/proto/common/blackparrot.tcl
View file
Open in desktop

Large diffs are not rendered by default.

4 changes: 4 additions & 0 deletions piton/tools/src/sims/sims,2.0
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2676,6 +2676,7 @@ sub parse_args
'gui!',
'log_all!',
'debug_all!',
'debug_pp!',
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this one vs debug_all? Can you just use it via -vcs_build_args=-debug_pp rather than adding a new flag?

'ibm!',
'ed_enable!',
'ed_sync_method=s',
Expand Down Expand Up @@ -2900,6 +2901,9 @@ sub parse_args
#
push (@{$opt{sim_run_args}}, "-gui") if ($opt{gui}) ;
push (@{$opt{vcs_build_args}}, "-debug_all") if ($opt{debug_all}) ;
push (@{$opt{vcs_build_args}}, "-debug_pp") if ($opt{debug_pp}) ;
push (@{$opt{vcs_build_args}}, "-CFLAGS \"-I/mnt/users/ssd1/homes/svijay97/BlackParrot/black-parrot-sdk/include -std=c++14\"") ;
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll change the path to a generic one

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do. It's also not being conditionally added?

push (@{$opt{vcs_build_args}}, "/mnt/users/ssd1/homes/svijay97/BlackParrot/black-parrot-sdk/lib/libdromajo_cosim.a") ;

# Push optional execution drafting enable
push (@{$opt{midas_args}}, "-DED_ENABLE") if ($opt{ed_enable});
Expand Down
2 changes: 1 addition & 1 deletion piton/verif/env/manycore/devices_blackparrot.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ Description: Peripheral address map for OpenPiton+BlackParrot configurations.
<name>mem</name>
<base>0x80000000</base>
<!-- 1 GB -->
<length>0x40000000</length>
<length>0x08000000</length>
</port>
<port>
<name>iob</name>
Expand Down
4 changes: 2 additions & 2 deletions piton/verif/env/manycore/pc_cmp.v.pyv
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -258,8 +258,8 @@ endtask // get_thread_status
end

always @(posedge clk) begin
spc0_inst_done <= `BLACKPARROT_CORE0.core.be.calculator.commit_pkt.instret;
spc0_phy_pc_w <= 48'($signed(48'(`BLACKPARROT_CORE0.core.be.calculator.commit_pkt.pc)));
spc0_inst_done <= `BLACKPARROT_CORE0.be.calculator.commit_pkt_cast_o.instret;
spc0_phy_pc_w <= 48'($signed(48'(`BLACKPARROT_CORE0.be.calculator.commit_pkt_cast_o.pc)));
end
`else
`ifdef RTL_PICO0
Expand Down