diff options
| author | Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk> | 2017-12-15 18:41:48 +0000 |
|---|---|---|
| committer | Jason Wang <jasowang@redhat.com> | 2017-12-22 10:00:47 +0800 |
| commit | 8f90bc2f8fbba3c98116e80afd537fb0ca90be2e (patch) | |
| tree | 39239882bcdd1878fba52fe4e79942680065b839 /hw/net/sungem.c | |
| parent | a89a6b052abff87871800406bd3c598fcf406426 (diff) | |
| download | focaccia-qemu-8f90bc2f8fbba3c98116e80afd537fb0ca90be2e.tar.gz focaccia-qemu-8f90bc2f8fbba3c98116e80afd537fb0ca90be2e.zip | |
sungem: fix multicast filter CRC calculation
From the Linux sungem driver, we know that the multicast filter CRC is implemented using ether_crc_le() which isn't the same as calling zlib's crc32() function (the zlib implementation requires a complemented initial value and also returns the complemented result). Fix the multicast filter by simply using the new net_crc32_le() function. 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/sungem.c')
| -rw-r--r-- | hw/net/sungem.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/hw/net/sungem.c b/hw/net/sungem.c index 6aa8d1117b..60f1e479f3 100644 --- a/hw/net/sungem.c +++ b/hw/net/sungem.c @@ -11,12 +11,11 @@ #include "hw/pci/pci.h" #include "qemu/log.h" #include "net/net.h" +#include "net/eth.h" #include "net/checksum.h" #include "hw/net/mii.h" #include "sysemu/sysemu.h" #include "trace.h" -/* For crc32 */ -#include <zlib.h> #define TYPE_SUNGEM "sungem" @@ -595,7 +594,7 @@ static ssize_t sungem_receive(NetClientState *nc, const uint8_t *buf, } /* Get MAC crc */ - mac_crc = crc32(~0, buf, 6); + mac_crc = net_crc32_le(buf, ETH_ALEN); /* Packet isn't for me ? */ rx_cond = sungem_check_rx_mac(s, buf, mac_crc); |