linux-kernel - Re: [PATCH 08/13] JFS: use list_move() instead of list_del()/list

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AANLkTin6Btn+btgSa1C+kM-7PQ0KTufmPwGZ6UpqLjHR@mail.gmail.com>
Date:	Mon, 21 Mar 2011 13:49:50 -0500
From:	Dave Kleikamp <dkleikamp@...il.com>
To:	"Kirill A. Shutemov" <kirill@...temov.name>
Cc:	linux-kernel@...r.kernel.org, jfs-discussion@...ts.sourceforge.net
Subject: Re: [PATCH 08/13] JFS: use list_move() instead of list_del()/list_add()
 combination

On Tue, Mar 15, 2011 at 5:53 PM, Kirill A. Shutemov
<kirill@...temov.name> wrote:
> Signed-off-by: Kirill A. Shutemov <kirill@...temov.name>

Acked-by: Dave Kleikamp <shaggy@...nel.org>

> ---
>  fs/jfs/jfs_txnmgr.c |    9 +++------
>  1 files changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/fs/jfs/jfs_txnmgr.c b/fs/jfs/jfs_txnmgr.c
> index 9466957..8cb7b92 100644
> --- a/fs/jfs/jfs_txnmgr.c
> +++ b/fs/jfs/jfs_txnmgr.c
> @@ -2979,12 +2979,9 @@ int jfs_sync(void *arg)
>                                 * put back on the anon_list.
>                                 */
>
> -                               /* Take off anon_list */
> -                               list_del(&jfs_ip->anon_inode_list);
> -
> -                               /* Put on anon_list2 */
> -                               list_add(&jfs_ip->anon_inode_list,
> -                                        &TxAnchor.anon_list2);
> +                               /* Move anon_list -> anon_list2 */
> +                               list_move(&jfs_ip->anon_inode_list,
> +                                         &TxAnchor.anon_list2);
>
>                                TXN_UNLOCK();
>                                iput(ip);
> --
> 1.7.4.1
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/