All of lore.kernel.org
 help / color / mirror / Atom feed
* - r-o-bind-mounts-fix-mounth-vs-fsh-include-quagmire.patch removed from -mm tree
@ 2007-02-15  8:32 akpm
  0 siblings, 0 replies; only message in thread
From: akpm @ 2007-02-15  8:32 UTC (permalink / raw)
  To: hansendc, haveblue, hch, miklos, viro, mm-commits


The patch titled
     r/o bind mounts: fix mount.h vs fs.h include quagmire
has been removed from the -mm tree.  Its filename was
     r-o-bind-mounts-fix-mounth-vs-fsh-include-quagmire.patch

This patch was dropped because it had testing failures

------------------------------------------------------
Subject: r/o bind mounts: fix mount.h vs fs.h include quagmire
From: Dave Hansen <hansendc@us.ibm.com>

I attempted to check the sb->s_flags (declared in fs.h)) variable in mount.h,
by including fs.h, but it blew up on me very quickly.

fs.h references current, which requires sched.h.  sched.h references mounts,
so it includes mount.h.  Circular include dependency fun.  Well, it appears
that fs.h is no stranger to these kinds of problems because it has #includes
spread out in quite a few places.

So, move the sched.h include in fs.h down to just before it is needed, waaaaay
after the 'struct superblock' definition that we need.  Do the same for
mount.h so that we can get at current from fs.h.

If there's a better way to solve this, I'm all ears.

Signed-off-by: Dave Hansen <haveblue@us.ibm.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Miklos Szeredi <miklos@szeredi.hu>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 include/linux/fs.h |    3 ++-
 linux/sched.h      |    0 
 2 files changed, 2 insertions(+), 1 deletion(-)

diff -puN include/linux/fs.h~r-o-bind-mounts-fix-mounth-vs-fsh-include-quagmire include/linux/fs.h
--- a/include/linux/fs.h~r-o-bind-mounts-fix-mounth-vs-fsh-include-quagmire
+++ a/include/linux/fs.h
@@ -280,7 +280,6 @@ extern int dir_notify_enable;
 #include <linux/prio_tree.h>
 #include <linux/init.h>
 #include <linux/pid.h>
-#include <linux/sched.h>
 #include <linux/mutex.h>
 
 #include <asm/atomic.h>
@@ -967,6 +966,8 @@ enum {
 #define vfs_check_frozen(sb, level) \
 	wait_event((sb)->s_wait_unfrozen, ((sb)->s_frozen < (level)))
 
+#include <linux/sched.h>
+
 #define get_fs_excl() atomic_inc(&current->fs_excl)
 #define put_fs_excl() atomic_dec(&current->fs_excl)
 #define has_fs_excl() atomic_read(&current->fs_excl)
diff -puN include/linux/sched.h~r-o-bind-mounts-fix-mounth-vs-fsh-include-quagmire include/linux/sched.h
_

Patches currently in -mm which might be from hansendc@us.ibm.com are

r-o-bind-mounts-fix-mounth-vs-fsh-include-quagmire.patch
r-o-bind-mounts-r-o-bind-mounts-stub-functions.patch
r-o-bind-mounts-ext3-remove-extra-is_rdonly-check.patch
r-o-bind-mounts-ext4-remove-extra-is_rdonly-check.patch
r-o-bind-mounts-r-o-bind-mounts-elevate-write-count-for-some-ioctls.patch
r-o-bind-mounts-elevate-writer-count-for-chown-and-friends.patch
r-o-bind-mounts-make-access-use-mnt-check.patch
r-o-bind-mounts-elevate-mnt-writers-for-callers-of-vfs_mkdir.patch
r-o-bind-mounts-elevate-write-count-during-entire-ncp_ioctl.patch
r-o-bind-mounts-elevate-write-count-for-link-and-symlink-calls.patch
r-o-bind-mounts-elevate-mount-count-for-extended-attributes.patch
r-o-bind-mounts-elevate-write-count-for-file_update_time.patch
r-o-bind-mounts-mount_is_safe-add-comment.patch
r-o-bind-mounts-unix_find_other-elevate-write-count-for-touch_atime.patch
r-o-bind-mounts-elevate-write-count-over-calls-to-vfs_rename.patch
r-o-bind-mounts-elevate-write-count-files-are-opened.patch
r-o-bind-mounts-nfs-check-mnt-instead-of-sb.patch
r-o-bind-mounts-elevate-writer-count-for-do_sys_truncate.patch
r-o-bind-mounts-elevate-write-count-for-do_utimes.patch
r-o-bind-mounts-elevate-write-count-for-do_sys_utime-and-touch_atime.patch
r-o-bind-mounts-sys_mknodat-elevate-write-count-for-vfs_mknod-create.patch
r-o-bind-mounts-elevate-mnt-writers-for-vfs_unlink-callers.patch
r-o-bind-mounts-do_rmdir-elevate-write-count.patch
r-o-bind-mounts-elevate-writer-count-for-custom-struct_file.patch
r-o-bind-mounts-remove-is_rdonly-from-permission.patch
r-o-bind-mounts-reiser4-remove-is_rdonly-checks.patch

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-02-15  8:44 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-02-15  8:32 - r-o-bind-mounts-fix-mounth-vs-fsh-include-quagmire.patch removed from -mm tree akpm

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.