search

ominux / vtr-verilog-to-routing

Automatically exported from code.google.com/p/vtr-verilog-to-routing
0 stars 0 forks source link

ODIN II: dual_port_ram memory depth not bounded #10

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. ./odin_II.exe -V test3.v -a sample_arch.xml 
2.
3.

What is the expected output? What do you see instead?
addr input width to dual_port_ram should be bounded.
Instead, overflow occurs:

Hard Logical Memory Distribution
============================
DPRAM: 32 width 26 depth

Total Logical Memory Blocks = 1 
Total Logical Memory bits = -2147483648 
Max Memory Width = 32 
Max Memory Depth = 26

and ODIN II eats up lots of memory during the "Performing Optimizations of the 
Netlist" stage.

What version of the product are you using? On what operating system?
ODIN II version 0.1 (from VTR1.0rc1) under 64-bit Linux

Please provide any additional information below.

Original issue reported on code.google.com by eddie.h...@gmail.com on 20 Jan 2012 at 1:08

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by jeffrey....@gmail.com on 20 Jan 2012 at 6:17

GoogleCodeExporter commented 9 years ago 
Thank you for reporting this issue and helping improve ODIN II!

The memories are undergoing a major overhaul now. I have fixed the overflow 
problem in the printout in my local version and it will be released to SVN as 
soon as we've published our results. 

The depth ODIN II can handle depends on what optimizations you are doing, the 
amount of RAM in your machine and which architecture you are mapping to. In 
your case, ODIN II was splitting all memories down to a data width of 1, which 
resulted in poor mapping performance on such a large memory block. If you 
remove the class="memory" from the memory specifications in the architecture 
file you supplied, it will prevent ODIN II from splitting all the memories down 
to 1 bit data. Then the compilation will succeed.

Original comment by andy16...@gmail.com on 5 Apr 2012 at 2:01

GoogleCodeExporter commented 9 years ago

Original comment by andy16...@gmail.com on 5 Apr 2012 at 2:09

GoogleCodeExporter commented 9 years ago 
Added a configurable bound to memories.h.

Original comment by andy16...@gmail.com on 5 Apr 2012 at 2:40