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 /drivers/md/dm-log-userspace-base.c | |
| 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>
Diffstat (limited to 'drivers/md/dm-log-userspace-base.c')
| -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 | } |
