Re: do we need 10 copies?

Dan Hollis (goemon@anime.net)
Sun, 23 Sep 2001 15:36:25 -0700 (PDT)


On Sun, 23 Sep 2001, Kai Germaschewski wrote:
> On Sun, 23 Sep 2001, Garst R. Reese wrote:
> > This table (512 bytes) and the code to implement crc-ccit is replicated
> > in 10 drivers. ppp-async even exports it. Surely there is a better way.
> As for the ISDN code (4 copies), there is the plan to use a a common HDLC
> en/decoding module, however that's a 2.5 thing. I'll take a look if I can
> find a generic solution then, but it might turn out difficult - having a
> module of its own just for that table wastes nearly a page, so that's
> probably worse than the current state of affairs.

I think the point is not so much that it tries to save memory, but
that it avoids duplicated code and errors. IIRC there were bugs in
previous kernels related to typos/errors in the individual crc tables...

-Dan

-- 
[-] Omae no subete no kichi wa ore no mono da. [-]

- 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/