ODIN II: dual_port_ram memory depth not bounded

[kmurray]

Originally reported on Google Code with ID 10

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.

Reported by eddie.hung on 2012-01-20 01:08:30

---

- _Attachment: [test3.v](https://storage.googleapis.com/google-code-attachments/vtr-verilog-to-routing/issue-10/comment-0/test3.v)_ - _Attachment: [sample_arch.xml](https://storage.googleapis.com/google-code-attachments/vtr-verilog-to-routing/issue-10/comment-0/sample_arch.xml)_

[kmurray]

Reported by jeffrey.goeders on 2012-01-20 18:17:53

• Labels added: Module-ODIN

[kmurray]

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.

Reported by andy16666 on 2012-04-05 14:01:37

[kmurray]

Reported by andy16666 on 2012-04-05 14:09:41

• Status changed: Fixed

[kmurray]

Added a configurable bound to memories.h. 

Reported by andy16666 on 2012-04-05 14:40:29