[libvirt] [RFC PATCHv2 5/5] WIP: smartcard: turn on qemu support
Eric Blake
eblake at redhat.com
Fri Jan 14 18:40:34 UTC 2011
On 01/14/2011 06:46 AM, Daniel P. Berrange wrote:
> On Fri, Jan 14, 2011 at 12:41:13PM +0000, Daniel P. Berrange wrote:
>> On Thu, Jan 13, 2011 at 05:34:37PM -0700, Eric Blake wrote:
>>
>>> + virCommandAddArgBuffer(cmd, &smartcard_buf);
>>> + break;
>>> + case VIR_DOMAIN_SMARTCARD_TYPE_PASSTHROUGH:
>>> + virCommandAddArg(cmd, "-chardev");
>>> + if (!(devstr = qemuBuildChrChardevStr(&smartcard->data.passthru,
>>> + smartcard->info.alias)))
>>> + goto error;
>>
>> The 'id' setting for the chardev should really have a prefix
>> on it, because we won't want it to be the same as the 'id'
>> setting used for the -device option.
>
> This is in fact a pre-existing flaw in our chardev handling
> which we need to fix. The 'qemuBuildChrChardevStr' API & its
> callers are buggy. They takes the assigned alias from the XML
> and use it directly for the chardev. So we end up with
>
> -chardev pty,id=$alias
> -device isa-serial,chardev=$alias
Another big flaw in our chardev handling is that we haven't yet got any
support in qemuDomainXMLToNative for parsing -chardev, which means it
all shows up as qemu:command elements instead. The last time we were
able to convert from qemu command line back to XML was for qemu 0.11.x.
--
Eric Blake eblake at redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110114/49fbb642/attachment-0001.sig>
More information about the libvir-list
mailing list