On Sat, May 09, 2020 at 07:01:20PM +0200, Christoph Hellwig wrote:
> Explain the bno field that is not actually part of the structure
> anymore.
>
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
> ---
> db/agfl.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/db/agfl.c b/db/agfl.c
> index 45e4d6f9..ce7a2548 100644
> --- a/db/agfl.c
> +++ b/db/agfl.c
> @@ -47,6 +47,7 @@ const field_t agfl_crc_flds[] = {
> { "uuid", FLDT_UUID, OI(OFF(uuid)), C1, 0, TYP_NONE },
> { "lsn", FLDT_UINT64X, OI(OFF(lsn)), C1, 0, TYP_NONE },
> { "crc", FLDT_CRC, OI(OFF(crc)), C1, 0, TYP_NONE },
> + /* the bno array really is behind the actual structure */
Er... the bno array comes /after/ the actual structure, right?
--D
> { "bno", FLDT_AGBLOCKNZ, OI(bitize(sizeof(struct xfs_agfl))),
> agfl_bno_size, FLD_ARRAY|FLD_COUNT, TYP_DATA },
> { NULL }
> --
> 2.26.2
>
