Comment cleanups

Move comment where it applies and remove unused attribe when the var
is actually used.

diff --git a/lib/activate/activate.c b/lib/activate/activate.c
index 67d0cc3ce3d62c9ff828a6abcc305ffc5d662138..e7dd45a77b533559412d54b5cc8fa3bfb2fe6dd9 100644 (file)
--- a/lib/activate/activate.c
+++ b/lib/activate/activate.c
@@ -1502,16 +1502,6 @@ int lv_suspend(struct cmd_context *cmd, const char *lvid_s)
 }
 ***********/
 
- /*
-  * _lv_resume
-  * @cmd
-  * @lvid_s
-  * @origin_only
-  * @exclusive:  This parameter only has an affect in cluster-context.
-  *             It forces local target type to be used (instead of
-  *             cluster-aware type).
-  * @error_if_not_active
-  */
 static int _lv_resume(struct cmd_context *cmd, const char *lvid_s,
                      struct lv_activate_opts *laopts, int error_if_not_active)
 {
@@ -1574,6 +1564,10 @@ out:
  * In a cluster, set exclusive to indicate that only one node is using the
  * device.  Any tables loaded may then use non-clustered targets.
  *
+ * @origin_only
+ * @exclusive   This parameter only has an affect in cluster-context.
+ *              It forces local target type to be used (instead of
+ *              cluster-aware type).
  * Returns success if the device is not active
  */
 int lv_resume_if_active(struct cmd_context *cmd, const char *lvid_s,

diff --git a/lib/thin/thin.c b/lib/thin/thin.c
index 629c5a53880b9520c2f558d3efdc654a01b2f685..e3747a0cd77a4d8d33635e8abf85d4c999429eb7 100644 (file)
--- a/lib/thin/thin.c
+++ b/lib/thin/thin.c
@@ -212,11 +212,11 @@ static int _thin_pool_text_export(const struct lv_segment *seg, struct formatter
 
 #ifdef DEVMAPPER_SUPPORT
 static int _thin_pool_add_target_line(struct dev_manager *dm,
-                                     struct dm_pool *mem __attribute__((unused)),
-                                     struct cmd_context *cmd __attribute__((unused)),
+                                     struct dm_pool *mem,
+                                     struct cmd_context *cmd,
                                      void **target_state __attribute__((unused)),
                                      struct lv_segment *seg,
-                                     const struct lv_activate_opts *laopts __attribute__((unused)),
+                                     const struct lv_activate_opts *laopts,
                                      struct dm_tree_node *node, uint64_t len,
                                      uint32_t *pvmove_mirror_count __attribute__((unused)))
 {