[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/8] hw/net/cadence_gem: Remove dead code
From: |
Peter Crosthwaite |
Subject: |
Re: [Qemu-devel] [PATCH 2/8] hw/net/cadence_gem: Remove dead code |
Date: |
Sat, 10 May 2014 08:50:22 +1000 |
On Fri, May 9, 2014 at 4:46 AM, Peter Maydell <address@hidden> wrote:
> Commit 191946c moved the code to handle padding to minimum
> length from after the handling of the CRC to before it.
> This means that the CRC code doesn't need to cope with the
> possibility that the size is less than 60; remove this
> dead code.
>
> Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Peter Crosthwaite <address@hidden>
> ---
> hw/net/cadence_gem.c | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/hw/net/cadence_gem.c b/hw/net/cadence_gem.c
> index cdb1825..afddc8a 100644
> --- a/hw/net/cadence_gem.c
> +++ b/hw/net/cadence_gem.c
> @@ -717,7 +717,6 @@ static ssize_t gem_receive(NetClientState *nc, const
> uint8_t *buf, size_t size)
> rxbuf_ptr = (void *)buf;
> } else {
> unsigned crc_val;
> - int crc_offset;
>
> /* The application wants the FCS field, which QEMU does not provide.
> * We must try and caclculate one.
> @@ -727,12 +726,7 @@ static ssize_t gem_receive(NetClientState *nc, const
> uint8_t *buf, size_t size)
> memset(rxbuf + size, 0, sizeof(rxbuf) - size);
> rxbuf_ptr = rxbuf;
> crc_val = cpu_to_le32(crc32(0, rxbuf, MAX(size, 60)));
> - if (size < 60) {
> - crc_offset = 60;
> - } else {
> - crc_offset = size;
> - }
> - memcpy(rxbuf + crc_offset, &crc_val, sizeof(crc_val));
> + memcpy(rxbuf + size, &crc_val, sizeof(crc_val));
>
> bytes_to_copy += 4;
> size += 4;
> --
> 1.9.2
>
>
- [Qemu-devel] [PATCH 7/8] hw/arm/stellaris: Correct handling of GPTM TAR register, (continued)
- [Qemu-devel] [PATCH 7/8] hw/arm/stellaris: Correct handling of GPTM TAR register, Peter Maydell, 2014/05/08
- [Qemu-devel] [PATCH 8/8] hw/arm/omap_gpmc: Avoid buffer overrun filling prefetch FIFO, Peter Maydell, 2014/05/08
- [Qemu-devel] [PATCH 4/8] hw/arm/omap1: Avoid unintended sign extension writing omap_rtc YEARS_REG, Peter Maydell, 2014/05/08
- [Qemu-devel] [PATCH 1/8] hw/intc/allwinner-a10-pic: Add missing 'break', Peter Maydell, 2014/05/08
- [Qemu-devel] [PATCH 2/8] hw/net/cadence_gem: Remove dead code, Peter Maydell, 2014/05/08
- Re: [Qemu-devel] [PATCH 2/8] hw/net/cadence_gem: Remove dead code,
Peter Crosthwaite <=
- [Qemu-devel] [PATCH 3/8] hw/display/pxa2xx_lcd: Fix 16bpp+alpha and 18bpp+alpha palette formats, Peter Maydell, 2014/05/08
- [Qemu-devel] [PATCH 6/8] hw/timer/exynos4210_mct: Avoid overflow in exynos4210_ltick_recalc_count, Peter Maydell, 2014/05/08
- [Qemu-devel] [PATCH 5/8] hw/dma/omap_dma: Add (uint32_t) casts when shifting uint16_t by 16, Peter Maydell, 2014/05/08