From mboxrd@z Thu Jan 1 00:00:00 1970 From: domen@coderock.org Subject: [patch 2/7] list_for_each_entry: fs-dquot.c Date: Sun, 06 Mar 2005 11:37:34 +0100 Message-ID: <20050306103734.C7AB81F23B@trashy.coderock.org> Cc: linux-fsdevel@vger.kernel.org, domen@coderock.org, janitor@sternwelten.at Received: from coderock.org ([193.77.147.115]:63145 "EHLO trashy.coderock.org") by vger.kernel.org with ESMTP id S261375AbVCFKhq (ORCPT ); Sun, 6 Mar 2005 05:37:46 -0500 To: viro@parcelfarce.linux.theplanet.co.uk Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org Make code more readable with list_for_each_entry_safe. Signed-off-by: Domen Puncer Signed-off-by: Maximilian Attems Signed-off-by: Domen Puncer --- kj-domen/fs/dquot.c | 7 ++----- 1 files changed, 2 insertions(+), 5 deletions(-) diff -puN fs/dquot.c~list-for-each-entry-safe-fs_dquot fs/dquot.c --- kj/fs/dquot.c~list-for-each-entry-safe-fs_dquot 2005-03-05 16:09:05.000000000 +0100 +++ kj-domen/fs/dquot.c 2005-03-05 16:09:05.000000000 +0100 @@ -409,13 +409,10 @@ out_dqlock: * for this sb+type at all. */ static void invalidate_dquots(struct super_block *sb, int type) { - struct dquot *dquot; - struct list_head *head; + struct dquot *dquot, *tmp; spin_lock(&dq_list_lock); - for (head = inuse_list.next; head != &inuse_list;) { - dquot = list_entry(head, struct dquot, dq_inuse); - head = head->next; + list_for_each_entry_safe(dquot, tmp, &inuse_list, dq_inuse) { if (dquot->dq_sb != sb) continue; if (dquot->dq_type != type) _