All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jamal Hadi Salim <jhs@mojatatu.com>
To: Daniel Borkmann <daniel@iogearbox.net>,
	Neil Horman <nhorman@tuxdriver.com>,
	Vlad Yasevich <vyasevic@redhat.com>
Cc: David Miller <davem@davemloft.net>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: [RFC net-next 1/1] : sctp: denoise deprecation log on SCTP_EVENTS
Date: Thu, 09 Jul 2015 06:38:15 -0400	[thread overview]
Message-ID: <559E4F17.9090107@mojatatu.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 642 bytes --]



In the newer kernels this message is extremely noisy. After a quick
discussion with Daniel it seems to me it will be very hard to get
existing apps that nobody is going to update to continue to work
(i.e no forward compat). And newer apps that desire to play in both
older kernels and new kernels will have to play some tricks to work
(i.e weak backward compatibility).  These are good reasons
to totally get rid of this message. At minimal to neutre it.
The attached change tries to do that. However, if you had multiple
apps, you will only get warning for the first one.

Will send proper patch when theres some consensus.

cheers,
jamal

[-- Attachment #2: sctp-denoise --]
[-- Type: text/plain, Size: 753 bytes --]

diff --git a/net/sctp/socket.c b/net/sctp/socket.c
index 5f6c4e6..0e8c937 100644
--- a/net/sctp/socket.c
+++ b/net/sctp/socket.c
@@ -2207,9 +2207,10 @@ static int sctp_setsockopt_events(struct sock *sk, char __user *optval,
 		return -EFAULT;
 
 	if (sctp_sk(sk)->subscribe.sctp_data_io_event)
-		pr_warn_ratelimited(DEPRECATED "%s (pid %d) "
-				    "Requested SCTP_SNDRCVINFO event.\n"
-				    "Use SCTP_RCVINFO through SCTP_RECVRCVINFO option instead.\n",
+		pr_warn_once(DEPRECATED "%s (pid %d) "
+				    "SCTP_EVENTS Option - DEPRECATED\n"
+				    "Use SCTP_EVENT Option instead\n"
+				    "Refer to RFC6458 section 6.2\n",
 				    current->comm, task_pid_nr(current));
 
 	/* At the time when a user app subscribes to SCTP_SENDER_DRY_EVENT,

             reply	other threads:[~2015-07-09 10:38 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-09 10:38 Jamal Hadi Salim [this message]
2015-07-09 11:54 ` [RFC net-next 1/1] : sctp: denoise deprecation log on SCTP_EVENTS Daniel Borkmann
2015-07-09 12:59   ` Marcelo Ricardo Leitner
2015-07-22 11:48 ` David Laight
2015-07-22 12:04   ` Daniel Borkmann
2015-07-22 12:30     ` Michael Tuexen
2015-07-22 14:36       ` Daniel Borkmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=559E4F17.9090107@mojatatu.com \
    --to=jhs@mojatatu.com \
    --cc=daniel@iogearbox.net \
    --cc=davem@davemloft.net \
    --cc=netdev@vger.kernel.org \
    --cc=nhorman@tuxdriver.com \
    --cc=vyasevic@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.