summaryrefslogtreecommitdiffstats
path: root/fs/xfs/xfs_dir2.c
diff options
context:
space:
mode:
authorOlof Johansson <olof@lixom.net>2013-09-09 10:34:23 -0700
committerLinus Torvalds <torvalds@linux-foundation.org>2013-09-09 10:47:42 -0700
commit45150c43b1b0c16e665fd0a5cdcca128b8192db1 (patch)
treee1faec166f3edb5974829d9f7ec4c25e02fd1572 /fs/xfs/xfs_dir2.c
parentef9a61bef917e38f8e096f6df303329aed6cf467 (diff)
downloadop-kernel-dev-45150c43b1b0c16e665fd0a5cdcca128b8192db1.zip
op-kernel-dev-45150c43b1b0c16e665fd0a5cdcca128b8192db1.tar.gz
direct-io: Use return from cmpxchg to decide of assignment happened
Not using the return value can in the generic case be racy, so it's in general good practice to check the return value instead. This also resolved the warning caused on ARM and other architectures: fs/direct-io.c: In function 'sb_init_dio_done_wq': fs/direct-io.c:557:2: warning: value computed is not used [-Wunused-value] Signed-off-by: Olof Johansson <olof@lixom.net> Reviewed-by: Jan Kara <jack@suse.cz> Cc: Geert Uytterhoeven <geert@linux-m68k.org> Cc: Stephen Rothwell <sfr@canb.auug.org.au> Cc: Al Viro <viro@zeniv.linux.org.uk> Cc: Christoph Hellwig <hch@infradead.org> Cc: Russell King <linux@arm.linux.org.uk> Cc: H Peter Anvin <hpa@zytor.com> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/xfs/xfs_dir2.c')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud