[Qemu-devel] qemu fail to parse command line with "-pcidevice 00:19.0"

[Qemu-devel] qemu fail to parse command line with "-pcidevice 00:19.0"

[Hao, Xudong]

When assign one PCI device, qemu fail to parse the command line:
qemu-system_x86 -smp 2 -m 1024 -hda /path/to/img -pcidevice host=00:19.0
Error:
qemu-system-x86_64: Parameter 'id' expects an identifier
Identifiers consist of letters, digits, '-', '.', '_', starting with a letter.
pcidevice argument parse error; please check the help text for usage
Could not add assigned device host=00:19.0

https://bugs.launchpad.net/qemu/+bug/597932

This issue caused by qemu-kvm commit b560a9ab9be06afcbb78b3791ab836dad208a239.

Best Regards,
Xudong Hao

[Qemu-devel] Re: qemu fail to parse command line with "-pcidevice 00:19.0"

[Markus Armbruster]

Note to qemu-devel: this issue is qemu-kvm only.

"Hao, Xudong" <xudong.hao@intel.com> writes:

> When assign one PCI device, qemu fail to parse the command line:
> qemu-system_x86 -smp 2 -m 1024 -hda /path/to/img -pcidevice host=00:19.0
> Error:
> qemu-system-x86_64: Parameter 'id' expects an identifier
> Identifiers consist of letters, digits, '-', '.', '_', starting with a letter.
> pcidevice argument parse error; please check the help text for usage
> Could not add assigned device host=00:19.0
>
> https://bugs.launchpad.net/qemu/+bug/597932
>
> This issue caused by qemu-kvm commit b560a9ab9be06afcbb78b3791ab836dad208a239.

The bug is in add_assigned_device():

    r = get_param_value(id, sizeof(id), "id", arg);
    if (!r)
        r = get_param_value(id, sizeof(id), "name", arg);
    if (!r)
        r = get_param_value(id, sizeof(id), "host", arg);

We end up with invalid ID "00:19.0".

Is there a technical reason why we must have an ID?  I doubt it.

Work-around: -device pci-assign,host=00:19.1

See the last section of docs/qdev-device-use.txt for details.

[Qemu-devel] RE: qemu fail to parse command line with "-pcidevice 00:19.0"

[Hao, Xudong]

>Work-around: -device pci-assign,host=00:19.1
OK, this new way can work when create guest with static assignment.
But how to hot add a pci device to guest? the old hot add command "pci_add pci_addr=auto host host=00:19.0" has the same parse error.

BTW: if we use add "-net none" in qemu command, guest can not be created and nothing error printed.

Do you have plan to fix this parse issue?

Thanks,
Xudong
-----Original Message-----
From: Markus Armbruster [mailto:armbru@redhat.com] 
Sent: 2010年6月24日 14:08
To: Hao, Xudong
Cc: qemu-devel@nongnu.org; aliguori@us.ibm.com; kvm@vger.kernel.org
Subject: Re: qemu fail to parse command line with "-pcidevice 00:19.0"

Note to qemu-devel: this issue is qemu-kvm only.

"Hao, Xudong" <xudong.hao@intel.com> writes:

> When assign one PCI device, qemu fail to parse the command line:
> qemu-system_x86 -smp 2 -m 1024 -hda /path/to/img -pcidevice host=00:19.0
> Error:
> qemu-system-x86_64: Parameter 'id' expects an identifier
> Identifiers consist of letters, digits, '-', '.', '_', starting with a letter.
> pcidevice argument parse error; please check the help text for usage
> Could not add assigned device host=00:19.0
>
> https://bugs.launchpad.net/qemu/+bug/597932
>
> This issue caused by qemu-kvm commit b560a9ab9be06afcbb78b3791ab836dad208a239.

The bug is in add_assigned_device():

    r = get_param_value(id, sizeof(id), "id", arg);
    if (!r)
        r = get_param_value(id, sizeof(id), "name", arg);
    if (!r)
        r = get_param_value(id, sizeof(id), "host", arg);

We end up with invalid ID "00:19.0".

Is there a technical reason why we must have an ID?  I doubt it.

Work-around: -device pci-assign,host=00:19.1

See the last section of docs/qdev-device-use.txt for details.

[Qemu-devel] Re: qemu fail to parse command line with "-pcidevice 00:19.0"

[Markus Armbruster]

"Hao, Xudong" <xudong.hao@intel.com> writes:

>>Work-around: -device pci-assign,host=00:19.1
> OK, this new way can work when create guest with static assignment.
> But how to hot add a pci device to guest? the old hot add command "pci_add pci_addr=auto host host=00:19.0" has the same parse error.

Command line's -device becomes monitor's device_add:

device_add pci-assign,host=00:19.1

> BTW: if we use add "-net none" in qemu command, guest can not be created and nothing error printed.
>
> Do you have plan to fix this parse issue?

Separate issue.  Fix posted:

Subject: [Qemu-devel] [PATCH] net: Fix VM start with '-net none'
Date: Tue, 15 Jun 2010 13:30:39 +0530
Message-Id: <22a96312232a0458fc04268b79d17828c824df42.1276588830.git.amit.shah@redhat.com>

You could have found this yourself :)

[Qemu-devel] RE: qemu fail to parse command line with "-pcidevice 00:19.0"

[Hao, Xudong]

Thanks, Mark. 

-----Original Message-----
From: Markus Armbruster [mailto:armbru@redhat.com] 
Sent: 2010年6月25日 12:58
To: Hao, Xudong
Cc: qemu-devel@nongnu.org; aliguori@us.ibm.com; kvm@vger.kernel.org
Subject: Re: qemu fail to parse command line with "-pcidevice 00:19.0"

"Hao, Xudong" <xudong.hao@intel.com> writes:

>>Work-around: -device pci-assign,host=00:19.1
> OK, this new way can work when create guest with static assignment.
> But how to hot add a pci device to guest? the old hot add command "pci_add pci_addr=auto host host=00:19.0" has the same parse error.

Command line's -device becomes monitor's device_add:

device_add pci-assign,host=00:19.1

> BTW: if we use add "-net none" in qemu command, guest can not be created and nothing error printed.
>
> Do you have plan to fix this parse issue?

Separate issue.  Fix posted:

Subject: [Qemu-devel] [PATCH] net: Fix VM start with '-net none'
Date: Tue, 15 Jun 2010 13:30:39 +0530
Message-Id: <22a96312232a0458fc04268b79d17828c824df42.1276588830.git.amit.shah@redhat.com>

You could have found this yourself :)

[Qemu-devel] Re: qemu fail to parse command line with "-pcidevice 00:19.0"

[Hidetoshi Seto]

(2010/06/24 15:08), Markus Armbruster wrote:
> Note to qemu-devel: this issue is qemu-kvm only.
> 
> "Hao, Xudong" <xudong.hao@intel.com> writes:
> 
>> When assign one PCI device, qemu fail to parse the command line:
>> qemu-system_x86 -smp 2 -m 1024 -hda /path/to/img -pcidevice host=00:19.0
>> Error:
>> qemu-system-x86_64: Parameter 'id' expects an identifier
>> Identifiers consist of letters, digits, '-', '.', '_', starting with a letter.
>> pcidevice argument parse error; please check the help text for usage
>> Could not add assigned device host=00:19.0
>>
>> https://bugs.launchpad.net/qemu/+bug/597932
>>
>> This issue caused by qemu-kvm commit b560a9ab9be06afcbb78b3791ab836dad208a239.
> 
> The bug is in add_assigned_device():
> 
>     r = get_param_value(id, sizeof(id), "id", arg);
>     if (!r)
>         r = get_param_value(id, sizeof(id), "name", arg);
>     if (!r)
>         r = get_param_value(id, sizeof(id), "host", arg);
> 
> We end up with invalid ID "00:19.0".

... Are there any strong reason why we cannot use ':' in the identifier?


Thanks,
H.Seto

[Qemu-devel] Re: qemu fail to parse command line with "-pcidevice 00:19.0"

[Markus Armbruster]

Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com> writes:

> (2010/06/24 15:08), Markus Armbruster wrote:
>> Note to qemu-devel: this issue is qemu-kvm only.
>> 
>> "Hao, Xudong" <xudong.hao@intel.com> writes:
>> 
>>> When assign one PCI device, qemu fail to parse the command line:
>>> qemu-system_x86 -smp 2 -m 1024 -hda /path/to/img -pcidevice host=00:19.0
>>> Error:
>>> qemu-system-x86_64: Parameter 'id' expects an identifier
>>> Identifiers consist of letters, digits, '-', '.', '_', starting with a letter.
>>> pcidevice argument parse error; please check the help text for usage
>>> Could not add assigned device host=00:19.0
>>>
>>> https://bugs.launchpad.net/qemu/+bug/597932
>>>
>>> This issue caused by qemu-kvm commit b560a9ab9be06afcbb78b3791ab836dad208a239.
>> 
>> The bug is in add_assigned_device():
>> 
>>     r = get_param_value(id, sizeof(id), "id", arg);
>>     if (!r)
>>         r = get_param_value(id, sizeof(id), "name", arg);
>>     if (!r)
>>         r = get_param_value(id, sizeof(id), "host", arg);
>> 
>> We end up with invalid ID "00:19.0".
>
> ... Are there any strong reason why we cannot use ':' in the identifier?

Paul Brook (cc'ed) objected.