Merge branch 'sctp_logspam'

Neil Horman says:

====================
sctp: Consolidate and ratelimit deprecation warnings

The SCTP protocol has several deprecation warnings in its setsockopt path that
can be triggered by unprivlidged users.  Since these are not ratelimited, we can
spam the logs quite easily here.  Since these are all deprecation warnings, and
that type of warning isn't uncommon in the rest of the kernel, lets make a
common pr_warn_deprecated macro to produce somewhat generalized ratelimited
deprecation warnings easily
====================

Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

2 files changed, 25 insertions, 12 deletions

diff --git a/include/linux/printk.h b/include/linux/printk.h
index 694925837a16..26fb95ce5080 100644
--- a/include/linux/printk.h
+++ b/include/linux/printk.h
@@ -88,6 +88,13 @@ struct va_format {
 #define HW_ERR          "[Hardware Error]: "
 
 /*
+ * DEPRECATED
+ * Add this to a message whenever you want to warn user space about the use
+ * of a deprecated aspect of an API so they can stop using it
+ */
+#define DEPRECATED      "[Deprecated]: "
+
+/*
  * Dummy printk for disabled debugging statements to use whilst maintaining
  * gcc's format and side-effect checking.
  */
diff --git a/net/sctp/socket.c b/net/sctp/socket.c
index adc5e3779b29..e9c5121e0fbf 100644
--- a/net/sctp/socket.c
+++ b/net/sctp/socket.c
@@ -2578,8 +2578,9 @@ static int sctp_setsockopt_delayed_ack(struct sock *sk,
                 if (params.sack_delay == 0 && params.sack_freq == 0)
                         return 0;
         } else if (optlen == sizeof(struct sctp_assoc_value)) {
-                pr_warn("Use of struct sctp_assoc_value in delayed_ack socket option deprecated\n");
-                pr_warn("Use struct sctp_sack_info instead\n");
+                pr_warn_ratelimited(DEPRECATED
+                                    "Use of struct sctp_assoc_value in delayed_ack socket option.\n"
+                                    "Use struct sctp_sack_info instead\n");
                 if (copy_from_user(&params, optval, optlen))
                         return -EFAULT;
 
@@ -2994,8 +2995,9 @@ static int sctp_setsockopt_maxseg(struct sock *sk, char __user *optval, unsigned
         int val;
 
         if (optlen == sizeof(int)) {
-                pr_warn("Use of int in maxseg socket option deprecated\n");
-                pr_warn("Use struct sctp_assoc_value instead\n");
+                pr_warn_ratelimited(DEPRECATED
+                                    "Use of int in maxseg socket option.\n"
+                                    "Use struct sctp_assoc_value instead\n");
                 if (copy_from_user(&val, optval, optlen))
                         return -EFAULT;
                 params.assoc_id = 0;
@@ -3252,8 +3254,9 @@ static int sctp_setsockopt_maxburst(struct sock *sk,
         int assoc_id = 0;
 
         if (optlen == sizeof(int)) {
-                pr_warn("Use of int in max_burst socket option deprecated\n");
-                pr_warn("Use struct sctp_assoc_value instead\n");
+                pr_warn_ratelimited(DEPRECATED
+                                    "Use of int in max_burst socket option deprecated.\n"
+                                    "Use struct sctp_assoc_value instead\n");
                 if (copy_from_user(&val, optval, optlen))
                         return -EFAULT;
         } else if (optlen == sizeof(struct sctp_assoc_value)) {
@@ -4573,8 +4576,9 @@ static int sctp_getsockopt_delayed_ack(struct sock *sk, int len,
                 if (copy_from_user(&params, optval, len))
                         return -EFAULT;
         } else if (len == sizeof(struct sctp_assoc_value)) {
-                pr_warn("Use of struct sctp_assoc_value in delayed_ack socket option deprecated\n");
-                pr_warn("Use struct sctp_sack_info instead\n");
+                pr_warn_ratelimited(DEPRECATED
+                                    "Use of struct sctp_assoc_value in delayed_ack socket option.\n"
+                                    "Use struct sctp_sack_info instead\n");
                 if (copy_from_user(&params, optval, len))
                         return -EFAULT;
         } else
@@ -5218,8 +5222,9 @@ static int sctp_getsockopt_maxseg(struct sock *sk, int len,
         struct sctp_association *asoc;
 
         if (len == sizeof(int)) {
-                pr_warn("Use of int in maxseg socket option deprecated\n");
-                pr_warn("Use struct sctp_assoc_value instead\n");
+                pr_warn_ratelimited(DEPRECATED
+                                    "Use of int in maxseg socket option.\n"
+                                    "Use struct sctp_assoc_value instead\n");
                 params.assoc_id = 0;
         } else if (len >= sizeof(struct sctp_assoc_value)) {
                 len = sizeof(struct sctp_assoc_value);
@@ -5310,8 +5315,9 @@ static int sctp_getsockopt_maxburst(struct sock *sk, int len,
         struct sctp_association *asoc;
 
         if (len == sizeof(int)) {
-                pr_warn("Use of int in max_burst socket option deprecated\n");
-                pr_warn("Use struct sctp_assoc_value instead\n");
+                pr_warn_ratelimited(DEPRECATED
+                                    "Use of int in max_burst socket option.\n"
+                                    "Use struct sctp_assoc_value instead\n");
                 params.assoc_id = 0;
         } else if (len >= sizeof(struct sctp_assoc_value)) {
                 len = sizeof(struct sctp_assoc_value);