linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [patch 1/7] list_for_each_entry: fs-namespace.c
@ 2005-03-06 10:37 domen
  0 siblings, 0 replies; 2+ messages in thread
From: domen @ 2005-03-06 10:37 UTC (permalink / raw)
  To: viro; +Cc: linux-fsdevel, domen, janitor




Make code more readable with list_for_each_entry.
Compile tested.

Signed-off-by: Domen Puncer <domen@coderock.org>
Signed-off-by: Maximilian Attems <janitor@sternwelten.at>
Signed-off-by: Domen Puncer <domen@coderock.org>
---


 kj-domen/fs/namespace.c |    4 +---
 1 files changed, 1 insertion(+), 3 deletions(-)

diff -puN fs/namespace.c~list-for-each-entry-fs_namespace fs/namespace.c
--- kj/fs/namespace.c~list-for-each-entry-fs_namespace	2005-03-05 16:09:03.000000000 +0100
+++ kj-domen/fs/namespace.c	2005-03-05 16:09:03.000000000 +0100
@@ -536,7 +536,6 @@ lives_below_in_same_fs(struct dentry *d,
 static struct vfsmount *copy_tree(struct vfsmount *mnt, struct dentry *dentry)
 {
 	struct vfsmount *res, *p, *q, *r, *s;
-	struct list_head *h;
 	struct nameidata nd;
 
 	res = q = clone_mnt(mnt, dentry);
@@ -545,8 +544,7 @@ static struct vfsmount *copy_tree(struct
 	q->mnt_mountpoint = mnt->mnt_mountpoint;
 
 	p = mnt;
-	for (h = mnt->mnt_mounts.next; h != &mnt->mnt_mounts; h = h->next) {
-		r = list_entry(h, struct vfsmount, mnt_child);
+	list_for_each_entry(r, &mnt->mnt_mounts, mnt_child) {
 		if (!lives_below_in_same_fs(r->mnt_mountpoint, dentry))
 			continue;
 
_

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [patch 1/7] list_for_each_entry: fs-namespace.c
@ 2005-06-20 21:57 domen
  0 siblings, 0 replies; 2+ messages in thread
From: domen @ 2005-06-20 21:57 UTC (permalink / raw)
  To: viro; +Cc: linux-fsdevel, Maximilian Attems, domen

[-- Attachment #1: list-for-each-entry-fs_namespace.patch --]
[-- Type: text/plain, Size: 1137 bytes --]

From: Domen Puncer <domen@coderock.org>



Make code more readable with list_for_each_entry.
Compile tested.

Signed-off-by: Domen Puncer <domen@coderock.org>
Signed-off-by: Maximilian Attems <janitor@sternwelten.at>
Signed-off-by: Domen Puncer <domen@coderock.org>
---
 namespace.c |    4 +---
 1 files changed, 1 insertion(+), 3 deletions(-)

Index: quilt/fs/namespace.c
===================================================================
--- quilt.orig/fs/namespace.c
+++ quilt/fs/namespace.c
@@ -536,7 +536,6 @@ lives_below_in_same_fs(struct dentry *d,
 static struct vfsmount *copy_tree(struct vfsmount *mnt, struct dentry *dentry)
 {
 	struct vfsmount *res, *p, *q, *r, *s;
-	struct list_head *h;
 	struct nameidata nd;
 
 	res = q = clone_mnt(mnt, dentry);
@@ -545,8 +544,7 @@ static struct vfsmount *copy_tree(struct
 	q->mnt_mountpoint = mnt->mnt_mountpoint;
 
 	p = mnt;
-	for (h = mnt->mnt_mounts.next; h != &mnt->mnt_mounts; h = h->next) {
-		r = list_entry(h, struct vfsmount, mnt_child);
+	list_for_each_entry(r, &mnt->mnt_mounts, mnt_child) {
 		if (!lives_below_in_same_fs(r->mnt_mountpoint, dentry))
 			continue;
 

--

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2005-06-20 22:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-03-06 10:37 [patch 1/7] list_for_each_entry: fs-namespace.c domen
  -- strict thread matches above, loose matches on Subject: below --
2005-06-20 21:57 domen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).