Re: Re: [PATCH] Fix restore handling checks

[Michal Novotny <minovotn@redhat.com>]

On 06/23/2010 02:10 PM, Paolo Bonzini wrote:
> On 06/23/2010 02:04 PM, Ian Jackson wrote:
>> Michal Novotny writes ("Re: [PATCH] Fix restore handling checks"):
>>> Are you saying that it's OK for administrators to violate the IDE specs
>>> and do it the way that is should never be done since this way it's not
>>> working on bare-metal systems ? This is the breach and it shouldn't be
>>> done this way so why to allow it? Shouldn't we care the code complies
>>> with the specifications to have it done the right way?
>>
>> The job of the programmer is to give effect to the wishes of the
>> users, not to comply with rules from elsewhere.  If the wishes of the
>> users conflict with rules from elsewhere, including specs, then the
>> programmer should do what the user wants.
>
> Telling him that he wanted something that cannot be _emulated_ 
> accurately is also a possibility.  But we can agree to disagree here 
> and Michal can remove this part of the patch.
>
> Paolo
>

Well, just one correction to this Paolo. It can be emulated but the 
emulation doesn't comply to the specifications and since it was never 
supported at least for HVM guests prior to my patch to fix read-only 
image handling so it shouldn't break anything so if it was not emulated 
before the patch I sent 13 days ago why not to emulate/implement it the 
right way? I don't see any reason to emulate it the wrong way since we 
know the right way that's been implemented/fixed just 13 days ago - at 
least for HVM guests. If users want to use read-only drives they can use 
SCSI drives so what's the problem here?

Surely, I can remove this part of the patch but I'm just trying to tell 
the readers of this thread that I don't see much point in having the 
wrong implementation. I don't think users want to do it the wrong way 
and I think they could understand that this is not supported and that 
they should use SCSI drive definition for read-only drives instead.

-- 
Michal Novotny<minovotn@redhat.com>, RHCE
Virtualization Team (xen userspace), Red Hat