]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
Fix several doxygen warnings, to minimize noise. Only changes comments.
authorTommi Virtanen <tommi.virtanen@dreamhost.com>
Tue, 10 Jan 2012 18:08:52 +0000 (10:08 -0800)
committerTommi Virtanen <tommi.virtanen@dreamhost.com>
Tue, 10 Jan 2012 19:20:56 +0000 (11:20 -0800)
Signed-off-by: Tommi Virtanen <tommi.virtanen@dreamhost.com>
src/client/Client.h
src/common/pick_address.h
src/crush/crush.c
src/crush/mapper.c
src/include/frag.h
src/mds/MDCache.cc
src/mds/MDCache.h
src/mds/Server.cc
src/os/HashIndex.h
src/osd/PG.h

index 5ffa0a522bf5198ed70c0250d88425d0b2703a57..1431bbc8a6020bfd85c60aee98c7f59af149950f 100644 (file)
@@ -524,7 +524,7 @@ public:
    * and the offset.
    *
    * Returns 0 if it reached the end of the directory.
-   * If @cb returns a negative error code, stop and return that.
+   * If @cb returns a negative error code, stop and return that.
    */
   int readdir_r_cb(dir_result_t *dirp, add_dirent_cb_t cb, void *p);
 
index e0b9bf9f6783c862404947f3ebef9474a2e4c442..50c2e53a87e81e560dbac10eb2e0f1907c9f5010 100644 (file)
@@ -31,9 +31,9 @@ void pick_addresses(CephContext *cct);
  *
  * check if any of the listed addresses is configured on the local host.
  *
- * @cct context
- * @ls list of addresses
- * @match [out] pointer to match, if an item in @ls is found configured locally.
+ * @param cct context
+ * @param ls list of addresses
+ * @param match [out] pointer to match, if an item in @a ls is found configured locally.
  */
 bool have_local_addr(CephContext *cct, const list<entity_addr_t>& ls, entity_addr_t *match);
 
index 0790b80facca6a9304830b9ae8d55f493a814471..2190d5bff65dcf449cf224fe7715bfd5d2c7b51b 100644 (file)
@@ -23,8 +23,8 @@ const char *crush_bucket_alg_name(int alg)
 
 /**
  * crush_get_bucket_item_weight - Get weight of an item in given bucket
- * @b: bucket pointer
- * @p: item index in bucket
+ * @param b bucket pointer
+ * @param p item index in bucket
  */
 int crush_get_bucket_item_weight(const struct crush_bucket *b, int p)
 {
@@ -46,7 +46,7 @@ int crush_get_bucket_item_weight(const struct crush_bucket *b, int p)
 
 /**
  * crush_calc_parents - Calculate parent vectors for the given crush map.
- * @map: crush_map pointer
+ * @param map crush_map pointer
  */
 void crush_calc_parents(struct crush_map *map)
 {
@@ -121,7 +121,7 @@ void crush_destroy_bucket(struct crush_bucket *b)
 
 /**
  * crush_destroy - Destroy a crush_map
- * @map: crush_map pointer
+ * @param map crush_map pointer
  */
 void crush_destroy(struct crush_map *map)
 {
index c955c0a51f346bc388f76d6d095975e68949f34c..1e475b4020e61c3058a399c5eb5d145da907ec35 100644 (file)
 
 /**
  * crush_find_rule - find a crush_rule id for a given ruleset, type, and size.
- * @map: the crush_map
- * @ruleset: the storage ruleset id (user defined)
- * @type: storage ruleset type (user defined)
- * @size: output set size
+ * @param map the crush_map
+ * @param ruleset the storage ruleset id (user defined)
+ * @param type storage ruleset type (user defined)
+ * @param size output set size
  */
 int crush_find_rule(const struct crush_map *map, int ruleset, int type, int size)
 {
@@ -276,16 +276,16 @@ static int is_out(const struct crush_map *map, const __u32 *weight, int item, in
 
 /**
  * crush_choose - choose numrep distinct items of given type
- * @map: the crush_map
- * @bucket: the bucket we are choose an item from
- * @x: crush input value
- * @numrep: the number of items to choose
- * @type: the type of item to choose
- * @out: pointer to output vector
- * @outpos: our position in that vector
- * @firstn: true if choosing "first n" items, false if choosing "indep"
- * @recurse_to_leaf: true if we want one device under each item of given type
- * @out2: second output vector for leaf items (if @recurse_to_leaf)
+ * @param map the crush_map
+ * @param bucket the bucket we are choose an item from
+ * @param x crush input value
+ * @param numrep the number of items to choose
+ * @param type the type of item to choose
+ * @param out pointer to output vector
+ * @param outpos our position in that vector
+ * @param firstn true if choosing "first n" items, false if choosing "indep"
+ * @param recurseto_leaf: true if we want one device under each item of given type
+ * @param out2 second output vector for leaf items (if @a recurse_to_leaf)
  */
 static int crush_choose(const struct crush_map *map,
                        struct crush_bucket *bucket,
@@ -449,12 +449,12 @@ reject:
 
 /**
  * crush_do_rule - calculate a mapping with the given input and rule
- * @map: the crush_map
- * @ruleno: the rule id
- * @x: hash input
- * @result: pointer to result vector
- * @result_max: maximum result size
- * @force: force initial replica choice; -1 for none
+ * @param map the crush_map
+ * @param ruleno the rule id
+ * @param x hash input
+ * @param result pointer to result vector
+ * @param resultmax: maximum result size
+ * @param force force initial replica choice; -1 for none
  */
 int crush_do_rule(const struct crush_map *map,
                  int ruleno, int x, int *result, int result_max,
index f52ebdd28708364b19e12d5f25a6f123ed706214..7e25712fbb685404b886857b3163c9544faead09 100644 (file)
@@ -238,8 +238,8 @@ public:
   }
 
   /**
-   * get_branch -- get branch point at OR above frag @x
-   *  - may be @x itself, if @x is a split
+   * get_branch -- get branch point at OR above frag @x
+   *  - may be @a x itself, if @a x is a split
    *  - may be root (frag_t())
    */
   frag_t get_branch(frag_t x) const {
@@ -251,9 +251,9 @@ public:
   }
 
   /**
-   * get_branch_above -- get a branch point above frag @x
+   * get_branch_above -- get a branch point above frag @x
    *  - may be root (frag_t())
-   *  - may NOT be @x, even if @x is a split.
+   *  - may NOT be @a x, even if @a x is a split.
    */
   frag_t get_branch_above(frag_t x) const {
     while (1) {
@@ -265,8 +265,8 @@ public:
 
 
   /**
-   * get_branch_or_leaf -- get branch or leaf point parent for frag @x
-   *  - may be @x itself, if @x is a split or leaf
+   * get_branch_or_leaf -- get branch or leaf point parent for frag @x
+   *  - may be @a x itself, if @a x is a split or leaf
    *  - may be root (frag_t())
    */
   frag_t get_branch_or_leaf(frag_t x) const {
@@ -300,7 +300,7 @@ public:
   }
 
   /**
-   * contains(fg) -- does fragtree contain the specific frag @x
+   * contains(fg) -- does fragtree contain the specific frag @x
    */
   bool contains(frag_t x) const {
     std::list<frag_t> q;
index 953a89e0d83f97e0b903399e81355fd003e51016..fb7fec66301f3ee108533b93de65f3af9f865de3 100644 (file)
@@ -3393,8 +3393,8 @@ void MDCache::rejoin_send_rejoins()
 /** 
  * rejoin_walk - build rejoin declarations for a subtree
  * 
- * @dir subtree root
- * @rejoin rejoin message
+ * @param dir subtree root
+ * @param rejoin rejoin message
  *
  * from a rejoining node:
  *  weak dirfrag
@@ -3739,8 +3739,8 @@ public:
 /**
  * parallel_fetch -- make a pass at fetching a bunch of paths in parallel
  *
- * @pathmap - map of inodeno to full pathnames.  we remove items from this map 
- *            as we discover we have them.
+ * @param pathmap map of inodeno to full pathnames.  we remove items
+ *            from this map as we discover we have them.
  *
  *           returns true if there is work to do, false otherwise.
  */
@@ -6919,9 +6919,9 @@ CInode *MDCache::cache_traverse(const filepath& fp)
 /**
  * open_remote_dir -- open up a remote dirfrag
  *
- * @diri - base inode
- * @approxfg - approximate fragment.
- * @fin - completion callback
+ * @param diri base inode
+ * @param approxfg approximate fragment.
+ * @param fin completion callback
  */
 void MDCache::open_remote_dirfrag(CInode *diri, frag_t approxfg, Context *fin) 
 {
@@ -6947,8 +6947,8 @@ void MDCache::open_remote_dirfrag(CInode *diri, frag_t approxfg, Context *fin)
 /** 
  * get_dentry_inode - get or open inode
  *
- * @dn the dentry
- * @mdr current request
+ * @param dn the dentry
+ * @param mdr current request
  *
  * will return inode for primary, or link up/open up remote link's inode as necessary.
  * If it's not available right now, puts mdr on wait list and returns null.
@@ -9343,9 +9343,9 @@ void MDCache::handle_dentry_unlink(MDentryUnlink *m)
 /** 
  * adjust_dir_fragments -- adjust fragmentation for a directory
  *
- * @diri - directory inode
- * @basefrag - base fragment
- * @bits - bit adjustment.  positive for split, negative for merge.
+ * @param diri directory inode
+ * @param basefrag base fragment
+ * @param bits bit adjustment.  positive for split, negative for merge.
  */
 void MDCache::adjust_dir_fragments(CInode *diri, frag_t basefrag, int bits,
                                   list<CDir*>& resultfrags, 
index 0662cfa28e7567cc56ff37948b8d391376f528a5..236f644458f693a62e5c8932ac635d5b927ad8ce 100644 (file)
@@ -665,10 +665,10 @@ public:
    * @param mdr The MDRequest associated with the path. Can be null.
    * @param req The Message associated with the path. Can be null.
    * @param fin The Context associated with the path. Can be null.
-   * @param filepath The path to traverse to.
-   * @pdnvec Data return parameter -- on success, contains a vector of dentries.
-   * On failure, is either empty or contains the full trace of traversable
-   * dentries.
+   * @param path The path to traverse to.
+   * @param pdnvec Data return parameter -- on success, contains a
+   * vector of dentries. On failure, is either empty or contains the
+   * full trace of traversable dentries.
    * @param pin Data return parameter -- if successful, points to the inode
    * associated with filepath. If unsuccessful, is null.
    * @param onfail Specifies different lookup failure behaviors. If set to
index d44bd7ea377a20f6cd0f06a4cf309c3241186abc..55e5a579efe146b0c661c5496e4efce720540679 100644 (file)
@@ -2041,10 +2041,10 @@ CDentry* Server::rdlock_path_xlock_dentry(MDRequest *mdr, int n,
 /**
  * try_open_auth_dirfrag -- open dirfrag, or forward to dirfrag auth
  *
- * @diri base indoe
- * @fg the exact frag we want
- * @mdr request
- * Returns: the pointer, or NULL if it had to be delayed (but mdr is taken care of)
+ * @param diri base inode
+ * @param fg the exact frag we want
+ * @param mdr request
+ * @returns the pointer, or NULL if it had to be delayed (but mdr is taken care of)
  */
 CDir* Server::try_open_auth_dirfrag(CInode *diri, frag_t fg, MDRequest *mdr)
 {
index 9ac471ba16fd8aba095dd2b51c13e163de9fbe71..8f6ebfb11233c72c34e697b907ce8ead2b9f85d1 100644 (file)
@@ -22,6 +22,8 @@
 
 /**
  * Implements collection prehashing.
+ *
+ * @verbatim
  *     (root) - 0 - 0
  *                - 1
  *                - E
@@ -31,6 +33,8 @@
  *            .
  *            .
  *            - F - 0
+ * @endverbatim
+ *
  * A file is located at the longest existing directory from the root 
  * given by the hex characters in the hash beginning with the least
  * significant.
index 53174c9a4230c975e664f14bf4ce3d026bb6ae64..2ce8554cb474a2b2b7a1664fe87423c776fc21b5 100644 (file)
@@ -885,10 +885,10 @@ public:
 
   struct PriorSet {
     set<int> probe; /// current+prior OSDs we need to probe.
-    set<int> down;  /// down osds that would normally be in @probe and might be interesting.
+    set<int> down;  /// down osds that would normally be in @probe and might be interesting.
     map<int,epoch_t> blocked_by;  /// current lost_at values for any OSDs in cur set for which (re)marking them lost would affect cur set
 
-    bool pg_down;   /// some down osds are included in @cur; the DOWN pg state bit should be set.
+    bool pg_down;   /// some down osds are included in @cur; the DOWN pg state bit should be set.
     PriorSet(const OSDMap &osdmap,
             const map<epoch_t, Interval> &past_intervals,
             const vector<int> &up,