Please find the series which moves ext2 direct-io to use modern iomap interface. RFCv2 -> RFCv3: 1. Addressed minor review comments related to extern, parameter naming in function declaration, removing not required braces and shorting overly long lines. 2. Added Reviewed-by from various reviewers. 3. Fixed a warning & couple of compilation errors in Patch-7 (ext2 trace points) related to CFLAGS_trace & second related to unable to find function definition for iov_iter_count(). (requires uio.h file) CFLAGS_trace is required in Makefile so that it can find trace.h file from tracepoint infrastructure. 4. Changed naming of IOCB_STRINGS TO TRACE_IOCB_STRINGS. 5. Shortened naming of tracepoint events for ext2 dio. 6. Added iomap DIO tracepoint events. 7. Disha tested this series internally against Power with "auto" group for 4k and 64k blocksize configuration. Added her "Tested-by" tag in all DIO related patches. No new failures were reported. Thanks everyone for the review and test. The series is looking good to me now. It has now been tested on x86 and Power with different configurations. Please let me know if anything else is required on this. v2: https://lore.kernel.org/all/ZDTybcM4kjYLSrGI@xxxxxxxxxxxxx/ Ritesh Harjani (IBM) (10): ext2/dax: Fix ext2_setsize when len is page aligned libfs: Add __generic_file_fsync_nolock implementation ext4: Use __generic_file_fsync_nolock implementation ext2: Use __generic_file_fsync_nolock implementation ext2: Move direct-io to use iomap fs.h: Add TRACE_IOCB_STRINGS for use in trace points ext2: Add direct-io trace points iomap: Remove IOMAP_DIO_NOSYNC unused dio flag iomap: Minor refactor of iomap_dio_rw iomap: Add trace points for DIO path fs/ext2/Makefile | 5 +- fs/ext2/ext2.h | 1 + fs/ext2/file.c | 127 +++++++++++++++++++++++++++++++++++++++++- fs/ext2/inode.c | 58 +++++++++++-------- fs/ext2/trace.c | 6 ++ fs/ext2/trace.h | 94 +++++++++++++++++++++++++++++++ fs/ext4/fsync.c | 31 +++++------ fs/iomap/direct-io.c | 16 ++++-- fs/iomap/trace.c | 1 + fs/iomap/trace.h | 90 ++++++++++++++++++++++++++++++ fs/libfs.c | 43 ++++++++++++++ include/linux/fs.h | 16 ++++++ include/linux/iomap.h | 6 -- 13 files changed, 444 insertions(+), 50 deletions(-) create mode 100644 fs/ext2/trace.c create mode 100644 fs/ext2/trace.h -- 2.39.2
