Re: [PATCH] Rename KVM_UPSTREAM to OBSOLETE_KVM_IMPL

[Jan Kiszka <jan.kiszka@web.de>]

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

Am 13.09.2010 20:56, Anthony Liguori wrote:
> On 09/13/2010 01:52 PM, Jan Kiszka wrote:
>> Am 13.09.2010 19:54, Avi Kivity wrote:
>>   
>>> The symbol KVM_UPSTREAM is used to mark sections of code that are
>>> part of
>>> the upstream kvm implemetation that is not used in qemu-kvm.  However
>>> the
>>> name becomes ambiguous if qemu-kvm is merged upstream.
>>>      
>> I doubt this is describing all cases correctly as well. Some changes
>> should rather happen the other way around (e.g. you surely don't want to
>> obsolete x86 kvm_arch_put/get_registers in favor of
>> kvm_arch_load/save_regs, do you?).
>>    
> 
> There's really no perfect name to describe what we're actually doing
> here.  It's probably not a detail worth worrying that much about.

I don't mind the name as long as it doesn't reflect the strategy (but
why this change at all then?).

Jan (who would prefer to have the time for doing the cleanups)


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 259 bytes --]