Re: [PATCH v7 08/17] xfs: merge freeing of mp names and mp

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

Subject: Re: [PATCH v7 08/17] xfs: merge freeing of mp names and mp
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Fri, 25 Oct 2019 16:37:37 +0200
Cc: linux-xfs <linux-xfs@xxxxxxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx>, "Darrick J. Wong" <darrick.wong@xxxxxxxxxx>, Brian Foster <bfoster@xxxxxxxxxx>, Eric Sandeen <sandeen@xxxxxxxxxxx>, David Howells <dhowells@xxxxxxxxxx>, Dave Chinner <dchinner@xxxxxxxxxx>, Al Viro <viro@xxxxxxxxxxxxxxxxxx>
In-reply-to: <157190347727.27074.15948763811572596699.stgit@fedora-28>
References: <157190333868.27074.13987695222060552856.stgit@fedora-28> <157190347727.27074.15948763811572596699.stgit@fedora-28>
User-agent: Mutt/1.10.1 (2018-07-13)

On Thu, Oct 24, 2019 at 03:51:17PM +0800, Ian Kent wrote:
> In all cases when struct xfs_mount (mp) fields m_rtname and m_logname
> are freed mp is also freed, so merge these into a single function
> xfs_mount_free().
> 
> Signed-off-by: Ian Kent <raven@xxxxxxxxxx>

Looks good,

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

References:
- [PATCH v7 00/17] xfs: mount API patch series
  - From: Ian Kent
- [PATCH v7 08/17] xfs: merge freeing of mp names and mp
  - From: Ian Kent

Prev by Date: Re: [PATCH v7 07/17] xfs: move xfs_mount_alloc to be with parsing code
Next by Date: Re: [PATCH v7 11/17] xfs: refactor suffix_kstrtoint()
Previous by thread: Re: [PATCH v7 08/17] xfs: merge freeing of mp names and mp
Next by thread: [PATCH v7 09/17] xfs: add xfs_remount_rw() helper
Index(es):
- Date
- Thread

[Index of Archives] [XFS Filesystem Development (older mail)] [Linux Filesystem Development] [Linux Audio Users] [Yosemite Trails] [Linux Kernel] [Linux RAID] [Linux SCSI]

Powered by Linux