From: "Michał Kępień" <kernel@kempniu.pl>
To: Miquel Raynal <miquel.raynal@bootlin.com>,
Richard Weinberger <richard@nod.at>,
Vignesh Raghavendra <vigneshr@ti.com>
Cc: Boris Brezillon <boris.brezillon@collabora.com>,
linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: [PATCH v3 0/4] mtdchar: add MEMREAD ioctl
Date: Tue, 25 Jan 2022 11:48:18 +0100 [thread overview]
Message-ID: <20220125104822.8420-1-kernel@kempniu.pl> (raw)
This patch series adds a new mtdchar ioctl, MEMREAD. Its purpose is to
serve as a read counterpart of the MEMWRITE ioctl, exposing a broader
set of capabilities for read operations (e.g. use of MTD_OPS_AUTO_OOB,
access to ECC statistics) to user-space applications making use of MTD
devices via /dev/mtd* character devices.
Changes from v2:
- Squashed patch 1/5 into patch 5/5 to prevent breaking bisectability
due to an incompatible ABI change between those two patches.
Revised commit messages accordingly.
Changes from v1:
- Added patches 2-5 which enable the new MEMREAD ioctl to report ECC
statistics for the read operation back to user space. (There are
obviously different ways these changes can be split up into separate
commits; I was aiming for maximum ease of review.)
- The 'retlen' and 'oobretlen' fields were not set in the struct
mtd_read_req returned to userspace. This was done properly in
Boris' original draft patch [1], but I missed it in my v1.
- Invalid IS_ERR() checks were replaced with NULL checks. This was an
artifact of copy-pasting mtdchar_write_ioctl() in v1: unlike
memdup_user() used therein, kmalloc() always returns NULL on error.
- Minor subject prefix adjustment for patch 1/5 ("mtd" -> "mtdchar").
Michał Kępień (4):
mtd: track maximum number of bitflips for each read request
mtd: always initialize 'stats' in struct mtd_oob_ops
mtd: add ECC error accounting for each read request
mtdchar: add MEMREAD ioctl
drivers/mtd/devices/docg3.c | 8 ++
drivers/mtd/inftlcore.c | 6 +-
drivers/mtd/mtdchar.c | 136 ++++++++++++++++++++++++
drivers/mtd/mtdcore.c | 5 +
drivers/mtd/mtdswap.c | 6 +-
drivers/mtd/nand/onenand/onenand_base.c | 16 ++-
drivers/mtd/nand/onenand/onenand_bbt.c | 2 +-
drivers/mtd/nand/raw/nand_base.c | 10 ++
drivers/mtd/nand/raw/nand_bbt.c | 8 +-
drivers/mtd/nand/raw/sm_common.c | 2 +-
drivers/mtd/nand/spi/core.c | 10 ++
drivers/mtd/nftlcore.c | 6 +-
drivers/mtd/sm_ftl.c | 4 +-
drivers/mtd/ssfdc.c | 2 +-
drivers/mtd/tests/nandbiterrs.c | 2 +-
drivers/mtd/tests/oobtest.c | 8 +-
drivers/mtd/tests/readtest.c | 2 +-
fs/jffs2/wbuf.c | 6 +-
include/linux/mtd/mtd.h | 7 ++
include/uapi/mtd/mtd-abi.h | 64 ++++++++++-
20 files changed, 276 insertions(+), 34 deletions(-)
--
2.34.1
next reply other threads:[~2022-01-25 10:52 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-25 10:48 Michał Kępień [this message]
2022-01-25 10:48 ` [PATCH v3 1/4] mtd: track maximum number of bitflips for each read request Michał Kępień
2022-01-25 10:48 ` [PATCH v3 2/4] mtd: always initialize 'stats' in struct mtd_oob_ops Michał Kępień
2022-01-25 10:48 ` [PATCH v3 3/4] mtd: add ECC error accounting for each read request Michał Kępień
2022-01-25 10:48 ` [PATCH v3 4/4] mtdchar: add MEMREAD ioctl Michał Kępień
2022-02-03 9:18 ` Richard Weinberger
2022-02-03 9:46 ` Miquel Raynal
2022-02-03 10:13 ` Richard Weinberger
2022-02-14 11:18 ` Michał Kępień
2022-02-14 11:15 ` Michał Kępień
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220125104822.8420-1-kernel@kempniu.pl \
--to=kernel@kempniu.pl \
--cc=boris.brezillon@collabora.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=miquel.raynal@bootlin.com \
--cc=richard@nod.at \
--cc=vigneshr@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox