The patch titled fs-introduce-write_begin-write_end-and-perform_write-aops fix 4 has been added to the -mm tree. Its filename is fs-introduce-write_begin-write_end-and-perform_write-aops-fix-4.patch *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find out what to do about this ------------------------------------------------------ Subject: fs-introduce-write_begin-write_end-and-perform_write-aops fix 4 From: Nick Piggin <npiggin@xxxxxxx> Dmitriy noticed that iov_iter_fault_in_readable could go past the end of the first iov in a multi-iov situation, and that could be considered an EFAULT by the caller. Fix and comment. Fixes: fs-introduce-write_begin-write_end-and-perform_write-aops.patch Signed-off-by: Nick Piggin <npiggin@xxxxxxx> Cc: Badari Pulavarty <pbadari@xxxxxxxxx> Cc: Dmitriy Monakhov <dmonakhov@xxxxx> Cc: Mark Fasheh <mark.fasheh@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- mm/filemap.c | 10 ++++++++++ 1 files changed, 10 insertions(+) diff -puN mm/filemap.c~fs-introduce-write_begin-write_end-and-perform_write-aops-fix-4 mm/filemap.c --- a/mm/filemap.c~fs-introduce-write_begin-write_end-and-perform_write-aops-fix-4 +++ a/mm/filemap.c @@ -1794,9 +1794,19 @@ void iov_iter_advance(struct iov_iter *i i->count -= bytes; } +/* + * Fault in the first iovec of the given iov_iter, to a maximum length + * of bytes. Returns 0 on success, or non-zero if the memory could not be + * accessed (ie. because it is an invalid address). + * + * writev-intensive code may want this to prefault several iovecs -- that + * would be possible (callers must not rely on the fact that _only_ the + * first iovec will be faulted with the current implementation). + */ int iov_iter_fault_in_readable(struct iov_iter *i, size_t bytes) { char __user *buf = i->iov->iov_base + i->iov_offset; + bytes = min(bytes, i->iov->iov_len - i->iov_offset); return fault_in_pages_readable(buf, bytes); } _ Patches currently in -mm which might be from npiggin@xxxxxxx are mm-kill-validate_anon_vma-to-avoid-mapcount-bug.patch slob-rework-freelist-handling.patch slob-remove-bigblock-tracking.patch slob-improved-alignment-handling.patch mm-fix-fault-vs-invalidate-race-for-linear-mappings.patch mm-fix-fault-vs-invalidate-race-for-linear-mappings-fix.patch mm-merge-populate-and-nopage-into-fault-fixes-nonlinear.patch mm-merge-nopfn-into-fault.patch mm-merge-nopfn-into-fault-spufs-fix.patch mm-remove-legacy-cruft.patch mm-debug-check-for-the-fault-vs-invalidate-race.patch mm-fix-clear_page_dirty_for_io-vs-fault-race.patch mm-revert-kernel_ds-buffered-write-optimisation.patch revert-81b0c8713385ce1b1b9058e916edcf9561ad76d6.patch revert-6527c2bdf1f833cc18e8f42bd97973d583e4aa83.patch mm-clean-up-buffered-write-code.patch mm-debug-write-deadlocks.patch mm-trim-more-holes.patch mm-buffered-write-cleanup.patch mm-write-iovec-cleanup.patch mm-fix-pagecache-write-deadlocks.patch mm-buffered-write-iterator.patch fs-fix-data-loss-on-error.patch fs-introduce-write_begin-write_end-and-perform_write-aops.patch fs-introduce-write_begin-write_end-and-perform_write-aops-fix.patch fs-introduce-write_begin-write_end-and-perform_write-aops-fix-3.patch fs-introduce-write_begin-write_end-and-perform_write-aops-fix-4.patch mm-restore-kernel_ds-optimisations.patch implement-simple-fs-aops.patch block_dev-convert-to-new-aops.patch ext2-convert-to-new-aops.patch ext3-convert-to-new-aops.patch ext4-convert-to-new-aops.patch xfs-convert-to-new-aops.patch fs-new-cont-helpers.patch fat-convert-to-new-aops.patch hfs-convert-to-new-aops.patch hfsplus-convert-to-new-aops.patch hpfs-convert-to-new-aops.patch bfs-convert-to-new-aops.patch qnx4-convert-to-new-aops.patch reiserfs-use-generic-write.patch reiserfs-convert-to-new-aops.patch reiserfs-use-generic_cont_expand_simple.patch with-reiserfs-no-longer-using-the-weird-generic_cont_expand-remove-it-completely.patch nfs-convert-to-new-aops.patch smb-convert-to-new-aops.patch fuse-convert-to-new-aops.patch hostfs-convert-to-new-aops.patch jffs2-convert-to-new-aops.patch ufs-convert-to-new-aops.patch udf-convert-to-new-aops.patch sysv-convert-to-new-aops.patch minix-convert-to-new-aops.patch jfs-convert-to-new-aops.patch fix-read-truncate-race.patch mm-document-fault_data-and-flags.patch fs-introduce-some-page-buffer-invariants.patch fs-reiserfs-cleanups.patch fs-introduce-write_begin-write_end-and-perform_write-aops-revoke.patch - To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html