* src/virsh.c: applied patch from Atsushi SAKAI to clarify

that memory values are in kilobytes
Daniel
This commit is contained in:
Daniel Veillard 2007-04-02 11:56:38 +00:00
parent 8ba930c380
commit b5d6258530
2 changed files with 19 additions and 14 deletions

View File

@ -1,3 +1,8 @@
Mon Apr 2 13:54:17 CEST 2007 Daniel Veillard <veillard@redhat.com>
* src/virsh.c: applied patch from Atsushi SAKAI to clarify
that memory values are in kilobytes
Wed Mar 30 17:25:33 IST 2007 Mark McLoughlin <markmc@redhat.com> Wed Mar 30 17:25:33 IST 2007 Mark McLoughlin <markmc@redhat.com>
* qemud/iptables.c: As suggested by danpb, make libvirt_qemud * qemud/iptables.c: As suggested by danpb, make libvirt_qemud

View File

@ -1437,7 +1437,7 @@ cmdSetvcpus(vshControl * ctl, vshCmd * cmd)
* "setmemory" command * "setmemory" command
*/ */
static vshCmdInfo info_setmem[] = { static vshCmdInfo info_setmem[] = {
{"syntax", "setmem <domain> <bytes>"}, {"syntax", "setmem <domain> <kilobytes>"},
{"help", gettext_noop("change memory allocation")}, {"help", gettext_noop("change memory allocation")},
{"desc", gettext_noop("Change the current memory allocation in the guest domain.")}, {"desc", gettext_noop("Change the current memory allocation in the guest domain.")},
{NULL, NULL} {NULL, NULL}
@ -1445,7 +1445,7 @@ static vshCmdInfo info_setmem[] = {
static vshCmdOptDef opts_setmem[] = { static vshCmdOptDef opts_setmem[] = {
{"domain", VSH_OT_DATA, VSH_OFLAG_REQ, gettext_noop("domain name, id or uuid")}, {"domain", VSH_OT_DATA, VSH_OFLAG_REQ, gettext_noop("domain name, id or uuid")},
{"bytes", VSH_OT_DATA, VSH_OFLAG_REQ, gettext_noop("number of bytes of memory")}, {"kilobytes", VSH_OT_DATA, VSH_OFLAG_REQ, gettext_noop("number of kilobytes of memory")},
{NULL, 0, 0, NULL} {NULL, 0, 0, NULL}
}; };
@ -1453,7 +1453,7 @@ static int
cmdSetmem(vshControl * ctl, vshCmd * cmd) cmdSetmem(vshControl * ctl, vshCmd * cmd)
{ {
virDomainPtr dom; virDomainPtr dom;
int bytes; int kilobytes;
int ret = TRUE; int ret = TRUE;
if (!vshConnectionUsability(ctl, ctl->conn, TRUE)) if (!vshConnectionUsability(ctl, ctl->conn, TRUE))
@ -1462,14 +1462,14 @@ cmdSetmem(vshControl * ctl, vshCmd * cmd)
if (!(dom = vshCommandOptDomain(ctl, cmd, "domain", NULL))) if (!(dom = vshCommandOptDomain(ctl, cmd, "domain", NULL)))
return FALSE; return FALSE;
bytes = vshCommandOptInt(cmd, "bytes", &bytes); kilobytes = vshCommandOptInt(cmd, "kilobytes", &kilobytes);
if (bytes <= 0) { if (kilobytes <= 0) {
virDomainFree(dom); virDomainFree(dom);
vshError(ctl, FALSE, _("Invalid value of %d for memory size"), bytes); vshError(ctl, FALSE, _("Invalid value of %d for memory size"), kilobytes);
return FALSE; return FALSE;
} }
if (virDomainSetMemory(dom, bytes) != 0) { if (virDomainSetMemory(dom, kilobytes) != 0) {
ret = FALSE; ret = FALSE;
} }
@ -1481,7 +1481,7 @@ cmdSetmem(vshControl * ctl, vshCmd * cmd)
* "setmaxmem" command * "setmaxmem" command
*/ */
static vshCmdInfo info_setmaxmem[] = { static vshCmdInfo info_setmaxmem[] = {
{"syntax", "setmaxmem <domain> <bytes>"}, {"syntax", "setmaxmem <domain> <kilobytes>"},
{"help", gettext_noop("change maximum memory limit")}, {"help", gettext_noop("change maximum memory limit")},
{"desc", gettext_noop("Change the maximum memory allocation limit in the guest domain.")}, {"desc", gettext_noop("Change the maximum memory allocation limit in the guest domain.")},
{NULL, NULL} {NULL, NULL}
@ -1489,7 +1489,7 @@ static vshCmdInfo info_setmaxmem[] = {
static vshCmdOptDef opts_setmaxmem[] = { static vshCmdOptDef opts_setmaxmem[] = {
{"domain", VSH_OT_DATA, VSH_OFLAG_REQ, gettext_noop("domain name, id or uuid")}, {"domain", VSH_OT_DATA, VSH_OFLAG_REQ, gettext_noop("domain name, id or uuid")},
{"bytes", VSH_OT_DATA, VSH_OFLAG_REQ, gettext_noop("maxmimum memory limit in bytes")}, {"kilobytes", VSH_OT_DATA, VSH_OFLAG_REQ, gettext_noop("maxmimum memory limit in kilobytes")},
{NULL, 0, 0, NULL} {NULL, 0, 0, NULL}
}; };
@ -1497,7 +1497,7 @@ static int
cmdSetmaxmem(vshControl * ctl, vshCmd * cmd) cmdSetmaxmem(vshControl * ctl, vshCmd * cmd)
{ {
virDomainPtr dom; virDomainPtr dom;
int bytes; int kilobytes;
int ret = TRUE; int ret = TRUE;
if (!vshConnectionUsability(ctl, ctl->conn, TRUE)) if (!vshConnectionUsability(ctl, ctl->conn, TRUE))
@ -1506,14 +1506,14 @@ cmdSetmaxmem(vshControl * ctl, vshCmd * cmd)
if (!(dom = vshCommandOptDomain(ctl, cmd, "domain", NULL))) if (!(dom = vshCommandOptDomain(ctl, cmd, "domain", NULL)))
return FALSE; return FALSE;
bytes = vshCommandOptInt(cmd, "bytes", &bytes); kilobytes = vshCommandOptInt(cmd, "kilobytes", &kilobytes);
if (bytes <= 0) { if (kilobytes <= 0) {
virDomainFree(dom); virDomainFree(dom);
vshError(ctl, FALSE, _("Invalid value of %d for memory size"), bytes); vshError(ctl, FALSE, _("Invalid value of %d for memory size"), kilobytes);
return FALSE; return FALSE;
} }
if (virDomainSetMaxMemory(dom, bytes) != 0) { if (virDomainSetMaxMemory(dom, kilobytes) != 0) {
ret = FALSE; ret = FALSE;
} }