Re: [PATCH][Trivial] f2fs: Use list_for_each_entry rather than list_for_each_entry_safe.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jianpeng,

Could you explain why this *should* be changed?
Thanks,

2013-05-14 (화), 20:06 +0800, majianpeng:
> Signed-off-by: Jianpeng Ma <majianpeng@xxxxxxxxx>
> ---
>  fs/f2fs/debug.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/fs/f2fs/debug.c b/fs/f2fs/debug.c
> index 8d99437..0d6c6aa 100644
> --- a/fs/f2fs/debug.c
> +++ b/fs/f2fs/debug.c
> @@ -175,12 +175,12 @@ get_cache:
>  
>  static int stat_show(struct seq_file *s, void *v)
>  {
> -    struct f2fs_stat_info *si, *next;
> +    struct f2fs_stat_info *si;
>      int i = 0;
>      int j;
>  
>      mutex_lock(&f2fs_stat_mutex);
> -    list_for_each_entry_safe(si, next, &f2fs_stat_list, stat_list) {
> +    list_for_each_entry(si, &f2fs_stat_list, stat_list) {
>          char devname[BDEVNAME_SIZE];
>  
>          update_general_status(si->sbi);

-- 
Jaegeuk Kim
Samsung

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux