Skip to content

nilfs2: add support for FITRIM ioctl

Andreas Rohner linux-nilfs 3 versions – Applied

Status from this message: Applied to my queue for upstream.

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v4 Latest2 patches 1 replyApplied
  2. v32 patches 2 repliesSuperseded
  3. v12 patches 3 repliesSuperseded

Changes in v4 from the cover letter

* Fix integer overflow
* Add comment

Patches in v4 Applied

#Subject and filesSizeRepliesApplied
1/2nilfs2: add nilfs_sufile_trim_fs to trim clean segsfs/nilfs2/sufile.c fs/nilfs2/sufile.h+153 -00 replies
2/2nilfs2: add FITRIM ioctl support for nilfs2fs/nilfs2/ioctl.c+45 -00 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v4 yet.

Read the v4 thread

Cover letter [PATCH v4 0/2] nilfs2: add support for FITRIM ioctl

Hi,

This patch adds support for the FITRIM ioctl, which allows user space tools like fstrim to issue TRIM/DISCARD requests to the underlying device. It takes a fstrim_range structure as a parameter and for every clean segment in the specified range the function blkdev_issue_discard is called. The range is truncated to file system block boundaries.

I tested it with a 32 bit and 64 bit kernel. On the 32 bit system CONFIG_LBDAF was disabled so that sector_t was 32 bit.

Best regards,
Andreas Rohner

---
v3->v4 (based on review by Ryusuke Konishi)
* Fix integer overflow
* Add comment
v2->v3 (based on review by Ryusuke Konishi)
* Fix integer overflow
* Round range to block boundary instead of sector boundary
* Move range check to nilfs_sufile_trim_fs()
v1->v2 (based on review by Ryusuke Konishi)
* Remove upper limit of minlen
* Add check for minlen
* Round range to sector boundary instead of segment boundary
* Fix minor bug
* Use kmap_atomic instead of kmap
* Move input checks to ioctl.c
* Use nilfs_sufile_segment_usages_in_block()
--

Andreas Rohner (2):
nilfs2: add nilfs_sufile_trim_fs to trim clean segs
nilfs2: add FITRIM ioctl support for nilfs2

 fs/nilfs2/ioctl.c  |  45 ++++++++++++++++
 fs/nilfs2/sufile.c | 152 +++++++++++++++++++++++++++++++++++++++++++++++++++++
 fs/nilfs2/sufile.h |   1 +
 3 files changed, 198 insertions(+)
Signature · 6 lines
1.9.0

--
To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Read the cover letter in its thread