Il 02/05/2012 00:18, Anthony Liguori ha scritto:
> anthony@titi:~/build/qemu$ x86_64-softmmu/qemu-system-x86_64 -device
> rtl8139,?
> <there is no output>
I don't think this is a fair comparison, or makes sense at all. The cause
of the bug in master is a cut-and-paste typo:
@@ -157,7 +157,7 @@ int qdev_device_help(QemuOpts *opts)
* for removal. This conditional should be removed along with
* it.
*/
- if (!prop->info->parse) {
+ if (!prop->info->set) {
continue; /* no way to set it, don't show */
}
error_printf("%s.%s=%s\n", driver, prop->name,
@@ -165,7 +165,7 @@ int qdev_device_help(QemuOpts *opts)
}
if (info->bus_info) {
for (prop = info->bus_info->props; prop&& prop->name; prop++) {
- if (!prop->info->parse) {
+ if (!prop->info->set) {
continue; /* no way to set it, don't show */
}
error_printf("%s.%s=%s\n", driver, prop->name,
while here the problem is due to a half-baked change in this series.
Since I redid the same change in my properties series, and I did it
correctly, the only sensible solution is to rebase these patches on
that one.
> So my series makes the situation better and I think it's easier to fix
> the full problem. I also don't view the current bug as a -rc0 blocker
> (although it's obviously a release blocker). I can send a proper patch
> later in the week but I'd still like to commit the bus changes before -rc0.
Please don't. I have barely started reviewing this series and I have
quite a few questions, though some may be trivial. In the meanwhile,
I'll separate the early pieces of my series and rebase this one on top.
Paolo
