[openib-general] RE: [PATCH] OpenSM: SA SMInfoRecord should support GetTable as well as Get method

[Yael Kalka]

The GetTable can be accepted. 
As I said - currently it doesn't mean anything.

As for the list - the client re-registration issue can be added. 
I can't think of anything else right now, I will let you know when
I have something to add there.

Yael

-----Original Message-----
From: Hal Rosenstock [mailto:halr at voltaire.com]
Sent: Wednesday, December 07, 2005 2:15 PM
To: Yael Kalka
Cc: openib-general at openib.org
Subject: RE: [PATCH] OpenSM: SA SMInfoRecord should support GetTable as
well as Get method


Hi Yael,

On Wed, 2005-12-07 at 06:59, Yael Kalka wrote:
> Hi Hal,
> If you look at the code - currently the What is returned is a record
> with the 
> local SMInfo record.
> The code should be fixed to return a table, or a requested SMInfo
record
> - 
> not only of the local port.
> So currently - a table is not returned, and the code isn't correct
with
> or
> without the patch...
> This issue should be added to our to-do list.

Are you saying that because of that the GetTable should not be accepted
until this is fixed or is it a separate issue ?

There is more to do here as you point out and I will track this on the
TODO list. Do you have other things for this list (see
management/osm/doc/todo) ?

-- Hal

> Yael 
> 
> -----Original Message-----
> From: Hal Rosenstock [mailto:halr at voltaire.com]
> Sent: Tuesday, December 06, 2005 4:06 PM
> To: Yael Kalka
> Cc: openib-general at openib.org
> Subject: [PATCH] OpenSM: SA SMInfoRecord should support GetTable as
well
> as Get method
> 
> 
> OpenSM: SA SMInfoRecord should support GetTable as well as Get method
> 
> Signed-off-by: Hal Rosenstock <halr at voltaire.com>
> 
> Index: osm_sa_sminfo_record.c
> ===================================================================
> --- osm_sa_sminfo_record.c      (revision 4323)
> +++ osm_sa_sminfo_record.c      (working copy)
> @@ -165,7 +165,8 @@ osm_smir_rcv_process(
>  
>    CL_ASSERT( p_sa_mad->attr_id == IB_MAD_ATTR_SMINFO_RECORD );
>  
> -  if (p_sa_mad->method != IB_MAD_METHOD_GET)
> +  if ( (p_sa_mad->method != IB_MAD_METHOD_GET) &&
> +       (p_sa_mad->method != IB_MAD_METHOD_GETTABLE) )
>    {
>      osm_log( p_rcv->p_log, OSM_LOG_ERROR,
>               "osm_smir_rcv_process: ERR 2804: "