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.

-- 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

Reply via email to