On Tue, Feb 17, 2015 at 04:03:53PM -0500, John Ferlan wrote: > Add the 'iothreads' command to display IOThread Info data. Allow for > [--live] or [--config] options in order to display live or config data > for an active domain. > > An active domain may return: > > $ virsh iothreads $dom > IOThread ID CPU Affinity Resource(s) > ----------------------------------------------------------------- > 1 2 /home/vm-images/f18 > 2 3 /home/vm-images/iothr-vol1 > 3 0 > > $ echo $? > 0 > > For domains which don't have IOThreads the following is returned: > > $ virsh iothreads $dom > error: No IOThreads found for the domain > > $ echo $? > 0
Printing an error, but not returning one seems strange. Do we do this
somewhere else too?
> +
> + vshPrintExtra(ctl, " %-15s %-15s %-15s\n",
> + _("IOThread ID"), _("CPU Affinity"), _("Resource(s)"));
> + vshPrintExtra(ctl,
> "-----------------------------------------------------------------\n");
> + for (i = 0; i < niothreads; i++) {
> +
> + vshPrintExtra(ctl, " %-15u ", info[i]->iothread_id);
vshPrint should be used for the actual data, to make the command work
with --quiet.
> + ignore_value(vshPrintPinInfo(info[i]->cpumap, info[i]->cpumaplen,
> + maxcpu, 0));
> + for (j = 0; j < info[i]->nresources; j++) {
> + if (j == 0)
> + vshPrintExtra(ctl, "\t\t %s", info[i]->resources[j]);
> + else
> + vshPrintExtra(ctl, "\n %-15s %s\t\t %s",
> + " ", " ", info[i]->resources[j]);
> + }
Jan
signature.asc
Description: Digital signature
-- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
