verilog-ethernet icon indicating copy to clipboard operation
verilog-ethernet copied to clipboard

fix overflow in parameter sizes for ARP timeouts

Open AaronCleaver opened this issue 4 years ago • 2 comments

ARP sends a request 4 times with a 2 second delay then should wait for 30s before dropping packet. The parameter declarations and literals default to integer (signed 32 bit). The 30 sec cycle count doesn't fit in 31 bits of integer and overflows, result is a 9min wait before dropping packet.

log2(125000000 * 2) = 27.89 log2(125000000 * 30) = 31.80 (overflow)

AaronCleaver avatar Apr 15 '21 05:04 AaronCleaver

Is this issue rectified the lastest repo

fpgapsyc avatar Jan 06 '22 13:01 fpgapsyc

Is this issue rectified the lastest repo

No, I don't believe it is.

AaronCleaver avatar Jan 07 '22 08:01 AaronCleaver