[rds-devel] [PATCH v3 net-next 3/3] rds: Extend RDS API for IPv6 support

Santosh Shilimkar santosh.shilimkar at oracle.com
Fri Jul 13 15:00:59 PDT 2018


Hi Dave,

On 7/13/2018 2:25 PM, David Miller wrote:
> From: Ka-Cheong Poon <ka-cheong.poon at oracle.com>
> Date: Fri, 13 Jul 2018 04:02:59 -0700
> 
>> @@ -52,7 +52,7 @@
>>   #define RDS_RECVERR			5
>>   #define RDS_CONG_MONITOR		6
>>   #define RDS_GET_MR_FOR_DEST		7
>> -#define SO_RDS_TRANSPORT		8
>> +#define SO_RDS_TRANSPORT		9
> 
> There is no way you can change this value without breaking
> applications.
> 
Downstream Oracle shipping application have been built
with 9 as a SO_RDS_TRANSPORT from beginning. 8 is used
for RDS_CONN_RESET but support for it doesn't exist
upstream yet. Ka-cheong was aligning them so that we
have same number upstream as well as in shipping products.

Ofcourse any application built using upstream header and
using SO_RDS_TRANSPORT will break but since this particular
option was added for special case(application wants to
upfront select transport instead letting bind figure it out),
our hope its not used by other application(s).

Regards,
Santosh



More information about the rds-devel mailing list