On 03/04/2015 12:48 PM, Ján Tomko wrote: > 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? > Hmm.. Cannot remember my reference on this... Searched on "No " in virsh-domain.c and found "No current block job".. I could change to using vshPrintExtra for the same text if that seems better. >> + >> + 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. > Over aggressive with copy-n-paste... tks, John >> + 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 > -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list