On Thu, Jan 11, 2018 at 01:37:34PM -0600, Eric Sandeen wrote:
> In addition to more closely matching the kernel, this will
> help us catch any leaks from these allocations.
>
> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>
> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxxx>
> ---
> libxfs/init.c | 4 ++++
> libxfs/trans.c | 29 ++++++++++++++++++-----------
> 2 files changed, 22 insertions(+), 11 deletions(-)
>
> diff --git a/libxfs/init.c b/libxfs/init.c
> index 302f088..7bde8b7 100644
> --- a/libxfs/init.c
> +++ b/libxfs/init.c
> @@ -384,6 +384,7 @@ manage_zones(int release)
> extern kmem_zone_t *xfs_da_state_zone;
> extern kmem_zone_t *xfs_btree_cur_zone;
> extern kmem_zone_t *xfs_bmap_free_item_zone;
> + extern kmem_zone_t *xfs_trans_zone;
> extern kmem_zone_t *xfs_log_item_desc_zone;
> extern void xfs_dir_startup();
>
> @@ -395,6 +396,7 @@ manage_zones(int release)
> kmem_free(xfs_da_state_zone);
> kmem_free(xfs_btree_cur_zone);
> kmem_free(xfs_bmap_free_item_zone);
> + kmem_free(xfs_trans_zone);
> kmem_free(xfs_log_item_desc_zone);
> return;
> }
> @@ -413,6 +415,8 @@ manage_zones(int release)
> xfs_bmap_free_item_zone = kmem_zone_init(
> sizeof(struct xfs_extent_free_item),
> "xfs_bmap_free_item");
> + xfs_trans_zone = kmem_zone_init(
> + sizeof(struct xfs_trans), "xfs_trans");
> xfs_log_item_desc_zone = kmem_zone_init(
> sizeof(struct xfs_log_item_desc), "xfs_log_item_desc");
> xfs_dir_startup();
> diff --git a/libxfs/trans.c b/libxfs/trans.c
> index 57ff3ea..035cc22 100644
> --- a/libxfs/trans.c
> +++ b/libxfs/trans.c
> @@ -36,6 +36,7 @@ static void xfs_trans_free_items(struct xfs_trans *tp);
> * Simple transaction interface
> */
>
> +kmem_zone_t *xfs_trans_zone;
> kmem_zone_t *xfs_log_item_desc_zone;
>
> /*
> @@ -143,6 +144,18 @@ libxfs_trans_roll(
> return 0;
> }
>
> +/*
> + * Free the transaction structure. If there is more clean up
> + * to do when the structure is freed, add it here.
> + */
> +static void
> +xfs_trans_free(
> + struct xfs_trans *tp)
> +{
> + kmem_zone_free(xfs_trans_zone, tp);
> + tp = NULL;
I don't think we need to NULL the local variable and then return.
--D
> +}
> +
> int
> libxfs_trans_alloc(
> struct xfs_mount *mp,
> @@ -166,11 +179,8 @@ libxfs_trans_alloc(
> return -ENOSPC;
> }
>
> - if ((ptr = calloc(sizeof(xfs_trans_t), 1)) == NULL) {
> - fprintf(stderr, _("%s: xact calloc failed (%d bytes): %s\n"),
> - progname, (int)sizeof(xfs_trans_t), strerror(errno));
> - exit(1);
> - }
> + ptr = kmem_zone_zalloc(xfs_trans_zone,
> + (flags & XFS_TRANS_NOFS) ? KM_NOFS : KM_SLEEP);
> ptr->t_mountp = mp;
> ptr->t_blk_res = blocks;
> INIT_LIST_HEAD(&ptr->t_items);
> @@ -212,8 +222,7 @@ libxfs_trans_cancel(
> #endif
> if (tp != NULL) {
> xfs_trans_free_items(tp);
> - free(tp);
> - tp = NULL;
> + xfs_trans_free(tp);
> }
> #ifdef XACT_DEBUG
> fprintf(stderr, "## cancelled transaction %p\n", otp);
> @@ -867,8 +876,7 @@ libxfs_trans_commit(
> fprintf(stderr, "committed clean transaction %p\n", tp);
> #endif
> xfs_trans_free_items(tp);
> - free(tp);
> - tp = NULL;
> + xfs_trans_free(tp);
> return 0;
> }
>
> @@ -891,7 +899,6 @@ libxfs_trans_commit(
> trans_committed(tp);
>
> /* That's it for the transaction structure. Free it. */
> - free(tp);
> - tp = NULL;
> + xfs_trans_free(tp);
> return 0;
> }
> --
> 1.8.3.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
