Re: [PATCH 3/4] xfs: Convert inode numbers to cpu endianity immediately after reading

[Andrei Borzenkov <arvidjaar@gmail.com>]

В Mon, 14 Jul 2014 17:21:30 +0200
Jan Kara <jack@suse.cz> пишет:

> Currently XFS driver converted inode numbers to native endianity only
> when using them to compute inode position. Although this works, it is
> somewhat confusing. So convert inode numbers when reading them from disk
> structures as every other field.
> 

Not sure whether it is better, but committed to avoid risk breaking
followup patch.

> Signed-off-by: Jan Kara <jack@suse.cz>
> ---
>  grub-core/fs/xfs.c | 13 ++++++-------
>  1 file changed, 6 insertions(+), 7 deletions(-)
> 
> diff --git a/grub-core/fs/xfs.c b/grub-core/fs/xfs.c
> index ef3bc787e968..7e247a32df5c 100644
> --- a/grub-core/fs/xfs.c
> +++ b/grub-core/fs/xfs.c
> @@ -180,14 +180,14 @@ static inline grub_uint64_t
>  GRUB_XFS_INO_INOINAG (struct grub_xfs_data *data,
>  		      grub_uint64_t ino)
>  {
> -  return (grub_be_to_cpu64 (ino) & ((1LL << GRUB_XFS_INO_AGBITS (data)) - 1));
> +  return (ino & ((1LL << GRUB_XFS_INO_AGBITS (data)) - 1));
>  }
>  
>  static inline grub_uint64_t
>  GRUB_XFS_INO_AG (struct grub_xfs_data *data,
>  		 grub_uint64_t ino)
>  {
> -  return (grub_be_to_cpu64 (ino) >> GRUB_XFS_INO_AGBITS (data));
> +  return (ino >> GRUB_XFS_INO_AGBITS (data));
>  }
>  
>  static inline grub_disk_addr_t
> @@ -511,13 +511,12 @@ grub_xfs_iterate_dir (grub_fshelp_node_t dir,
>  	if (smallino)
>  	  {
>  	    parent = grub_be_to_cpu32 (diro->inode.data.dir.dirhead.parent.i4);
> -	    parent = grub_cpu_to_be64 (parent);
>  	    /* The header is a bit smaller than usual.  */
>  	    de = (struct grub_xfs_dir_entry *) ((char *) de - 4);
>  	  }
>  	else
>  	  {
> -	    parent = diro->inode.data.dir.dirhead.parent.i8;
> +	    parent = grub_be_to_cpu64(diro->inode.data.dir.dirhead.parent.i8);
>  	  }
>  
>  	/* Synthesize the direntries for `.' and `..'.  */
> @@ -550,7 +549,6 @@ grub_xfs_iterate_dir (grub_fshelp_node_t dir,
>  		| (((grub_uint64_t) inopos[5]) << 16)
>  		| (((grub_uint64_t) inopos[6]) << 8)
>  		| (((grub_uint64_t) inopos[7]) << 0);
> -	    ino = grub_cpu_to_be64 (ino);
>  
>  	    c = de->name[de->len];
>  	    de->name[de->len] = '\0';
> @@ -632,7 +630,8 @@ grub_xfs_iterate_dir (grub_fshelp_node_t dir,
>  		   is not used by GRUB.  So it can be overwritten.  */
>  		filename[direntry->len] = '\0';
>  
> -		if (iterate_dir_call_hook (direntry->inode, filename, &ctx))
> +		if (iterate_dir_call_hook (grub_be_to_cpu64(direntry->inode), 
> +					   filename, &ctx))
>  		  {
>  		    grub_free (dirblock);
>  		    return 1;
> @@ -694,7 +693,7 @@ grub_xfs_mount (grub_disk_t disk)
>      goto fail;
>  
>    data->diropen.data = data;
> -  data->diropen.ino = data->sblock.rootino;
> +  data->diropen.ino = grub_be_to_cpu64(data->sblock.rootino);
>    data->diropen.inode_read = 1;
>    data->bsize = grub_be_to_cpu32 (data->sblock.bsize);
>    data->agsize = grub_be_to_cpu32 (data->sblock.agsize);