diff options
author | Nicholas Mc Guire <hofrat@osadl.org> | 2015-03-18 18:59:02 -0400 |
---|---|---|
committer | Mike Snitzer <snitzer@redhat.com> | 2015-04-15 12:10:20 -0400 |
commit | 18cc980ac8cf8c727d1f7d581b4576ed64bd78a6 (patch) | |
tree | 82e8b0bf26e893fca24d249aeebffbc2521f8425 | |
parent | c32a512fdf62de260ee8298436558ea50b94dfcb (diff) |
dm log userspace base: fix compile warning
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>
-rw-r--r-- | drivers/md/dm-log-userspace-base.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/md/dm-log-userspace-base.c b/drivers/md/dm-log-userspace-base.c index 39fa00733431..058256d2eeea 100644 --- a/drivers/md/dm-log-userspace-base.c +++ b/drivers/md/dm-log-userspace-base.c | |||
@@ -741,7 +741,6 @@ static int userspace_get_resync_work(struct dm_dirty_log *log, region_t *region) | |||
741 | static void userspace_set_region_sync(struct dm_dirty_log *log, | 741 | static void userspace_set_region_sync(struct dm_dirty_log *log, |
742 | region_t region, int in_sync) | 742 | region_t region, int in_sync) |
743 | { | 743 | { |
744 | int r; | ||
745 | struct log_c *lc = log->context; | 744 | struct log_c *lc = log->context; |
746 | struct { | 745 | struct { |
747 | region_t r; | 746 | region_t r; |
@@ -751,12 +750,12 @@ static void userspace_set_region_sync(struct dm_dirty_log *log, | |||
751 | pkg.r = region; | 750 | pkg.r = region; |
752 | pkg.i = (int64_t)in_sync; | 751 | pkg.i = (int64_t)in_sync; |
753 | 752 | ||
754 | r = userspace_do_request(lc, lc->uuid, DM_ULOG_SET_REGION_SYNC, | 753 | (void) userspace_do_request(lc, lc->uuid, DM_ULOG_SET_REGION_SYNC, |
755 | (char *)&pkg, sizeof(pkg), NULL, NULL); | 754 | (char *)&pkg, sizeof(pkg), NULL, NULL); |
756 | 755 | ||
757 | /* | 756 | /* |
758 | * It would be nice to be able to report failures. | 757 | * It would be nice to be able to report failures. |
759 | * However, it is easy emough to detect and resolve. | 758 | * However, it is easy enough to detect and resolve. |
760 | */ | 759 | */ |
761 | return; | 760 | return; |
762 | } | 761 | } |