[openib-general] RE: SDP and Socket Options

[Majumder, Rajib]

there's a spelling mistake in sdp_inet_create. 

conn = sdp_conn_alloc(GFP_KERNEL);
	if (!conn) {
		sdp_dbg_warn(conn, "SOCKET: failed to create socekt <%d:%d>",
			     sock->type, protocol);
		return -ENOMEM;
	}

the "socekt" in above sdp_dbg_warn should be changed to "socket".

thanks.

rajib

-----Original Message-----
From: Michael S. Tsirkin [mailto:mst at mellanox.co.il]
Sent: 30 August 2005 15:22
To: Majumder, Rajib
Cc: openib-general at openib.org
Subject: Re: SDP and Socket Options


Quoting Majumder, Rajib <rajib.majumder at csfb.com>:
> Subject: SDP and Socket Options
> 
> hello, 
> i am trying to figure out, as of now, what socket stuff are NOT supported by SDP. this will help me identify the gaps between the 2. 

When you say SDP, I assume you are talking about creating a socket
with the AF_INET_SDP family.

> can you point me toward some docs that may describe these gaps? 

You'll have to look at the source. Get it here:
https://openib.org/svn/gen2/trunk/src/linux-kernel/infiniband/ulp/sdp/

Its actually not that big: around 16000 lines (including comments).
sdp_inet.c is a good starting point - thats where we register
the new protocol family. That file is 1461 lines.

> 1) what socket system calls are not supported? 

I think all of them work at this point.

> 2) what socket options are not supported? 

IPv6 addressing is probably the biggest omission.
This actually shouldnt be hard to fix, given enough interest.

I didnt look into this in depth yet, but I think SO_{RCV,SND}BUF
just set sk_rcvbuf/sk_sendbuf in the socket structure, which
then isnt used in any way.

> 3) other stuff - keepalives etc 
> thanks. 
> rajib 

Keepalives is the only missing point that I know about.

-- 
MST

==============================================================================
Please access the attached hyperlink for an important electronic communications disclaimer: 

http://www.csfb.com/legal_terms/disclaimer_external_email.shtml

==============================================================================