Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use hubpack instead of ssmarshal for kipc #694

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

mkeeter
Copy link
Collaborator

@mkeeter mkeeter commented Aug 9, 2022

As part of Project Hubpack Everywhere, this is low-hanging fruit.

Kernel tests still pass, and I can use jefe as usual.

However, this increases flash usage significantly, at least on test/tests-stm32h7/app-h753.toml

Before (master, 7cace18d):

flash:
      ADDRESS  | PROGRAM |  USED |  SIZE | LIMIT
    0x08000000 | kernel | 32768 | 32768 | (fixed)
    0x08008000 | runner | 13456 | 16384 | 32768
    0x0800c000 | assist | 10240 | 16384 | 32768
    0x08010000 | suite  | 40428 | 65536 | 65536
    0x08020000 | idol   |  3700 |  4096 | 4096
    0x08021000 | idle   |    84 |   128 | 256
    0x08021080 | --end-- |       |       |

ram:
      ADDRESS  | PROGRAM |  USED |  SIZE | LIMIT
    0x20000000 | kernel |  2100 |  4096 | (fixed)
    0x20001000 | runner |  2184 |  4096 | 4096
    0x20002000 | suite  |  2176 |  4096 | 4096
    0x20003000 | assist |  2176 |  4096 | 4096
    0x20004000 | idol   |  1024 |  1024 | 1024
    0x20004400 | idle   |   256 |   256 | 256
    0x20004500 | --end-- |       |       |

After (44cb16f0):

flash:
      ADDRESS  | PROGRAM |  USED |  SIZE | LIMIT
    0x08000000 | kernel | 32768 | 32768 | (fixed)
    0x08008000 | runner | 20064 | 32768 | 32768
    0x08010000 | suite  | 40608 | 65536 | 65536
    0x08020000 | assist | 17292 | 32768 | 32768
    0x08028000 | idol   |  3732 |  4096 | 4096
    0x08029000 | idle   |    84 |   128 | 256
    0x08029080 | --end-- |       |       |

ram:
      ADDRESS  | PROGRAM |  USED |  SIZE | LIMIT
    0x20000000 | kernel |  2100 |  4096 | (fixed)
    0x20001000 | runner |  2184 |  4096 | 4096
    0x20002000 | suite  |  2176 |  4096 | 4096
    0x20003000 | assist |  2176 |  4096 | 4096
    0x20004000 | idol   |  1024 |  1024 | 1024
    0x20004400 | idle   |   256 |   256 | 256
    0x20004500 | --end-- |       |       |

runner increased by 6.6K, and assist by 7K.

@mkeeter mkeeter added the kernel Relates to the Hubris kernel label Aug 9, 2022
@mkeeter mkeeter requested review from cbiffle and jgallagher and removed request for jgallagher August 9, 2022 14:41
@mkeeter mkeeter marked this pull request as draft August 9, 2022 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel Relates to the Hubris kernel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant