linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Rientjes <rientjes@google.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Steven Whitehouse <swhiteho@redhat.com>,
	Christine Caulfield <ccaulfie@redhat.com>,
	David Teigland <teigland@redhat.com>,
	cluster-devel@redhat.com,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Michal Marek <mmarek@suse.cz>, Kees Cook <keescook@chromium.org>,
	"David S. Miller" <davem@davemloft.net>,
	Vlad Yasevich <vyasevich@gmail.com>
Subject: [patch for-3.8] net, sctp: remove CONFIG_EXPERIMENTAL
Date: Tue, 12 Feb 2013 16:24:56 -0800 (PST)	[thread overview]
Message-ID: <alpine.DEB.2.02.1302121619340.3805@chino.kir.corp.google.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1302121207250.25566@chino.kir.corp.google.com>

From: Kees Cook <keescook@chromium.org>

This config item has not carried much meaning for a while now and is
almost always enabled by default. As agreed during the Linux kernel
summit, remove it.

Acked-by: David S. Miller <davem@davemloft.net>
Acked-by: Vlad Yasevich <vyasevich@gmail.com>
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: David Rientjes <rientjes@google.com>
---
 Ah, look what I found in my mailbox from October 23.  The patch for dlm
 was merged, but this wasn't for some reason.  It's acked by the 
 maintainer so it should be good to go and fixes the reported build error:

        net/built-in.o: In function `sctp_crc32c':
        include/net/sctp/checksum.h:51: undefined reference to `crc32c'
        include/net/sctp/checksum.h:51: undefined reference to `crc32c'
        include/net/sctp/checksum.h:51: undefined reference to `crc32c'
        include/net/sctp/checksum.h:51: undefined reference to `crc32c'
        include/net/sctp/checksum.h:51: undefined reference to `crc32c'
        net/built-in.o:include/net/sctp/checksum.h:51: more undefined references to `crc32c' follow

 And I guess "dlm not being marked experimental for a long time" meant
 three months.

 net/sctp/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/sctp/Kconfig b/net/sctp/Kconfig
--- a/net/sctp/Kconfig
+++ b/net/sctp/Kconfig
@@ -3,8 +3,8 @@
 #
 
 menuconfig IP_SCTP
-	tristate "The SCTP Protocol (EXPERIMENTAL)"
-	depends on INET && EXPERIMENTAL
+	tristate "The SCTP Protocol"
+	depends on INET
 	depends on IPV6 || IPV6=n
 	select CRYPTO
 	select CRYPTO_HMAC

  reply	other threads:[~2013-02-13  0:25 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-11 21:48 [patch for-3.8] fs, dlm: fix build error when EXPERIMENTAL is disabled David Rientjes
2013-02-12  9:50 ` Steven Whitehouse
2013-02-12 16:56   ` Linus Torvalds
2013-02-12 20:13     ` David Rientjes
2013-02-13  0:24       ` David Rientjes [this message]
2013-02-13  9:53         ` [patch for-3.8] net, sctp: remove CONFIG_EXPERIMENTAL Steven Whitehouse
2013-02-13 18:57         ` David Miller

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=alpine.DEB.2.02.1302121619340.3805@chino.kir.corp.google.com \
    --to=rientjes@google.com \
    --cc=ccaulfie@redhat.com \
    --cc=cluster-devel@redhat.com \
    --cc=davem@davemloft.net \
    --cc=keescook@chromium.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mmarek@suse.cz \
    --cc=swhiteho@redhat.com \
    --cc=teigland@redhat.com \
    --cc=torvalds@linux-foundation.org \
    --cc=vyasevich@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).