Menu

#373 DISCiPLE and +D subpages not coalesced

v1.2.1
closed-accepted
5
2016-07-17
2016-06-11
No

Related to [bugs:#347], looks odd to me that DISCiPLE and +D RAM pages are not summarised like the ROM pages.

fuse --disciple --debugger-command "tbreak event disciple:page"
ADDRESS SOURCE WRITABLE CONTENDED
0x0000 DISCiPLE ROM 0 N N
0x2000 DISCiPLE RAM 0 Y N
0x2800 DISCiPLE RAM 0 Y N
0x3000 DISCiPLE RAM 0 Y N
0x3800 DISCiPLE RAM 0 Y N

I think we are failing to set the subpage offset for the DISCiPLE RAM page:

/* We expect the next page to have an increased offset */
offset += MEMORY_PAGE_SIZE;

The coalesced subpages should look like:

ADDRESS SOURCE WRITABLE CONTENDED
0x0000 DISCiPLE ROM 0 N N
0x2000 DISCiPLE RAM 0 Y N

The same happens for +D, but in this case I think we should use different memory sources too, like DISCiPLE:

fuse --plusd --debugger-command "tbreak event plusd:page"

actual:

ADDRESS SOURCE WRITABLE CONTENDED
0x0000 PlusD 0 N N
0x2000 PlusD 0 Y N
0x2800 PlusD 0 Y N
0x3000 PlusD 0 Y N
0x3800 PlusD 0 Y N

expected:

ADDRESS SOURCE WRITABLE CONTENDED
0x0000 PlusD ROM 0 N N
0x2000 PlusD RAM 0 Y N
1 Attachments

Related

Bugs: #347

Discussion

  • Sergio Baldoví

    Sergio Baldoví - 2016-07-06
    • Group: future --> next_release
     
  • Sergio Baldoví

    Sergio Baldoví - 2016-07-06

    I'll commit this for the next release, barring objections.

     
  • Sergio Baldoví

    Sergio Baldoví - 2016-07-08
    • status: open --> pending-accepted
    • assigned_to: Sergio Baldoví
     
  • Sergio Baldoví

    Sergio Baldoví - 2016-07-08

    Committed in [r5671].

     

    Related

    Commit: [r5671]

  • Sergio Baldoví

    Sergio Baldoví - 2016-07-17
    • status: pending-accepted --> closed-accepted
     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.