Matt Reppert wrote:
> Okay, I know, it's just a library module, doesn't need to ever be unloaded
> anyway. But error noise in dmesg annoys me, hence this patch.
I don't think you even need to set the use count at all for crc32:
As long as another module is using it, you can't unload it because
the exported symbols are used. When those symbols are not known
to other modules, it is also safe to unload crc32.
I noticed another small problem with init_crc: if crc32init_be()
fails, the memory allocated by crc32init_le() is never freed,
see below.
Arnd <><
--- 1.5/lib/crc32.c Mon Apr 8 22:22:00 2002
+++ edited/lib/crc32.c Mon Dec 2 14:25:37 2002
@@ -547,11 +547,13 @@
*/
static int __init init_crc32(void)
{
- int rc1, rc2, rc;
- rc1 = crc32init_le();
- rc2 = crc32init_be();
- rc = rc1 || rc2;
- if (!rc) MOD_INC_USE_COUNT;
+ int rc;
+ rc = crc32init_le();
+ if (rc)
+ return rc;
+ rc = crc32init_be();
+ if (rc)
+ crc32cleanup_le();
return rc;
}
-
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/
This archive was generated by hypermail 2b29 : Sat Dec 07 2002 - 22:00:12 EST