From: Luiz Capitulino <lcapitulino@redhat.com>
Date: Fri, 31 Jul 2009 18:15:41 +0000 (-0300)
Subject: Fix do_commit() behavior
X-Git-Tag: TizenStudio_2.0_p2.3.2~208^2~11040
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=73006d2a9a6f226486674c0ee92b5ea5bb1a2cf6;p=sdk%2Femulator%2Fqemu.git

Fix do_commit() behavior

Commit 751c6a17042b5d011013d6963c0505d671cf708e changed the monitor's
'commit' command to this behavior:

1. Any string you type as argument will cause do_commit() to
call bdrv_commit() to all devices

2. If you enter a device name, it will be the only one ignored
by do_commit() :)

The fix is to call bdrv_commit() to the specified device only and
ignore the others (when 'all' is not specified).

Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Message-Id:
---

diff --git a/monitor.c b/monitor.c
index aac8878..4b270d9 100644
--- a/monitor.c
+++ b/monitor.c
@@ -259,7 +259,7 @@ static void do_commit(Monitor *mon, const char *device)
     all_devices = !strcmp(device, "all");
     TAILQ_FOREACH(dinfo, &drives, next) {
         if (!all_devices)
-            if (!strcmp(bdrv_get_device_name(dinfo->bdrv), device))
+            if (strcmp(bdrv_get_device_name(dinfo->bdrv), device))
                 continue;
         bdrv_commit(dinfo->bdrv);
     }