On Mon, Mar 06, 2023 at 05:34:22PM +0100, Juergen Gross wrote:
+/* Build the cache_map containing the cache modes per memory range. */^^^^^^^^^^^^^^^^^^^
+void mtrr_build_map(void)
+{
+ unsigned int i;
+ u64 start, end, size;
+ u8 type;
+
+ if (!mtrr_state.enabled)
+ return;^^^^^^^^^^^^^^^^^^
+
+ /* Add fixed MTRRs, optimize for adjacent entries with same type. */
+ if (mtrr_state.enabled & MTRR_STATE_MTRR_FIXED_ENABLED) {
First check can go.
+ start = 0;
+ end = size = 0x10000;
+ type = mtrr_state.fixed_ranges[0];
+
+ for (i = 1; i < MTRR_NUM_FIXED_RANGES; i++) {
Loops starts at 1. Comment explaining why pls.
+ if (i == 8 || i == 24)
Magic numbers.
Ok, I'll stop here.
Please send a new version with the review comments incorporated. I need
to look more at that map thing more.
Attachment:
OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key
Attachment:
OpenPGP_signature
Description: OpenPGP digital signature