Re: [PATCH] x86_32: trim memory by updating e820 v3

From: Gabriel C
Date: Mon Apr 28 2008 - 19:10:53 EST


Yinghai Lu wrote:
> On Mon, Apr 28, 2008 at 2:19 PM, Gabriel C <nix.or.die@xxxxxxxxxxxxxx> wrote:
>> Gabriel C wrote:
>> > Gabriel C wrote:
>> >> Yinghai Lu wrote:
>> >>> On Mon, Apr 28, 2008 at 7:24 AM, Gabriel C <nix.or.die@xxxxxxxxxxxxxx> wrote:
>> >>>> Mika Fischer wrote:
>> >>>> > Hi Ingo,
>> >>>> >
>> >>>> > I'm having the same problem.
>> >>>> >
>> >>>> > Ingo Molnar schrieb:
>> >>>> >> excellent. So just to make sure: this box never had proper graphics
>> >>>> >> under Linux (under no previous kernel), due to the way the BIOS has set
>> >>>> >> up the MTRR's, right?
>> >>>> >
>> >>>> > Well, not quite. X still works fine, but since the video memory is
>> >>>> > overlapped by two of the existing MTRRs, X cannot add a write-combining
>> >>>> > range for the video memory. That makes X rather slow especially if you
>> >>>> > use DRI for Compiz etc.
>> >>>>
>> >>>> Well you are lucky then :)
>> >>>>
>> >>>> Yeah X 'worked' but it worked as slow as with vesa video driver here.
>> >>> [ 0.000000] rangeX: 0000000000000000 - 00000000d0000000
>> >>> [ 0.000000] Setting variable MTRR 0, base: 0MB, range: 2048MB, type WB
>> >>> [ 0.000000] Setting variable MTRR 1, base: 2048MB, range: 1024MB, type WB
>> >>> [ 0.000000] Setting variable MTRR 2, base: 3072MB, range: 256MB, type WB
>> >>> [ 0.000000] range0: 00000000cf800000 - 00000000cf800000
>> >>> [ 0.000000] range: 00000000cf800000 - 00000000d0000000
>> >>> [ 0.000000] Setting variable MTRR 3, base: 3320MB, range: 8MB, type WB
>> >>> [ 0.000000] range0: 0000000100000000 - 0000000120000000
>> >>> [ 0.000000] Setting variable MTRR 4, base: 4096MB, range: 512MB, type WB
>> >>> [ 0.000000] range: 0000000120000000 - 0000000130000000
>> >>> [ 0.000000] Setting variable MTRR 5, base: 4608MB, range: 256MB, type WB
>> >>> [ 0.000000] hole: 000000012c000000 - 0000000130000000
>> >>> [ 0.000000] Setting variable MTRR 6, base: 4800MB, range: 64MB, type UC
>> >>>
>> >>> so your X server need two entries for WB?
>> >>>
>> >>> can you send out /proc/mtrr with booting with disable_mtrr_cleanup?
>> >> I can just not right now , cannot reboot the box yet. In about 1h or so , maybe less.
>> >
>> > Here the output with v3 which is disabled by default:
>> >
>> > --($:~)-- cat /proc/mtrr
>> > reg00: base=0xd0000000 (3328MB), size= 256MB: uncachable, count=1
>> > reg01: base=0xe0000000 (3584MB), size= 512MB: uncachable, count=1
>> > reg02: base=0x00000000 ( 0MB), size=4096MB: write-back, count=1
>> > reg03: base=0x100000000 (4096MB), size= 512MB: write-back, count=1
>> > reg04: base=0x120000000 (4608MB), size= 128MB: write-back, count=1
>> > reg05: base=0x128000000 (4736MB), size= 64MB: write-back, count=1
>> > reg06: base=0xcf600000 (3318MB), size= 2MB: uncachable, count=1
>> >
>> > dmesg is saying now :
>> >
>> > [ 22.764595] mtrr: type mismatch for d0000000,10000000 old: write-back new: write-combining
>> >
>> >
>> > My card settings in BIOS ( that was default ) are the following :
>> >
>> > DVMT Mode -> DVMT Mode ( possible setting DVMT Mode or Fixed Mode )
>> > DVMT / Memory -> 256MB ( possible settings 128/256 MB or Maximum DVMT )
>> >
>> > Initiate Graphics Adapter -> PEG/PCI ( possible settings IGD , PCI/IGD , PCI/PEG , PEG/IGD )
>> > Internal Graphics Mode Select -> Enabled,8MB ( possible settings Enabled,8MB , Enabled,1MB maybe Disabled I forgot to look)
>> > PEG Port -> Auto ( possible settings Auto , Disabled )
>> > PEG Port Force x1 -> Disabled ( possible settings Enabled , Disabled )
>> >
>> > Of course these settings are only possible when the card is not disabled :)
>> >
>> > I'm gonna try v4 now and enable it. Please let me know if you need more infos.
>>
>> Hmm v4 doesn't work anymore here ( I've tested with all possible settings in BIOS ).
>> It takes 6 minutes to boot to :
>>
>
> so you card is using 256M and 8M? 0xd0000000-0xe0000000, where is
> another 8M address.

Looks like this , yes. Is using the 256MB Memory and the 8MB for Graphics Mode Select.

I'm not really sure why the 8MB are needed , BIOS book doesn't tell me.
I could try to disable and see what I get =)

>
> mtrr by BIOS is very interesting:
> before
>> > reg02: base=0x00000000 ( 0MB), size=4096MB: write-back, count=1
>> > reg06: base=0xcf600000 (3318MB), size= 2MB: uncachable, count=1
>> > reg00: base=0xd0000000 (3328MB), size= 256MB: uncachable, count=1
>> > reg01: base=0xe0000000 (3584MB), size= 512MB: uncachable, count=1
>> > reg03: base=0x100000000 (4096MB), size= 512MB: write-back, count=1
>> > reg04: base=0x120000000 (4608MB), size= 128MB: write-back, count=1
>> > reg05: base=0x128000000 (4736MB), size= 64MB: write-back, count=1
>
>
> after 256M chunk size got
>> >>> [ 0.000000] Setting variable MTRR 0, base: 0MB, range: 2048MB, type WB
>> >>> [ 0.000000] Setting variable MTRR 1, base: 2048MB, range: 1024MB, type WB
>> >>> [ 0.000000] Setting variable MTRR 2, base: 3072MB, range: 256MB, type WB
>> >>> [ 0.000000] Setting variable MTRR 3, base: 3320MB, range: 8MB, type WB
>> >>> [ 0.000000] Setting variable MTRR 4, base: 4096MB, range: 512MB, type WB
>> >>> [ 0.000000] Setting variable MTRR 5, base: 4608MB, range: 256MB, type WB
>> >>> [ 0.000000] Setting variable MTRR 6, base: 4800MB, range: 64MB, type UC
>
> so the convering is right..., need to spare another entry for your card.
>
> or we can dumping the
>> >>> [ 0.000000] Setting variable MTRR 3, base: 3320MB, range: 8MB, type WB
> for extra entra...
>
> but the mtrr trimming code need to be updated instead of only using highest_pfn
>
> YH
>

Gabriel

--
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/