[dm-devel] [Patch addition] dm-log-cluster addendum2
Jonathan Brassow
jbrassow at redhat.com
Thu Feb 12 16:37:30 UTC 2009
There is no need to case pointers that are 'void *' to begin
with.
This patch should be rolled into the dm-log-cluster.patch.
Signed-off-by: Jonathan Brassow <jbrassow at redhat.com>
Index: linux-2.6/drivers/md/dm-log-cluster.c
===================================================================
--- linux-2.6.orig/drivers/md/dm-log-cluster.c
+++ linux-2.6/drivers/md/dm-log-cluster.c
@@ -243,7 +243,7 @@ static int cluster_disk_ctr(struct dm_di
static void cluster_dtr(struct dm_dirty_log *log)
{
int r;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
r = dm_clog_consult_server(lc->uuid, DM_CLOG_DTR,
NULL, 0,
@@ -264,7 +264,7 @@ static void cluster_dtr(struct dm_dirty_
static int cluster_presuspend(struct dm_dirty_log *log)
{
int r;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
r = dm_clog_consult_server(lc->uuid, DM_CLOG_PRESUSPEND,
NULL, 0,
@@ -280,7 +280,7 @@ static int cluster_presuspend(struct dm_
static int cluster_postsuspend(struct dm_dirty_log *log)
{
int r;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
r = dm_clog_consult_server(lc->uuid, DM_CLOG_POSTSUSPEND,
NULL, 0,
@@ -296,7 +296,7 @@ static int cluster_postsuspend(struct dm
static int cluster_resume(struct dm_dirty_log *log)
{
int r;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
lc->in_sync_hint = 0;
r = dm_clog_consult_server(lc->uuid, DM_CLOG_RESUME,
@@ -316,7 +316,7 @@ static int cluster_resume(struct dm_dirt
*/
static uint32_t cluster_get_region_size(struct dm_dirty_log *log)
{
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
return lc->region_size;
}
@@ -336,7 +336,7 @@ static int cluster_is_clean(struct dm_di
int r;
int is_clean;
size_t rdata_size;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
rdata_size = sizeof(is_clean);
r = cluster_do_request(lc, lc->uuid, DM_CLOG_IS_CLEAN,
@@ -364,7 +364,7 @@ static int cluster_in_sync(struct dm_dir
int r;
int in_sync;
size_t rdata_size;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
/*
* We can never respond directly - even if in_sync_hint is
@@ -411,7 +411,7 @@ static int cluster_flush(struct dm_dirty
{
int r = 0;
unsigned long flags;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
LIST_HEAD(flush_list);
struct flush_entry *fe, *tmp_fe;
@@ -469,7 +469,7 @@ fail:
static void cluster_mark_region(struct dm_dirty_log *log, region_t region)
{
unsigned long flags;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
struct flush_entry *fe;
/* Wait for an allocation, but _never_ fail */
@@ -500,7 +500,7 @@ static void cluster_mark_region(struct d
static void cluster_clear_region(struct dm_dirty_log *log, region_t region)
{
unsigned long flags;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
struct flush_entry *fe;
/*
@@ -538,7 +538,7 @@ static int cluster_get_resync_work(struc
{
int r;
size_t rdata_size;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
struct {
int i;
region_t r;
@@ -569,7 +569,7 @@ static void cluster_set_region_sync(stru
region_t region, int in_sync)
{
int r;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
struct {
region_t r;
int i;
@@ -633,7 +633,7 @@ static int cluster_status(struct dm_dirt
{
int r = 0;
size_t sz = (size_t)maxlen;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
switch (status_type) {
case STATUSTYPE_INFO:
@@ -665,7 +665,7 @@ static int cluster_is_remote_recovering(
region_t region)
{
int r;
- struct log_c *lc = (struct log_c *)log->context;
+ struct log_c *lc = log->context;
static unsigned long long limit;
struct {
int is_recovering;
More information about the dm-devel
mailing list