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.

Sign up for GitHub

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

Jump to bottom

[PATCH v5] quell GCC 13.1 bound check warnings #1838

Merged
merged 3 commits into from
Dec 21, 2023
Merged

[PATCH v5] quell GCC 13.1 bound check warnings #1838

merged 3 commits into from
Dec 21, 2023

Commits on Dec 21, 2023

  1. quell false positive GCC 13.1 warnings 

    Quell false positive GCC 13.1 -Warray-bounds and -Wstringop-overflow
warnings.

The warnings still appear during LTO linking, because GCC does not
carry diagnostic pragmas to the linking stage.

Signed-off-by: Jere Leppänen <jere.leppanen@nokia.com>
Reviewed-by: Janne Peltonen <janne.peltonen@nokia.com>
    @JereLeppanen
    JereLeppanen committed Dec 21, 2023
    Configuration menu
    Copy the full SHA
    4e8212b View commit details
    Browse the repository at this point in the history

  2. linux-gen: pktio: fix flexible array member error in pedantic compila… 

    …tion

When compiling with GCC 13.1 and -Wpedantic, struct sset_info causes
the error "invalid use of structure with flexible array member
[-Werror=pedantic]". The flexible array member in this case is the
data member of struct ethtool_sset_info. In order to reserve space for
it, turn sset_info into a union, and make the next member in the union
the size of struct ethtool_sset_info and the data.

Signed-off-by: Jere Leppänen <jere.leppanen@nokia.com>
Reviewed-by: Janne Peltonen <janne.peltonen@nokia.com>
    @JereLeppanen
    JereLeppanen committed Dec 21, 2023
    Configuration menu
    Copy the full SHA
    013b0f8 View commit details
    Browse the repository at this point in the history

  3. configure: add -Wno-error=stringop-overflow also to CXXFLAGS 

    Also c++ programs need -Wno-error=stringop-overflow flag, in order to
avoid errors during linking, when using LTO.

Signed-off-by: Jere Leppänen <jere.leppanen@nokia.com>
Reviewed-by: Janne Peltonen <janne.peltonen@nokia.com>
    @JereLeppanen
    JereLeppanen committed Dec 21, 2023
    Configuration menu
    Copy the full SHA
    f669ef5 View commit details
    Browse the repository at this point in the history