[sheepdog] [PATCH v2 3/7] sheep: rename ctime to create_time in vdi.c

levin li levin108 at gmail.com
Mon Aug 6 10:40:24 CEST 2012


From: levin li <xingke.lwp at taobao.com>

'ctime' shadows a variable in pthread.h, in order to use pthread in
vdi.c, so we need to rename ctime to get rid of the compile warning

Signed-off-by: levin li <xingke.lwp at taobao.com>
---
 collie/vdi.c             |    6 +++---
 include/sheepdog_proto.h |    2 +-
 sheep/vdi.c              |   23 ++++++++++++-----------
 3 files changed, 16 insertions(+), 15 deletions(-)

diff --git a/collie/vdi.c b/collie/vdi.c
index 4c793c9..08aa0b6 100644
--- a/collie/vdi.c
+++ b/collie/vdi.c
@@ -77,7 +77,7 @@ static void print_vdi_list(uint32_t vid, char *name, char *tag, uint32_t snapid,
 	if (info && strcmp(name, info->name) != 0)
 		return;
 
-	ti = i->ctime >> 32;
+	ti = i->create_time >> 32;
 	if (raw_output) {
 		snprintf(dbuf, sizeof(dbuf), "%" PRIu64, (uint64_t) ti);
 	} else {
@@ -132,7 +132,7 @@ static void print_vdi_tree(uint32_t vid, char *name, char * tag, uint32_t snapid
 	if (is_current(i))
 		strcpy(buf, "(you are here)");
 	else {
-		ti = i->ctime >> 32;
+		ti = i->create_time >> 32;
 		localtime_r(&ti, &tm);
 
 		strftime(buf, sizeof(buf),
@@ -149,7 +149,7 @@ static void print_vdi_graph(uint32_t vid, char *name, char * tag, uint32_t snapi
 	struct tm tm;
 	char dbuf[128], tbuf[128], size_str[128];
 
-	ti = i->ctime >> 32;
+	ti = i->create_time >> 32;
 	localtime_r(&ti, &tm);
 
 	strftime(dbuf, sizeof(dbuf), "%Y-%m-%d", &tm);
diff --git a/include/sheepdog_proto.h b/include/sheepdog_proto.h
index f119f9c..7d96d63 100644
--- a/include/sheepdog_proto.h
+++ b/include/sheepdog_proto.h
@@ -155,7 +155,7 @@ struct sd_rsp {
 struct sheepdog_inode {
 	char name[SD_MAX_VDI_LEN];
 	char tag[SD_MAX_VDI_TAG_LEN];
-	uint64_t ctime;
+	uint64_t create_time;
 	uint64_t snap_ctime;
 	uint64_t vm_clock_nsec;
 	uint64_t vdi_size;
diff --git a/sheep/vdi.c b/sheep/vdi.c
index 28611b8..48aa9e6 100644
--- a/sheep/vdi.c
+++ b/sheep/vdi.c
@@ -107,7 +107,7 @@ static int create_vdi_obj(struct vdi_iocb *iocb, uint32_t new_vid,
 
 	strncpy(new->name, name, sizeof(new->name));
 	new->vdi_id = new_vid;
-	new->ctime = (uint64_t) tv.tv_sec << 32 | tv.tv_usec * 1000;
+	new->create_time = (uint64_t) tv.tv_sec << 32 | tv.tv_usec * 1000;
 	new->vdi_size = iocb->size;
 	new->copy_policy = 0;
 	new->nr_copies = iocb->nr_copies;
@@ -168,7 +168,7 @@ out:
 static int find_first_vdi(unsigned long start, unsigned long end, char *name,
 			  char *tag, uint32_t snapid, uint32_t *vid,
 			  unsigned long *deleted_nr, uint32_t *next_snap,
-			  unsigned int *inode_nr_copies, uint64_t *ctime)
+			  unsigned int *inode_nr_copies, uint64_t *create_time)
 {
 	struct sheepdog_inode *inode = NULL;
 	unsigned long i;
@@ -205,8 +205,8 @@ static int find_first_vdi(unsigned long start, unsigned long end, char *name,
 			*next_snap = inode->snap_id + 1;
 			*vid = inode->vdi_id;
 			*inode_nr_copies = inode->nr_copies;
-			if (ctime)
-				*ctime = inode->ctime;
+			if (create_time)
+				*create_time = inode->create_time;
 			ret = SD_RES_SUCCESS;
 			goto out_free_inode;
 		}
@@ -226,7 +226,7 @@ out:
 static int do_lookup_vdi(char *name, int namelen, uint32_t *vid, char *tag,
 		uint32_t snapid, uint32_t *next_snapid, unsigned long *right_nr,
 		unsigned long *deleted_nr, unsigned int *nr_copies,
-		uint64_t *ctime)
+		uint64_t *create_time)
 {
 	int ret;
 	unsigned long nr, start_nr;
@@ -245,7 +245,7 @@ static int do_lookup_vdi(char *name, int namelen, uint32_t *vid, char *tag,
 		/* look up on the right side of the hash point */
 		ret = find_first_vdi(nr - 1, start_nr, name,
 				     tag, snapid, vid, deleted_nr, next_snapid,
-				     nr_copies, ctime);
+				     nr_copies, create_time);
 		return ret;
 	} else {
 		/* round up... bitmap search from the head of the bitmap */
@@ -257,7 +257,8 @@ static int do_lookup_vdi(char *name, int namelen, uint32_t *vid, char *tag,
 			/* look up on the left side of the hash point */
 			ret = find_first_vdi(nr - 1, 0, name,
 					     tag, snapid, vid, deleted_nr,
-					     next_snapid, nr_copies, ctime);
+					     next_snapid, nr_copies,
+					     create_time);
 			if (ret == SD_RES_NO_VDI)
 				; /* we need to go to the right side */
 			else
@@ -270,14 +271,14 @@ static int do_lookup_vdi(char *name, int namelen, uint32_t *vid, char *tag,
 }
 
 int lookup_vdi(char *name, char *tag, uint32_t *vid, uint32_t snapid,
-	       unsigned int *nr_copies, uint64_t *ctime)
+	       unsigned int *nr_copies, uint64_t *create_time)
 {
 	uint32_t dummy0;
 	unsigned long dummy1, dummy2;
 
 	return do_lookup_vdi(name, strlen(name), vid, tag,
 			     snapid, &dummy0, &dummy1, &dummy2, nr_copies,
-			     ctime);
+			     create_time);
 }
 
 int add_vdi(struct vdi_iocb *iocb, uint32_t *new_vid)
@@ -677,7 +678,7 @@ err:
 }
 
 int get_vdi_attr(struct sheepdog_vdi_attr *vattr, int data_len,
-		 uint32_t vid, uint32_t *attrid, uint64_t ctime,
+		 uint32_t vid, uint32_t *attrid, uint64_t create_time,
 		 int wr, int excl, int delete)
 {
 	struct sheepdog_vdi_attr tmp_attr;
@@ -685,7 +686,7 @@ int get_vdi_attr(struct sheepdog_vdi_attr *vattr, int data_len,
 	uint32_t end;
 	int ret;
 
-	vattr->ctime = ctime;
+	vattr->ctime = create_time;
 
 	/* we cannot include value_len for calculating the hash value */
 	hval = fnv_64a_buf(vattr->name, sizeof(vattr->name), FNV1A_64_INIT);
-- 
1.7.1




More information about the sheepdog mailing list