<div dir="ltr"><div class="gmail_quote">On Mon, Apr 25, 2011 at 6:28 PM, Cole Robinson <span dir="ltr"><<a href="mailto:crobinso@redhat.com">crobinso@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On 04/25/2011 04:27 AM, Richard W.M. Jones wrote:<br>
> [Copying this to libvir-list]<br>
><br>
> On Mon, Apr 25, 2011 at 01:07:37AM +0400, Emre Erenoglu wrote:<br>
>> Hi,<br>
>><br>
>> I'm the package maintainer for virt-manager and related packages for Pardus<br>
>> distribution. While testing the latest libvirt, virtinst & virt-manager<br>
>> packages, I've come across a strange issue and I would like to get your<br>
>> valuable opinion.<br>
>><br>
>> I add all spice related devices and everything works good, except the<br>
>> vdagent inside the windows xp guest. The virtio serial driver is loaded<br>
>> correctly. As I track down the issue, I found out that libvirt is starting<br>
>> qemu-kvm with parameters which do not match the ones adviced by the spice<br>
>> people. Please see below email discussion with them on this. The offending<br>
>> line seems to be the chardev parameter.  qemu-kvm is started by virt-manager<br>
>> with the following parameter for chardev:<br>
>><br>
>> -chardev null,id=channel0<br>
>><br>
>> and the full spice related parameters are:<br>
>><br>
>> -device virtio-serial-pci,id=virtio-serial0,bus=pci.0,addr=0x8 -chardev<br>
>> null,id=channel0 -device<br>
>> virtserialport,bus=virtio-serial0.0,nr=0,chardev=channel0,name=com.redhat.spice.0<br>
>> -usb -device usb-tablet,id=input0 -spice<br>
>> port=5900,addr=127.0.0.1,disable-ticketing -vga qxl -device<br>
>> qxl,id=video1,bus=pci.0,addr=0x7<br>
>><br>
>> while spice people adviced:<br>
>><br>
>> -chardev spicevmc,id=channel0,name=vdagent<br>
>><br>
>> and the rest of the parameters to match it. See below mail on the details.<br>
>> I don't know if this is really the issue, but I also recognize the following<br>
>> inside the domain XML:<br>
>><br>
>>     <channel type='null'><br>
>>       <target type='virtio' name='com.redhat.spice.0'/><br>
>>       <address type='virtio-serial' controller='0' bus='0' port='0'/><br>
>>     </channel><br>
>><br>
>> the "channel type" is listed as "null", while I assume it should have been<br>
>> listed as "spicevmc". (not sure of this, I saw this in some other<br>
>> websites).  When I edit the domain xml with virsh edit, it saves my changes<br>
>> but the "null" stays the same how many times I try to change it.<br>
>><br>
>> Please note that I've applied the following patches to virtinst 0.500.6:<br>
>><br>
>> constrain-spicevmc-usage-correct.patch<br>
>> virtinst-fix-channel-parse.patch<br>
>> virtinst-spicevmc-fixes.patch<br>
>><br>
>> which I obtained from the git. I also patched virt-manager 0.8.7 with the<br>
>> following I obtained from the git:<br>
>><br>
>> chardev-hide-unsupported-params-for-selected-type.patch<br>
>> only-show-relevant-char-device-fields.patch<br>
>> show-char-device-target-name.patch<br>
>> chardev-propose-to-add-remove-spice-agent.patch<br>
>> allow-setting-char-device-target-name.patch<br>
>> fix-adding-removing-channel-device.patch<br>
>><br>
>> Any idea what I might be missing to get the vdagent run inside the windows<br>
>> guest?<br>
>><br>
>> Many thanks,<br>
>><br>
>> Emre Erenoglu<br>
<br>
What libvirt version are you using? spicevmc requires libvirt 0.8.8. However<br>
if libvirt is silently reverting to 'null' it's a bug either way.<br>
<font color="#888888"><br></font></blockquote><div><br>Hi Cole,<br><br>I recognized that I was using 0.8.7 since our iptables version does not support the newly added "CHECKSUM" parameter that libvirt used for iptables commands. I disabled this by patching it and now libvirt 0.9.0 runs good, with the vdagent in the guest also working OK.<br>
<br>Many thanks for all who spent their time to fix this issue.<br><br>yes, libvirt 0.8.7 was silently reverting the change I made manually in the xml file. Neither virsh edit nor editing the xml and re-defining it worked. Once libvirt saw the "spicevmc" there, it just removed it and put "null" instead.<br>
</div></div><br>-- <br>Emre<br>
</div>