Re: [PATCH] trim memory not covered by WB MTRRs

2007-07-05 Thread Justin Piszcz
On Thu, 5 Jul 2007, Pavel Machek wrote: Hi! On some machines, buggy BIOSes don't properly setup WB MTRRs to cover all available RAM, meaning the last few megs (or even gigs) of memory will be marked uncached. Since Linux tends to allocate from high memory addresses first, this causes the

Re: [PATCH] trim memory not covered by WB MTRRs

2007-07-05 Thread Pavel Machek
Hi! > On some machines, buggy BIOSes don't properly setup WB MTRRs to > cover all available RAM, meaning the last few megs (or even gigs) > of memory will be marked uncached. Since Linux tends to allocate > from high memory addresses first, this causes the machine to be > unusably slow as soon

Re: [PATCH] trim memory not covered by WB MTRRs

2007-07-05 Thread Pavel Machek
Hi! On some machines, buggy BIOSes don't properly setup WB MTRRs to cover all available RAM, meaning the last few megs (or even gigs) of memory will be marked uncached. Since Linux tends to allocate from high memory addresses first, this causes the machine to be unusably slow as soon as the

Re: [PATCH] trim memory not covered by WB MTRRs

2007-07-05 Thread Justin Piszcz
On Thu, 5 Jul 2007, Pavel Machek wrote: Hi! On some machines, buggy BIOSes don't properly setup WB MTRRs to cover all available RAM, meaning the last few megs (or even gigs) of memory will be marked uncached. Since Linux tends to allocate from high memory addresses first, this causes the

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Pim Zandbergen
Jesse Barnes wrote: Yeah, that's what I needed. end_pfn looks ok, but I guess my test is a little too precise. It should be if ((highest_addr >> PAGE_SHIFT) < end_pfn) rather than !=. Right. That made the message disappear. Nice to know my BIOS is really fixed. Thanks, Pim - To

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Jesse Barnes
On Wednesday, June 27, 2007 10:02:35 Pim Zandbergen wrote: > Jesse Barnes wrote: > > It looks like end_pfn might be ~0UL now... can you print that out > > in your configuration? > > Er, do you need the value of end_pfn ? > Here's what I changed: > > if ((highest_addr >> PAGE_SHIFT) != end_pfn) { >

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Pim Zandbergen
Jesse Barnes wrote: It looks like end_pfn might be ~0UL now... can you print that out in your configuration? Er, do you need the value of end_pfn ? Here's what I changed: if ((highest_addr >> PAGE_SHIFT) != end_pfn) { printk(KERN_WARNING "***\n");

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Jesse Barnes
On Wednesday, June 27, 2007 9:00:33 Pim Zandbergen wrote: > Jesse Barnes wrote: > > Yeah, you're right I should use an unsigned format string. Pim, if > > you change it to %lu does the printk in your dmesg look better? > > Er, no. > > MTRRs don't cover all of memory, trimmed

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Jesse Barnes
On Wednesday, June 27, 2007 9:00:33 Pim Zandbergen wrote: > Jesse Barnes wrote: > > Yeah, you're right I should use an unsigned format string. Pim, if > > you change it to %lu does the printk in your dmesg look better? > > Er, no. > > MTRRs don't cover all of memory, trimmed

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Pim Zandbergen
Jesse Barnes wrote: Yeah, you're right I should use an unsigned format string. Pim, if you change it to %lu does the printk in your dmesg look better? Er, no. MTRRs don't cover all of memory, trimmed 18446744073709486080 pages Thanks, Pim - To unsubscribe from this list: send the line

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Jesse Barnes
On Wednesday, June 27, 2007 7:22:24 Mauro Giachero wrote: > On 6/27/07, Pim Zandbergen <[EMAIL PROTECTED]> wrote: > > Now: > > Jesse released a new patch and I tried if for fun on 2.6.22-rc6 > > It looks like the patch is releasing memory rather than trimming > > it: > > > > [...] > > Jun 27

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Mauro Giachero
On 6/27/07, Pim Zandbergen <[EMAIL PROTECTED]> wrote: Now: Jesse released a new patch and I tried if for fun on 2.6.22-rc6 It looks like the patch is releasing memory rather than trimming it: [...] Jun 27 12:22:56 corneille kernel: MTRRs don't cover all of memory, trimmed -65536 pages

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Justin Piszcz
On Wed, 27 Jun 2007, Pim Zandbergen wrote: Andi Kleen wrote: That's impossible. Either it limited your RAM or it didn't change anything. OK, maybe it's cosmetic, but I would not expect a negative number With old BIOS it printed MTRRs don't cover all of memory, trimmed 196608 pages

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Pim Zandbergen
Andi Kleen wrote: That's impossible. Either it limited your RAM or it didn't change anything. OK, maybe it's cosmetic, but I would not expect a negative number With old BIOS it printed MTRRs don't cover all of memory, trimmed 196608 pages with new BIOS it prints MTRRs don't cover

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Andi Kleen
On Wed, Jun 27, 2007 at 12:44:01PM +0200, Pim Zandbergen wrote: > Jesse saved my life by releasing a patch that made my GigaByte Intel G33 > based motherboard use all of its 8GB RAM and not be slow as hell. That's impossible. Either it limited your RAM or it didn't change anything. -Andi - To

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Pim Zandbergen
First: Jesse saved my life by releasing a patch that made my GigaByte Intel G33 based motherboard use all of its 8GB RAM and not be slow as hell. Then: GigaByte released a BIOS update that fixed the root of the problem. I went back from patched vanilla kernel to "official" Fedora kernel. Now:

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Pim Zandbergen
First: Jesse saved my life by releasing a patch that made my GigaByte Intel G33 based motherboard use all of its 8GB RAM and not be slow as hell. Then: GigaByte released a BIOS update that fixed the root of the problem. I went back from patched vanilla kernel to official Fedora kernel. Now:

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Andi Kleen
On Wed, Jun 27, 2007 at 12:44:01PM +0200, Pim Zandbergen wrote: Jesse saved my life by releasing a patch that made my GigaByte Intel G33 based motherboard use all of its 8GB RAM and not be slow as hell. That's impossible. Either it limited your RAM or it didn't change anything. -Andi - To

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Pim Zandbergen
Andi Kleen wrote: That's impossible. Either it limited your RAM or it didn't change anything. OK, maybe it's cosmetic, but I would not expect a negative number With old BIOS it printed MTRRs don't cover all of memory, trimmed 196608 pages with new BIOS it prints MTRRs don't cover

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Justin Piszcz
On Wed, 27 Jun 2007, Pim Zandbergen wrote: Andi Kleen wrote: That's impossible. Either it limited your RAM or it didn't change anything. OK, maybe it's cosmetic, but I would not expect a negative number With old BIOS it printed MTRRs don't cover all of memory, trimmed 196608 pages

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Mauro Giachero
On 6/27/07, Pim Zandbergen [EMAIL PROTECTED] wrote: Now: Jesse released a new patch and I tried if for fun on 2.6.22-rc6 It looks like the patch is releasing memory rather than trimming it: [...] Jun 27 12:22:56 corneille kernel: MTRRs don't cover all of memory, trimmed -65536 pages [...]

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Jesse Barnes
On Wednesday, June 27, 2007 7:22:24 Mauro Giachero wrote: On 6/27/07, Pim Zandbergen [EMAIL PROTECTED] wrote: Now: Jesse released a new patch and I tried if for fun on 2.6.22-rc6 It looks like the patch is releasing memory rather than trimming it: [...] Jun 27 12:22:56 corneille

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Pim Zandbergen
Jesse Barnes wrote: Yeah, you're right I should use an unsigned format string. Pim, if you change it to %lu does the printk in your dmesg look better? Er, no. MTRRs don't cover all of memory, trimmed 18446744073709486080 pages Thanks, Pim - To unsubscribe from this list: send the line

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Jesse Barnes
On Wednesday, June 27, 2007 9:00:33 Pim Zandbergen wrote: Jesse Barnes wrote: Yeah, you're right I should use an unsigned format string. Pim, if you change it to %lu does the printk in your dmesg look better? Er, no. MTRRs don't cover all of memory, trimmed 18446744073709486080

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Jesse Barnes
On Wednesday, June 27, 2007 9:00:33 Pim Zandbergen wrote: Jesse Barnes wrote: Yeah, you're right I should use an unsigned format string. Pim, if you change it to %lu does the printk in your dmesg look better? Er, no. MTRRs don't cover all of memory, trimmed 18446744073709486080

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Pim Zandbergen
Jesse Barnes wrote: It looks like end_pfn might be ~0UL now... can you print that out in your configuration? Er, do you need the value of end_pfn ? Here's what I changed: if ((highest_addr PAGE_SHIFT) != end_pfn) { printk(KERN_WARNING ***\n); printk(KERN_WARNING

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Jesse Barnes
On Wednesday, June 27, 2007 10:02:35 Pim Zandbergen wrote: Jesse Barnes wrote: It looks like end_pfn might be ~0UL now... can you print that out in your configuration? Er, do you need the value of end_pfn ? Here's what I changed: if ((highest_addr PAGE_SHIFT) != end_pfn) {

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-27 Thread Pim Zandbergen
Jesse Barnes wrote: Yeah, that's what I needed. end_pfn looks ok, but I guess my test is a little too precise. It should be if ((highest_addr PAGE_SHIFT) end_pfn) rather than !=. Right. That made the message disappear. Nice to know my BIOS is really fixed. Thanks, Pim - To

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Yinghai Lu
On 6/26/07, Andi Kleen <[EMAIL PROTECTED]> wrote: RevE had a new memory remapping scheme (memory hoisting) at least, which I think you refered to earlier. There might have been more changes in F. yes, REV E has hardware memory remapping. Rev F you don't need to use mtrr to set MEM to WE above

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Andi Kleen
On Tue, Jun 26, 2007 at 10:06:41AM -0600, Eric W. Biederman wrote: > Andi Kleen <[EMAIL PROTECTED]> writes: > > >> For the K7 and K8 cores AMD systems are exactly like Intel systems > >> with respect to MTRRs (although AMD systems also have additional registers) > >> For the K9 core (i.e. AMD

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Eric W. Biederman
Andi Kleen <[EMAIL PROTECTED]> writes: >> For the K7 and K8 cores AMD systems are exactly like Intel systems >> with respect to MTRRs (although AMD systems also have additional registers) >> For the K9 core (i.e. AMD socket F or the K8 with DDR2 support) there > > It's called K8RevE, not K9 revF

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Yinghai Lu
On 6/26/07, Andi Kleen <[EMAIL PROTECTED]> wrote: > For the K7 and K8 cores AMD systems are exactly like Intel systems > with respect to MTRRs (although AMD systems also have additional registers) > For the K9 core (i.e. AMD socket F or the K8 with DDR2 support) there It's called K8RevE, not K9

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Andi Kleen
> For the K7 and K8 cores AMD systems are exactly like Intel systems > with respect to MTRRs (although AMD systems also have additional registers) > For the K9 core (i.e. AMD socket F or the K8 with DDR2 support) there It's called K8RevE, not K9 > is an additional mechanism that makes everything

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Jesse Barnes
On Tuesday, June 26, 2007 8:02:49 am Andi Kleen wrote: > On Mon, Jun 25, 2007 at 04:36:52PM -0700, Jesse Barnes wrote: > > On Monday, June 25, 2007 4:34:33 Andi Kleen wrote: > > > > This patch fixes a bug in the last patch that caused the code to > > > > run on non-Intel machines (AMD machines

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Jesse Barnes
On Tuesday, June 26, 2007 8:07:45 am Jesse Barnes wrote: > On Tuesday, June 26, 2007 8:03:48 am Andi Kleen wrote: > > On Mon, Jun 25, 2007 at 08:30:52PM -0700, Jesse Barnes wrote: > > > Oh, and FYI I've seen new systems with a default mapping type of WB, > > > with a few uncached holes for MMIO.

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Jesse Barnes
On Tuesday, June 26, 2007 8:03:48 am Andi Kleen wrote: > On Mon, Jun 25, 2007 at 08:30:52PM -0700, Jesse Barnes wrote: > > Oh, and FYI I've seen new systems with a default mapping type of WB, with > > a few uncached holes for MMIO. That means PAT will be absolutely > > required on those systems

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Andi Kleen
On Mon, Jun 25, 2007 at 08:30:52PM -0700, Jesse Barnes wrote: > Oh, and FYI I've seen new systems with a default mapping type of WB, with a > few uncached holes for MMIO. That means PAT will be absolutely required on > those systems if we want to use WC for the framebuffer or other memory.

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Andi Kleen
On Mon, Jun 25, 2007 at 04:36:52PM -0700, Jesse Barnes wrote: > On Monday, June 25, 2007 4:34:33 Andi Kleen wrote: > > > This patch fixes a bug in the last patch that caused the code to > > > run on non-Intel machines (AMD machines apparently don't need it > > > > Actually the problem can happen

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Andi Kleen
On Mon, Jun 25, 2007 at 04:36:52PM -0700, Jesse Barnes wrote: On Monday, June 25, 2007 4:34:33 Andi Kleen wrote: This patch fixes a bug in the last patch that caused the code to run on non-Intel machines (AMD machines apparently don't need it Actually the problem can happen on AMD too,

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Andi Kleen
On Mon, Jun 25, 2007 at 08:30:52PM -0700, Jesse Barnes wrote: Oh, and FYI I've seen new systems with a default mapping type of WB, with a few uncached holes for MMIO. That means PAT will be absolutely required on those systems if we want to use WC for the framebuffer or other memory. Why?

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Jesse Barnes
On Tuesday, June 26, 2007 8:03:48 am Andi Kleen wrote: On Mon, Jun 25, 2007 at 08:30:52PM -0700, Jesse Barnes wrote: Oh, and FYI I've seen new systems with a default mapping type of WB, with a few uncached holes for MMIO. That means PAT will be absolutely required on those systems if we

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Jesse Barnes
On Tuesday, June 26, 2007 8:07:45 am Jesse Barnes wrote: On Tuesday, June 26, 2007 8:03:48 am Andi Kleen wrote: On Mon, Jun 25, 2007 at 08:30:52PM -0700, Jesse Barnes wrote: Oh, and FYI I've seen new systems with a default mapping type of WB, with a few uncached holes for MMIO. That

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Jesse Barnes
On Tuesday, June 26, 2007 8:02:49 am Andi Kleen wrote: On Mon, Jun 25, 2007 at 04:36:52PM -0700, Jesse Barnes wrote: On Monday, June 25, 2007 4:34:33 Andi Kleen wrote: This patch fixes a bug in the last patch that caused the code to run on non-Intel machines (AMD machines apparently

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Andi Kleen
For the K7 and K8 cores AMD systems are exactly like Intel systems with respect to MTRRs (although AMD systems also have additional registers) For the K9 core (i.e. AMD socket F or the K8 with DDR2 support) there It's called K8RevE, not K9 is an additional mechanism that makes everything

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Yinghai Lu
On 6/26/07, Andi Kleen [EMAIL PROTECTED] wrote: For the K7 and K8 cores AMD systems are exactly like Intel systems with respect to MTRRs (although AMD systems also have additional registers) For the K9 core (i.e. AMD socket F or the K8 with DDR2 support) there It's called K8RevE, not K9

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Eric W. Biederman
Andi Kleen [EMAIL PROTECTED] writes: For the K7 and K8 cores AMD systems are exactly like Intel systems with respect to MTRRs (although AMD systems also have additional registers) For the K9 core (i.e. AMD socket F or the K8 with DDR2 support) there It's called K8RevE, not K9 revF not revE.

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Andi Kleen
On Tue, Jun 26, 2007 at 10:06:41AM -0600, Eric W. Biederman wrote: Andi Kleen [EMAIL PROTECTED] writes: For the K7 and K8 cores AMD systems are exactly like Intel systems with respect to MTRRs (although AMD systems also have additional registers) For the K9 core (i.e. AMD socket F or the

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-26 Thread Yinghai Lu
On 6/26/07, Andi Kleen [EMAIL PROTECTED] wrote: RevE had a new memory remapping scheme (memory hoisting) at least, which I think you refered to earlier. There might have been more changes in F. yes, REV E has hardware memory remapping. Rev F you don't need to use mtrr to set MEM to WE above

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Jesse Barnes
On Monday, June 25, 2007 8:29:35 pm Jesse Barnes wrote: > Is there an is_cpu() check to differentiate between those? Anyway I'd > rather not enable it unless we see reports though... So far I've only seen > reports of this problem on some recent Intel based systems. Oh, and FYI I've seen new

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Jesse Barnes
On Monday, June 25, 2007 5:54:49 pm Eric W. Biederman wrote: > Jesse Barnes <[EMAIL PROTECTED]> writes: > > On Monday, June 25, 2007 4:34:33 Andi Kleen wrote: > >> > This patch fixes a bug in the last patch that caused the code to > >> > run on non-Intel machines (AMD machines apparently don't

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Eric W. Biederman
Jesse Barnes <[EMAIL PROTECTED]> writes: > On Monday, June 25, 2007 4:34:33 Andi Kleen wrote: >> > This patch fixes a bug in the last patch that caused the code to >> > run on non-Intel machines (AMD machines apparently don't need it >> >> Actually the problem can happen on AMD too, but the

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Jesse Barnes
On Monday, June 25, 2007 4:34:33 Andi Kleen wrote: > > This patch fixes a bug in the last patch that caused the code to > > run on non-Intel machines (AMD machines apparently don't need it > > Actually the problem can happen on AMD too, but the symptoms can > be different and there can be more

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Andi Kleen
> This patch fixes a bug in the last patch that caused the code to > run on non-Intel machines (AMD machines apparently don't need it Actually the problem can happen on AMD too, but the symptoms can be different and there can be more wrong than just the MTRRs. -Andi - To unsubscribe from this

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Justin Piszcz
On Mon, 25 Jun 2007, Jesse Barnes wrote: On Monday, June 25, 2007 3:01:27 Andrew Morton wrote: On Mon, 25 Jun 2007 14:34:42 -0700 Jesse Barnes <[EMAIL PROTECTED]> wrote: akpm -- this one should replace all the mtrr patches currently in your tree. fear, uncertainty, doubt.

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Jesse Barnes
On Monday, June 25, 2007 3:01:27 Andrew Morton wrote: > On Mon, 25 Jun 2007 14:34:42 -0700 > > Jesse Barnes <[EMAIL PROTECTED]> wrote: > > akpm -- this one should replace all the mtrr patches currently > > in your tree. > > fear, uncertainty, doubt. > > box:/usr/src/25> grep mtrr series >

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Andrew Morton
On Mon, 25 Jun 2007 14:34:42 -0700 Jesse Barnes <[EMAIL PROTECTED]> wrote: > akpm -- this one should replace all the mtrr patches currently > in your tree. fear, uncertainty, doubt. box:/usr/src/25> grep mtrr series x86_64-mm-bug-in-i386-mtrr-initialization.patch

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Justin Piszcz
Will try this patch shortly w/ 2.6.22-rc6. p34:/usr/src/linux-2.6.22-rc6# patch -p1 < ../mtrr-v3.patch patching file Documentation/kernel-parameters.txt patching file arch/i386/kernel/cpu/mtrr/generic.c patching file arch/i386/kernel/cpu/mtrr/if.c patching file arch/i386/kernel/cpu/mtrr/main.c

[PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Jesse Barnes
On some machines, buggy BIOSes don't properly setup WB MTRRs to cover all available RAM, meaning the last few megs (or even gigs) of memory will be marked uncached. Since Linux tends to allocate from high memory addresses first, this causes the machine to be unusably slow as soon as the kernel

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Justin Piszcz
Impressive. Jesse, can you touch base with Intel's BIOS department? Also, what are the chances of that patch making it into 2.6.22-rc6/7 if it hasn't already? On Mon, 25 Jun 2007, Pim Zandbergen wrote: Pim Zandbergen wrote I reported this to GigaByte, and lo and behold, they sent me a

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Pim Zandbergen
Pim Zandbergen wrote I reported this to GigaByte, and lo and behold, they sent me a fixed BIOS within 48 hours. Kudos to Taipeh! They sent the BIOS image in a private message, so it might take a while before it's available on their website. It is now, and it is described as "Fix Vista boot

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Pim Zandbergen
Pim Zandbergen wrote I reported this to GigaByte, and lo and behold, they sent me a fixed BIOS within 48 hours. Kudos to Taipeh! They sent the BIOS image in a private message, so it might take a while before it's available on their website. It is now, and it is described as Fix Vista boot

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Justin Piszcz
Impressive. Jesse, can you touch base with Intel's BIOS department? Also, what are the chances of that patch making it into 2.6.22-rc6/7 if it hasn't already? On Mon, 25 Jun 2007, Pim Zandbergen wrote: Pim Zandbergen wrote I reported this to GigaByte, and lo and behold, they sent me a

[PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Jesse Barnes
On some machines, buggy BIOSes don't properly setup WB MTRRs to cover all available RAM, meaning the last few megs (or even gigs) of memory will be marked uncached. Since Linux tends to allocate from high memory addresses first, this causes the machine to be unusably slow as soon as the kernel

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Justin Piszcz
Will try this patch shortly w/ 2.6.22-rc6. p34:/usr/src/linux-2.6.22-rc6# patch -p1 ../mtrr-v3.patch patching file Documentation/kernel-parameters.txt patching file arch/i386/kernel/cpu/mtrr/generic.c patching file arch/i386/kernel/cpu/mtrr/if.c patching file arch/i386/kernel/cpu/mtrr/main.c

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Andrew Morton
On Mon, 25 Jun 2007 14:34:42 -0700 Jesse Barnes [EMAIL PROTECTED] wrote: akpm -- this one should replace all the mtrr patches currently in your tree. fear, uncertainty, doubt. box:/usr/src/25 grep mtrr series x86_64-mm-bug-in-i386-mtrr-initialization.patch

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Jesse Barnes
On Monday, June 25, 2007 3:01:27 Andrew Morton wrote: On Mon, 25 Jun 2007 14:34:42 -0700 Jesse Barnes [EMAIL PROTECTED] wrote: akpm -- this one should replace all the mtrr patches currently in your tree. fear, uncertainty, doubt. box:/usr/src/25 grep mtrr series

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Justin Piszcz
On Mon, 25 Jun 2007, Jesse Barnes wrote: On Monday, June 25, 2007 3:01:27 Andrew Morton wrote: On Mon, 25 Jun 2007 14:34:42 -0700 Jesse Barnes [EMAIL PROTECTED] wrote: akpm -- this one should replace all the mtrr patches currently in your tree. fear, uncertainty, doubt. box:/usr/src/25

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Andi Kleen
This patch fixes a bug in the last patch that caused the code to run on non-Intel machines (AMD machines apparently don't need it Actually the problem can happen on AMD too, but the symptoms can be different and there can be more wrong than just the MTRRs. -Andi - To unsubscribe from this

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Jesse Barnes
On Monday, June 25, 2007 4:34:33 Andi Kleen wrote: This patch fixes a bug in the last patch that caused the code to run on non-Intel machines (AMD machines apparently don't need it Actually the problem can happen on AMD too, but the symptoms can be different and there can be more wrong than

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Eric W. Biederman
Jesse Barnes [EMAIL PROTECTED] writes: On Monday, June 25, 2007 4:34:33 Andi Kleen wrote: This patch fixes a bug in the last patch that caused the code to run on non-Intel machines (AMD machines apparently don't need it Actually the problem can happen on AMD too, but the symptoms can be

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Jesse Barnes
On Monday, June 25, 2007 5:54:49 pm Eric W. Biederman wrote: Jesse Barnes [EMAIL PROTECTED] writes: On Monday, June 25, 2007 4:34:33 Andi Kleen wrote: This patch fixes a bug in the last patch that caused the code to run on non-Intel machines (AMD machines apparently don't need it

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-25 Thread Jesse Barnes
On Monday, June 25, 2007 8:29:35 pm Jesse Barnes wrote: Is there an is_cpu() check to differentiate between those? Anyway I'd rather not enable it unless we see reports though... So far I've only seen reports of this problem on some recent Intel based systems. Oh, and FYI I've seen new

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-21 Thread Jesse Barnes
On Thursday, June 21, 2007 12:40:58 Yinghai Lu wrote: > On 6/7/07, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > On some machines, buggy BIOSes don't properly setup WB MTRRs to > > cover all available RAM, meaning the last few megs (or even gigs) > > of memory will be marked uncached. Since Linux

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-21 Thread Yinghai Lu
On 6/7/07, Jesse Barnes <[EMAIL PROTECTED]> wrote: On some machines, buggy BIOSes don't properly setup WB MTRRs to cover all available RAM, meaning the last few megs (or even gigs) of memory will be marked uncached. Since Linux tends to allocate from high memory addresses first, this causes the

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-21 Thread Justin Piszcz
On Thu, 21 Jun 2007, Pim Zandbergen wrote: Jesse Barnes wrote: What, are you going to report this to GigaByte? No, but you should if you haven't already. I think GigaByte probably gets its BIOS from another BIOS vendor (maybe Intel), so when that vendor provides them with an update,

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-21 Thread Pim Zandbergen
Jesse Barnes wrote: What, are you going to report this to GigaByte? No, but you should if you haven't already. I think GigaByte probably gets its BIOS from another BIOS vendor (maybe Intel), so when that vendor provides them with an update, they'll probably provide it on their

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-21 Thread Pim Zandbergen
Jesse Barnes wrote: What, are you going to report this to GigaByte? No, but you should if you haven't already. I think GigaByte probably gets its BIOS from another BIOS vendor (maybe Intel), so when that vendor provides them with an update, they'll probably provide it on their

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-21 Thread Justin Piszcz
On Thu, 21 Jun 2007, Pim Zandbergen wrote: Jesse Barnes wrote: What, are you going to report this to GigaByte? No, but you should if you haven't already. I think GigaByte probably gets its BIOS from another BIOS vendor (maybe Intel), so when that vendor provides them with an update,

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-21 Thread Yinghai Lu
On 6/7/07, Jesse Barnes [EMAIL PROTECTED] wrote: On some machines, buggy BIOSes don't properly setup WB MTRRs to cover all available RAM, meaning the last few megs (or even gigs) of memory will be marked uncached. Since Linux tends to allocate from high memory addresses first, this causes the

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-21 Thread Jesse Barnes
On Thursday, June 21, 2007 12:40:58 Yinghai Lu wrote: On 6/7/07, Jesse Barnes [EMAIL PROTECTED] wrote: On some machines, buggy BIOSes don't properly setup WB MTRRs to cover all available RAM, meaning the last few megs (or even gigs) of memory will be marked uncached. Since Linux tends to

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-20 Thread Andi Kleen
> I assume this cannot be fixed by the simple approach > of echoing some useful numbers into /proc/mtrr like > we used to do for video memory? (Before X did this > automatically?) > > An extra bootscript seems better than loosing memory. In some cases it probably can, in other cases not because

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-20 Thread Pim Zandbergen
Jesse Barnes wrote: On Friday, June 15, 2007 3:17:11 Pim Zandbergen wrote: Not that it matters much, as the current i810/intel xorg driver does not yet support the GMA3100, so I'm using the vesa driver. I *think* the latest trees support that chip. If you're feeling brave, checkout

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-20 Thread Helge Hafting
Jesse Barnes wrote: On some machines, buggy BIOSes don't properly setup WB MTRRs to cover all available RAM, meaning the last few megs (or even gigs) of memory will be marked uncached. Since Linux tends to allocate from high memory addresses first, this causes the machine to be unusably slow as

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-20 Thread Helge Hafting
Jesse Barnes wrote: On some machines, buggy BIOSes don't properly setup WB MTRRs to cover all available RAM, meaning the last few megs (or even gigs) of memory will be marked uncached. Since Linux tends to allocate from high memory addresses first, this causes the machine to be unusably slow as

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-20 Thread Pim Zandbergen
Jesse Barnes wrote: On Friday, June 15, 2007 3:17:11 Pim Zandbergen wrote: Not that it matters much, as the current i810/intel xorg driver does not yet support the GMA3100, so I'm using the vesa driver. I *think* the latest trees support that chip. If you're feeling brave, checkout

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-20 Thread Andi Kleen
I assume this cannot be fixed by the simple approach of echoing some useful numbers into /proc/mtrr like we used to do for video memory? (Before X did this automatically?) An extra bootscript seems better than loosing memory. In some cases it probably can, in other cases not because the

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-15 Thread Jesse Barnes
On Friday, June 15, 2007 3:17:11 Pim Zandbergen wrote: > Not that it matters much, as the current i810/intel xorg driver does > not yet support the GMA3100, so I'm using the vesa driver. I *think* the latest trees support that chip. If you're feeling brave, checkout the latest version of the

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-15 Thread Jesse Barnes
On Friday, June 15, 2007 3:21:17 Pim Zandbergen wrote: > Jesse Barnes wrote: > > Thanks for testing, Pim. Glad it works for you. > > The pleasure was all on my side. > > > Keep an eye out for BIOS upgrades, the next version might fix it. > > What, are you going to report this to GigaByte? No,

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-15 Thread Justin Piszcz
On Fri, 15 Jun 2007, Pim Zandbergen wrote: Justin Piszcz wrote: That's strange, I guess different chipsets 'chew' up different amounts of memory OR you have your DVT(?) (video-card memory/aperature) set to 256MB? I have mine set to 128MB, in top: Mem: 8039576k total, 6187304k used,

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-15 Thread Pim Zandbergen
Jesse Barnes wrote: Thanks for testing, Pim. Glad it works for you. The pleasure was all on my side. Keep an eye out for BIOS upgrades, the next version might fix it. What, are you going to report this to GigaByte? Thanks, Pim - To unsubscribe from this list: send the line

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-15 Thread Pim Zandbergen
Justin Piszcz wrote: That's strange, I guess different chipsets 'chew' up different amounts of memory OR you have your DVT(?) (video-card memory/aperature) set to 256MB? I have mine set to 128MB, in top: Mem: 8039576k total, 6187304k used, 1852272k free, 696k buffers Me: Mem:

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-15 Thread Pim Zandbergen
Justin Piszcz wrote: That's strange, I guess different chipsets 'chew' up different amounts of memory OR you have your DVT(?) (video-card memory/aperature) set to 256MB? I have mine set to 128MB, in top: Mem: 8039576k total, 6187304k used, 1852272k free, 696k buffers Me: Mem:

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-15 Thread Pim Zandbergen
Jesse Barnes wrote: Thanks for testing, Pim. Glad it works for you. The pleasure was all on my side. Keep an eye out for BIOS upgrades, the next version might fix it. What, are you going to report this to GigaByte? Thanks, Pim - To unsubscribe from this list: send the line

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-15 Thread Justin Piszcz
On Fri, 15 Jun 2007, Pim Zandbergen wrote: Justin Piszcz wrote: That's strange, I guess different chipsets 'chew' up different amounts of memory OR you have your DVT(?) (video-card memory/aperature) set to 256MB? I have mine set to 128MB, in top: Mem: 8039576k total, 6187304k used,

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-15 Thread Jesse Barnes
On Friday, June 15, 2007 3:21:17 Pim Zandbergen wrote: Jesse Barnes wrote: Thanks for testing, Pim. Glad it works for you. The pleasure was all on my side. Keep an eye out for BIOS upgrades, the next version might fix it. What, are you going to report this to GigaByte? No, but you

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-15 Thread Jesse Barnes
On Friday, June 15, 2007 3:17:11 Pim Zandbergen wrote: Not that it matters much, as the current i810/intel xorg driver does not yet support the GMA3100, so I'm using the vesa driver. I *think* the latest trees support that chip. If you're feeling brave, checkout the latest version of the

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-14 Thread Jesse Barnes
On Thursday, June 14, 2007 2:21:16 Justin Piszcz wrote: > To Intel, > > When will HECI be supported via the kernel? When it becomes > supported, would that alter the MTRR map at all? I *think* HECI is related to our IT remote management stuff, but I don't work on it. It *may* affect the MTRR

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-14 Thread Justin Piszcz
On Thu, 14 Jun 2007, Jesse Barnes wrote: On Thursday, June 14, 2007 1:26:07 Justin Piszcz wrote: On Thu, 14 Jun 2007, Pim Zandbergen wrote: Thanks for this patch. I was having the exact same symptoms as Justin Piszcz, on a different, but similar motherboard: Motherboard: GigaByte

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-14 Thread Jesse Barnes
On Thursday, June 14, 2007 1:26:07 Justin Piszcz wrote: > On Thu, 14 Jun 2007, Pim Zandbergen wrote: > > Thanks for this patch. I was having the exact same symptoms as > > Justin Piszcz, on a different, but similar motherboard: > > > > Motherboard: GigaByte GA-G33-DS3R > > BIOS rev: F2 > >

Re: [PATCH] trim memory not covered by WB MTRRs

2007-06-14 Thread Justin Piszcz
On Thu, 14 Jun 2007, Pim Zandbergen wrote: Thanks for this patch. I was having the exact same symptoms as Justin Piszcz, on a different, but similar motherboard: Motherboard: GigaByte GA-G33-DS3R BIOS rev: F2 Chipset: Intel G33 Memory: 8GB Distro: Fedora 7 x86_64 Kernel:

  1   2   3   >