lkml.org 
[lkml]   [2021]   [Mar]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.11 297/775] ubifs: replay: Fix high stack usage, again
    Date
    From: Arnd Bergmann <arnd@arndb.de>

    [ Upstream commit 410b6de702ef84fea6e7abcb6620ef8bfc112fae ]

    An earlier commit moved out some functions to not be inlined by gcc, but
    after some other rework to remove one of those, clang started inlining
    the other one and ran into the same problem as gcc did before:

    fs/ubifs/replay.c:1174:5: error: stack frame size of 1152 bytes in function 'ubifs_replay_journal' [-Werror,-Wframe-larger-than=]

    Mark the function as noinline_for_stack to ensure it doesn't happen
    again.

    Fixes: f80df3851246 ("ubifs: use crypto_shash_tfm_digest()")
    Fixes: eb66eff6636d ("ubifs: replay: Fix high stack usage")
    Signed-off-by: Arnd Bergmann <arnd@arndb.de>
    Reviewed-by: Nathan Chancellor <natechancellor@gmail.com>
    Signed-off-by: Richard Weinberger <richard@nod.at>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    fs/ubifs/replay.c | 4 +++-
    1 file changed, 3 insertions(+), 1 deletion(-)

    diff --git a/fs/ubifs/replay.c b/fs/ubifs/replay.c
    index 79801c9a5b874..0f8a6a16421b4 100644
    --- a/fs/ubifs/replay.c
    +++ b/fs/ubifs/replay.c
    @@ -559,7 +559,9 @@ static int is_last_bud(struct ubifs_info *c, struct ubifs_bud *bud)
    }

    /* authenticate_sleb_hash is split out for stack usage */
    -static int authenticate_sleb_hash(struct ubifs_info *c, struct shash_desc *log_hash, u8 *hash)
    +static int noinline_for_stack
    +authenticate_sleb_hash(struct ubifs_info *c,
    + struct shash_desc *log_hash, u8 *hash)
    {
    SHASH_DESC_ON_STACK(hash_desc, c->hash_tfm);

    --
    2.27.0


    \
     
     \ /
      Last update: 2021-03-02 00:40    [W:4.106 / U:0.436 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site