diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2009-07-19 08:44:27 -0500 |
|---|---|---|
| committer | Steven Rostedt <rostedt@rostedt.homelinux.com> | 2013-02-25 13:32:56 -0500 |
| commit | 6444ba4c1bcd9b8c923cf83a2ad26a643954cabf (patch) | |
| tree | 3c1e1eea67300974d510738d3ba108be88ef6163 /fs | |
| parent | f02b42801052dab41069e83b9f5c48aad6e9d12a (diff) | |
fs: namespace preemption fix
On RT we cannot loop with preemption disabled here as
mnt_make_readonly() might have been preempted. We can safely enable
preemption while waiting for MNT_WRITE_HOLD to be cleared. Safe on !RT
as well.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/namespace.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/fs/namespace.c b/fs/namespace.c index b3d8f51c6fa3..5e39984416f9 100644 --- a/fs/namespace.c +++ b/fs/namespace.c @@ -341,8 +341,14 @@ int mnt_want_write(struct vfsmount *mnt) * incremented count after it has set MNT_WRITE_HOLD. */ smp_mb(); - while (mnt->mnt_flags & MNT_WRITE_HOLD) + /* + * No need to keep preemption disabled accross the spin loop. + */ + while (mnt->mnt_flags & MNT_WRITE_HOLD) { + preempt_enable(); cpu_relax(); + preempt_disable(); + } /* * After the slowpath clears MNT_WRITE_HOLD, mnt_is_readonly will * be set to match its requirements. So we must not load that until @@ -352,9 +358,7 @@ int mnt_want_write(struct vfsmount *mnt) if (__mnt_is_readonly(mnt)) { mnt_dec_writers(mnt); ret = -EROFS; - goto out; } -out: preempt_enable(); return ret; } |