On 2024/12/18 22:54, Kemeng Shi wrote:
> After we remove ext4 journal callback, transaction->t_private_list is
> not used anymore. Just remove unused transaction->t_private_list.
>
> Signed-off-by: Kemeng Shi <shikemeng@xxxxxxxxxxxxxxx>
Looks good to me.
Reviewed-by: Zhang Yi <yi.zhang@xxxxxxxxxx>
> ---
> Documentation/filesystems/journalling.rst | 4 +---
> fs/jbd2/transaction.c | 1 -
> include/linux/jbd2.h | 6 ------
> 3 files changed, 1 insertion(+), 10 deletions(-)
>
> diff --git a/Documentation/filesystems/journalling.rst b/Documentation/filesystems/journalling.rst
> index 0254f7d57429..863e93e623f7 100644
> --- a/Documentation/filesystems/journalling.rst
> +++ b/Documentation/filesystems/journalling.rst
> @@ -111,9 +111,7 @@ a callback function when the transaction is finally committed to disk,
> so that you can do some of your own management. You ask the journalling
> layer for calling the callback by simply setting
> ``journal->j_commit_callback`` function pointer and that function is
> -called after each transaction commit. You can also use
> -``transaction->t_private_list`` for attaching entries to a transaction
> -that need processing when the transaction commits.
> +called after each transaction commit.
>
> JBD2 also provides a way to block all transaction updates via
> jbd2_journal_lock_updates() /
> diff --git a/fs/jbd2/transaction.c b/fs/jbd2/transaction.c
> index 66513c18ca29..9fe17e290c21 100644
> --- a/fs/jbd2/transaction.c
> +++ b/fs/jbd2/transaction.c
> @@ -92,7 +92,6 @@ static void jbd2_get_transaction(journal_t *journal,
> atomic_set(&transaction->t_outstanding_revokes, 0);
> atomic_set(&transaction->t_handle_count, 0);
> INIT_LIST_HEAD(&transaction->t_inode_list);
> - INIT_LIST_HEAD(&transaction->t_private_list);
>
> /* Set up the commit timer for the new transaction. */
> journal->j_commit_timer.expires = round_jiffies_up(transaction->t_expires);
> diff --git a/include/linux/jbd2.h b/include/linux/jbd2.h
> index 50f7ea8714bf..90c802e48e23 100644
> --- a/include/linux/jbd2.h
> +++ b/include/linux/jbd2.h
> @@ -700,12 +700,6 @@ struct transaction_s
>
> /* Disk flush needs to be sent to fs partition [no locking] */
> int t_need_data_flush;
> -
> - /*
> - * For use by the filesystem to store fs-specific data
> - * structures associated with the transaction
> - */
> - struct list_head t_private_list;
> };
>
> struct transaction_run_stats_s {
