Messages in this thread |  | | Subject | Re: crc32 cleanups | Date | Fri, 12 Oct 2001 15:45:32 -0400 | From | Horst von Brand <> |
| |
Matt Domsch <Matt_Domsch@dell.com> said: > In trying to get my EFI GUID Partition Table patch included into the stock > kernel, Andreas Dilger suggested it was time for some crc32 cleanup, as > the GPT patch added yet another copy of the common crc32 function. So, > here's my first pass at it. Comments welcome.
[...]
> This patch (appended below), makes include/linux/crc32.h and > lib/crc32.c. It generates a table based on the commonly used polynomial > at init time provided a driver needs it. It changes ether_crc_le() to use > this table. It renames the commonly seen ether_crc() to be > ether_crc_be(), and puts it in crc32.h (allowing lots of copies to be > deleted).
You could just place the functions (each with its table) into separate .c files, and place them in the library. The linker will then include them iff they are referenced somewhere. OTOH, they _are_ all over the place right now, so they could just be included unconditionally. Thinking of ways to set up to have them included in case only modules use them makes me shiver... -- Dr. Horst H. von Brand Usuario #22616 counter.li.org Departamento de Informatica Fono: +56 32 654431 Universidad Tecnica Federico Santa Maria +56 32 654239 Casilla 110-V, Valparaiso, Chile Fax: +56 32 797513 - 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/
|  |