Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
fix some doxygen warnings
authorMartin Quinson <martin.quinson@loria.fr>
Wed, 28 Dec 2016 16:13:32 +0000 (17:13 +0100)
committerMartin Quinson <martin.quinson@loria.fr>
Wed, 28 Dec 2016 16:13:32 +0000 (17:13 +0100)
28 files changed:
doc/doxygen/FAQ.doc
doc/doxygen/module-index.doc
doc/doxygen/module-s4u.doc
doc/doxygen/module-smpi.doc
doc/doxygen/module-xbt.doc
doc/doxygen/options.doc
doc/doxygen/platform.doc
doc/doxygen/uhood_arch.doc
examples/msg/README.doc
include/simgrid/s4u/Actor.hpp
include/simgrid/s4u/file.hpp
include/simgrid/s4u/host.hpp
src/kernel/routing/DijkstraZone.hpp
src/kernel/routing/FatTreeZone.hpp
src/kernel/routing/NetZoneImpl.hpp
src/mc/Process.cpp
src/mc/compare.cpp
src/mc/mc_checkpoint.cpp
src/mc/mc_dwarf.cpp
src/msg/msg_gos.cpp
src/msg/msg_vm.cpp
src/s4u/s4u_host.cpp
src/simdag/sd_task.cpp
src/simix/ActorImpl.cpp
src/simix/libsmx.cpp
src/simix/smx_global.cpp
src/simix/smx_network.cpp
src/surf/xml/surfxml_sax_cb.cpp

index f6d96b5..6613818 100644 (file)
@@ -141,7 +141,7 @@ MSG_task_isend() and MSG_task_irecv();
 MSG_comm_test(), MSG_comm_wait(), MSG_comm_waitall() and MSG_comm_waitany();
 MSG_comm_destroy(). 
 
-There is even a specific example section on \ref msg_ex_asynchronous_communications.
+There is even a specific example section on \ref msg_ex_async .
 
 \subsubsection faq_MIA_thread_synchronization How to synchronize my user processes?
 
index b89f0d6..5c272c3 100644 (file)
@@ -72,10 +72,10 @@ bypass mechanism.
 
 The <b>intra-zone level</b> is naturally handled by the netzones. Each
 netzone have to specify the routing algorithm it uses for that.
-@ref{FullZone} netzones have complete matrix where matrix(a,b)
+@ref simgrid::kernel::routing::FullZone "FullZone" netzones have complete matrix where matrix(a,b)
 represents the full path (the list of links) between the hosts a and
-b. @ref{FloydZone} apply the Floyd-Warshall algorithm to compute the
-paths. @ref{ClusterZone} model classical switched or hub networks,
+b. @ref simgrid::kernel::routing::FloydZone "FloydZone" apply the Floyd-Warshall algorithm to compute the
+paths. @ref simgrid::kernel::routing::ClusterZone "ClusterZone" model classical switched or hub networks,
 where each component is connected through a private link onto a common
 backbone. Many other routing algorithms are provided to model the
 classical needs, but you can naturally define your own routing if the
index 6b8ffa2..7ee18cf 100644 (file)
@@ -22,7 +22,7 @@ For sake of simplicity, we use
 everywhere in S4U. This is an idiom where resources are automatically
 managed through the context. Provided that you never manipulate
 objects of type Foo directly but always FooPtr references (which are
-[boost::intrusive_ptr](http://www.boost.org/doc/libs/1_61_0/libs/smart_ptr/intrusive_ptr.html)<Foo>),
+[boost::intrusive_ptr](http://www.boost.org/doc/libs/1_61_0/libs/smart_ptr/intrusive_ptr.html)&lt;Foo&gt;),
 you will never have to explicitely release the resource that you use
 nor to free the memory of unused objects.
 
index 5d4f2ae..7eef9f9 100644 (file)
@@ -63,7 +63,7 @@ SMPI_PRETEND_CC=1 ./configure # here come the configure parameters
 make
 @endverbatim
 
-\warn
+\warning
   Again, make sure that SMPI_PRETEND_CC is not set when you actually 
   compile your application. It is just a work-around for some configure-scripts
   and replaces some internals by "return 0;". Your simulation will not
index b61583c..fe02319 100644 (file)
@@ -16,8 +16,6 @@
     - Data structures
       - \ref XBT_dynar
       - \ref XBT_dict
-      - \ref XBT_set
-        Data are associated to both an ID (0(1) search) and a name
       - \ref XBT_fifo
       - \ref XBT_swag
       - \ref XBT_heap
index 1cfd2df..16d9b75 100644 (file)
@@ -915,7 +915,7 @@ SMPI implements more than 100 different algorithms for MPI collective communicat
 simulate the behavior of most of the existing MPI libraries. The \b smpi/coll-selector item can be used
  to use the decision logic of either OpenMPI or MPICH libraries (values: ompi or mpich, by default SMPI
 uses naive version of collective operations). Each collective operation can be manually selected with a
-\b smpi/collective_name:algo_name. Available algorithms are listed in \ref SMPI_collective_algorithms .
+\b smpi/collective_name:algo_name. Available algorithms are listed in \ref SMPI_use_colls .
 
 \subsection options_model_smpi_iprobe smpi/iprobe: Inject constant times for calls to MPI_Iprobe
 
index 6aebdf5..ba3bb76 100644 (file)
@@ -101,10 +101,10 @@ Here is an illustration of these concepts:
 
 \subsection  pf_As Platform: The &lt;AS&gt; tag
 
-The concept of an AS was already outlined above (Section \ref pf_basics);
-recall that the AS is so important because it groups other resources (such
-as routers/hosts) together (in fact, these resources must be contained by
-an AS).
+For historical reasons, the XML files use the expression AS for
+NetZones. Netzones are very important because they group other resources (such
+as routers/hosts) together (in fact, any such resource must be
+contained in a NetZone).
 
 Available attributes :
 
@@ -312,7 +312,7 @@ limiter_link      | no        | int    | Bandwidth for limiter link (if any). Th
 loopback_bw       | no      | int    | Bandwidth for loopback (if any). See <b>link</b> section for syntax/details. If loopback_bw and loopback_lat (see below) attributes are omitted, no loopback link is created and all intra-node communication will use the main network link of the node. Loopback link is a \ref pf_sharing_policy_fatpipe "\b FATPIPE".
 loopback_lat      | no      | int    | Latency for loopback (if any). See <b>link</b> section for syntax/details. See loopback_bw for more info.
 topology          | no      | FLAT\|TORUS\|FAT_TREE\|DRAGONFLY (default: FLAT) | Network topology to use. SimGrid currently supports FLAT (with or without backbone, as described before), <a href="http://en.wikipedia.org/wiki/Torus_interconnect">TORUS </a>, FAT_TREE, and DRAGONFLY attributes for this tag.
-topo_parameters   | no      | string | Specific parameters to pass for the topology defined in the topology tag. For torus networks, comma-separated list of the number of nodes in each dimension of the torus. For fat trees, refer to \ref simgrid::kernel::routing::AsClusterFatTree "AsClusterFatTree documentation". For dragonfly, refer to \ref simgrid::kernel::routing::AsClusterDragonfly "AsClusterDragonfly documentation".
+topo_parameters   | no      | string | Specific parameters to pass for the topology defined in the topology tag. For torus networks, comma-separated list of the number of nodes in each dimension of the torus. Please refer to the specific documentation for \ref simgrid::kernel::routing::FatTreeZone "FatTree NetZone", \ref simgrid::kernel::routing::DragonflyZone "Dragonfly NetZone".
 
 
 the router name is defined as the resulting String in the following
@@ -404,18 +404,14 @@ etc.
 
 \subsubsection pf_peer The &lt;peer&gt; tag
 
-This tag represents a peer, as in Peer-to-Peer (P2P) networks. However, internally,
-SimGrid transforms a peer into an AS (similar to Cluster). Hence, this tag
-is virtually only a shortcut that comes with some pre-defined resources
-and values. These are:
+This tag represents a peer, as in Peer-to-Peer (P2P) networks. This
+can only be used in Vivaldi NetZones. It creates the  following
+resources to the NetZone:
 
-\li A tiny AS whose routing type is cluster is created
 \li A host
 \li Two links: One for download and one for upload. This is
     convenient to use and simulate stuff under the last mile model (e.g., ADSL peers).
 \li It connects the two links to the host
-\li It creates a router (a gateway) that serves as an entry point for this peer zone.
-    This router has coordinates.
 
 #### Attributes ####
 
@@ -423,37 +419,14 @@ Attribute name  | Mandatory | Values | Description
 --------------- | --------- | ------ | -----------
 id              | yes       | string | The identifier of the peer. Facilitates referring to this peer.
 speed           | yes       | int    | See the description of the ``host`` tag for this attribute
-bw_in           | yes       | int    | Bandwidth downstream
-bw_out          | yes       | int    | Bandwidth upstream
+bw_in           | yes       | int    | Bandwidth of the private downstream link
+bw_out          | yes       | int    | Bandwidth of the private upstream link
 lat             | yes       | double | Latency for both up- and downstream, in seconds.
 coordinates     | no        | string | Coordinates of the gateway for this peer. Example value: 12.8 14.4 6.4
 sharing_policy  | no        | SHARED\|FULLDUPLEX (default: FULLDUPLEX) | Sharing policy for links. See <b>link</b> description for details.
 availability_file| no       | string | Availability file for the peer. Same as host availability file. See <b>host</b> description for details.
 state_file      | no        | string | State file for the peer. Same as host state file. See <b>host</b> description for details.
 
-Internally, SimGrid transforms any ``\<peer/\>`` construct such as
-\verbatim
-<peer id="FOO"
-  coordinates="12.8 14.4 6.4"
-  speed="1.5Gf"
-  bw_in="2.25GBps"
-  bw_out="2.25GBps"
-  lat="500us" />
-\endverbatim
-into an ``\<AS\>`` (see Sections \ref pf_basics and \ref pf_As). In fact, this example of the ``\<peer/\>`` tag
-is completely equivalent to the following declaration:
-
-\verbatim
-<AS id="as_FOO" routing="Cluster">
-   <host id="peer_FOO" speed="1.5Gf"/>
-   <link id="link_FOO_UP" bandwidth="2.25GBps" latency="500us"/>
-   <link id="link_FOO_DOWN" bandwidth="2.25GBps" latency="500us"/>
-   <router id="router_FOO" coordinates="25.5 9.4 1.4"/>
-   <host_link id="peer_FOO" up="link_FOO_UP" down="link_FOO_DOWN"/>
-</AS>
-\endverbatim
-
-
 \subsection pf_ne Network equipments: links and routers
 
 There are two tags at all times available to represent network entities and
@@ -1183,12 +1156,9 @@ routing model (the path is given relative to SimGrid's source directory):
 For more information on how to use the [Vivaldi Coordinates](https://en.wikipedia.org/wiki/Vivaldi_coordinates),
 see also Section \ref pf_P2P_tags "P2P tags".
 
-For documentation on how to activate this model (as some initialization must be done
-in the simulator), see Section \ref options_model_network_coord "Activating Coordinate Based Routing".
-
 Note that it is possible to combine the Vivaldi routing model with other routing models;
 an example can be found in the file \c examples/platforms/cloud.xml. This
-examples models an AS using Vivaldi that contains other ASes that use different
+examples models a NetZone using Vivaldi that contains other NetZones that use different
 routing models.
 
 #### Example platform files ####
index db93f20..1a97f1d 100644 (file)
@@ -75,7 +75,7 @@ It is a portable library providing some grounding features such as \ref
 XBT_log, \ref XBT_ex and \ref XBT_config.
 
 XBT also encompass the following convenient C data structures:
-\ref XBT_dynar, \ref XBT_fifo, \ref XBT_dict, \ref XBT_heap, \ref XBT_set and
+\ref XBT_dynar, \ref XBT_fifo, \ref XBT_dict, \ref XBT_heap, and
 \ref XBT_swag. The code is being migrated in C++ so you should probably want
 to use standard C++ containers instead of them if possible.
 
index ef998ed..1115a22 100644 (file)
@@ -43,7 +43,7 @@ documentation, but it should remain readable directly.
 @section msg_ex_async Asynchronous communications
 
 In addition to the fully documented example of @ref
-msg_ex_asynchronous_communications, there are several other examples
+MSG_ex_asynchronous_communications, there are several other examples
 shipped in the archive:
   
  - <b>Basic asynchronous communications</b>. 
index 7bc02e7..b157771 100644 (file)
@@ -130,7 +130,7 @@ namespace s4u {
  *  @{
  */
 
-/** @brief Simulation Agent (see \ref s4u_actor)*/
+/** @brief Simulation Agent */
 XBT_PUBLIC_CLASS Actor {
   friend Mailbox;
   friend simgrid::simix::ActorImpl;
index 5c1565c..ecb52c9 100644 (file)
@@ -20,7 +20,7 @@ class Storage;
  *
  * Used to simulate the time it takes to access to a file, but does not really store any information.
  *
- * They are located on @link{simgrid::s4u::Storage} that are accessed from a given @link{simgrid::s4u::Host} through mountpoints.
+ * They are located on @ref simgrid::s4u::Storage that are accessed from a given @ref simgrid::s4u::Host through mountpoints.
  * For now, you cannot change the mountpoints programatically, and must declare them from your platform file.
  */
 XBT_PUBLIC_CLASS File {
index ea5d00a..b037535 100644 (file)
@@ -33,7 +33,7 @@ namespace s4u {
  * An host represents some physical resource with computing and networking capabilities.
  *
  * All hosts are automatically created during the call of the method
- * @link{simgrid::s4u::Engine::loadPlatform()}.
+ * @ref simgrid::s4u::Engine::loadPlatform().
  * You cannot create a host yourself.
  *
  * You can retrieve a particular host using simgrid::s4u::Host::byName()
index 83786e3..bf443fb 100644 (file)
@@ -36,8 +36,7 @@ namespace routing {
  *  The path between components is computed each time you request it,
  *  using the Dijkstra algorithm. A cache can be used to reduce the computation.
  *
- *  This result in rather small platform file, very fast initialization, and intermediate memory requirements
- *  (somewhere between the one of @DijkstraZone and the one of @FullZone).
+ *  This result in rather small platform file, very fast initialization, and very low memory requirements, but somehow long path resolution times.
  */
 class XBT_PRIVATE DijkstraZone : public RoutedZone {
 public:
index 9661646..5077667 100644 (file)
@@ -14,7 +14,7 @@ namespace routing {
 
 class XBT_PRIVATE FatTreeLink;
 
-/** \brief A node in a fat tree (@ref AsClusterFatTree).
+/** \brief A node in a fat tree (@ref FatTreeZone).
  * A FatTreeNode can either be a switch or a processing node. Switches are
  * identified by a negative ID. This class is closely related to fat
  */
@@ -54,7 +54,7 @@ public:
   FatTreeNode(sg_platf_cluster_cbarg_t cluster, int id, int level, int position);
 };
 
-/** \brief Link in a fat tree (@ref AsClusterFatTree).
+/** \brief Link in a fat tree (@ref FatTreeZone).
  *
  * Represents a single, duplex link in a fat tree. This is necessary to have a tree.
  * It is equivalent to a physical link.
index bc4341b..1dc9520 100644 (file)
@@ -40,8 +40,8 @@ class BypassRoute;
  * Finding the path between two nodes is rather complex because we navigate a hierarchy of netzones, each of them
  * being a full network. In addition, the routing can declare shortcuts (called bypasses), either within a NetZone
  * at the route level or directly between NetZones. Also, each NetZone can use a differing routing algorithm, depending
- * on its class. @ref{FullZone} have a full matrix giving explicitly the path between any pair of their
- * contained nodes, while @ref{DijkstraZone} or @ref{FloydZone} rely on a shortest path algorithm. @ref{VivaldiZone}
+ * on its class. @ref FullZone have a full matrix giving explicitly the path between any pair of their
+ * contained nodes, while @ref DijkstraZone or @ref FloydZone rely on a shortest path algorithm. @ref VivaldiZone
  * does not even have any link but only use only coordinate information to compute the latency.
  *
  * So NetZoneImpl::getGlobalRoute builds the path recursively asking its specific information to each traversed NetZone
index 663e486..2fc57aa 100644 (file)
@@ -508,10 +508,9 @@ const void *Process::read_bytes(void* buffer, std::size_t size,
 
 /** Write data to a process memory
  *
- *  @param process the process
- *  @param local   local memory address (source)
- *  @param remote  target process memory address (target)
- *  @param len     data size
+ *  @param buffer   local memory address (source)
+ *  @param len      data size
+ *  @param address  target process memory address (target)
  */
 void Process::write_bytes(const void* buffer, size_t len, RemotePtr<void> address)
 {
index e672746..c3a81f0 100644 (file)
@@ -194,7 +194,6 @@ struct StateComparator {
 
   /** Check whether two blocks are known to be matching
    *
-   *  @param state  State used
    *  @param b1     Block of state 1
    *  @param b2     Block of state 2
    *  @return       if the blocks are known to be matching
@@ -207,7 +206,6 @@ struct StateComparator {
 
   /** Check whether two fragments are known to be matching
    *
-   *  @param state  State used
    *  @param b1     Block of state 1
    *  @param f1     Fragment of state 1
    *  @param b2     Block of state 2
index 29afd75..d17abc8 100644 (file)
@@ -57,7 +57,7 @@ namespace mc {
 
 /** @brief Restore a region from a snapshot
  *
- *  @param reg     Target region
+ *  @param region     Target region
  */
 static void restore(mc_mem_region_t region)
 {
@@ -270,7 +270,7 @@ void find_object_address(
  *  A variable may be defined only from a given value of IP.
  *
  *  \param var   Variable description
- *  \param frame Scope description
+ *  \param scope Scope description
  *  \param ip    Instruction pointer
  *  \return      true if the variable is valid
  * */
index c177458..89951e4 100644 (file)
@@ -358,13 +358,13 @@ static bool MC_dwarf_attr_flag(Dwarf_Die * die, int attribute, bool integrate)
   return result;
 }
 
-/** \brief Find the default lower bound for a given language
+/** @brief Find the default lower bound for a given language
  *
  *  The default lower bound of an array (when DW_TAG_lower_bound
  *  is missing) depends on the language of the compilation unit.
  *
- *  \param lang Language of the compilation unit (values defined in the DWARF spec)
- *  \return     Default lower bound of an array in this compilation unit
+ *  @param lang Language of the compilation unit (values defined in the DWARF spec)
+ *  @return     Default lower bound of an array in this compilation unit
  * */
 static uint64_t MC_dwarf_default_lower_bound(int lang)
 {
index 6ac5ddf..0d63305 100644 (file)
@@ -703,7 +703,7 @@ msg_task_t MSG_comm_get_task(msg_comm_t comm)
 
 /**
  * \brief This function is called by SIMIX in kernel mode to copy the data of a comm.
- * \param comm the comm
+ * \param synchro the comm
  * \param buff the data copied
  * \param buff_size size of the buffer
  */
index 873e41a..64e13a6 100644 (file)
@@ -45,7 +45,7 @@ void MSG_vm_set_params(msg_vm_t vm, vm_params_t params)
 /** \ingroup m_vm_management
  * \brief Get the parameters of a given host
  *
- * \param host a host
+ * \param vm the vm you are interested into
  * \param params a prameter object
  */
 void MSG_vm_get_params(msg_vm_t vm, vm_params_t params)
index 72c3dfc..e58eead 100644 (file)
@@ -126,10 +126,10 @@ int Host::pstatesCount() const {
  * \brief Find a route toward another host
  *
  * \param dest [IN] where to
- * \param route [OUT] where to store the list of links (must exist, cannot be nullptr).
+ * \param links [OUT] where to store the list of links (must exist, cannot be nullptr).
  * \param latency [OUT] where to store the latency experienced on the path (or nullptr if not interested)
  *                It is the caller responsibility to initialize latency to 0 (we add to provided route)
- * \pre route!=nullptr
+ * \pre links!=nullptr
  *
  * walk through the routing components tree and find a route between hosts
  * by calling each "get_route" function in each routing component.
index 0006c32..8de2276 100644 (file)
@@ -718,7 +718,7 @@ static inline void SD_task_do_schedule(SD_task_t task)
  *
  * \param task the task you want to schedule
  * \param host_count number of hosts on which the task will be executed
- * \param workstation_list the hosts on which the task will be executed
+ * \param host_list the hosts on which the task will be executed
  * \param flops_amount computation amount for each hosts (i.e., an array of host_count doubles)
  * \param bytes_amount communication amount between each pair of hosts (i.e., a matrix of host_count*host_count doubles)
  * \param rate task execution speed rate
index e51e517..b66efd1 100644 (file)
@@ -483,6 +483,7 @@ void SIMIX_process_kill(smx_actor_t process, smx_actor_t issuer) {
 
 /** @brief Ask another process to raise the given exception
  *
+ * @param process The process that should raise that exception
  * @param cat category of exception
  * @param value value associated to the exception
  * @param msg string information associated to the exception
@@ -961,7 +962,7 @@ void SIMIX_segment_index_set(smx_actor_t proc, int index){
  * \param code the main function of the process
  * \param data a pointer to any data one may want to attach to the new object. It is for user-level information and can be nullptr.
  * It can be retrieved with the function \ref simcall_process_get_data.
- * \param hostname name of the host where the new agent is executed.
+ * \param host where the new agent is executed.
  * \param kill_time time when the process is killed
  * \param argc first argument passed to \a code
  * \param argv second argument passed to \a code
index c92f4e3..e9fb2be 100644 (file)
@@ -526,7 +526,7 @@ void simcall_comm_wait(smx_activity_t comm, double timeout)
  * \brief Set the category of an synchro.
  *
  * This functions changes the category only. It calls a surf function.
- * \param execution The execution synchro
+ * \param synchro The execution synchro
  * \param category The tracing category
  */
 void simcall_set_category(smx_activity_t synchro, const char *category)
index 95c67da..30eec19 100644 (file)
@@ -571,7 +571,7 @@ void SIMIX_run()
  *
  * Set the date to execute the function on the surf.
  *   \param date Date to execute function
- *   \param function Function to be executed
+ *   \param callback Function to be executed
  *   \param arg Parameters of the function
  *
  */
index 409a20d..1a4b577 100644 (file)
@@ -764,8 +764,8 @@ void SIMIX_comm_copy_buffer_callback(smx_activity_t synchro, void* buff, size_t
 
 
 /**
- *  \brief Copy the communication data from the sender's buffer to the receiver's one
- *  \param comm The communication
+ *  @brief Copy the communication data from the sender's buffer to the receiver's one
+ *  @param synchro The communication
  */
 void SIMIX_comm_copy_data(smx_activity_t synchro)
 {
index a3fc7d9..8292cf7 100644 (file)
@@ -384,7 +384,7 @@ void ETag_surfxml_include() {
  * error message in that case.
  *
  * Yeah, that's terribly hackish, but it works. A better solution should be dealed with in flexml
- * directly: a command line flag could instruct it to do the correct thing when #include is encountered
+ * directly: a command line flag could instruct it to do the correct thing when the include directive is encountered
  * on a line. One day maybe, if the maya allow it.
  */
 int ETag_surfxml_include_state()