On Wed, 18 Aug 2010 14:58:44 -0700 Yevgeny Kliteynik <klit...@gmail.com> wrote:
> On 18-Aug-10 7:06 PM, Ira Weiny wrote: > > On Wed, 18 Aug 2010 06:38:14 -0700 > > Yevgeny Kliteynik<klit...@dev.mellanox.co.il> wrote: > > > >> Don't bother reporting events to plug-ins while SM is exiting. > > > > Does this fix a bug? > > Don't think so (at least not that I'm aware of it). > For me, the issue here is optimization. Does this delay exit? Is that the optimization. I wonder if plugins would want to know the SM is shutting down. But there is a "delete" function... ? Ira > > -- Yevgeny > > > Ira > > > >> > >> Signed-off-by: Yevgeny Kliteynik<klit...@dev.mellanox.co.il> > >> --- > >> opensm/opensm/osm_opensm.c | 2 +- > >> 1 files changed, 1 insertions(+), 1 deletions(-) > >> > >> diff --git a/opensm/opensm/osm_opensm.c b/opensm/opensm/osm_opensm.c > >> index d3dc02e..50e8763 100644 > >> --- a/opensm/opensm/osm_opensm.c > >> +++ b/opensm/opensm/osm_opensm.c > >> @@ -475,7 +475,7 @@ void osm_opensm_report_event(osm_opensm_t *osm, > >> osm_epi_event_id_t event_id, > >> cl_list_item_t *item; > >> > >> for (item = cl_qlist_head(&osm->plugin_list); > >> - item != cl_qlist_end(&osm->plugin_list); > >> + (!osm_exit_flag)&& (item != cl_qlist_end(&osm->plugin_list)); > >> item = cl_qlist_next(item)) { > >> osm_epi_plugin_t *p = (osm_epi_plugin_t *)item; > >> if (p->impl->report) > >> -- > >> 1.6.2.4 > >> > >> -- > >> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > >> the body of a message to majord...@vger.kernel.org > >> More majordomo info at http://**vger.kernel.org/majordomo-info.html > >> > > > > > > -- Ira Weiny Math Programmer/Computer Scientist Lawrence Livermore National Lab 925-423-8008 wei...@llnl.gov -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html