shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-21 13:34:40 +03:00
Code Releases Activity

Move close few lines

Browse Source 
Since the function dev_close() has code path, which really could close
file (for unlocked vg) and destroy dev handler, stay on safe side and move
the close few lines later, even our current use case shouldn't trigger
such scenario.
This commit is contained in:
Zdenek Kabelac 2012-02-08 11:15:38 +00:00
parent 02aeb23f1f
commit 1a39fa0073
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/device/dev-io.c
View File

@ -1,6 +1,6 @@
/* /*
* Copyright (C) 2001-2004 Sistina Software, Inc. All rights reserved. * Copyright (C) 2001-2004 Sistina Software, Inc. All rights reserved.
* Copyright (C) 2004-2007 Red Hat, Inc. All rights reserved. * Copyright (C) 2004-2012 Red Hat, Inc. All rights reserved.
* *
* This file is part of LVM2. * This file is part of LVM2.
* *
@ -292,15 +292,15 @@ static int _dev_read_ahead_dev(struct device *dev, uint32_t *read_ahead)
return 0; return 0;
} }
if (!dev_close(dev))
stack;
*read_ahead = (uint32_t) read_ahead_long; *read_ahead = (uint32_t) read_ahead_long;
dev->read_ahead = read_ahead_long; dev->read_ahead = read_ahead_long;
log_very_verbose("%s: read_ahead is %u sectors", log_very_verbose("%s: read_ahead is %u sectors",
dev_name(dev), *read_ahead); dev_name(dev), *read_ahead);
if (!dev_close(dev))
stack;
return 1; return 1;
} }