netdev - Re: [e1000][net-2.6 tree] Regression: driver doesn't detect card on my node.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 06 Feb 2008 11:41:15 +0300
From:	Pavel Emelyanov <xemul@...nvz.org>
To:	Yinghai Lu <Yinghai.Lu@....COM>
CC:	"Kok, Auke" <auke-jan.h.kok@...el.com>, jesse.brandeburg@...el.com,
	jeffrey.t.kirsher@...el.com, john.ronciak@...el.com,
	David Miller <davem@...emloft.net>,
	Linux Netdev List <netdev@...r.kernel.org>,
	e1000-devel@...ts.sourceforge.net, Denis Lunev <den@...nvz.org>,
	Justin Piszcz <jpiszcz@...idpixels.com>,
	Ingo Molnar <mingo@...e.hu>,
	Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [e1000][net-2.6 tree] Regression: driver doesn't detect card
 on my node.

Yinghai Lu wrote:
> 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@...ulnb) (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@...ulnb) (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.

OK, here's the dmesg diff:
--- bad-log     2008-02-06 10:01:21.000000000 +0300
+++ bad-log-info        2008-02-06 11:37:51.000000000 +0300
@@ -19,6 +19,14 @@ 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
+i=0 c0000 - 100000 0
+i=1 0 - 100000 6
+i=2 100000 - 140000 6
+i=3 0 - 0 0
+i=4 0 - 0 0
+i=5 0 - 0 0
+i=6 0 - 0 0
+i=7 0 - 0 0
 ***************
 **** WARNING: likely BIOS bug
 **** MTRRs don't cover all of memory, trimmed 786432 pages


> 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;
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html