All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Jiang <dave.jiang@intel.com>
To: darrick.wong@oracle.com
Cc: linux-nvdimm@lists.01.org, david@fromorbit.com,
	linux-xfs@vger.kernel.org, linux-ext4@vger.kernel.org
Subject: [PATCH v6 0/3] minimal DAX support for XFS realtime device
Date: Fri, 16 Feb 2018 10:04:09 -0700	[thread overview]
Message-ID: <151880058592.43131.558230052922233871.stgit@djiang5-desk3.ch.intel.com> (raw)

Darrick,
After reading the comments from you, Dave Chinner, and Dan, it looks like
the dyanmic S_DAX flag support won't be coming or not any time soon at the
least. Here are the the collection of patches so far to address yours and
Dave C's comments for minimal support. Please let me know what else I am
missing. Thanks!

v6:
- Removed excess () per Christoph comment.

v5:
- Removed sb parameter for bdev_dax_supported() since we only use it for
  debug output per Christoph comment.

v4:
- Removed setting of error return in ext2 and ext4 per Ross's comments
- Rebased against 4.16-rc1 with updates

---

Darrick J. Wong (1):
      fs: allow per-device dax status checking for filesystems

Dave Jiang (2):
      dax: change bdev_dax_supported() to support boolean returns
      xfs: reject removal of realtime flag when datadev doesn't support DAX


 drivers/dax/super.c |   44 ++++++++++++++++++++++----------------------
 fs/ext2/super.c     |    3 +--
 fs/ext4/super.c     |    3 +--
 fs/xfs/xfs_ioctl.c  |   17 ++++++++++++++++-
 fs/xfs/xfs_iops.c   |   30 +++++++++++++++++++++++++-----
 fs/xfs/xfs_super.c  |   10 ++++++++--
 include/linux/dax.h |   12 ++++--------
 7 files changed, 77 insertions(+), 42 deletions(-)

--
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm

WARNING: multiple messages have this Message-ID (diff)
From: Dave Jiang <dave.jiang@intel.com>
To: darrick.wong@oracle.com
Cc: linux-nvdimm@lists.01.org, david@fromorbit.com,
	linux-xfs@vger.kernel.org, ross.zwisler@linux.intel.com,
	linux-ext4@vger.kernel.org, dan.j.williams@intel.com
Subject: [PATCH v6 0/3] minimal DAX support for XFS realtime device
Date: Fri, 16 Feb 2018 10:04:09 -0700	[thread overview]
Message-ID: <151880058592.43131.558230052922233871.stgit@djiang5-desk3.ch.intel.com> (raw)

Darrick,
After reading the comments from you, Dave Chinner, and Dan, it looks like
the dyanmic S_DAX flag support won't be coming or not any time soon at the
least. Here are the the collection of patches so far to address yours and
Dave C's comments for minimal support. Please let me know what else I am
missing. Thanks!

v6:
- Removed excess () per Christoph comment.

v5:
- Removed sb parameter for bdev_dax_supported() since we only use it for
  debug output per Christoph comment.

v4:
- Removed setting of error return in ext2 and ext4 per Ross's comments
- Rebased against 4.16-rc1 with updates

---

Darrick J. Wong (1):
      fs: allow per-device dax status checking for filesystems

Dave Jiang (2):
      dax: change bdev_dax_supported() to support boolean returns
      xfs: reject removal of realtime flag when datadev doesn't support DAX


 drivers/dax/super.c |   44 ++++++++++++++++++++++----------------------
 fs/ext2/super.c     |    3 +--
 fs/ext4/super.c     |    3 +--
 fs/xfs/xfs_ioctl.c  |   17 ++++++++++++++++-
 fs/xfs/xfs_iops.c   |   30 +++++++++++++++++++++++++-----
 fs/xfs/xfs_super.c  |   10 ++++++++--
 include/linux/dax.h |   12 ++++--------
 7 files changed, 77 insertions(+), 42 deletions(-)

--

             reply	other threads:[~2018-02-16 16:58 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-16 17:04 Dave Jiang [this message]
2018-02-16 17:04 ` [PATCH v6 0/3] minimal DAX support for XFS realtime device Dave Jiang
2018-02-16 17:04 ` [PATCH v6 1/3] fs: allow per-device dax status checking for filesystems Dave Jiang
2018-02-16 17:04   ` Dave Jiang
2018-02-16 17:04   ` Dave Jiang
2018-02-20  0:24   ` kbuild test robot
2018-02-20  0:24     ` kbuild test robot
2018-02-20  0:24     ` kbuild test robot
2018-02-16 17:04 ` [PATCH v6 2/3] dax: change bdev_dax_supported() to support boolean returns Dave Jiang
2018-02-16 17:04   ` Dave Jiang
2018-02-16 17:04   ` Dave Jiang
2018-02-16 17:26   ` Darrick J. Wong
2018-02-16 17:26     ` Darrick J. Wong
2018-02-16 17:26     ` Darrick J. Wong
2018-02-16 17:04 ` [PATCH v6 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX Dave Jiang
2018-02-16 17:04   ` Dave Jiang
2018-02-16 17:22   ` Darrick J. Wong
2018-02-16 17:22     ` Darrick J. Wong
2018-02-18  0:23     ` Dave Chinner
2018-02-18  0:23       ` Dave Chinner
2018-02-20 23:01       ` Darrick J. Wong
2018-02-20 23:01         ` Darrick J. Wong
2018-02-20 23:01         ` Darrick J. Wong
2018-02-20 23:15         ` Dave Chinner
2018-02-20 23:15           ` Dave Chinner
2018-02-20 23:15           ` Dave Chinner
2018-02-20 23:23           ` Darrick J. Wong
2018-02-20 23:23             ` Darrick J. Wong
2018-02-20 23:23             ` Darrick J. Wong
2018-02-27 16:46             ` Dave Jiang
2018-02-27 16:46               ` Dave Jiang
2018-02-27 16:46               ` Dave Jiang
2018-02-27 17:14               ` Darrick J. Wong
2018-02-27 17:14                 ` Darrick J. Wong
2018-02-27 17:14                 ` Darrick J. Wong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=151880058592.43131.558230052922233871.stgit@djiang5-desk3.ch.intel.com \
    --to=dave.jiang@intel.com \
    --cc=darrick.wong@oracle.com \
    --cc=david@fromorbit.com \
    --cc=linux-ext4@vger.kernel.org \
    --cc=linux-nvdimm@lists.01.org \
    --cc=linux-xfs@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.