Wedson Almeida Filho <wedsonaf@xxxxxxxxx> writes: > +/// The number of an inode. > +pub type Ino = u64; Would it be possible to use a descriptive name such as `INodeNumber`? > + /// Returns the super-block that owns the inode. > + pub fn super_block(&self) -> &SuperBlock<T> { > + // SAFETY: `i_sb` is immutable, and `self` is guaranteed to be valid by the existence of a > + // shared reference (&self) to it. > + unsafe { &*(*self.0.get()).i_sb.cast() } > + } I think the safety comment should talk about the pointee rather than the pointer? "The pointee of `i_sb` is immutable, and ..." BR Andreas