virtualization.lists.linux-foundation.org archive mirror
 help / color / mirror / Atom feed
* PATCH: Fix VMI and COMPAT_VDSO for 2.6.21
@ 2007-03-10  7:00 Zachary Amsden
  2007-03-10  7:01 ` Zachary Amsden
  0 siblings, 1 reply; 4+ messages in thread
From: Zachary Amsden @ 2007-03-10  7:00 UTC (permalink / raw)
  To: Linus Torvalds, Andrew Morton, Andi Kleen, Ingo Molnar,
	Rusty Russell, Jeremy Fitzhardinge, Chris Wright,
	Linux Kernel Mailing List, Virtualization Mailing List

VMI is broken under COMPAT_VDSO, as Xen and other non hardware assisted 
hypervisors will be.  I have been working on a fix for this which works 
for older glibcs that panic when the new relocatable VDSO is used.  
However, I believe at this time that the fix is going to be too radical 
to consider at this stage in the release of 2.6.21.  We don't expect 
this config option to be turned on by vendors for new distributions, so 
at this point we are willing to drop support for it when VMI is compiled 
in, and work on a patch for 2.6.22 which more fully addresses the problem.

Please apply, thanks.

Zach

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: PATCH: Fix VMI and COMPAT_VDSO for 2.6.21
  2007-03-10  7:00 PATCH: Fix VMI and COMPAT_VDSO for 2.6.21 Zachary Amsden
@ 2007-03-10  7:01 ` Zachary Amsden
  2007-03-10  8:47   ` Ingo Molnar
  0 siblings, 1 reply; 4+ messages in thread
From: Zachary Amsden @ 2007-03-10  7:01 UTC (permalink / raw)
  To: Zachary Amsden
  Cc: Andrew Morton, Chris Wright, Linus Torvalds,
	Virtualization Mailing List, Linux Kernel Mailing List,
	Ingo Molnar

[-- Attachment #1: Type: text/plain, Size: 708 bytes --]

Zachary Amsden wrote:
> VMI is broken under COMPAT_VDSO, as Xen and other non hardware 
> assisted hypervisors will be.  I have been working on a fix for this 
> which works for older glibcs that panic when the new relocatable VDSO 
> is used.  However, I believe at this time that the fix is going to be 
> too radical to consider at this stage in the release of 2.6.21.  We 
> don't expect this config option to be turned on by vendors for new 
> distributions, so at this point we are willing to drop support for it 
> when VMI is compiled in, and work on a patch for 2.6.22 which more 
> fully addresses the problem.
>
> Please apply, thanks.

I suppose it would be better if I attached the patch.

Zach

[-- Attachment #2: vmi-depends-on-not-compat-vdso.patch --]
[-- Type: text/plain, Size: 592 bytes --]

Signed-off-by: Zachary Amsden <zach@vmware.com>

Index: linux-2.6.21/arch/i386/Kconfig
===================================================================
--- linux-2.6.21.orig/arch/i386/Kconfig	2007-03-06 18:51:33.000000000 -0800
+++ linux-2.6.21/arch/i386/Kconfig	2007-03-09 22:36:15.000000000 -0800
@@ -220,7 +220,7 @@ config PARAVIRT
 
 config VMI
 	bool "VMI Paravirt-ops support"
-	depends on PARAVIRT
+	depends on PARAVIRT && !COMPAT_VDSO
 	help
 	  VMI provides a paravirtualized interface to the VMware ESX server
 	  (it could be used by other hypervisors in theory too, but is not

[-- Attachment #3: Type: text/plain, Size: 165 bytes --]

_______________________________________________
Virtualization mailing list
Virtualization@lists.osdl.org
https://lists.osdl.org/mailman/listinfo/virtualization

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: PATCH: Fix VMI and COMPAT_VDSO for 2.6.21
  2007-03-10  7:01 ` Zachary Amsden
@ 2007-03-10  8:47   ` Ingo Molnar
  2007-03-10 10:32     ` Zachary Amsden
  0 siblings, 1 reply; 4+ messages in thread
From: Ingo Molnar @ 2007-03-10  8:47 UTC (permalink / raw)
  To: Zachary Amsden
  Cc: Andrew Morton, Chris Wright, Linus Torvalds,
	Virtualization Mailing List, Linux Kernel Mailing List


* Zachary Amsden <zach@vmware.com> wrote:

> Zachary Amsden wrote:
> >VMI is broken under COMPAT_VDSO, as Xen and other non hardware 
> >assisted hypervisors will be.  I have been working on a fix for this 
> >which works for older glibcs that panic when the new relocatable VDSO 
> >is used.  However, I believe at this time that the fix is going to be 
> >too radical to consider at this stage in the release of 2.6.21.  We 
> >don't expect this config option to be turned on by vendors for new 
> >distributions, so at this point we are willing to drop support for it 
> >when VMI is compiled in, and work on a patch for 2.6.22 which more 
> >fully addresses the problem.
> >
> >Please apply, thanks.

makes sense. We can do Jan's relocatable-COMPAT_VDSO thing in v2.6.22, 
but for v2.6.21 that's way too intrusive.

Acked-by: Ingo Molnar <mingo@elte.hu>

> I suppose it would be better if I attached the patch.

it certainly helps ;-)

	Ingo

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: PATCH: Fix VMI and COMPAT_VDSO for 2.6.21
  2007-03-10  8:47   ` Ingo Molnar
@ 2007-03-10 10:32     ` Zachary Amsden
  0 siblings, 0 replies; 4+ messages in thread
From: Zachary Amsden @ 2007-03-10 10:32 UTC (permalink / raw)
  To: Ingo Molnar
  Cc: Linus Torvalds, Andrew Morton, Andi Kleen, Rusty Russell,
	Jeremy Fitzhardinge, Chris Wright, Linux Kernel Mailing List,
	Virtualization Mailing List

Ingo Molnar wrote:
> makes sense. We can do Jan's relocatable-COMPAT_VDSO thing in v2.6.22, 
> but for v2.6.21 that's way too intrusive.
>   

Agree.  I think we can clean up some of the strange build magic though, 
by adding boot time ELF magic instead.  We'll see which works out better.

Zach

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2007-03-10 10:32 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-03-10  7:00 PATCH: Fix VMI and COMPAT_VDSO for 2.6.21 Zachary Amsden
2007-03-10  7:01 ` Zachary Amsden
2007-03-10  8:47   ` Ingo Molnar
2007-03-10 10:32     ` Zachary Amsden

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).