[ofa-general] [PATCH 11/10] libibmad:infiniband-diags: deprecate madrpc_set_[retries|timeout] WAS: [PATCH 1/10] libibmad: Clean up "new" interface

[Sasha Khapyorsky]

On 11:11 Mon 02 Mar     , Ira Weiny wrote:
> 
> I see this as well.  Right now libibmad is designed around the "run and exit"
> diag model but we are moving it toward a library which can be used in more
> complex applications.  We should push to do this once and as correct as
> possible.

We can support both models, but we shouldn't complicate existing one, so
enforcing all existing utils to call explicitly mad_rpc_set_timeout()
is not a good idea IMO.

BTW, this patch now breaks most of infiniband-diags tools when timeout is
not specified in command line explicitly with '-t'. The default used is
ibd_timeout, which is '0' and umad_send() fails with 'Resource
temporarily unavailable'.

Sasha