<div dir="ltr">Hi Jan,<div><br></div><div>Thanks for the replies.<br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 15, 2013 at 1:29 PM, Ján Tomko <span dir="ltr"><<a href="mailto:jtomko@redhat.com" target="_blank">jtomko@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">On 11/15/2013 04:20 AM, Li Zhang wrote:<br>

> On 2013年11月14日 18:02, Ján Tomko wrote:<br>
>> On 10/28/2013 10:20 AM, Shivaprasad G Bhat wrote:<br>
>>> The bus type IDE being enum Zero, the bus type on pseries system appears as<br>
>>> IDE for all the disk types. Pseries platform needs this to appear as SCSI<br>
>>> instead of IDE.<br>
>>><br>
>>> Signed-off-by: Shivaprasad G Bhat <<a href="mailto:sbhat@linux.vnet.ibm.com">sbhat@linux.vnet.ibm.com</a>><br>
>>> ---<br>
>>>   src/qemu/qemu_domain.c |   11 +++++++++++<br>
>>>   1 file changed, 11 insertions(+)<br>
>>><br>
</div>...<br>
<div class="im">>> This would also affect XML parsing, as these PostParse functions are called<br>
>> when parsing the XML too, not just when doing a XML->native translation.<br>
>><br>
>> This also affect all the disks specified on the command line. You shouldn't<br>
>> assume the disk is SCSI when IDE was explicitly specified.<br>
><br></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">> This patch is to parse "-hda" as SCSI device type because Power doesn't<br>

> support IDE device.<br>
> I am not sure whether it's necessary to keep IDE device type for Power in<br>
> libvirt.<br>
> Currently, QEMU set "-hda" as SCSI for Power platform.<br>
><br>
<br>
</div>Yes, since QEMU treats it as SCSI, we could do the same for -hdX.<br>
I meant that we shouldn't correct command line like<br>
-device ide-cd,drive=ddd,bus=ide.0 to SCSI.<br></blockquote><div> </div><div>I checked that the domxml-to-native doesnt process -device option in qemuParseCommandLine. It simply add an arg entry into the xml.</div><div><div>
 <qemu:arg value='-device'/></div><div>    <qemu:arg value='scsi-cd,bus=scsi0.0,channel=0,scsi-id=0,lun=2,drive=drive-scsi0-0-0-2,id=scsi0-0-0-2,bootindex=1'/></div></div><div><br></div><div>May be we should check these ide strings for device option in qemuParseCommandLine() and error out. Let me know if this is the right approach than doing in PostParsing.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
But it seems domxml-to-native can't even parse the command line libvirt<br>
outputs for disks at the moment:<br>
error: internal error: missing index/unit/bus parameter in drive<br>
'file=/var/iso/f19.iso,if=none,id=drive-ide0-1-0,readonly=on,format=raw'<br>
<br></blockquote><div>The -drive option parsing doesnt process the id="XXXXXX" attribute today. The bus, index, unitid need to be derived from the string "drive-ide0-1-0". Let me know if you want me to add the parsing code. Otherwise, I see there is no mention of id="" in any of the common usage documentation. Link that i referred is <a href="http://wiki.qemu.org/download/qemu-doc.html">http://wiki.qemu.org/download/qemu-doc.html</a>. Is that a hidden attribute? </div>
<div><br></div><div>So, I assume only the "un-documented" options("id=drive-ide0-1-0") may be "many" of them are not working.<br></div><div><br></div><div>Otherwise options "<span style="color:rgb(0,0,0)">drive file=file,if=scsi,bus=0,unit=6" or "</span><span style="color:rgb(0,0,0)">qemu -drive file=file,index=0,if=floppy" are working.</span></div>
<div><br></div><div><pre class="" style="color:rgb(0,0,0)">The documented usage "-drive file=a" and "-drive file=b" Fails though. Even the id attribute parsing doesnt help here :) </pre><pre class="" style="color:rgb(0,0,0)">
<br></pre><pre class="" style="color:rgb(0,0,0)">Thanks and Regards</pre><pre class="" style="color:rgb(0,0,0)">Shiva</pre></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Jan<br></blockquote></div></div></div></div>