[sheepdog] [PATCH v3] trivial: misc fixes of comments
Hitoshi Mitake
mitake.hitoshi at lab.ntt.co.jp
Thu May 9 03:35:58 CEST 2013
From: Hitoshi Mitake <mitake.hitoshi at gmail.com>
This patch fixes some incorrect comments:
1. group.c: colosync -> corosync
2. sockfd_cache.c: vnode -> node
3. sheep_priv.h: past -> part
1 is a very trivial mistake. 2 wouldn't cause a serious confusion but
the description is not correct. vnode is a concept of consisntent
hashing and it is not a something connect to. 3 would be
confusing. Because the way of dealing with exceptional nodes is subtle
and hard to understand.
Signed-off-by: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>
---
v3: s/Rlease/Release/
v2: eliminate the modification for the comment of leave_cluster()
sheep/group.c | 2 +-
sheep/sheep_priv.h | 2 +-
sheep/sockfd_cache.c | 10 +++++-----
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/sheep/group.c b/sheep/group.c
index d554ac1..bc397a0 100644
--- a/sheep/group.c
+++ b/sheep/group.c
@@ -980,7 +980,7 @@ enum cluster_join_result sd_check_join_cb(const struct sd_node *joining,
uint32_t epoch;
/*
- * If I'm the first sheep joins in colosync, I
+ * If I'm the first sheep joins in corosync, I
* becomes the master without sending JOIN.
*/
diff --git a/sheep/sheep_priv.h b/sheep/sheep_priv.h
index a8a9ff9..5062be9 100644
--- a/sheep/sheep_priv.h
+++ b/sheep/sheep_priv.h
@@ -77,7 +77,7 @@ struct cluster_info {
uint64_t disk_space;
/*
- * List of nodes that were past of the last epoch before a shutdown,
+ * List of nodes that were part of the last epoch before a shutdown,
* but failed to join.
*/
struct list_head failed_nodes;
diff --git a/sheep/sockfd_cache.c b/sheep/sockfd_cache.c
index 1b9644f..6b3d750 100644
--- a/sheep/sockfd_cache.c
+++ b/sheep/sockfd_cache.c
@@ -388,7 +388,7 @@ grab:
goto out;
}
- /* Create a new cached connection for this vnode */
+ /* Create a new cached connection for this node */
sd_dprintf("create cache connection %s:%d idx %d", name, port, idx);
fd = connect_to(name, port);
if (fd < 0) {
@@ -450,7 +450,7 @@ static void sockfd_cache_close(const struct node_id *nid, int idx)
}
/*
- * Return a sockfd connected to the vnode to the caller
+ * Return a sockfd connected to the node to the caller
*
* Try to get a 'long' FD as best, which is cached and never closed. If no FD
* available, we return a 'short' FD which is supposed to be closed by
@@ -480,7 +480,7 @@ struct sockfd *sheep_get_sockfd(const struct node_id *nid)
}
/*
- * Rlease a sockfd connected to the vnode, which is acquired from
+ * Release a sockfd connected to the node, which is acquired from
* sheep_get_sockfd()
*
* If it is a long FD, just decrease the refcount to make it available again.
@@ -504,10 +504,10 @@ void sheep_put_sockfd(const struct node_id *nid, struct sockfd *sfd)
}
/*
- * Delete a sockfd connected to the vnode, when vnode is crashed.
+ * Delete a sockfd connected to the node, when node is crashed.
*
* If it is a long FD, de-refcount it and tres to destroy all the cached FDs of
- * this vnode in the cache.
+ * this node in the cache.
* If it is a short FD, just close it.
*/
void sheep_del_sockfd(const struct node_id *nid, struct sockfd *sfd)
--
1.7.2.5
More information about the sheepdog
mailing list