Re: [PATCH] iget_locked [3/6]

[Alexander Viro]

On Fri, 10 May 2002, Jan Harkes wrote:

> @@ -156,11 +165,12 @@
>  {
>  	int error = -ENOMEM;
>  
> -	*inode = iget(sb, CTL_INO);
> -	if ( *inode ) {
> +	*inode = iget_locked(sb, CTL_INO);
> +	if ( *inode && ((*inode)->i_state & I_NEW) ) {
>  		(*inode)->i_op = &coda_ioctl_inode_operations;
>  		(*inode)->i_fop = &coda_ioctl_operations;
>  		(*inode)->i_mode = 0444;
> +		unlock_new_inode(*inode);
>  		error = 0;

Ehhh....  Do we need this guy hashed, in the first place?

>    destroy_inode: reiserfs_destroy_inode,
>    read_inode: reiserfs_read_inode,
> -  read_inode2: reiserfs_read_inode2,

Why do we keep ->read_inode() here?