Re: [patch 1/1] list_for_each_entry: fs-dquot.c

[Jan Kara]

  Hello,

> Make code more readable with list_for_each_entry_safe.
> (Didn't compile before, doesn't compile now)
  What do you mean by "didn't compile before"? Which kernel have you
tried?

> Signed-off-by: Maximilian Attems <janitor@xxxxxxxxxxxxxx>
> Signed-off-by: Domen Puncer <domen@xxxxxxxxxxxx>
> ---
> 
> 
>  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-01-10 17:59:46.000000000 +0100
> +++ kj-domen/fs/dquot.c	2005-01-10 17:59:46.000000000 +0100
> @@ -406,13 +406,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)


								Honza
-- 
Jan Kara <jack@xxxxxxx>
SuSE CR Labs
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/