lowRISC / opentitan

OpenTitan: Open source silicon root of trust
https://www.opentitan.org
Apache License 2.0
2.53k stars 754 forks source link

[dv/mem_bkdr_util] Improve memory write performance #17086

Open matutem opened 1 year ago

matutem commented 1 year ago

Description

A vcs time profile for chip_sw_pwrmgr_smoketest shows 50% of the time is spent in uvm_hdl_deposit for mem_bkdr_util::write. This is excessive, and we should be able to reduce it significantly.

I think we should explore alternative implementations, perhaps including direct updates of the memory arrays as in $readmemh, to avoid uvm_hdl_deposit.

andreaskurth commented 1 year ago

Triaged for pwrmgr. Assigning https://github.com/lowRISC/opentitan/labels/Type%3AIcebox as not critical for M2.5.

matutem commented 1 month ago

This performance optimization is not indispensable, and should not get in the way of milestones.