Re: [PATCH] x86: mtrr cleanup for converting continuous todiscrete layout v3

From: Ingo Molnar
Date: Mon Apr 28 2008 - 16:15:33 EST



* Yinghai Lu <yhlu.kernel.send@xxxxxxxxx> wrote:

> some BIOS like to use continus MTRR layout, and some X driver can not
> add WB entries for graphical cards when 4g or more RAM installed.
>
> the patch will change MTRR to discrete.
>
> mtrr_chunk_size= could be used to have smaller continuous block to
> hold holes. default is 256m, could be set according to size of
> graphics card memory.
>
> v2: fix -1 for UC checking

> v3: default to disable, and need use enable_mtrr_cleanup to enable
> this feature
> skip the var state change warning.
> remove next_basek in range_to_mtrr()

a boot option is often inconvenient though - could you perhaps also make
this a Kconfig option, with it defaulting to off? Something like
CONFIG_MTRR_SANITIZE=y or so? That way distros can make a conscious
decision as well whether they want this feature (for the Xorg they pick)
or not.

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/