diff options
| author | Mark Cave-Ayland | 2017-12-15 19:41:43 +0100 |
|---|---|---|
| committer | Jason Wang | 2017-12-22 02:55:03 +0100 |
| commit | eaba8f34f0dac5e9296df9c2046d9601264c46a4 (patch) | |
| tree | 47497a0be3a2e626b1dcc62eb36d0cc64f36e898 /hw/net/eepro100.c | |
| parent | e1000: Separate TSO and non-TSO contexts, fixing UDP TX corruption (diff) | |
| download | qemu-eaba8f34f0dac5e9296df9c2046d9601264c46a4.tar.gz qemu-eaba8f34f0dac5e9296df9c2046d9601264c46a4.tar.xz qemu-eaba8f34f0dac5e9296df9c2046d9601264c46a4.zip | |
net: move CRC32 calculation from compute_mcast_idx() into its own net_crc32() function
Separate out the standard ethernet CRC32 calculation into a new net_crc32()
function, renaming the constant POLYNOMIAL to POLYNOMIAL_BE to make it clear
that this is a big-endian CRC32 calculation.
As part of the constant rename, remove the duplicate definition of POLYNOMIAL
from eepro100.c and use the new POLYNOMIAL_BE constant instead.
Once this is complete remove the existing CRC32 implementation from
compute_mcast_idx() and call the new net_crc32() function in its place.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Jason Wang <jasowang@redhat.com>
Diffstat (limited to 'hw/net/eepro100.c')
| -rw-r--r-- | hw/net/eepro100.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/hw/net/eepro100.c b/hw/net/eepro100.c index 1c0def555b..71cddfece4 100644 --- a/hw/net/eepro100.c +++ b/hw/net/eepro100.c @@ -323,8 +323,6 @@ static const uint16_t eepro100_mdi_mask[] = { 0xffff, 0xffff, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, }; -#define POLYNOMIAL 0x04c11db6 - static E100PCIDeviceInfo *eepro100_get_class(EEPRO100State *s); /* From FreeBSD (locally modified). */ @@ -342,7 +340,7 @@ static unsigned e100_compute_mcast_idx(const uint8_t *ep) crc <<= 1; b >>= 1; if (carry) { - crc = ((crc ^ POLYNOMIAL) | carry); + crc = ((crc ^ POLYNOMIAL_BE) | carry); } } } |