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
next prev parent 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).