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.

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

Incorrect Density for HBM2_8Gb Preset in Memory Configuration #71

Open
Ozen7 opened this issue Nov 16, 2024 · 1 comment
Open

Incorrect Density for HBM2_8Gb Preset in Memory Configuration #71

Ozen7 opened this issue Nov 16, 2024 · 1 comment

Comments

@Ozen7
Copy link

Ozen7 commented Nov 16, 2024

{"HBM2_8Gb", {6<<10, 128, {1, 2, 4, 4, 1<<15, 1<<6}}},

Hello,

The HBM2_8Gb preset defines a density of 6<<10 (6144 Mb or 6 Gb) per channel. However, the preset name (HBM2_8Gb) suggests the density should be 8<<10 (8192 Mb or 8 Gb) per channel. This mismatch leads to errors when using this preset in configurations expecting 8 Gb density.

Steps to Reproduce

  1. Configure Ramulator with the following setup:
DRAM:
  impl: HBM2
  org:
    preset: HBM2_8Gb
    channel: 8
    rank: 1
  timing:
    preset: HBM2_2000
  1. Run a simulation with this configuration.
  2. Observe the error indicating a mismatch between the calculated density and the preset density:
    An error occurred: Calculated HBM2 channel density 8192 Mb does not equal the provided density 6144 Mb!

Expected Behavior

The HBM2_8Gb preset should define a density of 8<<10 (8192 Mb) per channel, matching the preset name.

Thank you!

@Ozen7
Copy link
Author

Ozen7 commented Nov 16, 2024

Also, out of pure curiosity, can HBM be used with a closed-row policy? I realize it doesn't make much sense from a real-world use perspective, but I was wondering if it was at all possible.

For reference, the configuration errors out when you try to implement this since the closed-row policy requires a rank from the memory implementation (and HBM of course doesn't have one).

m_rank_level = m_dram->m_levels("rank");

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

No branches or pull requests

1 participant