Re: [PATCH 04/37] xfs: introduce realtime rmap btree ondisk definitions

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

 



On Thu, Dec 12, 2024 at 05:01:37PM -0800, Darrick J. Wong wrote:
> From: Darrick J. Wong <djwong@xxxxxxxxxx>
> 
> Add the ondisk structure definitions for realtime rmap btrees. The
> realtime rmap btree will be rooted from a hidden inode so it needs to
> have a separate btree block magic and pointer format.
> 
> Next, add everything needed to read, write and manipulate rmap btree
> blocks. This prepares the way for connecting the btree operations
> implementation.

This isn't really the entire on-disk format, because the metadata
btrees in data forks comes later in the series.  Not sure how to
best word this, though.

> +#ifndef __XFS_RTRMAP_BTREE_H__
> +#define	__XFS_RTRMAP_BTREE_H__

The mix of a space for ifndef and a tab for define is a bit weird.
І'd suggest to pick one and stick to it.

Otherwise looks good:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>




[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