diff options
author | Andrew Morton <akpm@osdl.org> | 2006-09-29 05:01:44 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-09-29 12:18:25 -0400 |
commit | 327dcaadc0bc08ad081aa8e36b6ec7ad7aa45e30 (patch) | |
tree | 68bec9183fcee79370e076244be347dd20d5dd9f | |
parent | 74d392aaabfc890cc1f0e80fc5ff13e5d3bcf4c9 (diff) |
[PATCH] expand_fdtable(): remove pointless unlock+lock
This unlock/lock on a super-unlikely path isn't worth the kernel text.
Cc: Vadim Lobanov <vlobanov@speakeasy.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | fs/file.c | 2 |
1 files changed, 0 insertions, 2 deletions
@@ -317,9 +317,7 @@ static int expand_fdtable(struct files_struct *files, int nr) | |||
317 | free_fdtable(cur_fdt); | 317 | free_fdtable(cur_fdt); |
318 | } else { | 318 | } else { |
319 | /* Somebody else expanded, so undo our attempt */ | 319 | /* Somebody else expanded, so undo our attempt */ |
320 | spin_unlock(&files->file_lock); | ||
321 | __free_fdtable(new_fdt); | 320 | __free_fdtable(new_fdt); |
322 | spin_lock(&files->file_lock); | ||
323 | } | 321 | } |
324 | return 1; | 322 | return 1; |
325 | } | 323 | } |