netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Yinghai Lu <Yinghai.Lu@Sun.COM>
To: Pavel Emelyanov <xemul@openvz.org>
Cc: "Kok, Auke" <auke-jan.h.kok@intel.com>,
	jesse.brandeburg@intel.com, jeffrey.t.kirsher@intel.com,
	john.ronciak@intel.com, David Miller <davem@davemloft.net>,
	Linux Netdev List <netdev@vger.kernel.org>,
	e1000-devel@lists.sourceforge.net, Denis Lunev <den@openvz.org>,
	Justin Piszcz <jpiszcz@lucidpixels.com>,
	Ingo Molnar <mingo@elte.hu>, Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [e1000][net-2.6 tree] Regression: driver doesn't detect card on my node.
Date: Wed, 06 Feb 2008 00:20:24 -0800	[thread overview]
Message-ID: <200802060020.25492.yinghai.lu@sun.com> (raw)
In-Reply-To: <47A95D60.5090203@openvz.org>

On Tuesday 05 February 2008 11:10:24 pm Pavel Emelyanov wrote:
> [snip]
> 
> >>> What other info from me is required?
> > 
> > boot log please. with the patch and reverting the patch.
> 
> Here they are (attached).
> BTW, I found, that the sky2 adapter is broken by this patch as well.
> To simplify the analysis, here's their diff:
> 
> --- bad-log	2008-02-06 10:06:40.000000000 +0300
> +++ good-log	2008-02-06 10:06:40.000000000 +0300
> @@ -1,4 +1,4 @@
> -Linux version 2.6.24 (pavel@xemulnb) (gcc version 4.1.3 20070724 (prerelease) (SUSE Linux)) #22 SMP Tue Feb 5 20:09:11 MSK 2008
> +Linux version 2.6.24 (pavel@xemulnb) (gcc version 4.1.3 20070724 (prerelease) (SUSE Linux)) #23 SMP Wed Feb 6 10:03:53 MSK 2008
>  BIOS-provided physical RAM map:
>   BIOS-e820: 0000000000000000 - 000000000009f800 (usable)
>   BIOS-e820: 000000000009f800 - 00000000000a0000 (reserved)
> @@ -19,29 +19,6 @@ Scan SMP from c0000000 for 1024 bytes.
>  Scan SMP from c009fc00 for 1024 bytes.
>  Scan SMP from c00f0000 for 65536 bytes.
>  found SMP MP-table at [c00ff780] 000ff780
> -***************
> -**** WARNING: likely BIOS bug
> -**** MTRRs don't cover all of memory, trimmed 786432 pages
> -***************
> -update e820 for mtrr
> -modified physical RAM map:
> - modified: 0000000000000000 - 000000000009f800 (usable)
> - modified: 000000000009f800 - 00000000000a0000 (reserved)
> - modified: 00000000000e0000 - 0000000000100000 (reserved)
> - modified: 0000000000100000 - 0000000040000000 (usable)
> - modified: 0000000040000000 - 00000000bfff0000 (reserved)
> - modified: 00000000bfff0000 - 00000000bffff000 (ACPI data)
> - modified: 00000000bffff000 - 00000000c0000000 (ACPI NVS)
> - modified: 00000000c0000000 - 0000000100000000 (reserved)
> - modified: 0000000100000000 - 0000000140000000 (usable)

the root cause could be your mtrr last entry for RAM has strange type

please apply the following line to print the mtrr before trimming to current linus tree.

Thanks

YH

diff --git a/arch/x86/kernel/cpu/mtrr/main.c b/arch/x86/kernel/cpu/mtrr/main.c
index 1e27b69..c0c44e0 100644
--- a/arch/x86/kernel/cpu/mtrr/main.c
+++ b/arch/x86/kernel/cpu/mtrr/main.c
@@ -680,6 +680,7 @@ int __init mtrr_trim_uncached_memory(unsigned long end_pfn)
 	/* Find highest cached pfn */
 	for (i = 0; i < num_var_ranges; i++) {
 		mtrr_if->get(i, &base, &size, &type);
+		printk(KERN_INFO "i=%d %lx - %lx %d\n", i, base, base + size, type);
 		if (type != MTRR_TYPE_WRBACK)
 			continue;
 		base <<= PAGE_SHIFT;

  reply	other threads:[~2008-02-06  8:12 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-05 17:18 [e1000][net-2.6 tree] Regression: driver doesn't detect card on my node Pavel Emelyanov
2008-02-05 17:24 ` Kok, Auke
2008-02-05 17:43   ` Yinghai Lu
2008-02-06  7:10     ` Pavel Emelyanov
2008-02-06  8:20       ` Yinghai Lu [this message]
2008-02-06  8:41         ` Pavel Emelyanov
2008-02-06  9:19           ` Yinghai Lu
2008-02-06  9:30             ` Pavel Emelyanov
2008-02-06  9:49               ` [PATCH] x86_32: fix regression caused by trim ram according to mtrr on system with 4G more RAM Yinghai Lu
2008-02-06  9:51                 ` Ingo Molnar
2008-02-06  7:27     ` [e1000][net-2.6 tree] Regression: driver doesn't detect card on my node Pavel Emelyanov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200802060020.25492.yinghai.lu@sun.com \
    --to=yinghai.lu@sun.com \
    --cc=auke-jan.h.kok@intel.com \
    --cc=davem@davemloft.net \
    --cc=den@openvz.org \
    --cc=e1000-devel@lists.sourceforge.net \
    --cc=jeffrey.t.kirsher@intel.com \
    --cc=jesse.brandeburg@intel.com \
    --cc=john.ronciak@intel.com \
    --cc=jpiszcz@lucidpixels.com \
    --cc=mingo@elte.hu \
    --cc=netdev@vger.kernel.org \
    --cc=tglx@linutronix.de \
    --cc=xemul@openvz.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).