Commit 3fd091e7 authored by Vladislav Yasevich's avatar Vladislav Yasevich Committed by David S. Miller

[SCTP]: Remove multiple levels of msecs to jiffies conversions.

The SCTP sysctl entries are displayed in milliseconds, but stored
internally in jiffies. This results in multiple levels of msecs to
jiffies conversion and as a result produces a truncation error. This
patch makes things consistent in that we store and display defaults
in milliseconds and only convert once for use by association.
This patch also adds some sane min/max values so that we don't go off
the deep end.
Signed-off-by: default avatarVladislav Yasevich <vladislav.yasevich@hp.com>
Signed-off-by: default avatarSridhar Samudrala <sri@us.ibm.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ce556b3a
...@@ -264,10 +264,10 @@ enum { SCTP_MAX_DUP_TSNS = 16 }; ...@@ -264,10 +264,10 @@ enum { SCTP_MAX_DUP_TSNS = 16 };
enum { SCTP_MAX_GABS = 16 }; enum { SCTP_MAX_GABS = 16 };
/* Heartbeat interval - 30 secs */ /* Heartbeat interval - 30 secs */
#define SCTP_DEFAULT_TIMEOUT_HEARTBEAT (30 * HZ) #define SCTP_DEFAULT_TIMEOUT_HEARTBEAT (30*1000)
/* Delayed sack timer - 200ms */ /* Delayed sack timer - 200ms */
#define SCTP_DEFAULT_TIMEOUT_SACK ((200 * HZ) / 1000) #define SCTP_DEFAULT_TIMEOUT_SACK (200)
/* RTO.Initial - 3 seconds /* RTO.Initial - 3 seconds
* RTO.Min - 1 second * RTO.Min - 1 second
...@@ -275,9 +275,9 @@ enum { SCTP_MAX_GABS = 16 }; ...@@ -275,9 +275,9 @@ enum { SCTP_MAX_GABS = 16 };
* RTO.Alpha - 1/8 * RTO.Alpha - 1/8
* RTO.Beta - 1/4 * RTO.Beta - 1/4
*/ */
#define SCTP_RTO_INITIAL (3 * HZ) #define SCTP_RTO_INITIAL (3 * 1000)
#define SCTP_RTO_MIN (1 * HZ) #define SCTP_RTO_MIN (1 * 1000)
#define SCTP_RTO_MAX (60 * HZ) #define SCTP_RTO_MAX (60 * 1000)
#define SCTP_RTO_ALPHA 3 /* 1/8 when converted to right shifts. */ #define SCTP_RTO_ALPHA 3 /* 1/8 when converted to right shifts. */
#define SCTP_RTO_BETA 2 /* 1/4 when converted to right shifts. */ #define SCTP_RTO_BETA 2 /* 1/4 when converted to right shifts. */
...@@ -290,8 +290,7 @@ enum { SCTP_MAX_GABS = 16 }; ...@@ -290,8 +290,7 @@ enum { SCTP_MAX_GABS = 16 };
#define SCTP_DEF_MAX_INIT 6 #define SCTP_DEF_MAX_INIT 6
#define SCTP_DEF_MAX_SEND 10 #define SCTP_DEF_MAX_SEND 10
#define SCTP_DEFAULT_COOKIE_LIFE_SEC 60 /* seconds */ #define SCTP_DEFAULT_COOKIE_LIFE (60 * 1000) /* 60 seconds */
#define SCTP_DEFAULT_COOKIE_LIFE_USEC 0 /* microseconds */
#define SCTP_DEFAULT_MINWINDOW 1500 /* default minimum rwnd size */ #define SCTP_DEFAULT_MINWINDOW 1500 /* default minimum rwnd size */
#define SCTP_DEFAULT_MAXWINDOW 65535 /* default rwnd size */ #define SCTP_DEFAULT_MAXWINDOW 65535 /* default rwnd size */
......
...@@ -128,9 +128,9 @@ extern struct sctp_globals { ...@@ -128,9 +128,9 @@ extern struct sctp_globals {
* RTO.Alpha - 1/8 (3 when converted to right shifts.) * RTO.Alpha - 1/8 (3 when converted to right shifts.)
* RTO.Beta - 1/4 (2 when converted to right shifts.) * RTO.Beta - 1/4 (2 when converted to right shifts.)
*/ */
unsigned long rto_initial; unsigned int rto_initial;
unsigned long rto_min; unsigned int rto_min;
unsigned long rto_max; unsigned int rto_max;
/* Note: rto_alpha and rto_beta are really defined as inverse /* Note: rto_alpha and rto_beta are really defined as inverse
* powers of two to facilitate integer operations. * powers of two to facilitate integer operations.
...@@ -145,13 +145,13 @@ extern struct sctp_globals { ...@@ -145,13 +145,13 @@ extern struct sctp_globals {
int cookie_preserve_enable; int cookie_preserve_enable;
/* Valid.Cookie.Life - 60 seconds */ /* Valid.Cookie.Life - 60 seconds */
unsigned long valid_cookie_life; unsigned int valid_cookie_life;
/* Delayed SACK timeout 200ms default*/ /* Delayed SACK timeout 200ms default*/
unsigned long sack_timeout; unsigned int sack_timeout;
/* HB.interval - 30 seconds */ /* HB.interval - 30 seconds */
unsigned long hb_interval; unsigned int hb_interval;
/* Association.Max.Retrans - 10 attempts /* Association.Max.Retrans - 10 attempts
* Path.Max.Retrans - 5 attempts (per destination address) * Path.Max.Retrans - 5 attempts (per destination address)
......
...@@ -1049,7 +1049,7 @@ SCTP_STATIC __init int sctp_init(void) ...@@ -1049,7 +1049,7 @@ SCTP_STATIC __init int sctp_init(void)
sctp_rto_beta = SCTP_RTO_BETA; sctp_rto_beta = SCTP_RTO_BETA;
/* Valid.Cookie.Life - 60 seconds */ /* Valid.Cookie.Life - 60 seconds */
sctp_valid_cookie_life = 60 * HZ; sctp_valid_cookie_life = SCTP_DEFAULT_COOKIE_LIFE;
/* Whether Cookie Preservative is enabled(1) or not(0) */ /* Whether Cookie Preservative is enabled(1) or not(0) */
sctp_cookie_preserve_enable = 1; sctp_cookie_preserve_enable = 1;
......
...@@ -3045,14 +3045,14 @@ SCTP_STATIC int sctp_init_sock(struct sock *sk) ...@@ -3045,14 +3045,14 @@ SCTP_STATIC int sctp_init_sock(struct sock *sk)
sp->initmsg.sinit_num_ostreams = sctp_max_outstreams; sp->initmsg.sinit_num_ostreams = sctp_max_outstreams;
sp->initmsg.sinit_max_instreams = sctp_max_instreams; sp->initmsg.sinit_max_instreams = sctp_max_instreams;
sp->initmsg.sinit_max_attempts = sctp_max_retrans_init; sp->initmsg.sinit_max_attempts = sctp_max_retrans_init;
sp->initmsg.sinit_max_init_timeo = jiffies_to_msecs(sctp_rto_max); sp->initmsg.sinit_max_init_timeo = sctp_rto_max;
/* Initialize default RTO related parameters. These parameters can /* Initialize default RTO related parameters. These parameters can
* be modified for with the SCTP_RTOINFO socket option. * be modified for with the SCTP_RTOINFO socket option.
*/ */
sp->rtoinfo.srto_initial = jiffies_to_msecs(sctp_rto_initial); sp->rtoinfo.srto_initial = sctp_rto_initial;
sp->rtoinfo.srto_max = jiffies_to_msecs(sctp_rto_max); sp->rtoinfo.srto_max = sctp_rto_max;
sp->rtoinfo.srto_min = jiffies_to_msecs(sctp_rto_min); sp->rtoinfo.srto_min = sctp_rto_min;
/* Initialize default association related parameters. These parameters /* Initialize default association related parameters. These parameters
* can be modified with the SCTP_ASSOCINFO socket option. * can be modified with the SCTP_ASSOCINFO socket option.
...@@ -3061,8 +3061,7 @@ SCTP_STATIC int sctp_init_sock(struct sock *sk) ...@@ -3061,8 +3061,7 @@ SCTP_STATIC int sctp_init_sock(struct sock *sk)
sp->assocparams.sasoc_number_peer_destinations = 0; sp->assocparams.sasoc_number_peer_destinations = 0;
sp->assocparams.sasoc_peer_rwnd = 0; sp->assocparams.sasoc_peer_rwnd = 0;
sp->assocparams.sasoc_local_rwnd = 0; sp->assocparams.sasoc_local_rwnd = 0;
sp->assocparams.sasoc_cookie_life = sp->assocparams.sasoc_cookie_life = sctp_valid_cookie_life;
jiffies_to_msecs(sctp_valid_cookie_life);
/* Initialize default event subscriptions. By default, all the /* Initialize default event subscriptions. By default, all the
* options are off. * options are off.
...@@ -3072,10 +3071,10 @@ SCTP_STATIC int sctp_init_sock(struct sock *sk) ...@@ -3072,10 +3071,10 @@ SCTP_STATIC int sctp_init_sock(struct sock *sk)
/* Default Peer Address Parameters. These defaults can /* Default Peer Address Parameters. These defaults can
* be modified via SCTP_PEER_ADDR_PARAMS * be modified via SCTP_PEER_ADDR_PARAMS
*/ */
sp->hbinterval = jiffies_to_msecs(sctp_hb_interval); sp->hbinterval = sctp_hb_interval;
sp->pathmaxrxt = sctp_max_retrans_path; sp->pathmaxrxt = sctp_max_retrans_path;
sp->pathmtu = 0; // allow default discovery sp->pathmtu = 0; // allow default discovery
sp->sackdelay = jiffies_to_msecs(sctp_sack_timeout); sp->sackdelay = sctp_sack_timeout;
sp->param_flags = SPP_HB_ENABLE | sp->param_flags = SPP_HB_ENABLE |
SPP_PMTUD_ENABLE | SPP_PMTUD_ENABLE |
SPP_SACKDELAY_ENABLE; SPP_SACKDELAY_ENABLE;
......
...@@ -45,9 +45,10 @@ ...@@ -45,9 +45,10 @@
#include <net/sctp/sctp.h> #include <net/sctp/sctp.h>
#include <linux/sysctl.h> #include <linux/sysctl.h>
static ctl_handler sctp_sysctl_jiffies_ms; static int zero = 0;
static long rto_timer_min = 1; static int one = 1;
static long rto_timer_max = 86400000; /* One day */ static int timer_max = 86400000; /* ms in one day */
static int int_max = INT_MAX;
static long sack_timer_min = 1; static long sack_timer_min = 1;
static long sack_timer_max = 500; static long sack_timer_max = 500;
...@@ -56,45 +57,45 @@ static ctl_table sctp_table[] = { ...@@ -56,45 +57,45 @@ static ctl_table sctp_table[] = {
.ctl_name = NET_SCTP_RTO_INITIAL, .ctl_name = NET_SCTP_RTO_INITIAL,
.procname = "rto_initial", .procname = "rto_initial",
.data = &sctp_rto_initial, .data = &sctp_rto_initial,
.maxlen = sizeof(long), .maxlen = sizeof(unsigned int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_doulongvec_ms_jiffies_minmax, .proc_handler = &proc_dointvec_minmax,
.strategy = &sctp_sysctl_jiffies_ms, .strategy = &sysctl_intvec,
.extra1 = &rto_timer_min, .extra1 = &one,
.extra2 = &rto_timer_max .extra2 = &timer_max
}, },
{ {
.ctl_name = NET_SCTP_RTO_MIN, .ctl_name = NET_SCTP_RTO_MIN,
.procname = "rto_min", .procname = "rto_min",
.data = &sctp_rto_min, .data = &sctp_rto_min,
.maxlen = sizeof(long), .maxlen = sizeof(unsigned int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_doulongvec_ms_jiffies_minmax, .proc_handler = &proc_dointvec_minmax,
.strategy = &sctp_sysctl_jiffies_ms, .strategy = &sysctl_intvec,
.extra1 = &rto_timer_min, .extra1 = &one,
.extra2 = &rto_timer_max .extra2 = &timer_max
}, },
{ {
.ctl_name = NET_SCTP_RTO_MAX, .ctl_name = NET_SCTP_RTO_MAX,
.procname = "rto_max", .procname = "rto_max",
.data = &sctp_rto_max, .data = &sctp_rto_max,
.maxlen = sizeof(long), .maxlen = sizeof(unsigned int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_doulongvec_ms_jiffies_minmax, .proc_handler = &proc_dointvec_minmax,
.strategy = &sctp_sysctl_jiffies_ms, .strategy = &sysctl_intvec,
.extra1 = &rto_timer_min, .extra1 = &one,
.extra2 = &rto_timer_max .extra2 = &timer_max
}, },
{ {
.ctl_name = NET_SCTP_VALID_COOKIE_LIFE, .ctl_name = NET_SCTP_VALID_COOKIE_LIFE,
.procname = "valid_cookie_life", .procname = "valid_cookie_life",
.data = &sctp_valid_cookie_life, .data = &sctp_valid_cookie_life,
.maxlen = sizeof(long), .maxlen = sizeof(unsigned int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_doulongvec_ms_jiffies_minmax, .proc_handler = &proc_dointvec_minmax,
.strategy = &sctp_sysctl_jiffies_ms, .strategy = &sysctl_intvec,
.extra1 = &rto_timer_min, .extra1 = &one,
.extra2 = &rto_timer_max .extra2 = &timer_max
}, },
{ {
.ctl_name = NET_SCTP_MAX_BURST, .ctl_name = NET_SCTP_MAX_BURST,
...@@ -102,7 +103,10 @@ static ctl_table sctp_table[] = { ...@@ -102,7 +103,10 @@ static ctl_table sctp_table[] = {
.data = &sctp_max_burst, .data = &sctp_max_burst,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec_minmax,
.strategy = &sysctl_intvec,
.extra1 = &zero,
.extra2 = &int_max
}, },
{ {
.ctl_name = NET_SCTP_ASSOCIATION_MAX_RETRANS, .ctl_name = NET_SCTP_ASSOCIATION_MAX_RETRANS,
...@@ -110,7 +114,10 @@ static ctl_table sctp_table[] = { ...@@ -110,7 +114,10 @@ static ctl_table sctp_table[] = {
.data = &sctp_max_retrans_association, .data = &sctp_max_retrans_association,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec_minmax,
.strategy = &sysctl_intvec,
.extra1 = &one,
.extra2 = &int_max
}, },
{ {
.ctl_name = NET_SCTP_SNDBUF_POLICY, .ctl_name = NET_SCTP_SNDBUF_POLICY,
...@@ -118,7 +125,8 @@ static ctl_table sctp_table[] = { ...@@ -118,7 +125,8 @@ static ctl_table sctp_table[] = {
.data = &sctp_sndbuf_policy, .data = &sctp_sndbuf_policy,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec,
.strategy = &sysctl_intvec
}, },
{ {
.ctl_name = NET_SCTP_RCVBUF_POLICY, .ctl_name = NET_SCTP_RCVBUF_POLICY,
...@@ -126,7 +134,8 @@ static ctl_table sctp_table[] = { ...@@ -126,7 +134,8 @@ static ctl_table sctp_table[] = {
.data = &sctp_rcvbuf_policy, .data = &sctp_rcvbuf_policy,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec,
.strategy = &sysctl_intvec
}, },
{ {
.ctl_name = NET_SCTP_PATH_MAX_RETRANS, .ctl_name = NET_SCTP_PATH_MAX_RETRANS,
...@@ -134,7 +143,10 @@ static ctl_table sctp_table[] = { ...@@ -134,7 +143,10 @@ static ctl_table sctp_table[] = {
.data = &sctp_max_retrans_path, .data = &sctp_max_retrans_path,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec_minmax,
.strategy = &sysctl_intvec,
.extra1 = &one,
.extra2 = &int_max
}, },
{ {
.ctl_name = NET_SCTP_MAX_INIT_RETRANSMITS, .ctl_name = NET_SCTP_MAX_INIT_RETRANSMITS,
...@@ -142,18 +154,21 @@ static ctl_table sctp_table[] = { ...@@ -142,18 +154,21 @@ static ctl_table sctp_table[] = {
.data = &sctp_max_retrans_init, .data = &sctp_max_retrans_init,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec_minmax,
.strategy = &sysctl_intvec,
.extra1 = &one,
.extra2 = &int_max
}, },
{ {
.ctl_name = NET_SCTP_HB_INTERVAL, .ctl_name = NET_SCTP_HB_INTERVAL,
.procname = "hb_interval", .procname = "hb_interval",
.data = &sctp_hb_interval, .data = &sctp_hb_interval,
.maxlen = sizeof(long), .maxlen = sizeof(unsigned int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_doulongvec_ms_jiffies_minmax, .proc_handler = &proc_dointvec_minmax,
.strategy = &sctp_sysctl_jiffies_ms, .strategy = &sysctl_intvec,
.extra1 = &rto_timer_min, .extra1 = &one,
.extra2 = &rto_timer_max .extra2 = &timer_max
}, },
{ {
.ctl_name = NET_SCTP_PRESERVE_ENABLE, .ctl_name = NET_SCTP_PRESERVE_ENABLE,
...@@ -161,23 +176,26 @@ static ctl_table sctp_table[] = { ...@@ -161,23 +176,26 @@ static ctl_table sctp_table[] = {
.data = &sctp_cookie_preserve_enable, .data = &sctp_cookie_preserve_enable,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec,
.strategy = &sysctl_intvec
}, },
{ {
.ctl_name = NET_SCTP_RTO_ALPHA, .ctl_name = NET_SCTP_RTO_ALPHA,
.procname = "rto_alpha_exp_divisor", .procname = "rto_alpha_exp_divisor",
.data = &sctp_rto_alpha, .data = &sctp_rto_alpha,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0444,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec,
.strategy = &sysctl_intvec
}, },
{ {
.ctl_name = NET_SCTP_RTO_BETA, .ctl_name = NET_SCTP_RTO_BETA,
.procname = "rto_beta_exp_divisor", .procname = "rto_beta_exp_divisor",
.data = &sctp_rto_beta, .data = &sctp_rto_beta,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0444,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec,
.strategy = &sysctl_intvec
}, },
{ {
.ctl_name = NET_SCTP_ADDIP_ENABLE, .ctl_name = NET_SCTP_ADDIP_ENABLE,
...@@ -185,7 +203,8 @@ static ctl_table sctp_table[] = { ...@@ -185,7 +203,8 @@ static ctl_table sctp_table[] = {
.data = &sctp_addip_enable, .data = &sctp_addip_enable,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec,
.strategy = &sysctl_intvec
}, },
{ {
.ctl_name = NET_SCTP_PRSCTP_ENABLE, .ctl_name = NET_SCTP_PRSCTP_ENABLE,
...@@ -193,7 +212,8 @@ static ctl_table sctp_table[] = { ...@@ -193,7 +212,8 @@ static ctl_table sctp_table[] = {
.data = &sctp_prsctp_enable, .data = &sctp_prsctp_enable,
.maxlen = sizeof(int), .maxlen = sizeof(int),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_dointvec .proc_handler = &proc_dointvec,
.strategy = &sysctl_intvec
}, },
{ {
.ctl_name = NET_SCTP_SACK_TIMEOUT, .ctl_name = NET_SCTP_SACK_TIMEOUT,
...@@ -201,8 +221,8 @@ static ctl_table sctp_table[] = { ...@@ -201,8 +221,8 @@ static ctl_table sctp_table[] = {
.data = &sctp_sack_timeout, .data = &sctp_sack_timeout,
.maxlen = sizeof(long), .maxlen = sizeof(long),
.mode = 0644, .mode = 0644,
.proc_handler = &proc_doulongvec_ms_jiffies_minmax, .proc_handler = &proc_dointvec_minmax,
.strategy = &sctp_sysctl_jiffies_ms, .strategy = &sysctl_intvec,
.extra1 = &sack_timer_min, .extra1 = &sack_timer_min,
.extra2 = &sack_timer_max, .extra2 = &sack_timer_max,
}, },
...@@ -242,37 +262,3 @@ void sctp_sysctl_unregister(void) ...@@ -242,37 +262,3 @@ void sctp_sysctl_unregister(void)
{ {
unregister_sysctl_table(sctp_sysctl_header); unregister_sysctl_table(sctp_sysctl_header);
} }
/* Strategy function to convert jiffies to milliseconds. */
static int sctp_sysctl_jiffies_ms(ctl_table *table, int __user *name, int nlen,
void __user *oldval, size_t __user *oldlenp,
void __user *newval, size_t newlen, void **context) {
if (oldval) {
size_t olen;
if (oldlenp) {
if (get_user(olen, oldlenp))
return -EFAULT;
if (olen != sizeof (int))
return -EINVAL;
}
if (put_user((*(int *)(table->data) * 1000) / HZ,
(int __user *)oldval) ||
(oldlenp && put_user(sizeof (int), oldlenp)))
return -EFAULT;
}
if (newval && newlen) {
int new;
if (newlen != sizeof (int))
return -EINVAL;
if (get_user(new, (int __user *)newval))
return -EFAULT;
*(int *)(table->data) = (new * HZ) / 1000;
}
return 1;
}
...@@ -75,7 +75,7 @@ static struct sctp_transport *sctp_transport_init(struct sctp_transport *peer, ...@@ -75,7 +75,7 @@ static struct sctp_transport *sctp_transport_init(struct sctp_transport *peer,
* parameter 'RTO.Initial'. * parameter 'RTO.Initial'.
*/ */
peer->rtt = 0; peer->rtt = 0;
peer->rto = sctp_rto_initial; peer->rto = msecs_to_jiffies(sctp_rto_initial);
peer->rttvar = 0; peer->rttvar = 0;
peer->srtt = 0; peer->srtt = 0;
peer->rto_pending = 0; peer->rto_pending = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment