2010/8/11 Daniel Veillard <veillard@redhat.com>:
On Mon, Aug 09, 2010 at 04:44:14PM -0600, Eric Blake wrote:
On 08/05/2010 06:45 PM, Matthias Bolte wrote:
floppy0.present defaults to true for GSX. Therefore, it needs to be explicitly disabled for GSX. --- src/esx/esx_vmx.c | 10 ++++++++++ 1 files changed, 10 insertions(+), 0 deletions(-)
If I understand correctly, you want to uniformly make floppy not present except when the user asked for it. The other ESX clients meet this goal, but for GSX, you have to add an extra argument to make it so. That being the case, I agree with the approach.
That's the intention, but it didn't struck me right away that ESX and GSX being different here is odd. I noticed that floppy0.present defaults to true on GSX while investigating some other GSX related issues in the VMX handling code. The comments in the esx_vmx.c file indicate that ESX would default to false in that case, but that's wrong, as I know now. floppy0.present defaults to true for GSX _and_ ESX, so the assumed default for ESX was wrong all the time. I probably screwed it when I wrote the initial VMX handling code. Therefore, this patch is incomplete and we need a v2 to address the ESX issue too.
@@ -2539,6 +2544,11 @@ esxVMX_FormatConfig(esxVMX_Context *ctx, virCapsPtr caps, virDomainDefPtr def, } }
+ if (!hasFloppyDevice && productVersion & esxVI_ProductVersion_GSX) {
I'd add an extra () here to make precedence clear:
if (!hasFloppyDevice && (productVersion & esxVI_ProductVersion_GSX)) {
ACK with that nit fixed.
v2 won't have the product version check anymore.
Agreed, ACK
Matthias you push it ?
Daniel
Well, no, as we need a v2 for this :) Matthias