bugSimulavr: an AVR simulator - Bugs: bug #34287, commit...

 
 

bug #34287: commit f658676ad64c4e10f9dff8da808869ee26b0a2e3 breaks python simulation

Submitter:  Marek Pietrzak <mpie>
Submitted:  Tue 13 Sep 2011 07:19:05 PM UTC
   
 
Category:  Simulation Severity:  3 - Normal
Item Group:  None Status:  Ready For Test
Privacy:  Public Assigned to:  petrh
Open/Closed:  Closed Component Version:  * simulavrxx
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 14 Sep 2011 07:05:36 PM UTC, comment #1: 

The patch in question: http://git.savannah.gnu.org/cgit/simulavr.git/commit/?id=f658676ad64c4e10f9dff8da808869ee26b0a2e3

You are right. The commit changed behavior when cpuCycles > 0, the old code relied on `int bpFlag = 0;'. Therefore SystemClock::RunTimeRange() incorrectly returned after executing a single instruction.

Fixed in http://git.savannah.gnu.org/cgit/simulavr.git/commit/?id=a46be63133461aef9258e928dcc6db469b16be93

Petr Hluzin <petrh>
Group Member
Tue 13 Sep 2011 07:19:05 PM UTC, original submission:  

Commit submitted on Thu Aug 11 21:55:24 2011 +0200 results in SystemClock::Step() returning non-zero in a normal situation (without breakpoint). It breaks python simulation examples. Also calling SystemClock::RunTimeRange() will return immediately without running the expected time range.

Marek Pietrzak <mpie>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

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 petrh (Posted a comment)
  • -email is unavailable- added by mpie (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-09-14 petrh StatusNone Ready For Test
        Assigned toNone petrh
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code