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
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list