Re: [PATCH v6 3/4] Manual conversion of ->i_state uses

From: Christian Brauner

Date: Mon Sep 29 2025 - 05:30:41 EST


On Thu, Sep 25, 2025 at 12:07:13PM +0200, Mateusz Guzik wrote:
> allmodconfig build was done on this patchset but somehow one failure was missed:
>
> diff --git a/fs/afs/inode.c b/fs/afs/inode.c
> index e9538e91f848..71ec043f7569 100644
> --- a/fs/afs/inode.c
> +++ b/fs/afs/inode.c
> @@ -427,7 +427,7 @@ static void afs_fetch_status_success(struct
> afs_operation *op)
> struct afs_vnode *vnode = vp->vnode;
> int ret;
>
> - if (vnode->netfs.inode.i_state & I_NEW) {
> + if (inode_state_read(&vnode->netfs.inode) & I_NEW) {
> ret = afs_inode_init_from_status(op, vp, vnode);
> afs_op_set_error(op, ret);
> if (ret == 0)
>
>
> I reran the thing with this bit and now it's all clean. I think this
> can be folded into the manual fixup patch (the one i'm responding to)
> instead of resending the patchset

Folded, thanks!