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.

Submitted by:  Dag Midling Larsen <dagml>
Submitted on:  Wed 13 Mar 2019 02:42:49 PM UTC  
 
Category:  apps Severity:  3 - Normal
Item Group:  Faulty Behaviour Status:  Duplicate
Privacy:  Public Assigned to:  Simon Goldschmidt <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>
Project AdministratorIn charge of this item.
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>
Project 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>
Project AdministratorIn charge of this item.
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>

 

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)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

     

     

    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.5