Skip to content

xfs: seek_data/seek_hole refinements

Jeff Liu linux-xfs 5 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v8 Latest4 patches 2 repliesIn review
  2. v74 patches 12 repliesSuperseded
  3. v64 patches 13 repliesSuperseded
Earlier versions · 2
VersionPostedPatchesRepliesStatus
v5 posting 2/20 of 4 patches0 repliesSuperseded
v5 posting 1/24 patches13 repliesSuperseded

Changes from v8 to v7 from the cover letter

- xfs_find_get_pgoff()
. Sample the next page index without page lock. Also, add comments to clarify why it's safe to check page->index without page lock. . Remove (pgoff_t) PAGEVEC_SIZE.

- xfs_seek_data()/xfs_seek_hole(), ASSERT(i > 1).

Patches in v8 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4xfs: Remove type argument from xfs_seek_data()/xfs_seek_hole()fs/xfs/xfs_file.c+4 -60 replies
2/4xfs: Introduce a helper routine to probe data or hole offset from page cachefs/xfs/xfs_file.c+219 -02 replies
3/4xfs: xfs_seek_data() refinement with unwritten extents check up from page cachefs/xfs/xfs_file.c+54 -180 replies
4/4xfs: xfs_seek_hole() refinement with hole searching from page cache for unwritten extentsfs/xfs/xfs_file.c+67 -110 replies

Reviews trailers per patch

All 4 patches have a Reviewed-by or Acked-by.

PatchMark TinguelyDave Chinner
1/4R-b R-b
2/4R-b R-b
3/4R-b R-b
4/4R-b R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v8 thread

Cover letter [PATCH v8 0/4] xfs: seek_data/seek_hole refinements

Hello,

Here is the updated patch set of refine xfs_seek_data/xfs_seek_hole with page cache lookup for unwritten extents. Hopefully, this version is qualified to be merged. :)

Changes from v8 to v7:

- xfs_find_get_pgoff()
. Sample the next page index without page lock. Also, add comments to clarify why it's safe to check page->index without page lock. . Remove (pgoff_t) PAGEVEC_SIZE.

- xfs_seek_data()/xfs_seek_hole(), ASSERT(i > 1).

Old version:

Version 7 of this posting can be found at
http://www.spinics.net/lists/xfs/msg13171.html

Version 6 of this posting can be found at
http://oss.sgi.com/archives/xfs/2012-08/msg00028.html

Version 5 of this posting can be found at
http://www.spinics.net/lists/xfs/msg12987.html

Version 4 of this posting can be found at
http://oss.sgi.com/archives/xfs/2012-07/msg00279.html

Version 3 of this posting can be found at
http://oss.sgi.com/archives/xfs/2012-07/msg00050.html

Version 2 of this posting can be found at
http://oss.sgi.com/archives/xfs/2012-06/msg00407.html

Version 1 of this posting can be found at
http://www.digipedia.pl/usenet/thread/11980/26875/

Thanks,
-Jeff

_______________________________________________
xfs mailing list
[email protected]
http://oss.sgi.com/mailman/listinfo/xfs