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

sof: coherent: Don't mark "struct coherent" as packed #8528

Merged
merged 1 commit into from
Nov 28, 2023

Commits on Nov 27, 2023

  1. sof: coherent: Don't mark "struct coherent" as packed

    On 64-bit systems (i.e: i.MX93), marking "struct coherent"
    as packed leads to compilation warnings such as:
    
    "warning: taking address of packed member of 'struct coherent'
    may result in an unaligned pointer value"
    
    This is because the "list" attribute is found at offset 36,
    which is not 8B-aligned. In contrast, 32-bit systems
    don't have this problem as the "list" attribute is found at
    offset 24 which is 4B-aligned.
    
    Since marking "struct coherent" as packed doesn't seem to
    be necessary, remove this to fix the compilation warnings.
    On 32-bit systems, 'pahole' shows no difference between
    the 'packed' and the non-'packed' cases as the fields are
    already naturally aligned at the moment.
    
    A more detailed discussion regarding this issue can be
    found here: thesofproject#8521.
    
    Signed-off-by: Laurentiu Mihalcea <laurentiu.mihalcea@nxp.com>
    LaurentiuM1234 committed Nov 27, 2023
    Configuration menu
    Copy the full SHA
    c112d04 View commit details
    Browse the repository at this point in the history