buglwIP - A Lightweight TCP/IP stack - Bugs: bug #38586, netif_loop_output()...

 
 

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

bug #38586: netif_loop_output() "deadlocks"

Submitted by:  Emil Ljungdahl <emil_ljungdahl>
Submitted on:  Mon 25 Mar 2013 10:53:00 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: Faulty BehaviourStatus: Fixed
Privacy: PublicAssigned to: Simon Goldschmidt <goldsimon>
Open/Closed: ClosedPlanned Release: None
lwIP version: git head

Wed 15 Jan 2014 08:31:32 AM UTC, comment #2:

But what happens to the looped back packets when tcpip_callback_with_block fails? Won't the packets get stalled until someone else calls netif_poll?

Matthias Pfaller <matthiaspfaller>
Wed 24 Apr 2013 07:38:32 PM UTC, comment #1:

Fixed, thanks for reporting.

Simon Goldschmidt <goldsimon>
Project AdministratorIn charge of this item.
Mon 25 Mar 2013 10:53:00 AM UTC, original submission:

I ran into a problem where the lwip core task hanged in sys_mbox_post() because of a full message queue. The problems seems to be that core/netif.c:netif_loop_output() calls tcpip_callback() which will wait for message to be posted, but since the current task is the only one eating messages we are stuck...

Am I right the call should be replaced by a tcpip_callback_with_block((tcpip_callback_fn)netif_poll, netif, 0);
?

Emil Ljungdahl <emil_ljungdahl>

 

Attached Files
file #27674:  netif_c.diff added by emil_ljungdahl (510B - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by matthiaspfaller (Posted a comment)
  • -unavailable- added by goldsimon (Posted a comment)
  • -unavailable- added by emil_ljungdahl (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 24 Apr 2013 07:38:32 PM UTCgoldsimonStatusNone=>Fixed
      Assigned toNone=>goldsimon
      Open/ClosedOpen=>Closed
    Mon 25 Mar 2013 10:53:00 AM UTCemil_ljungdahlAttached File-=>Added netif_c.diff, #27674

    Back to the top


    Powered by Savane 3.1-cleanup1