[SOLVED] SystemView's bugs

This site uses cookies. By continuing to browse this site, you are agreeing to our Cookie Policy.

  • [SOLVED] SystemView's bugs

    1.The TimeStamp have possible to overrange:
    Could you add these code to detect?
    SEGGER_SYSVIEW.C
    _TrySendOverflowPacket(void):
    U32 Delta;
    ......
    TimeStamp = SEGGER_SYSVIEW_GET_TIMESTAMP();
    +if(_SYSVIEW_Globals.LastTxTimeStamp > TimeStamp)
    +{
    + Delta = 0xffffffff - _SYSVIEW_Globals.LastTxTimeStamp + TimeStamp;
    +}
    +else
    +{
    + Delta = TimeStamp - _SYSVIEW_Globals.LastTxTimeStamp ;
    +}
    ......

    _SendPacket():
    ......
    TimeStamp = SEGGER_SYSVIEW_GET_TIMESTAMP();
    +if(_SYSVIEW_Globals.LastTxTimeStamp > TimeStamp)
    +{
    + Delta = 0xffffffff - _SYSVIEW_Globals.LastTxTimeStamp + TimeStamp;
    +}
    +else
    +{
    + Delta = TimeStamp - _SYSVIEW_Globals.LastTxTimeStamp ;
    +}
    ......

    2.When we set the SEGGER_SYSVIEW_POST_MORTEM_MODE = 1
    and then try to build.
    We will get the error:
    can't find the _HandleIncomingPacket();

    Could we fix it like these:
    SEGGER_SYSVIEW_IsStarted():
    +#if(SEGGER_SYSVIEW_POST_MORTEM_MODE != 1)
    if(SEGGER_RTT_HASDATA(CHANNEL_ID_DOWN))
    {
    ......
    }
    +#endif

  • Question 3:

    Is there a way to fix the BUG by myself?
    Can I fix the BUG base on the SYSVIEW source code and only open source the SYSVIEW source code?
    I have list what I want to change the source code, as above~

    Help~~~~~~~~
    SOS~~~~~~~~~

  • Hello,

    Thank you for your inquiry.
    1: Will be fixed in a future SystemView source release.
    2: Is not reproducible with the currently available SystemView sources: segger.com/downloads/jlink/systemview_target_src
    Which version are you using?


    rise.cai wrote:

    If the LGPL license will protect the sample.c and config.c under SystemView/Sample and SystemView/Config doc?
    You can find the licensing terms in the files headers.


    rise.cai wrote:

    Is there a way to fix the BUG by myself?
    Currently not.

    Best regards,
    Nino
    Please read the forum rules before posting.

    Keep in mind, this is *not* a support forum.
    Our engineers will try to answer your questions between their projects if possible but this can be delayed by longer periods of time.
    Should you be entitled to support you can contact us via our support system: segger.com/ticket/
    Or you can contact us via e-mail.

  • SEGGER - Nino wrote:

    Hello,

    Thank you for your inquiry.
    1: Will be fixed in a future SystemView source release.
    2: Is not reproducible with the currently available SystemView sources: https://www.segger.com/downloads/jlink/systemview_target_src
    Which version are you using?


    rise.cai wrote:

    If the LGPL license will protect the sample.c and config.c under SystemView/Sample and SystemView/Config doc?
    You can find the licensing terms in the files headers.

    rise.cai wrote:

    Is there a way to fix the BUG by myself?
    Currently not.
    Best regards,
    Nino
    I use the SystemView_V252d.
    The Bugs have not been fixed in lastest version V303.

    Thanks for answer my sample questions~~

    :) :) :) :)
    Rose
    Best regards
    Rise