Re: [ANNOUNCE] Open-FCoE - Fibre Channel over Ethernet Project

From: Christoph Hellwig
Date: Wed Nov 28 2007 - 02:44:46 EST


I just did a very quick glance over the tree. Some extremly highlevel
comments to start with before actually starting the source review:


- why do you need your own libcrc? lib/crc32.c has a crc32_le
- libsa should go. Much of it is just wrappers of kernel functions
that should be used directly. Other like that hash, even or state
helpers might either be opencoded in the caller or made completely
generic in lib/. Probably the former but we'll have to see.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/