qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* Re : [Qemu-devel] [PATCH 1/1] Make vmport an optional feature at run time.
@ 2008-08-25 11:21 Sylvain Petreolle
  2008-08-25 14:14 ` Anthony Liguori
  0 siblings, 1 reply; 8+ messages in thread
From: Sylvain Petreolle @ 2008-08-25 11:21 UTC (permalink / raw)
  To: qemu-devel

----- Message d'origine ----

> De : Anthony Liguori <anthony@codemonkey.ws>
> À : Ian Kirk <blob@blob.co.uk>
> Cc : qemu-devel@nongnu.org
> Envoyé le : Lundi, 25 Août 2008, 2h23mn 59s
> Objet : Re: [Qemu-devel] [PATCH 1/1] Make vmport an optional feature at run time.
> 
> Ian Kirk wrote:
> > Anthony Liguori wrote:
> >
> >  
> >> Ian Kirk wrote:
> >>    
> >>> Hi,
> >>>
> >>> This is my first time sending a patch, so apologies for any errors.
> >>>
> >>> Patch makes vmport optionally initiated.
> >>>
> >>>      
> >> Why should it be optional?
> >>    
> >
> > I believe that VMware ESXi (and perhaps other hypervisors/emulators/etc)
> > doesn't work when it thinks it is running within virtual enviroment, as it
> > talks to vmport when booting (and fails in the current implemtnation -
> > perhaps it only works under itself?)
> >  
> 
> That's most likely because the vmport emulation isn't complete enough.
> 
> > If I comment init_vmport() out, it definately progresses further along the
> > boot sequence.
> >
> > Also, I guess, it gives you the option to better emulate a real PC (which
> > I assume doesn't have vmport).
> >  
> 
> I don't think that's very valuable in and of itself.  Moreover, there 
> are probably a lot more issues with respect to getting ESXi to run under 
> QEMU.  Adding another command line option to support something that we 
> don't know will ever work worries me.  It's just another knob for 
> someone to accidentally tweak.
> 
> Regards,
> 
> Anthony Liguori

Apart ESXi, there are software that behave differently if they detect a virtual environment.
Some of them completely refuse to work.


Regards,

Sylvain Petreolle


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

end of thread, other threads:[~2008-08-28 21:42 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-08-25 11:21 Re : [Qemu-devel] [PATCH 1/1] Make vmport an optional feature at run time Sylvain Petreolle
2008-08-25 14:14 ` Anthony Liguori
2008-08-26 18:06   ` Ian Kirk
2008-08-28 20:19     ` Anthony Liguori
2008-08-28 20:42       ` Ian Kirk
2008-08-28 20:47         ` Anthony Liguori
2008-08-28 20:52           ` Ian Kirk
2008-08-28 21:41             ` Anthony Liguori

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