All of lore.kernel.org
 help / color / mirror / Atom feed
* [Ocfs2-devel] [PATCH] ocfs2: o2hb: remove useless force cast
@ 2016-03-02  7:58 Junxiao Bi
  2016-03-02  9:38 ` Joseph Qi
  0 siblings, 1 reply; 2+ messages in thread
From: Junxiao Bi @ 2016-03-02  7:58 UTC (permalink / raw)
  To: ocfs2-devel

Signed-off-by: Junxiao Bi <junxiao.bi@oracle.com>
---
 fs/ocfs2/cluster/heartbeat.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/fs/ocfs2/cluster/heartbeat.c b/fs/ocfs2/cluster/heartbeat.c
index 8ec85cac894e..023c72d35498 100644
--- a/fs/ocfs2/cluster/heartbeat.c
+++ b/fs/ocfs2/cluster/heartbeat.c
@@ -1315,19 +1315,19 @@ static int o2hb_debug_open(struct inode *inode, struct file *file)
 
 	case O2HB_DB_TYPE_REGION_LIVENODES:
 		spin_lock(&o2hb_live_lock);
-		reg = (struct o2hb_region *)db->db_data;
+		reg = db->db_data;
 		memcpy(map, reg->hr_live_node_bitmap, db->db_size);
 		spin_unlock(&o2hb_live_lock);
 		break;
 
 	case O2HB_DB_TYPE_REGION_NUMBER:
-		reg = (struct o2hb_region *)db->db_data;
+		reg = db->db_data;
 		out += snprintf(buf + out, PAGE_SIZE - out, "%d\n",
 				reg->hr_region_num);
 		goto done;
 
 	case O2HB_DB_TYPE_REGION_ELAPSED_TIME:
-		reg = (struct o2hb_region *)db->db_data;
+		reg = db->db_data;
 		lts = reg->hr_last_timeout_start;
 		/* If 0, it has never been set before */
 		if (lts)
@@ -1336,7 +1336,7 @@ static int o2hb_debug_open(struct inode *inode, struct file *file)
 		goto done;
 
 	case O2HB_DB_TYPE_REGION_PINNED:
-		reg = (struct o2hb_region *)db->db_data;
+		reg = db->db_data;
 		out += snprintf(buf + out, PAGE_SIZE - out, "%u\n",
 				!!reg->hr_item_pinned);
 		goto done;
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Ocfs2-devel] [PATCH] ocfs2: o2hb: remove useless force cast
  2016-03-02  7:58 [Ocfs2-devel] [PATCH] ocfs2: o2hb: remove useless force cast Junxiao Bi
@ 2016-03-02  9:38 ` Joseph Qi
  0 siblings, 0 replies; 2+ messages in thread
From: Joseph Qi @ 2016-03-02  9:38 UTC (permalink / raw)
  To: ocfs2-devel

On 2016/3/2 15:58, Junxiao Bi wrote:
> Signed-off-by: Junxiao Bi <junxiao.bi@oracle.com>
Looks good, thanks.

Reviewed-by: Joseph Qi <joseph.qi@huawei.com>
> ---
>  fs/ocfs2/cluster/heartbeat.c |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/fs/ocfs2/cluster/heartbeat.c b/fs/ocfs2/cluster/heartbeat.c
> index 8ec85cac894e..023c72d35498 100644
> --- a/fs/ocfs2/cluster/heartbeat.c
> +++ b/fs/ocfs2/cluster/heartbeat.c
> @@ -1315,19 +1315,19 @@ static int o2hb_debug_open(struct inode *inode, struct file *file)
>  
>  	case O2HB_DB_TYPE_REGION_LIVENODES:
>  		spin_lock(&o2hb_live_lock);
> -		reg = (struct o2hb_region *)db->db_data;
> +		reg = db->db_data;
>  		memcpy(map, reg->hr_live_node_bitmap, db->db_size);
>  		spin_unlock(&o2hb_live_lock);
>  		break;
>  
>  	case O2HB_DB_TYPE_REGION_NUMBER:
> -		reg = (struct o2hb_region *)db->db_data;
> +		reg = db->db_data;
>  		out += snprintf(buf + out, PAGE_SIZE - out, "%d\n",
>  				reg->hr_region_num);
>  		goto done;
>  
>  	case O2HB_DB_TYPE_REGION_ELAPSED_TIME:
> -		reg = (struct o2hb_region *)db->db_data;
> +		reg = db->db_data;
>  		lts = reg->hr_last_timeout_start;
>  		/* If 0, it has never been set before */
>  		if (lts)
> @@ -1336,7 +1336,7 @@ static int o2hb_debug_open(struct inode *inode, struct file *file)
>  		goto done;
>  
>  	case O2HB_DB_TYPE_REGION_PINNED:
> -		reg = (struct o2hb_region *)db->db_data;
> +		reg = db->db_data;
>  		out += snprintf(buf + out, PAGE_SIZE - out, "%u\n",
>  				!!reg->hr_item_pinned);
>  		goto done;
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-03-02  9:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-02  7:58 [Ocfs2-devel] [PATCH] ocfs2: o2hb: remove useless force cast Junxiao Bi
2016-03-02  9:38 ` Joseph Qi

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.