buglwIP - A Lightweight TCP/IP stack - Bugs: bug #37353, [ppp-new] save thread...

 
 

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

bug #37353: [ppp-new] save thread ppp_input_thread() handle

Submitted by:  hanhui <hanhui03>
Submitted on:  Fri 14 Sep 2012 06:34:30 PM UTC  
 
Category: PPPSeverity: 3 - Normal
Item Group: Change RequestStatus: Wont Fix
Privacy: PublicAssigned to: None
Open/Closed: ClosedPlanned Release: None
lwIP version: Other

Fri 26 Apr 2013 08:05:21 PM UTC, comment #2:

PPP_INPROC_OWNTHREAD feature removed in ppp-new, which make this bug not relevant anymore.

Git commit 25f9f55878f008113bc1f4a0068d4827ed4595b9.

Sylvain Rochet <gradator>
Project Member
Tue 18 Sep 2012 09:38:32 PM UTC, comment #1:

Well, I am more in favor of removing the broken PPP_INPROC_OWNTHREAD feature and moving PPP_INPROC_MULTITHREADED to PPPAPI in a better way (ppp_tcpip_callback_header structure moved to pppapi_msg).

Sylvain Rochet <gradator>
Project Member
Fri 14 Sep 2012 06:34:30 PM UTC, original submission:

Sylvain Rochet

In sys_arch.txt line171

function sys_thread_new() will return a handle of thread. like this

sys_thread_t sys_thread_new(char name, void ( thread)(void arg), void arg, int stacksize, int prio);

so I suggest that ppp_cb save ppp_input_thread handle in PPPoS mode, so before I called pppapi_delete() I must decide whether this thread was completely exit!

hanhui

hanhui <hanhui03>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by goldsimon (Updated the item)
  • -unavailable- added by gradator (Posted a comment)
  • -unavailable- added by hanhui03 (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 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 26 Apr 2013 08:05:21 PM UTCgradatorStatusNone=>Wont Fix
      Open/ClosedOpen=>Closed
    Mon 14 Jan 2013 09:07:34 PM UTCgoldsimonCategoryNone=>PPP

    Back to the top


    Powered by Savane 3.1-cleanup1