diff options
| author | Rasmus Villemoes <linux@rasmusvillemoes.dk> | 2016-09-15 00:25:03 +0200 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2016-09-27 21:45:46 -0400 |
| commit | de04e76935ad5985d318fbce298a17e9dd2092b7 (patch) | |
| tree | 6f334950fb361d3e1fe57dd264e66876887d3ac8 /fs/aio.c | |
| parent | be218aa2e3f7aa698cdce5a4efb1e178677db8fd (diff) | |
| download | lwn-de04e76935ad5985d318fbce298a17e9dd2092b7.tar.gz lwn-de04e76935ad5985d318fbce298a17e9dd2092b7.zip | |
fs/aio.c: eliminate redundant loads in put_aio_ring_file
Using a local variable we can prevent gcc from reloading
aio_ring_file->f_inode->i_mapping twice, eliminating 2x2 dependent
loads.
Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/aio.c')
| -rw-r--r-- | fs/aio.c | 9 |
1 files changed, 6 insertions, 3 deletions
@@ -274,14 +274,17 @@ __initcall(aio_setup); static void put_aio_ring_file(struct kioctx *ctx) { struct file *aio_ring_file = ctx->aio_ring_file; + struct address_space *i_mapping; + if (aio_ring_file) { truncate_setsize(aio_ring_file->f_inode, 0); /* Prevent further access to the kioctx from migratepages */ - spin_lock(&aio_ring_file->f_inode->i_mapping->private_lock); - aio_ring_file->f_inode->i_mapping->private_data = NULL; + i_mapping = aio_ring_file->f_inode->i_mapping; + spin_lock(&i_mapping->private_lock); + i_mapping->private_data = NULL; ctx->aio_ring_file = NULL; - spin_unlock(&aio_ring_file->f_inode->i_mapping->private_lock); + spin_unlock(&i_mapping->private_lock); fput(aio_ring_file); } |