Skip to content

ccp_priv_state takes up a lot more memory than it needs to #34

@fcangialosi

Description

@fcangialosi

currently, sizeof(struct ccp_priv_state) = 1952, which is dominated by assuming a maximum of 110 control registers and 110 report registers, which are each 8 bytes.

  1. This seems like way more than necessary. We could make this smaller by default and configurable as a kernel module option if someone needs an especially large program.

  2. Better yet, now that init_programs is only run once at the beginning, we could dynamically allocate the space for registers when programs are installed, that way we store exactly as much space for registers as we need.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions