46 lines
1.3 KiB
Diff
46 lines
1.3 KiB
Diff
From: Thomas Gleixner <tglx@linutronix.de>
|
|
Date: Sun, 19 Jul 2009 08:44:27 -0500
|
|
Subject: 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>
|
|
|
|
---
|
|
fs/namespace.c | 10 +++++++---
|
|
1 file changed, 7 insertions(+), 3 deletions(-)
|
|
|
|
Index: linux-3.2/fs/namespace.c
|
|
===================================================================
|
|
--- linux-3.2.orig/fs/namespace.c
|
|
+++ linux-3.2/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;
|
|
}
|