From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1KXPt2-0003qT-EX
	for qemu-devel@nongnu.org; Sun, 24 Aug 2008 20:24:48 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1KXPt0-0003q7-5W
	for qemu-devel@nongnu.org; Sun, 24 Aug 2008 20:24:47 -0400
Received: from [199.232.76.173] (port=47372 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1KXPsz-0003pw-WC
	for qemu-devel@nongnu.org; Sun, 24 Aug 2008 20:24:46 -0400
Received: from yx-out-1718.google.com ([74.125.44.155]:46035)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <anthony@codemonkey.ws>) id 1KXPt0-0006xt-2W
	for qemu-devel@nongnu.org; Sun, 24 Aug 2008 20:24:46 -0400
Received: by yx-out-1718.google.com with SMTP id 3so795956yxi.82
	for <qemu-devel@nongnu.org>; Sun, 24 Aug 2008 17:24:44 -0700 (PDT)
Message-ID: <48B1FB9F.3090405@codemonkey.ws>
Date: Sun, 24 Aug 2008 19:23:59 -0500
From: Anthony Liguori <anthony@codemonkey.ws>
MIME-Version: 1.0
Subject: Re: [Qemu-devel] [PATCH 1/1] Make vmport an optional feature at run
	time.
References: <1219578445-25610-1-git-send-email-blob@blob.co.uk>
	<48B1B8C5.4040902@codemonkey.ws>
	<alpine.LFD.1.10.0808242052020.13622@bcny.fcbq.bet>
In-Reply-To: <alpine.LFD.1.10.0808242052020.13622@bcny.fcbq.bet>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Reply-To: qemu-devel@nongnu.org
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Ian Kirk <blob@blob.co.uk>
Cc: qemu-devel@nongnu.org

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