The patch titled Subject: fs/9p: remove unnecessary new_valid_dev() checks has been added to the -mm tree. Its filename is fs-vfs-remove-unnecessary-new_valid_dev-check.patch This patch should soon appear at http://ozlabs.org/~akpm/mmots/broken-out/fs-vfs-remove-unnecessary-new_valid_dev-check.patch and later at http://ozlabs.org/~akpm/mmotm/broken-out/fs-vfs-remove-unnecessary-new_valid_dev-check.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Yaowei Bai <bywxiaobai@xxxxxxx> Subject: fs/9p: remove unnecessary new_valid_dev() checks new_valid_dev() always returns 1, so the !new_valid_dev() check is not needed. Remove it. Signed-off-by: Yaowei Bai <bywxiaobai@xxxxxxx> Cc: Alexander Viro <viro@xxxxxxxxxxxxxxxxxx> Cc: Eric Van Hensbergen <ericvh@xxxxxxxxx> Cc: Ron Minnich <rminnich@xxxxxxxxxx> Cc: Latchesar Ionkov <lucho@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- fs/9p/vfs_inode.c | 3 --- fs/9p/vfs_inode_dotl.c | 3 --- 2 files changed, 6 deletions(-) diff -puN fs/9p/vfs_inode.c~fs-vfs-remove-unnecessary-new_valid_dev-check fs/9p/vfs_inode.c --- a/fs/9p/vfs_inode.c~fs-vfs-remove-unnecessary-new_valid_dev-check +++ a/fs/9p/vfs_inode.c @@ -1368,9 +1368,6 @@ v9fs_vfs_mknod(struct inode *dir, struct dir->i_ino, dentry, mode, MAJOR(rdev), MINOR(rdev)); - if (!new_valid_dev(rdev)) - return -EINVAL; - /* build extension */ if (S_ISBLK(mode)) sprintf(name, "b %u %u", MAJOR(rdev), MINOR(rdev)); diff -puN fs/9p/vfs_inode_dotl.c~fs-vfs-remove-unnecessary-new_valid_dev-check fs/9p/vfs_inode_dotl.c --- a/fs/9p/vfs_inode_dotl.c~fs-vfs-remove-unnecessary-new_valid_dev-check +++ a/fs/9p/vfs_inode_dotl.c @@ -829,9 +829,6 @@ v9fs_vfs_mknod_dotl(struct inode *dir, s dir->i_ino, dentry, omode, MAJOR(rdev), MINOR(rdev)); - if (!new_valid_dev(rdev)) - return -EINVAL; - v9ses = v9fs_inode2v9ses(dir); dir_dentry = dentry->d_parent; dfid = v9fs_fid_lookup(dir_dentry); _ Patches currently in -mm which might be from bywxiaobai@xxxxxxx are kernel-watchdog-is_hardlockup-can-be-boolean.patch mm-page_alloc-remove-unused-parameter-in-init_currently_empty_zone.patch mm-vmscan-make-inactive_anon_is_low_global-return-directly.patch mm-oom_kill-introduce-is_sysrq_oom-helper.patch mm-compaction-add-an-is_via_compact_memory-helper-function.patch mm-vmscan-make-inactive_anon-file_is_low-return-bool.patch mm-memcontrol-make-mem_cgroup_inactive_anon_is_low-return-bool.patch fs-kdev_t-remove-unused-huge_valid_dev-function.patch fs-kdev_t-old-new_valid_dev-can-be-boolean.patch fs-vfs-remove-unnecessary-new_valid_dev-check.patch fs-btrfs-remove-unnecessary-new_valid_dev-check.patch fs-exofs-remove-unnecessary-new_valid_dev-check.patch fs-ext2-remove-unnecessary-new_valid_dev-check.patch fs-ext4-remove-unnecessary-new_valid_dev-check.patch fs-f2fs-remove-unnecessary-new_valid_dev-check.patch fs-hpfs-remove-unnecessary-new_valid_dev-check.patch fs-jfs-remove-unnecessary-new_valid_dev-check.patch fs-ncpfs-remove-unnecessary-new_valid_dev-check.patch fs-nfs-remove-unnecessary-new_valid_dev-check.patch fs-nilfs2-remove-unnecessary-new_valid_dev-check.patch fs-reiserfs-remove-unnecessary-new_valid_dev-check.patch fs-stat-remove-unnecessary-new_valid_dev-check.patch fs-ubifs-remove-unnecessary-new_valid_dev-check.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