Re: [PATCH v5 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX

To: Dave Jiang <dave.jiang@xxxxxxxxx>
Subject: Re: [PATCH v5 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Thu, 15 Feb 2018 13:47:08 -0800
Cc: darrick.wong@xxxxxxxxxx, linux-nvdimm@xxxxxxxxxxxx, david@xxxxxxxxxxxxx, linux-xfs@xxxxxxxxxxxxxxx, ross.zwisler@xxxxxxxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, dan.j.williams@xxxxxxxxx
In-reply-to: <151873085471.53899.4916392112937742603.stgit@djiang5-desk3.ch.intel.com>
References: <151873070632.53899.3320294279108510082.stgit@djiang5-desk3.ch.intel.com> <151873085471.53899.4916392112937742603.stgit@djiang5-desk3.ch.intel.com>
User-agent: Mutt/1.9.2 (2017-12-15)

> +	if ((XFS_IS_REALTIME_INODE(ip)) &&

No need for the inner braces.

Except for that the whole series looks fine to me.

Follow-Ups:
- Re: [PATCH v5 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX
  - From: Dave Jiang

References:
- [PATCH v5 0/3] minimal DAX support for XFS realtime device
  - From: Dave Jiang
- [PATCH v5 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX
  - From: Dave Jiang