Messages in this thread Patch in this message | | | From | Leon Romanovsky <> | Subject | [PATCH] fs: Delete always true compilation define | Date | Thu, 11 Jun 2020 16:02:37 +0300 |
| |
From: Leon Romanovsky <leonro@mellanox.com>
In commit 1027abe8827b ("[PATCH] merge locate_fd() and get_unused_fd()") the existing "#if 1" was moved from one place to another. Originally that compilation define was set for the sanity check and more than 12 years later it is safe to remove it.
Signed-off-by: Leon Romanovsky <leonro@mellanox.com> --- fs/file.c | 2 -- 1 file changed, 2 deletions(-)
diff --git a/fs/file.c b/fs/file.c index c8a4e4c86e55..cab9d55765dd 100644 --- a/fs/file.c +++ b/fs/file.c @@ -522,13 +522,11 @@ int __alloc_fd(struct files_struct *files, else __clear_close_on_exec(fd, fdt); error = fd; -#if 1 /* Sanity check */ if (rcu_access_pointer(fdt->fd[fd]) != NULL) { printk(KERN_WARNING "alloc_fd: slot %d not NULL!\n", fd); rcu_assign_pointer(fdt->fd[fd], NULL); } -#endif
out: spin_unlock(&files->file_lock); -- 2.26.2
| |