From: Dave Chinner <david@fromorbit.com>
To: Boaz Harrosh <openosd@gmail.com>
Cc: linux-fsdevel@vger.kernel.org, willy@linux.intel.com,
jack@suse.cz, xfs@oss.sgi.com
Subject: Re: [PATCH 3/6] xfs: add DAX file operations support
Date: Thu, 5 Mar 2015 09:03:48 +1100 [thread overview]
Message-ID: <20150304220348.GW18360@dastard> (raw)
In-Reply-To: <54F71CBA.2@gmail.com>
On Wed, Mar 04, 2015 at 04:54:50PM +0200, Boaz Harrosh wrote:
> On 03/04/2015 03:01 PM, Dave Chinner wrote:
> > On Wed, Mar 04, 2015 at 12:09:40PM +0200, Boaz Harrosh wrote:
> <>
> >
> > So, we definitely need splice to/from DAX enabled inodes to be
> > rejected. I'll have a look at that...
> >
>
> default_file_splice_read uses kernel_readv which I think might actually
> work. Do you know what xfstest(s) exercise splice?
We have a rudimentary one only because I discovered a while back
none existed at all. i.e. splice is effectively untested by
xfstests. If you want to write some tests to execise it, that'd be
great....
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
WARNING: multiple messages have this Message-ID (diff)
From: Dave Chinner <david@fromorbit.com>
To: Boaz Harrosh <openosd@gmail.com>
Cc: xfs@oss.sgi.com, linux-fsdevel@vger.kernel.org, jack@suse.cz,
willy@linux.intel.com
Subject: Re: [PATCH 3/6] xfs: add DAX file operations support
Date: Thu, 5 Mar 2015 09:03:48 +1100 [thread overview]
Message-ID: <20150304220348.GW18360@dastard> (raw)
In-Reply-To: <54F71CBA.2@gmail.com>
On Wed, Mar 04, 2015 at 04:54:50PM +0200, Boaz Harrosh wrote:
> On 03/04/2015 03:01 PM, Dave Chinner wrote:
> > On Wed, Mar 04, 2015 at 12:09:40PM +0200, Boaz Harrosh wrote:
> <>
> >
> > So, we definitely need splice to/from DAX enabled inodes to be
> > rejected. I'll have a look at that...
> >
>
> default_file_splice_read uses kernel_readv which I think might actually
> work. Do you know what xfstest(s) exercise splice?
We have a rudimentary one only because I discovered a while back
none existed at all. i.e. splice is effectively untested by
xfstests. If you want to write some tests to execise it, that'd be
great....
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
next prev parent reply other threads:[~2015-03-04 22:03 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-03 23:30 [RFC PATCH 0/6] xfs: DAX support Dave Chinner
2015-03-03 23:30 ` Dave Chinner
2015-03-03 23:30 ` [PATCH 1/6] dax: don't abuse get_block mapping for endio callbacks Dave Chinner
2015-03-03 23:30 ` Dave Chinner
2015-03-04 15:54 ` Jan Kara
2015-03-04 15:54 ` Jan Kara
2015-03-04 22:29 ` Dave Chinner
2015-03-04 22:29 ` Dave Chinner
2015-03-03 23:30 ` [PATCH 2/6] xfs: add DAX block zeroing support Dave Chinner
2015-03-03 23:30 ` Dave Chinner
2015-03-03 23:30 ` [PATCH 3/6] xfs: add DAX file operations support Dave Chinner
2015-03-03 23:30 ` Dave Chinner
2015-03-04 10:09 ` Boaz Harrosh
2015-03-04 10:09 ` Boaz Harrosh
2015-03-04 13:01 ` Dave Chinner
2015-03-04 13:01 ` Dave Chinner
2015-03-04 14:54 ` Boaz Harrosh
2015-03-04 14:54 ` Boaz Harrosh
2015-03-04 22:03 ` Dave Chinner [this message]
2015-03-04 22:03 ` Dave Chinner
2015-03-24 4:27 ` Dave Chinner
2015-03-24 4:27 ` Dave Chinner
2015-03-24 7:01 ` Christoph Hellwig
2015-03-24 7:01 ` Christoph Hellwig
2015-03-24 8:13 ` Boaz Harrosh
2015-03-24 8:13 ` Boaz Harrosh
2015-03-04 16:18 ` Jan Kara
2015-03-04 16:18 ` Jan Kara
2015-03-04 22:00 ` Dave Chinner
2015-03-04 22:00 ` Dave Chinner
2015-03-05 11:05 ` Jan Kara
2015-03-05 11:05 ` Jan Kara
2015-03-22 23:02 ` Dave Chinner
2015-03-22 23:02 ` Dave Chinner
2015-03-03 23:30 ` [PATCH 4/6] xfs: add DAX truncate support Dave Chinner
2015-03-03 23:30 ` Dave Chinner
2015-03-03 23:30 ` [PATCH 5/6] xfs: add DAX IO path support Dave Chinner
2015-03-03 23:30 ` Dave Chinner
2015-03-03 23:30 ` [PATCH 6/6] xfs: add initial DAX support Dave Chinner
2015-03-03 23:30 ` Dave Chinner
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=20150304220348.GW18360@dastard \
--to=david@fromorbit.com \
--cc=jack@suse.cz \
--cc=linux-fsdevel@vger.kernel.org \
--cc=openosd@gmail.com \
--cc=willy@linux.intel.com \
--cc=xfs@oss.sgi.com \
/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.