[openib-general] Re: opensm and SIGINT

[Eitan Zahavi]

Hal Rosenstock wrote:
> Hi Eitan,
> 
> On Fri, 2005-09-23 at 12:19, Eitan Zahavi wrote:
> 
>>Hi Hal, Viswa,
>>
>>Sorry I'm joining late on this thread due to the weekend (which starts
> 
> 
>>here on Friday ending Saturday night).
>>Is there any conclusion on this one?
> 
> 
> No.
> 
> 
>>The only log I have seen was from osmtest failing to send a MAD.
> 
> 
> True.
> 
> 
>>Looks like a umad issue?
> 
> 
> Not sure why you say that. There are other possibilities I'm aware of
> here:
> 
> Note that that failed sent MAD is one which has a response expected so
> this means that the response was not received. It also goes through the
> transmit retry strategy (I could see this on the SA side). So the only
> thing I can say at this point is that for some reason, the response does
> not make it back from the SA to the SA client (osmtest). That's where
> this one is right now.
Thanks for the update.

> 
> -- Hal
> 
> 
>>Eitan
>>
>>Hal Rosenstock wrote:
>>
>>>Hi again Viswa,
>>>
>>>On Wed, 2005-09-21 at 21:00, Hal Rosenstock wrote:
>>>
>>>
>>>>Hi Viswa,
>>>>
>>>>On Wed, 2005-09-21 at 20:23, Viswanath Krishnamurthy wrote:
>>>>
>>>>
>>>>>Currently opensm traps SIGINT. There was some discussion to remove
>>>
>>>it.
>>>
>>>
>>>>>I have currently running some tests on opensm
>>>>>by killing (SIGKILL) and restarting opensm. So far I ahve not found
>>>>>any resource leak issues. Is ther a plan to remove that
>>>>>signal handler. Ideally it should not exist.
>>>>
>>>>Eitan stated that this was historical in nature for gen1 drivers
> 
> which
> 
>>>>had resource tracking problems: "if OpenSM left without cleaning up
>>>
>>>all
>>>
>>>
>>>>used resources (like MAD buffers and UD-AVs), the driver oops'ed."
>>>>
>>>>I think that (eliminating the handler for SIGINT) can at least be
> 
> done
> 
>>>>for OSM_VENDOR_INTF_OPENIB and leave it there for the other vendor
>>>>layers for starters. I will experiment with gen2 and let you know.
>>>
>>>
>>>Does the patch below do what you want ? Can you try it ?
>>>
>>>-- Hal
>>>
>>>Index: opensm/osm_opensm.c
>>>===================================================================
>>>--- opensm/osm_opensm.c (revision 3513)
>>>+++ opensm/osm_opensm.c (working copy)
>>>@@ -182,7 +182,9 @@ osm_reg_sig_handler(
>>>    IN osm_opensm_t * const p_osm )
>>> {
>>>    __p_osm_to_signal = p_osm;
>>>+#ifndef OSM_VENDOR_INTF_OPENIB 
>>>    cl_reg_sig_hdl( SIGINT, __sig_handler );
>>>+#endif
>>>    cl_reg_sig_hdl( SIGTERM, __sig_handler );
>>>    cl_reg_sig_hdl( SIGHUP, __sig_handler );
>>>    osm_exit_flag = 0;
>>>
>>>
>>>_______________________________________________
>>>openib-general mailing list
>>>openib-general at openib.org
>>>http://openib.org/mailman/listinfo/openib-general
>>>
>>>To unsubscribe, please visit
>>>http://openib.org/mailman/listinfo/openib-general
>>>
>>
>