Re: [PATCH] x86: trim ram need to check if mtrr is there v2

view
thread

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

From: Yinghai Lu

Subject: Re: [PATCH] x86: trim ram need to check if mtrr is there v2

Date: Friday, January 25, 2008 - 1:39 am

On Jan 25, 2008 12:13 AM, Yinghai Lu <yhlu.kernel@gmail.com> wrote:
quoted text>
> On Jan 24, 2008 11:43 PM, Jeremy Fitzhardinge <jeremy@goop.org> wrote:
> >
> > Yinghai Lu wrote:
> > > [PATCH] x86: trim ram need to check if mtrr is there v2
> > >
> > >
> > >>> Jeremy Fitzhardinge wrote:
> > >>> When booting a current x86.git kernel under kvm, I get this:
> > >>>
> > >>> (qemu) Linux version 2.6.24-rc8 (jeremy@ezr) (gcc version 4.1.2 20070925
> > >>> (Red Hat 4.1.2-33)) #1928 SMP PREEMPT Thu Jan 24 17:09:04 PST 2008
> > >>> early_ioremap_init()
> > >>> BIOS-provided physical RAM map:
> > >>> BIOS-e820: 0000000000000000 - 000000000009fc00 (usable)
> > >>> BIOS-e820: 000000000009fc00 - 00000000000a0000 (reserved)
> > >>> BIOS-e820: 00000000000e8000 - 0000000000100000 (reserved)
> > >>> BIOS-e820: 0000000000100000 - 000000001fff0000 (usable)
> > >>> BIOS-e820: 000000001fff0000 - 0000000020000000 (ACPI data)
> > >>> BIOS-e820: 00000000fffc0000 - 0000000100000000 (reserved)
> > >>> console [earlyser0] enabled
> > >>> 0MB HIGHMEM available.
> > >>> 511MB LOWMEM available.
> > >>> Scan SMP from c0000000 for 1024 bytes.
> > >>> Scan SMP from c009fc00 for 1024 bytes.
> > >>> Scan SMP from c00f0000 for 65536 bytes.
> > >>> Scan SMP from c009fc00 for 1024 bytes.
> > >>> ***************
> > >>> **** WARNING: likely BIOS bug
> > >>> **** MTRRs don't cover all of memory, trimmed 131056 pages
> > >>> ***************
> > >>>
> > >> H. Peter Anvin wrote:
> > >> Looks like the code doesn't check that the CPU *has* MTRRs...
> > >>
> > >
> > > so check it mtrr is there, also check if mem less 4G and is AMD as early
> > >
> >
> > Thanks, this gets me to usermode under kvm.
>
> i think the is 0xffffffff to 0xffffffffUL make the difference.
>
> the cpu mode in KVM/qemu may cpu_has_mtrr, but doesn't really have
> return mtrr msr with correct value.
>
> is_cpu(INTEL) already make sure we have mtrr_if got assigned and cpu_has_mtrr.
>
> may need to fix qemu instead...
>
> Can you try post /proc/mtrr for your guest in kvm/qemu?

jeremy,

can you try v3 patch?

Thanks

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

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:

x86.git: mtrr trimming removes all memory under kvm, Jeremy Fitzhardinge, (Thu Jan 24, 6:44 pm)

Re: x86.git: mtrr trimming removes all memory under kvm, H. Peter Anvin, (Thu Jan 24, 6:49 pm)

Re: x86.git: mtrr trimming removes all memory under kvm, Yinghai Lu, (Thu Jan 24, 7:21 pm)

Re: x86.git: mtrr trimming removes all memory under kvm, Yinghai Lu, (Thu Jan 24, 7:32 pm)

[PATCH] x86: trim ram need to check if mtrr is there v2, Yinghai Lu, (Thu Jan 24, 8:47 pm)

Re: [PATCH] x86: trim ram need to check if mtrr is there v2, H. Peter Anvin, (Thu Jan 24, 10:44 pm)

Re: [PATCH] x86: trim ram need to check if mtrr is there v2, Jeremy Fitzhardinge, (Fri Jan 25, 12:43 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v2, Yinghai Lu, (Fri Jan 25, 12:50 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v2, Yinghai Lu, (Fri Jan 25, 1:13 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v2, Yinghai Lu, (Fri Jan 25, 1:39 am)

[PATCH] x86: trim ram need to check if mtrr is there v3, Yinghai Lu, (Fri Jan 25, 1:42 am)

Re: x86.git: mtrr trimming removes all memory under kvm, Andi Kleen, (Fri Jan 25, 3:52 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Ingo Molnar, (Fri Jan 25, 4:09 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v2, Ingo Molnar, (Fri Jan 25, 4:12 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Ingo Molnar, (Fri Jan 25, 7:21 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Andi Kleen, (Fri Jan 25, 7:57 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Ingo Molnar, (Fri Jan 25, 8:10 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v2, H. Peter Anvin, (Fri Jan 25, 8:47 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, H. Peter Anvin, (Fri Jan 25, 8:57 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Jeremy Fitzhardinge, (Fri Jan 25, 11:55 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Jeremy Fitzhardinge, (Fri Jan 25, 11:59 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Ingo Molnar, (Fri Jan 25, 11:59 am)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Jeremy Fitzhardinge, (Fri Jan 25, 12:01 pm)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Ingo Molnar, (Fri Jan 25, 12:02 pm)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Ingo Molnar, (Fri Jan 25, 12:04 pm)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Jeremy Fitzhardinge, (Fri Jan 25, 12:04 pm)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Yinghai Lu, (Fri Jan 25, 12:19 pm)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Jeremy Fitzhardinge, (Fri Jan 25, 12:27 pm)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Ingo Molnar, (Fri Jan 25, 12:30 pm)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Yinghai Lu, (Fri Jan 25, 12:32 pm)

Re: [PATCH] x86: trim ram need to check if mtrr is there v3, Jeremy Fitzhardinge, (Fri Jan 25, 12:39 pm)

Navigation

Popular discussions


linux-kernel:
Ingo Molnar	Re: [PATCH 0/3] v2 Make hierarchical RCU less IPI-happy and add more tracing
Jeremy Fitzhardinge	Re: Linux 2.6.28.10 and Linux 2.6.29.6 XEN Guest Support Broken x86_64 in BUILD
Nick Piggin	Re: [patch] CFS (Completely Fair Scheduler), v2
Gary Hade	Re: [PATCH 0/5][RFC] Physical PCI slot objects
Dave Johnson	Re: expected behavior of PF_PACKET on NETIF_F_HW_VLAN_RX device?
linux-netdev:
Arnd Bergmann	Re: 64-bit net_device_stats
Stephens, Allan	RE: [PATCH]: tipc: Fix oops on send prior to entering networked mode
frank.blaschka	[patch 3/5] [PATCH] qeth: support z/VM VSWITCH Port Isolation
Wu Fengguang	Re: [PATCH] dm9601: handle corrupt mac address
David Miller	Re: [PATCH net-2.6.24] Fix refcounting problem with netif_rx_reschedule()
git:
Junio C Hamano	Re: [PATCH] [RFC] add Message-ID field to log on git-am operation
Junio C Hamano	Re: Handling large files with GIT
Karl	Re: [ANNOUNCE] pg - A patch porcelain for GIT
Josh Triplett	Re: [RFC][PATCH 00/10] Sparse: Git's "make check" target
Pierre Habouzit	Re: [PATCH] git-daemon: more powerful base-path/user-path settings, using formats.
git-commits-head:
Linux Kernel Mailing List	MIPS: RBTX4939: Fix IOC pin-enable register updating
Linux Kernel Mailing List	regulator: update email address for Liam Girdwood
Linux Kernel Mailing List	[SCSI] ipr: add message to error table
Linux Kernel Mailing List	powerpc/32: Wire up the trampoline code for kdump
Linux Kernel Mailing List	USB: omap_udc: sync with OMAP tree
openbsd-misc:
Josh Grosse	Re: error : pkg add phpMyAdmin
Brian Candler	Re: OBSD's perspective on SELinux
Jacob Meuser	Re: /dev/audio: Device busy
David Vasek	Re: Inexpensive, low power, "wall wart" computer
William Boshuck	Re: Richard Stallman...

Colocation donated by:

Syndicate