[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Add support for vmware vmdk scsi images to qemu
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH] Add support for vmware vmdk scsi images to qemu-img |
Date: |
Wed, 12 Aug 2009 15:48:15 +0200 |
User-agent: |
Thunderbird 2.0.0.21 (X11/20090320) |
Anthony Liguori schrieb:
> Pantelis Koukousoulas wrote:
>> diff --git a/block_int.h b/block_int.h
>> index 830b7e9..0311198 100644
>> --- a/block_int.h
>> +++ b/block_int.h
>> @@ -30,10 +30,12 @@
>> #define BLOCK_FLAG_ENCRYPT 1
>> #define BLOCK_FLAG_COMPRESS 2
>> #define BLOCK_FLAG_COMPAT6 4
>> +#define BLOCK_FLAG_SCSI 8
>
> The flag space is limited and I don't want to pollute it with options
> that are of no use to QEMU. If you were able to limit the impact of
> this patch to just block/vmdk.c, we would be more likely to consider
> it. That may require a greater refactoring of how options work though.
Well, yes, a refactoring as great as using a local int scsi instead of
defining a global constant to be or-ed into a local int flags... ;-)
Actually I think we could drop these global BLOCK_FLAG_* completely
without losing anything.
And otherwise the patch already is completely local to VMDK.
Kevin
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH] Add support for vmware vmdk scsi images to qemu-img,
Kevin Wolf <=