Re: ether_crc32_[bl]e()
From: M. Warner Losh (imp_at_bsdimp.com)
Date: 05/31/04
- Previous message: Brian Feldman: "Re: ether_crc32_[bl]e()"
- In reply to: Brian Feldman: "Re: ether_crc32_[bl]e()"
- Next in thread: M. Warner Losh: "Re: ether_crc32_[bl]e()"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Mon, 31 May 2004 11:20:57 -0600 (MDT) To: green@freebsd.org
In message: <20040531164514.GA7776@green.homeunix.org>
Brian Feldman <green@freebsd.org> writes:
: On Sun, May 30, 2004 at 06:42:11PM +0000, Christian Weisgerber wrote:
: > NetBSD and OpenBSD have two helper functions in if_ethersubr.c
: >
: > ether_crc32_le(const u_int8_t *buf, size_t len)
: > ether_crc32_be(const u_int8_t *buf, size_t len)
: > [...]
: > The patch below adds the functions to if_ethersubr.c and, as an
: > example, switches re(4) to make use of this.
: >
: > 1. Do we want this?
:
: Yeah :)
I think so too. I had something like this on my list after I found a
very badly implemented big endian crc routine whose provenance was in
question. So this is excellent timing.
: I'm not sure if
: it is a style bug to perpetuate BSD u_intsize_t when uintsize_t is
: standardized and available as well.
I'd say that it is, since style(9) says we're moving towards that as
well.
Warner
_______________________________________________
freebsd-arch@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-arch
To unsubscribe, send any mail to "freebsd-arch-unsubscribe@freebsd.org"
- Previous message: Brian Feldman: "Re: ether_crc32_[bl]e()"
- In reply to: Brian Feldman: "Re: ether_crc32_[bl]e()"
- Next in thread: M. Warner Losh: "Re: ether_crc32_[bl]e()"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
- [PATCH 1/4] disk-protect: Add disk shock protection helpers to libata
... This patch adds some helper functions to libata in order to provide low ...
The user interface to this ... functionality will be implemented in the block layer in
a subsequent patch. ... unsigned int wait, unsigned int can_sleep); ... (Linux-Kernel) - Re: [patch] NetBSD disklabel support for geom_bsd
... it's the geom_bsd module with small patch like your one. ... NetBSD, OpenBSD
and DragonFlyBSD partitions. ... (freebsd-current) - Re: Errno man page
... Somebody supplied a patch but it ... NetBSD and OpenBSD both have
the correct values as ... To unsubscribe, ... (freebsd-current) - Re: Errno man page
... Ivan Voras wrote: ... Somebody supplied a patch but it ... NetBSD
and OpenBSD both have the correct values as ... (freebsd-current) - Re: ether_crc32_[bl]e()
... NetBSD and OpenBSD have two helper functions in if_ethersubr.c ... These CRCs
are used all over our network drivers, ... NetBSD has done so, ... (freebsd-arch)