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
