From: David Woodhouse <dwmw2@infradead.org>
To: "J.A. Magallon" <jamagallon@able.es>
Cc: marcelo@conectiva.com.br, alan@lxorguk.ukuu.org.uk,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Shared crc32 for 2.4.
Date: Thu, 22 May 2003 22:11:05 +0100 [thread overview]
Message-ID: <1053637864.21582.700.camel@imladris.demon.co.uk> (raw)
In-Reply-To: <20030521235033.GA2470@werewolf.able.es>
On Thu, 2003-05-22 at 00:50, J.A. Magallon wrote:
> This works if crc32.o is compiled as a module, but does not work if it is
> built into the kernel. For example, crc32_le symbol does not appear
> in System.map nor vmlinux.
Oh bollocks; sorry I'd forgotten about that.
Added to the tree from which Marcelo will pull for 2.4.22-pre1...
# This is a BitKeeper generated patch for the following project:
# Project Name: Linux kernel tree
# This patch format is intended for GNU patch command version 2.5 or higher.
# This patch includes the following deltas:
# ChangeSet 1.1211 -> 1.1212
# lib/crc32.c 1.1 -> 1.2
# kernel/ksyms.c 1.67 -> 1.68
# lib/Makefile 1.10 -> 1.11
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 03/05/22 dwmw2@dwmw2.baythorne.internal 1.1212
# Fix CONFIG_CRC32=y when nothing in-kernel uses CRC32 functions
# by exporting the symbol from kernel/ksyms.c instead of lib/crc32.c,
# hence forcing lib/crc32.o to get pulled in during the final link.
# --------------------------------------------
#
diff -Nru a/kernel/ksyms.c b/kernel/ksyms.c
--- a/kernel/ksyms.c Thu May 22 22:08:02 2003
+++ b/kernel/ksyms.c Thu May 22 22:08:02 2003
@@ -48,6 +48,7 @@
#include <linux/completion.h>
#include <linux/seq_file.h>
#include <linux/dnotify.h>
+#include <linux/crc32.h>
#include <asm/checksum.h>
#if defined(CONFIG_PROC_FS)
@@ -557,6 +558,12 @@
EXPORT_SYMBOL(strnicmp);
EXPORT_SYMBOL(strspn);
EXPORT_SYMBOL(strsep);
+
+#ifdef CONFIG_CRC32
+EXPORT_SYMBOL(crc32_le);
+EXPORT_SYMBOL(crc32_be);
+EXPORT_SYMBOL(bitreverse);
+#endif
/* software interrupts */
EXPORT_SYMBOL(tasklet_hi_vec);
diff -Nru a/lib/Makefile b/lib/Makefile
--- a/lib/Makefile Thu May 22 22:08:02 2003
+++ b/lib/Makefile Thu May 22 22:08:02 2003
@@ -8,7 +8,7 @@
L_TARGET := lib.a
-export-objs := cmdline.o dec_and_lock.o rwsem-spinlock.o rwsem.o rbtree.o crc32.o
+export-objs := cmdline.o dec_and_lock.o rwsem-spinlock.o rwsem.o rbtree.o
obj-y := errno.o ctype.o string.o vsprintf.o brlock.o cmdline.o \
bust_spinlocks.o rbtree.o dump_stack.o
diff -Nru a/lib/crc32.c b/lib/crc32.c
--- a/lib/crc32.c Thu May 22 22:08:02 2003
+++ b/lib/crc32.c Thu May 22 22:08:02 2003
@@ -266,10 +266,6 @@
return x;
}
-EXPORT_SYMBOL(crc32_le);
-EXPORT_SYMBOL(crc32_be);
-EXPORT_SYMBOL(bitreverse);
-
/*
* A brief CRC tutorial.
*
--
dwmw2
next prev parent reply other threads:[~2003-05-22 20:58 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-05-17 17:43 David Woodhouse
2003-05-19 7:24 ` Duncan Sands
2003-05-19 15:06 ` David Woodhouse
2003-05-21 23:50 ` J.A. Magallon
2003-05-22 21:11 ` David Woodhouse [this message]
2003-05-23 9:28 ` David Woodhouse
2003-05-22 20:23 ` Marcelo Tosatti
2003-05-23 6:44 ` David Woodhouse
[not found] <03ee01c320b4$c3040ad0$020120b0@jockeXP>
2003-05-27 11:26 ` David Woodhouse
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=1053637864.21582.700.camel@imladris.demon.co.uk \
--to=dwmw2@infradead.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=jamagallon@able.es \
--cc=linux-kernel@vger.kernel.org \
--cc=marcelo@conectiva.com.br \
/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
all inboxes | Powered by JetHome®