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

[Jaegeuk Kim <jaegeuk.kim@samsung.com>]

[-- Attachment #1: Type: text/plain, Size: 907 bytes --]

Hi Jianpeng,

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

2013-05-14 (화), 20:06 +0800, majianpeng:
> Signed-off-by: Jianpeng Ma <majianpeng@gmail.com>
> ---
>  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 #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]