[libvirt] [PATCHv2 5/5] Enable the virDomain{S, G}etInterfaceParameters in virsh
Osier Yang
jyang at redhat.com
Wed Dec 28 07:50:10 UTC 2011
On 2011年12月23日 15:09, Hu Tao wrote:
> Add a new command domiftune to get/set interface parameters.
>
> * tools/virsh.c: implement the new command
> * tools/virsh.pod: documentation of the new command
> ---
> tools/virsh.c | 198 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
> tools/virsh.pod | 21 ++++++
> 2 files changed, 219 insertions(+), 0 deletions(-)
>
> diff --git a/tools/virsh.c b/tools/virsh.c
> index 02f2e0d..d88fa95 100644
> --- a/tools/virsh.c
> +++ b/tools/virsh.c
> @@ -427,6 +427,8 @@ static void *_vshCalloc(vshControl *ctl, size_t nmemb, size_t sz, const char *fi
> static char *_vshStrdup(vshControl *ctl, const char *s, const char *filename, int line);
> #define vshStrdup(_ctl, _s) _vshStrdup(_ctl, _s, __FILE__, __LINE__)
>
> +static int parseRateStr(const char *rateStr, virNetDevBandwidthRatePtr rate);
> +
> static void *
> _vshMalloc(vshControl *ctl, size_t size, const char *filename, int line)
> {
> @@ -1601,6 +1603,201 @@ cleanup:
> return ret;
> }
>
> +/* "domiftune" command
> + */
> +static const vshCmdInfo info_domiftune[] = {
> + {"help", N_("get link state of a virtual interface")},
> + {"desc", N_("Get link state of a domain's virtual interface.")},
> + {NULL,NULL}
> +};
Improper help strings, should be from Copy & Paste. :-)
> +
> +static const vshCmdOptDef opts_domiftune[] = {
> + {"domain", VSH_OT_DATA, VSH_OFLAG_REQ, N_("domain name, id or uuid")},
> + {"interface", VSH_OT_DATA, VSH_OFLAG_REQ, N_("interface device")},
> + {"inbound", VSH_OT_DATA, VSH_OFLAG_NONE, N_("control domain's incoming traffics")},
> + {"outbound", VSH_OT_DATA, VSH_OFLAG_NONE, N_("control domain's outgoing traffics")},
> + {"config", VSH_OT_BOOL, VSH_OFLAG_NONE, N_("affect next boot")},
> + {"live", VSH_OT_BOOL, VSH_OFLAG_NONE, N_("affect running domain")},
> + {"current", VSH_OT_BOOL, VSH_OFLAG_NONE, N_("affect current domain")},
> + {NULL, 0, 0, NULL}
> +};
> +
> +static bool
> +cmdDomIftune(vshControl *ctl, const vshCmd *cmd)
> +{
> + virDomainPtr dom;
> + const char *name = NULL, *device = NULL,
> + *inboundStr = NULL, *outboundStr = NULL;
> + unsigned int flags = 0;
> + int nparams = 0;
> + virTypedParameterPtr params = NULL;
> + bool ret = false;
> + int current = vshCommandOptBool(cmd, "current");
> + int config = vshCommandOptBool(cmd, "config");
> + int live = vshCommandOptBool(cmd, "live");
> + virNetDevBandwidthRate inbound, outbound;
> + int i;
> +
> + if (current) {
> + if (live || config) {
> + vshError(ctl, "%s", _("--current must be specified exclusively"));
> + return false;
> + }
> + flags = VIR_DOMAIN_AFFECT_CURRENT;
> + } else {
> + if (config)
> + flags |= VIR_DOMAIN_AFFECT_CONFIG;
> + if (live)
> + flags |= VIR_DOMAIN_AFFECT_LIVE;
> + }
> +
> + if (!vshConnectionUsability(ctl, ctl->conn))
> + return false;
> +
> + if (!(dom = vshCommandOptDomain(ctl, cmd,&name)))
> + return false;
> +
> + if (vshCommandOptString(cmd, "interface",&device)<= 0) {
> + virDomainFree(dom);
> + return false;
> + }
> +
> + if (vshCommandOptString(cmd, "inbound",&inboundStr)< 0 ||
> + vshCommandOptString(cmd, "outbound",&outboundStr)< 0) {
> + vshError(ctl, "missing argument");
> + goto cleanup;
> + }
> +
> + memset(&inbound, 0, sizeof(inbound));
> + memset(&outbound, 0, sizeof(outbound));
> +
> + if (inboundStr) {
> + if (parseRateStr(inboundStr,&inbound)< 0) {
> + vshError(ctl, _("inbound format is incorrect"));
> + goto cleanup;
> + }
> + if (inbound.average == 0) {
> + vshError(ctl, _("inbound average is mandatory"));
> + goto cleanup;
> + }
> + nparams++; /* average */
> + if (inbound.peak) nparams++;
> + if (inbound.burst) nparams++;
> + }
> + if (outboundStr) {
> + if (parseRateStr(outboundStr,&outbound)< 0) {
> + vshError(ctl, _("outbound format is incorrect"));
> + goto cleanup;
> + }
> + if (outbound.average == 0) {
> + vshError(ctl, _("outbound average is mandatory"));
> + goto cleanup;
> + }
> + nparams++; /* average */
> + if (outbound.peak) nparams++;
> + if (outbound.burst) nparams++;
> + }
> +
> + if (nparams == 0) {
> + /* get the number of interface parameters */
> + if (virDomainGetInterfaceParameters(dom, device, NULL,&nparams, flags) != 0) {
> + vshError(ctl, "%s",
> + _("Unable to get number of interface parameters"));
> + goto cleanup;
> + }
> +
> + if (nparams == 0) {
> + /* nothing to output */
> + ret = true;
> + goto cleanup;
> + }
> +
> + /* get all interface parameters */
> + params = vshCalloc(ctl, nparams, sizeof(*params));
> + if (!params) {
> + virReportOOMError();
> + goto cleanup;
> + }
> + if (virDomainGetInterfaceParameters(dom, device, params,&nparams, flags) != 0) {
> + vshError(ctl, "%s", _("Unable to get interface parameters"));
> + goto cleanup;
> + }
> +
> + for (i = 0; i< nparams; i++) {
> + char *str = vshGetTypedParamValue(ctl,¶ms[i]);
> + vshPrint(ctl, "%-15s: %s\n", params[i].field, str);
> + VIR_FREE(str);
> + }
> + } else {
> + /* set the interface parameters */
> + params = vshCalloc(ctl, nparams, sizeof(*params));
> + if (!params) {
> + virReportOOMError();
> + goto cleanup;
> + }
> +
> + for (i = 0; i< nparams; i++)
> + params[i].type = VIR_TYPED_PARAM_UINT;
> +
> + i = 0;
> + if (inbound.average&& i< nparams) {
> + if (!virStrcpy(params[i].field, VIR_DOMAIN_BANDWIDTH_IN_AVERAGE,
> + sizeof(params[i].field)))
> + goto cleanup;
> + params[i].value.ui = inbound.average;
> + i++;
> + }
> + if (inbound.peak&& i< nparams) {
> + if (!virStrcpy(params[i].field, VIR_DOMAIN_BANDWIDTH_IN_PEAK,
> + sizeof(params[i].field)))
> + goto cleanup;
> + params[i].value.ui = inbound.peak;
> + i++;
> + }
> + if (inbound.burst&& i< nparams) {
> + if (!virStrcpy(params[i].field, VIR_DOMAIN_BANDWIDTH_IN_BURST,
> + sizeof(params[i].field)))
> + goto cleanup;
> + params[i].value.ui = inbound.burst;
> + i++;
> + }
> + if (outbound.average&& i< nparams) {
> + if (!virStrcpy(params[i].field, VIR_DOMAIN_BANDWIDTH_OUT_AVERAGE,
> + sizeof(params[i].field)))
> + goto cleanup;
> + params[i].value.ui = outbound.average;
> + i++;
> + }
> + if (outbound.peak&& i< nparams) {
> + if (!virStrcpy(params[i].field, VIR_DOMAIN_BANDWIDTH_OUT_PEAK,
> + sizeof(params[i].field)))
> + goto cleanup;
> + params[i].value.ui = outbound.peak;
> + i++;
> + }
> + if (outbound.burst&& i< nparams) {
> + if (!virStrcpy(params[i].field, VIR_DOMAIN_BANDWIDTH_OUT_BURST,
> + sizeof(params[i].field)))
> + goto cleanup;
> + params[i].value.ui = outbound.burst;
> + i++;
> + }
> +
> + if (virDomainSetInterfaceParameters(dom, device, params, nparams, flags) != 0) {
> + vshError(ctl, "%s", _("Unable to set interface parameters"));
> + goto cleanup;
> + }
> + }
> +
> + ret = true;
> +
> +cleanup:
> + virTypedParameterArrayClear(params, nparams);
> + VIR_FREE(params);
> + virDomainFree(dom);
> + return ret;
> +}
> +
> /*
> * "dommemstats" command
> */
> @@ -15589,6 +15786,7 @@ static const vshCmdDef domManagementCmds[] = {
> info_detach_interface, 0},
> {"domid", cmdDomid, opts_domid, info_domid, 0},
> {"domif-setlink", cmdDomIfSetLink, opts_domif_setlink, info_domif_setlink, 0},
> + {"domiftune", cmdDomIftune, opts_domiftune, info_domiftune, 0},
> {"domjobabort", cmdDomjobabort, opts_domjobabort, info_domjobabort, 0},
> {"domjobinfo", cmdDomjobinfo, opts_domjobinfo, info_domjobinfo, 0},
> {"domname", cmdDomname, opts_domname, info_domname, 0},
> diff --git a/tools/virsh.pod b/tools/virsh.pod
> index 6c10245..d1c5cbe 100644
> --- a/tools/virsh.pod
> +++ b/tools/virsh.pod
> @@ -554,6 +554,27 @@ configuration of the domain is modified.
> Query link state of the domain's virtual interface. If --persistent
> is specified, query the persistent configuration.
>
> +=item B<domiftune> I<domain> I<interface-device>
> +[[I<--config>] [I<--live>] | [I<--current>]]
> +[I<--inbound average,peak,burst>]
> +[I<--outbound average,peak,burst>]
> +
> +Set or query the domain's network interface's bandwidth parameters.
> +I<interface-device> can be the interface name (<target dev='name'/>),
It might be better to document as:
interface => interface's target name
> +or the mac address if the name is not specified in xml.
s/mac/MAC/
"if the name is not specified in xml" should be removed, as
it can cause one think MAC address could be only used when
the target name is not specified.
> +
> +If no I<--inbound> or I<--outbound> is specified, this command will
> +query and show the bandwidth settings. Otherwise, it will set the
> +inbound or outbound bandwidth. I<average,peak,burst> is the same as
> +in command I<attach-interface>.
> +
> +If I<--live> is specified, affect a running guest.
> +If I<--config> is specified, affect the next boot of a persistent guest.
> +If I<--current> is specified, affect the current guest state.
> +Both I<--live> and I<--current> flags may be given, but I<--current> is
> +exclusive. If no flag is specified, behavior is different depending
> +on hypervisor.
> +
> =item B<dommemstat> I<domain>
>
> Get memory stats for a running domain.
ACK with those nits fixed.
Regards,
Osier
More information about the libvir-list
mailing list