On Sep 18, 2007 20:03 -0700, Andrew Morton wrote: > On Tue, 18 Sep 2007 17:25:31 -0700 Avantika Mathur <mathur@xxxxxxxxxxxxxxxxxx> wrote: > > > +#if !defined(CONFIG_CRC16) > > +/** CRC table for the CRC-16. The poly is 0x8005 (x16 + x15 + x2 + 1) */ > > +__u16 const crc16_table[256] = { > > + 0x0000, 0xC0C1, 0xC181, 0x0140, 0xC301, 0x03C0, 0x0280, 0xC241, > > That's rather sad. A plain old "depends on" would be better. My bad. We wrote this patch and it had to run on older kernels that might not even have lib/crc16.c (it was added around 2.6.15 or so, so e.g. RHEL4 doesn't have it). I forgot to remove it from the upstream submission, and since it didn't cause problems nobody else complained... Cheers, Andreas -- Andreas Dilger Principal Software Engineer Cluster File Systems, Inc. - To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html