Querying the GID from a Provider Library

Querying the GID from a Provider Library

[Amrani, Ram]

Hi Doug, Yishay,
I need to translate sgid_index to sgid in the library to check for loopback.

While invoking ibv_query_gid does the job, it seems to me somewhat
inadequate that a provider library invokes an ibv function and not
an ibv_cmd function. What are your thoughts on this?

I would have invoked the  ibv_cmd_query_gid, but there is no such function
because ibv_query_gid does not require the kernel. However there's an old
*declaration* of the ibv_cmd_query_gid. Perhaps I will define the function
in a similar way to ibv_query_gid?!

Thanks,
Ram


--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: Querying the GID from a Provider Library

[Yishai Hadas]

On 2/1/2017 2:26 PM, Amrani, Ram wrote:
> Hi Doug, Yishay,
> I need to translate sgid_index to sgid in the library to check for loopback.
>
> While invoking ibv_query_gid does the job, it seems to me somewhat
> inadequate that a provider library invokes an ibv function and not
> an ibv_cmd function. What are your thoughts on this?
>
> I would have invoked the  ibv_cmd_query_gid, but there is no such function
> because ibv_query_gid does not require the kernel. However there's an old
> *declaration* of the ibv_cmd_query_gid. Perhaps I will define the function
> in a similar way to ibv_query_gid?!

ibv_query_gid is used all around by vendors, it serves as some helper 
function which is exported from libibverbs, you can just use it.

Re 'ibv_cmd_query_gid' looks like some 'dead' code that should be delted.


--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: Querying the GID from a Provider Library

[Jason Gunthorpe]

On Wed, Feb 01, 2017 at 06:11:07PM +0200, Yishai Hadas wrote:

> Re 'ibv_cmd_query_gid' looks like some 'dead' code that should be delted.

Yes, please send a patch..

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

RE: Querying the GID from a Provider Library

[Amrani, Ram]

> ibv_query_gid is used all around by vendors, it serves as some helper
> function which is exported from libibverbs, you can just use it.

OK, thanks.
 
> Re 'ibv_cmd_query_gid' looks like some 'dead' code that should be delted.

I'll send a patch on Sunday.

Thanks,
Ram


--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

RE: Querying the GID from a Provider Library

[Amrani, Ram]

> 
> Yes, please send a patch..
> 

Sure

Ram.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html