dm log userspace base: fix compile warning
authorNicholas Mc Guire <hofrat@osadl.org>
Wed, 18 Mar 2015 22:59:02 +0000 (18:59 -0400)
committerMike Snitzer <snitzer@redhat.com>
Wed, 15 Apr 2015 16:10:20 +0000 (12:10 -0400)
This fixes up a compile warning [-Wunused-but-set-variable] - given the
comment in userspace_set_region_sync() the non-reporting of errors is
intentional so the return value can be dropped to make gcc happy.

Also, fix typo in comment.

Signed-off-by: Nicholas Mc Guire <hofrat@osadl.org>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-log-userspace-base.c

index 39fa007..058256d 100644 (file)
@@ -741,7 +741,6 @@ static int userspace_get_resync_work(struct dm_dirty_log *log, region_t *region)
 static void userspace_set_region_sync(struct dm_dirty_log *log,
                                      region_t region, int in_sync)
 {
-       int r;
        struct log_c *lc = log->context;
        struct {
                region_t r;
@@ -751,12 +750,12 @@ static void userspace_set_region_sync(struct dm_dirty_log *log,
        pkg.r = region;
        pkg.i = (int64_t)in_sync;
 
-       r = userspace_do_request(lc, lc->uuid, DM_ULOG_SET_REGION_SYNC,
-                                (char *)&pkg, sizeof(pkg), NULL, NULL);
+       (void) userspace_do_request(lc, lc->uuid, DM_ULOG_SET_REGION_SYNC,
+                                   (char *)&pkg, sizeof(pkg), NULL, NULL);
 
        /*
         * It would be nice to be able to report failures.
-        * However, it is easy emough to detect and resolve.
+        * However, it is easy enough to detect and resolve.
         */
        return;
 }