On Fri, Mar 26, 2021 at 01:53:20PM +0100, Anthony Iliopoulos wrote:
> Use of the flag has had no effect since kernel commit 288699fecaff
> ("xfs: drop dmapi hooks") which removed all dmapi related code, so
> we can remove it.
>
> Signed-off-by: Anthony Iliopoulos <ailiop@xxxxxxxx>
> ---
> dump/content.c | 1 -
> dump/inomap.c | 1 -
> 2 files changed, 2 deletions(-)
>
> diff --git a/dump/content.c b/dump/content.c
> index 75b79220daf6..a40b47101a12 100644
> --- a/dump/content.c
> +++ b/dump/content.c
> @@ -4287,7 +4287,6 @@ init_extent_group_context(jdm_fshandle_t *fshandlep,
> gcp->eg_bmap[0].bmv_offset = 0;
> gcp->eg_bmap[0].bmv_length = -1;
> gcp->eg_bmap[0].bmv_count = BMAP_LEN;
> - gcp->eg_bmap[0].bmv_iflags = BMV_IF_NO_DMAPI_READ;
> gcp->eg_nextbmapp = &gcp->eg_bmap[1];
> gcp->eg_endbmapp = &gcp->eg_bmap[1];
> gcp->eg_bmapix = 0;
> diff --git a/dump/inomap.c b/dump/inomap.c
> index 85d61c353cf0..1333ca5bb8a8 100644
> --- a/dump/inomap.c
> +++ b/dump/inomap.c
> @@ -1647,7 +1647,6 @@ quantity2offset(jdm_fshandle_t *fshandlep, struct xfs_bstat *statp, off64_t qty)
> bmap[0].bmv_offset = 0;
> bmap[0].bmv_length = -1;
> bmap[0].bmv_count = BMAP_LEN;
> - bmap[0].bmv_iflags = BMV_IF_NO_DMAPI_READ;
I don't think bmap[0] is guaranteed to be initialized at this point,
so deleting this statement leaves bmv_iflags with random stack junk.
--D
> bmap[0].bmv_entries = -1;
> fd = jdm_open(fshandlep, statp, O_RDONLY);
> if (fd < 0) {
> --
> 2.31.0
>
