tillitis-key1 icon indicating copy to clipboard operation
tillitis-key1 copied to clipboard

Change names of ADDR_RAM_ASLR and ADDR_RAM_SCRAMBLE

Open mchack-work opened this issue 9 months ago • 1 comments

To remove potentially confusing users we might want to change some names.

Quoting @secworks

I suggest that we change both address and data randomization to ADDR_RAM_ADDR_RAND and ADDR_RAM_DATA_RAND. We are performing basically the same randomization operation. for both. Scrambling is possibly more correct now, but is longer. And if move to a PRNG it is randomization.

This potentially changes both Verilog and tk1_mem.h.

I think it's more important to change the Verilog to avoid using the confusing "ASLR". Such a change doesn't affect software, either, but does affect forks and testbenches.

Changing the header file affects others. We need to be careful here. In the header file we currently have:

// Deprecated - use _ADDR_RAND instead
#define TK1_MMIO_TK1_RAM_ASLR 0xff000100

#define TK1_MMIO_TK1_RAM_ADDR_RAND 0xff000100
#define TK1_MMIO_TK1_RAM_SCRAMBLE 0xff000104

If we do a change to the header file, too, we will probably keep the old names for a while, but add the new names, too, just like the old RAM_ASLR.

mchack-work avatar Apr 25 '24 07:04 mchack-work