From mboxrd@z Thu Jan 1 00:00:00 1970 From: domen@coderock.org Subject: [patch 1/7] list_for_each_entry: fs-namespace.c Date: Mon, 20 Jun 2005 23:57:00 +0200 Message-ID: <20050620215700.432387000@nd47.coderock.org> Cc: linux-fsdevel@vger.kernel.org, Maximilian Attems , domen@coderock.org Return-path: Received: from coderock.org ([193.77.147.115]:17563 "EHLO trashy.coderock.org") by vger.kernel.org with ESMTP id S261646AbVFTWEr (ORCPT ); Mon, 20 Jun 2005 18:04:47 -0400 To: viro@parcelfarce.linux.theplanet.co.uk Content-Disposition: inline; filename=list-for-each-entry-fs_namespace.patch Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org From: Domen Puncer Make code more readable with list_for_each_entry. Compile tested. Signed-off-by: Domen Puncer Signed-off-by: Maximilian Attems Signed-off-by: Domen Puncer --- 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; --