>From 97b0a66b8e7a4ce16e5d7a10f48c08c9663d2d5c Mon Sep 17 00:00:00 2001
From: Ira Weiny <[email protected]>
Date: Fri, 12 Dec 2008 16:57:12 -0800
Subject: [PATCH] opensm/opensm/osm_console.c: move reporting of plugins to 
"status" command.

   Since plugins are now generic, it does not make sence to have them printed
   under the perfmgr status command.

Signed-off-by: Ira Weiny <[email protected]>
---
 opensm/opensm/osm_console.c |   33 ++++++++++++++++++++-------------
 1 files changed, 20 insertions(+), 13 deletions(-)

diff --git a/opensm/opensm/osm_console.c b/opensm/opensm/osm_console.c
index 5727cea..c6e8e59 100644
--- a/opensm/opensm/osm_console.c
+++ b/opensm/opensm/osm_console.c
@@ -324,16 +324,31 @@ static char *sa_state_str(osm_sa_state_t state)
 
 static void print_status(osm_opensm_t * p_osm, FILE * out)
 {
+       cl_list_item_t *item;
+
        if (out) {
                cl_plock_acquire(&p_osm->lock);
-               fprintf(out, "   OpenSM Version: %s\n", p_osm->osm_version);
-               fprintf(out, "   SM State      : %s\n",
+               fprintf(out, "   OpenSM Version       : %s\n", 
p_osm->osm_version);
+               fprintf(out, "   SM State             : %s\n",
                        sm_state_str(p_osm->subn.sm_state));
-               fprintf(out, "   SA State      : %s\n",
+               fprintf(out, "   SA State             : %s\n",
                        sa_state_str(p_osm->sa.state));
-               fprintf(out, "   Routing Engine: %s\n",
+               fprintf(out, "   Routing Engine       : %s\n",
                        osm_routing_engine_type_str(p_osm->
                                                    routing_engine_used));
+
+               fprintf(out, "   Loaded event plugins :");
+               if (cl_qlist_head(&p_osm->plugin_list) ==
+                       cl_qlist_end(&p_osm->plugin_list)) {
+                       fprintf(out, " <none>");
+               }
+               for (item = cl_qlist_head(&p_osm->plugin_list);
+                    item != cl_qlist_end(&p_osm->plugin_list);
+                    item = cl_qlist_next(item))
+                       fprintf(out, " %s",
+                               ((osm_epi_plugin_t *)item)->plugin_name);
+               fprintf(out, "\n");
+
 #ifdef ENABLE_OSM_PERF_MGR
                fprintf(out, "\n   PerfMgr state/sweep state : %s/%s\n",
                        osm_perfmgr_get_state_str(&(p_osm->perfmgr)),
@@ -1128,24 +1143,16 @@ static void perfmgr_parse(char **p_last, osm_opensm_t * 
p_osm, FILE * out)
                        fprintf(out, "\"%s\" option not found\n", p_cmd);
                }
        } else {
-               cl_list_item_t *item;
                fprintf(out, "Performance Manager status:\n"
                        "state                   : %s\n"
                        "sweep state             : %s\n"
                        "sweep time              : %us\n"
-                       "outstanding queries/max : %d/%u\n"
-                       "loaded event plugin     :",
+                       "outstanding queries/max : %d/%u\n",
                        osm_perfmgr_get_state_str(&(p_osm->perfmgr)),
                        osm_perfmgr_get_sweep_state_str(&(p_osm->perfmgr)),
                        osm_perfmgr_get_sweep_time_s(&(p_osm->perfmgr)),
                        p_osm->perfmgr.outstanding_queries,
                        p_osm->perfmgr.max_outstanding_queries);
-               for (item = cl_qlist_head(&p_osm->plugin_list);
-                    item != cl_qlist_end(&p_osm->plugin_list);
-                    item = cl_qlist_next(item))
-                       fprintf(out, " %s",
-                               ((osm_epi_plugin_t *)item)->plugin_name);
-               fprintf(out, "\n");
        }
 }
 #endif                         /* ENABLE_OSM_PERF_MGR */
-- 
1.5.4.5

_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to