Skip to content

Cleanups in extent_io callbacks

David Sterba linux-btrfs 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4btrfs: let writepage_end_io_hook return voidfs/btrfs/extent_io.c fs/btrfs/extent_io.h fs/btrfs/inode.c+6 -111 reply
2/4btrfs: document existence of extent_io ops callbacksfs/btrfs/disk-io.c fs/btrfs/extent_io.h fs/btrfs/inode.c+26 -110 replies
3/4btrfs: drop checks for mandatory extent_io_ops callbacksfs/btrfs/extent_io.c+3 -40 replies
4/4btrfs: add dummy callback for readpage_io_failed and drop checksfs/btrfs/disk-io.c fs/btrfs/extent_io.c fs/btrfs/extent_io.h +1 more+10 -31 reply

Reviews trailers per patch

1 of 4 patches has a Reviewed-by or Acked-by.

PatchLiu Bo
1/4R-b
2/4
3/4
4/4

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

Read the v1 thread

Cover letter [PATCH 0/4] Cleanups in extent_io callbacks

Some of the checks for extent_io callbacks can be safely dropped as they're always defined, plus some dummy callback additions so more checks can be dropped. There's more potential for the same cleanup in other callbacks but this would need more evaluation wheather dummy callbacks vs existence checks are really worth it.

David Sterba (4):
btrfs: let writepage_end_io_hook return void
btrfs: document existence of extent_io ops callbacks
btrfs: drop checks for mandatory extent_io_ops callbacks
btrfs: add dummy callback for readpage_io_failed and drop checks

 fs/btrfs/disk-io.c   |  7 +++++--
 fs/btrfs/extent_io.c | 18 +++++++-----------
 fs/btrfs/extent_io.h | 25 +++++++++++++++++--------
 fs/btrfs/inode.c     | 20 ++++++++++++++------
 4 files changed, 43 insertions(+), 27 deletions(-)