Messages in this thread | | | Subject | Re: [patch for-3.8] fs, dlm: fix build error when EXPERIMENTAL is disabled | From | Steven Whitehouse <> | Date | Tue, 12 Feb 2013 09:50:05 +0000 |
| |
Hi,
On Mon, 2013-02-11 at 13:48 -0800, David Rientjes wrote: > CONFIG_IP_SCTP relies on being able to select things like CONFIG_CRC32C to > build. Thus, nothing should be selecting CONFIG_IP_SCTP that does not > meet its requirements. > > For example, if CONFIG_EXPERIMENTAL is disabled and CONFIG_DLM is enabled, > the build fails at link time: > > 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 > > Fix this by making CONFIG_DLM depend on CONFIG_EXPERIMENTAL so that > CONFIG_IP_SCTP properly builds. > That doesn't seem right to me... DLM has not been experimental for a long time now. Why not just select CRC32 in addition to IP_SCTP ?
Steve.
> Signed-off-by: David Rientjes <rientjes@google.com> > --- > fs/dlm/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/fs/dlm/Kconfig b/fs/dlm/Kconfig > --- a/fs/dlm/Kconfig > +++ b/fs/dlm/Kconfig > @@ -2,6 +2,7 @@ menuconfig DLM > tristate "Distributed Lock Manager (DLM)" > depends on INET > depends on SYSFS && CONFIGFS_FS && (IPV6 || IPV6=n) > + depends on EXPERIMENTAL > select IP_SCTP > help > A general purpose distributed lock manager for kernel or userspace > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/
| |