[PATCH] xfs: synchronously write the superblock on unmount

[Christoph Hellwig]

>  /*
> - * xfs_log_sbcount
> + * xfs_write_sbcount

Please drop these function name comment line in anything you touch.

>   *
>   * Sync the superblock counters to disk.
>   *
> - * Note this code can be called during the process of freezing, so
> - * we may need to use the transaction allocator which does not
> - * block when the transaction subsystem is in its frozen state.
>   */

Can you add a little commt here on why we write it out synchronously?
Basically a shortened version of the commit message.

>  	if (!xfs_sb_version_haslazysbcount(&mp->m_sb))
> -		return 0;
> +	return 0;

As mentioned by carlos the indendation here got messed up a bit.

> -extern int	xfs_log_sbcount(xfs_mount_t *);
> +extern int
> +xfs_write_sbcount(
> +	struct xfs_mount *mp);
>  extern __uint64_t xfs_default_resblks(xfs_mount_t *mp);

For the header I'd suggest to keep the simple indentation style.

Otherwise the changes look good to, thanks a lot!


Reviewed-by: Christoph Hellwig <hch at lst.de>