buglwIP - A Lightweight TCP/IP stack - Bugs: bug #55910, MEMP_SYS_TIMEOUT pool runs empty...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #55910: MEMP_SYS_TIMEOUT pool runs empty in sntp scenario.

Submitter:  Dag Midling Larsen <dagml>
Submitted:  Wed 13 Mar 2019 02:42:49 PM UTC
   
 
Category:  apps Severity:  3 - Normal
Item Group:  Faulty Behaviour Status:  Duplicate
Privacy:  Public Assigned to:  goldsimon
Open/Closed:  Closed Planned Release:  None
lwIP version:  2.1.1

Fri 10 Jan 2020 09:13:50 PM UTC, comment #4: 

Ok, this one was first, but I just close bug #56431 and patch #9878 fixing just this.

-> closing as duplicate

Simon Goldschmidt <goldsimon>
Group administrator
Wed 20 Mar 2019 04:35:03 PM UTC, comment #3: 

It could be because LWIP_NUM_SYS_TIMEOUT_INTERNAL does not take SNTP into account, and that is not obvious until you run into the mempool running empty (like Dag is seeing).

I use:

#define MEMP_NUM_SYS_TIMEOUT (LWIP_NUM_SYS_TIMEOUT_INTERNAL + LWIP_SNTP)

Josh <inojosh>
Wed 13 Mar 2019 07:29:45 PM UTC, comment #2: 

AFAIK there is nothing in NTP protocol to tell that a server is "out of sync", most NTP servers implementation just discard all requests until the server itself consider that it is synchronized.

So, what we should read is "When the target is connected to a ntp server which does not answer" instead of "When the target is connected to a ntp server which is out of sync".

Sylvain

Sylvain Rochet <gradator>
Group Member
Wed 13 Mar 2019 07:24:15 PM UTC, comment #1: 

Could you provide more detail please?
- when does the pool run empty? Do you have MEMP_NUM_SYS_TIMEOUT set too low or is there a memory leak?
- what does "out of sync" mean for an ntp server?

Simon Goldschmidt <goldsimon>
Group administrator
Wed 13 Mar 2019 02:42:49 PM UTC, original submission:  

Hi

When the target is connected to a ntp server which is out of sync, the MEMP_SYS_TIMEOUT pool runs empty.
(assert line 190 in timouts.c )

NOTE: present in lwIP version 2.1.2 which is not in the dropdown list (and an older lwIP version).

Best Regards,
Dag

Dag Midling Larsen <dagml>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by inojosh (Posted a comment)
  • -email is unavailable- added by gradator (Posted a comment)
  • -email is unavailable- added by goldsimon (Posted a comment)
  • -email is unavailable- added by dagml (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-01-10 goldsimon StatusNone Duplicate
        Assigned toNone goldsimon
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code