Re: [PATCH] fuse: add fusex filesystem

[Luis Henriques <luis@igalia.com>]

Hi Miklos!

I'm not sure if you have a more up-to-date version of your patch (or
patchset, as you mentioned you'd be splitting it).  But here's a small
issue I have fixed locally a while back, when I started playing with
fusex:

On Wed, Apr 29 2026, Miklos Szeredi wrote:

<snip>

> +static struct inode *fusex_get_inode(struct super_block *sb, struct fusex_id *id)
> +{
> +	struct inode *inode = fusex_iget(sb, id);
> +	struct fuse_statx_out statx;
> +	int err;
> +
> +	if (!inode)
> +		return ERR_PTR(-ENOMEM);
> +
> +	if (inode_state_read_once(inode) & I_NEW) {
> +		inode_state_set(inode, I_DONTCACHE);

inode_state_set() assumes the caller holds inode->i_lock, so we will see a
kernel splat from a lockdep assertion.

Cheers,
-- 
Luís

> +		err = fusex_send_statx(inode, &statx);
> +		if (err) {
> +			discard_new_inode(inode);
> +			return ERR_PTR(err);
> +		}
> +		fusex_set_attr(inode, &statx.stat);
> +		fusex_init_inode(inode);
> +		unlock_new_inode(inode);
> +	}
> +	return inode;
> +}