tasklwIP - A Lightweight TCP/IP stack - Tasks: task #14798, PPP: improve PPP_DEBUG macro...

 
 

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

task #14798: PPP: improve PPP_DEBUG macro support

Submitted by:  Marco <marcoe>
Submitted on:  Thu 04 Jan 2018 03:27:27 PM UTC  
 
Category:  PPP Should Start On:  Thu 04 Jan 2018 12:00:00 AM UTC
Should be Finished on:  Thu 04 Jan 2018 12:00:00 AM UTC Priority:  3 - Low
Status:  None Privacy:  Public
Percent Complete:  0% Assigned to:  None
Open/Closed:  Open Planned Release:  None
Effort:  1.00

Fri 05 Jan 2018 10:30:11 AM UTC, comment #4: 

Ok, the summary could become "PPP: improve PPP_DEBUG macro support (now it doesn't suppress all PPP debug messages)".

Marco <marcoe>
Thu 04 Jan 2018 04:46:52 PM UTC, comment #3: 

Ok, a task is fine then. But it needs a summary that tells us what to do when viewing the task list.

Simon Goldschmidt <goldsimon>
Project Administrator
Thu 04 Jan 2018 04:44:05 PM UTC, comment #2: 

I was thinking it could be an enhancement and, as a consequence, that a task was needed to proceed with a little cleanup of this part of code. Sorry if this is not the intended meaning for "task". I don't think this is a bug because as far as I know PPP is fully working, also if not size-optimized. If you prefer to have a bug report instead I will close here and open a bug report, tell me how to proceed.

In any case, I'm using lwIP 2.0.3 release version (92f23d6ca0971a32f2085b9480e738d34174417b).

Marco <marcoe>
Thu 04 Jan 2018 04:15:36 PM UTC, comment #1: 

If this should be a task, you should have given it a summary that tells us what to do. "PPP log messages" is not a task. Do you think it's a bug that 28th PPP debug disabled, too many things are linked in? If so, a bug report would have been better.

Which version of lwip are you talking about anyway?

Simon Goldschmidt <goldsimon>
Project Administrator
Thu 04 Jan 2018 03:27:27 PM UTC, original submission:  

Hello all,
I've noticed that also with PPP_DEBUG = 0 some kiloBytes of flash memory are occupied by ppp_vslprintf() code and by many PPP debug strings.

In fact, debugging messages are not consistent across PPP code because:
- in some points log messages are correctly produced using macros like LCPDEBUG(a) defined in pppdebug.h
- in other points there are direct calls to ppp_warn(), ppp_notice(), ...
- in other points the calls to ppp_warn(), ppp_notice(), ... are inside a #if PPP_DEBUG ... #endif block

I think it would be nice to have the possibility to completely remove these log messages and related helper functions in a production environment as well as we can do it for all the other debug messages inside lwIP.

Bye

Marco <marcoe>

 

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 goldsimon (Posted a comment)
  • -email is unavailable- added by marcoe (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.

     

     

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-01-05 goldsimon SummaryPPP log messages => PPP: improve PPP_DEBUG macro support

    Back to the top


    Powered by Savane 3.5