esp8266-ping icon indicating copy to clipboard operation
esp8266-ping copied to clipboard

Observed issues with multiple IPs when the u32_t requests > 9

Open 2BECKHAM2 opened this issue 6 years ago • 5 comments

Hi bluemurder,

While checking the ping with multiple IP I observed that when u32_t requests > 9, I face issues with Ping summary, seqno, Reply from IP. I have attached the file for your reference.

  1. LOG_PING_requests=9.txt - The results observed were OK as expected.

  2. LOG_PING_requests=10.txt - Observed issues with the printing of Ping summary. (Not available for all the IPs)

  3. LOG_PING_requests=50.txt - Observed issues with the printing of Ping summary and the icmp_sequence no.

  4. LOG_PING_requests=110.txt - Observed issues with the Reply from 192.168.31.1, printing of Ping summary, icmp_sequence no, and the number of pings initiated.

The above conditions were tested by changing the value of requests = XX in the Pinger.h.

LOG_PING_requests = 9.txt LOG_PING_requests = 10.txt LOG_PING_requests = 50.txt LOG_PING_requests = 110.txt

Thanks in advance.

2BECKHAM2 avatar May 24 '18 16:05 2BECKHAM2

It is critic the delay between different ping calls. In your case it is possible that the onend callback is called when the next ping sequence is already started. Please send your main .ino file

bluemurder avatar May 28 '18 05:05 bluemurder

Many thanks Bluemurder.

Find attached files.

  1. Ping.ino (The example file)

  2. PingModified.ino - where the IP, Information and number of ping count which are configurable.

I've tried delay(20000); in the above two files also, observed the same results.

Thanks again.

Ping.ino.txt PingModified.ino.txt

2BECKHAM2 avatar May 28 '18 11:05 2BECKHAM2

Hi,

I've tried with Serial.begin(9600); also - Same results observed.

2BECKHAM2 avatar May 28 '18 15:05 2BECKHAM2

Thanks for the excellent library Blumurder.

Do you have any update on the issue raised.

Thanks in advance.

2BECKHAM2 avatar Jul 10 '18 07:07 2BECKHAM2

No updates yet.

bluemurder avatar Jul 10 '18 09:07 bluemurder