On Wed, Dec 08, 2021 at 09:06:14AM +0000, Johannes Thumshirn wrote:

On 07/12/2021 20:08, David Sterba wrote:

quoted

All internal API should use struct btrfs_inode, applied with the
following diff:

Ah ok, good to know. I wanted  to have a bit less pointer casing 
with using 'struct inode'. Anyways I don't mind.

Now it's a mix of both and needs more works to unify but at least new
code should be done the right way. The pointer chasing will go
eventually and I don't think it has a measurable impact.