On 19-Aug-10 3:59 AM, Ira Weiny wrote:
> 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.
Yes.
When SM catches SIGTERM, it marks osm_exit_flag.
However, it doesn't mean that SM will stop doing
whatever it's doing and start exiting - if SM is
in the middle of calling plug-ins, it will
continue calling them.
> I wonder if plugins would want to know the SM is shutting down. But there is
> a "delete" function... ?
The "delete" function allows plug-ins to know when
SM is exiting (if they want to know it).
-- Yevgeny
> 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
>>>>
>>>
>>>
>>
>>
>
>
--
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
