Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   =?UTF-8?q?Micha=C5=82=20K=C4=99pie=C5=84?= <kernel@kempniu.pl>
To:     Miquel Raynal <miquel.raynal@bootlin.com>,
        Richard Weinberger <richard@nod.at>,
        Vignesh Raghavendra <vigneshr@ti.com>
Cc:     linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org
Message-ID: <20220516070601.11428-3-kernel@kempniu.pl>
Subject: [PATCH 2/2] mtdchar: use kvmalloc() for potentially large allocations
Date:   Mon, 16 May 2022 09:06:01 +0200
In-Reply-To: <20220516070601.11428-1-kernel@kempniu.pl>
References: <20220516070601.11428-1-kernel@kempniu.pl>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

mtdchar_write_ioctl() calls kmalloc() with the 'size' argument set to
the smaller of two values: the write request's data/OOB length provided
by user space and the erase block size of the MTD device.  If the latter
is large, kmalloc() may not be able to serve such allocation requests.
Use kvmalloc() instead.  Correspondingly, replace kfree() calls with
kvfree() calls.

Suggested-by: Richard Weinberger <richard@nod.at>
Signed-off-by: Micha=C5=82 K=C4=99pie=C5=84 <kernel@kempniu.pl>
---
 drivers/mtd/mtdchar.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/mtd/mtdchar.c b/drivers/mtd/mtdchar.c
index b2700f8467ff..05860288a7af 100644
--- a/drivers/mtd/mtdchar.c
+++ b/drivers/mtd/mtdchar.c
@@ -623,16 +623,16 @@ static int mtdchar_write_ioctl(struct mtd_info *mtd,
=20
 =09datbuf_len =3D min_t(size_t, req.len, mtd->erasesize);
 =09if (datbuf_len > 0) {
-=09=09datbuf =3D kmalloc(datbuf_len, GFP_KERNEL);
+=09=09datbuf =3D kvmalloc(datbuf_len, GFP_KERNEL);
 =09=09if (!datbuf)
 =09=09=09return -ENOMEM;
 =09}
=20
 =09oobbuf_len =3D min_t(size_t, req.ooblen, mtd->erasesize);
 =09if (oobbuf_len > 0) {
-=09=09oobbuf =3D kmalloc(oobbuf_len, GFP_KERNEL);
+=09=09oobbuf =3D kvmalloc(oobbuf_len, GFP_KERNEL);
 =09=09if (!oobbuf) {
-=09=09=09kfree(datbuf);
+=09=09=09kvfree(datbuf);
 =09=09=09return -ENOMEM;
 =09=09}
 =09}
@@ -682,8 +682,8 @@ static int mtdchar_write_ioctl(struct mtd_info *mtd,
 =09=09usr_oob +=3D ops.oobretlen;
 =09}
=20
-=09kfree(datbuf);
-=09kfree(oobbuf);
+=09kvfree(datbuf);
+=09kvfree(oobbuf);
=20
 =09return ret;
 }
--=20
2.36.1