linux-kernel - Re: 2.6.25-rc5-mm1 (paravirt/vsmp/no PCI)

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 for Android: free password hash cracker in your pocket

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

Message-ID: <47D73500.5080205@oracle.com>
Date:	Tue, 11 Mar 2008 18:42:24 -0700
From:	Randy Dunlap <randy.dunlap@...cle.com>
To:	Ravikiran G Thirumalai <kiran@...lex86.org>
CC:	Jeremy Fitzhardinge <jeremy@...p.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	virtualization@...ts.osdl.org, linux-kernel@...r.kernel.org,
	Glauber de Oliveira Costa <glommer@...il.com>,
	shai@...lex86.org
Subject: Re: 2.6.25-rc5-mm1 (paravirt/vsmp/no PCI)

Ravikiran G Thirumalai wrote:
> On Tue, Mar 11, 2008 at 11:18:44AM -0700, Jeremy Fitzhardinge wrote:
>> Randy Dunlap wrote:
>>
>> Added Cc:
>>
>>> On Tue, 11 Mar 2008 01:14:34 -0700 Andrew Morton wrote:
>>>
>>>
>>>> ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.25-rc5/2.6.25-rc5-mm1/
>>>>
>>> randconfig (x86_64) with
>>> PCI=n
>>> PARAVIRT=y
>>> VSMP=n
>>>
>>> ends with
>>>
>>> arch/x86/kernel/built-in.o: In function `is_vsmp_box':
>>> (.text+0x1178d): undefined reference to `early_pci_allowed'
>>> arch/x86/kernel/built-in.o: In function `is_vsmp_box':
>>> (.text+0x117a9): undefined reference to `read_pci_config'
>>> arch/x86/kernel/built-in.o: In function `vsmp_init':
>>> (.init.text+0x4fcc): undefined reference to `early_pci_allowed'
>>> arch/x86/kernel/built-in.o: In function `vsmp_init':
>>> (.init.text+0x501a): undefined reference to `read_pci_config'
>>> make[1]: *** [.tmp_vmlinux1] Error 1
>>>
>>> config attached.
>>   J
> 
> 
> Would anyone have objection to have PARAVIRT depend on PCI, since the
> vsmp paravirt bits depend on PCI cfg space to determine if the system is
> vsmp?   If not, this patch would suffice.

Works for me.
Acked-by: Randy Dunlap <randy.dunlap@...cle.com>

Thanks.

> Glauber?
> 
> Thanks,
> Kiran
> 
> ---
> 
> Make PARAVIRT depend on PCI.
> 
> vSMP PARAVIRT ops probe the pci config space to determine if the
> system is indeed a ScaleMP vSMP box.  Hence, depend on PCI to enable
> PARAVIRT.
> 
> Signed-off-by: Ravikiran Thirumalai <kiran@...lex86.org>
> 
> Index: linux-2.6.24/arch/x86/Kconfig
> ===================================================================
> --- linux-2.6.24.orig/arch/x86/Kconfig	2008-03-11 16:38:26.000000000 -0700
> +++ linux-2.6.24/arch/x86/Kconfig	2008-03-11 16:50:52.000000000 -0700
> @@ -384,7 +384,7 @@ source "arch/x86/lguest/Kconfig"
>  
>  config PARAVIRT
>  	bool "Enable paravirtualization code"
> -	depends on !(X86_VISWS || X86_VOYAGER)
> +	depends on !(X86_VISWS || X86_VOYAGER) && PCI
>  	help
>  	  This changes the kernel so it can modify itself when it is run
>  	  under a hypervisor, potentially improving performance significantly


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