linux-kernel - Re: [PATCH] jffs2: reduce stack usage in jffs2_build_xattr

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20230515-sinnbild-boten-fa2d5510a52c@brauner>
Date:   Mon, 15 May 2023 12:46:50 +0200
From:   Christian Brauner <brauner@...nel.org>
To:     David Woodhouse <dwmw2@...radead.org>,
        Richard Weinberger <richard@....at>,
        Fabian Frederick <fabf@...net.be>,
        Nick Desaulniers <ndesaulniers@...gle.com>,
        KaiGai Kohei <kaigai@...jp.nec.com>,
        linux-mtd@...ts.infradead.org, linux-kernel@...r.kernel.org,
        Christian Marangi <ansuelsmth@...il.com>
Cc:     Christian Brauner <brauner@...nel.org>,
        Tim Gardner <tim.gardner@...onical.com>,
        kernel test robot <lkp@...el.com>,
        Ron Economos <re@...z.net>,
        Nathan Chancellor <nathan@...nel.org>, stable@...r.kernel.org
Subject: Re: [PATCH] jffs2: reduce stack usage in jffs2_build_xattr_subsystem()

On Sat, 06 May 2023 06:56:12 +0200, Christian Marangi wrote:
> Use kcalloc() for allocation/flush of 128 pointers table to
> reduce stack usage.
> 
> Function now returns -ENOMEM or 0 on success.
> 
> stackusage
> Before:
> ./fs/jffs2/xattr.c:775  jffs2_build_xattr_subsystem     1208
> dynamic,bounded
> 
> [...]

I assume I was Cced to pick this up. I'm happy to do that. If this is
rather supposed to go through the jffs2 tree then please tell me so we
can drop it.

Applied to the vfs.misc branch of the vfs/vfs.git tree.
Patches in the vfs.misc branch should appear in linux-next soon.

Please report any outstanding bugs that were missed during review in a
new review to the original patch series allowing us to drop it.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git
branch: vfs.misc

[1/1] jffs2: reduce stack usage in jffs2_build_xattr_subsystem()
      https://git.kernel.org/vfs/vfs/c/493e7cebb906