linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ric Wheeler <rwheeler@redhat.com>
To: "Myklebust, Trond" <Trond.Myklebust@netapp.com>
Cc: Miklos Szeredi <miklos@szeredi.hu>, Zach Brown <zab@redhat.com>,
	"J. Bruce Fields" <bfields@fieldses.org>,
	Anna Schumaker <schumaker.anna@gmail.com>,
	Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linux-Fsdevel <linux-fsdevel@vger.kernel.org>,
	"linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>,
	"Schumaker, Bryan" <Bryan.Schumaker@netapp.com>,
	"Martin K. Petersen" <mkp@mkp.net>, Jens Axboe <axboe@kernel.dk>,
	Mark Fasheh <mfasheh@suse.com>, Joel Becker <jlbec@evilplan.org>,
	Eric Wong <normalperson@yhbt.net>
Subject: Re: [RFC] extending splice for copy offloading
Date: Sat, 28 Sep 2013 17:20:57 -0400	[thread overview]
Message-ID: <52474839.2080201@redhat.com> (raw)
In-Reply-To: <4FA345DA4F4AE44899BD2B03EEEC2FA9467EF2D7@SACEXCMBX04-PRD.hq.netapp.com>

On 09/28/2013 11:20 AM, Myklebust, Trond wrote:
>> -----Original Message-----
>> From: Miklos Szeredi [mailto:miklos@szeredi.hu]
>> Sent: Saturday, September 28, 2013 12:50 AM
>> To: Zach Brown
>> Cc: J. Bruce Fields; Ric Wheeler; Anna Schumaker; Kernel Mailing List; Linux-
>> Fsdevel; linux-nfs@vger.kernel.org; Myklebust, Trond; Schumaker, Bryan;
>> Martin K. Petersen; Jens Axboe; Mark Fasheh; Joel Becker; Eric Wong
>> Subject: Re: [RFC] extending splice for copy offloading
>>
>> On Fri, Sep 27, 2013 at 10:50 PM, Zach Brown <zab@redhat.com> wrote:
>>>> Also, I don't get the first option above at all.  The argument is
>>>> that it's safer to have more copies?  How much safety does another
>>>> copy on the same disk really give you?  Do systems that do dedup
>>>> provide interfaces to turn it off per-file?
>> I don't see the safety argument very compelling either.  There are real
>> semantic differences, however: ENOSPC on a write to a
>> (apparentlíy) already allocated block.  That could be a bit unexpected.  Do we
>> need a fallocate extension to deal with shared blocks?
> The above has been the case for all enterprise storage arrays ever since the invention of snapshots. The NFSv4.2 spec does allow you to set a per-file attribute that causes the storage server to always preallocate enough buffers to guarantee that you can rewrite the entire file, however the fact that we've lived without it for said 20 years leads me to believe that demand for it is going to be limited. I haven't put it top of the list of features we care to implement...
>
> Cheers,
>     Trond

I agree - this has been common behaviour for a very long time in the array 
space. Even without an array,  this is the same as overwriting a block in btrfs 
or any file system with a read-write LVM snapshot.

Regards,

Ric

  reply	other threads:[~2013-09-28 21:20 UTC|newest]

Thread overview: 67+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-11 17:06 [RFC] extending splice for copy offloading Zach Brown
2013-09-11 21:17 ` Eric Wong
     [not found]   ` <20130911211722.GA9725-yBiyF41qdooeIZ0/mPfg9Q@public.gmane.org>
2013-09-16 19:44     ` Rob Landley
2013-09-19 12:59   ` Jeff Layton
     [not found] ` <1378919210-10372-1-git-send-email-zab-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-09-11 17:06   ` [PATCH 1/3] splice: add DIRECT flag for splicing between files Zach Brown
2013-09-11 17:06   ` [PATCH 2/3] splice: add f_op->splice_direct Zach Brown
2013-09-11 17:06   ` [PATCH 3/3] btrfs: implement .splice_direct extent copying Zach Brown
2013-09-20  9:49   ` [RFC] extending splice for copy offloading Szeredi Miklos
     [not found]     ` <CAELBmZBGD4rph=gjLCPKCdEj+nzEQ-F=DExoL+h3vRm7qF7dCQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-09-25 18:38       ` Zach Brown
2013-09-25 19:02         ` Anna Schumaker
     [not found]           ` <CAFX2JfnyF8kyMYzCdqdr2JkoyQCom1bFLpFj89wODjoju54-Ow-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-09-25 19:06             ` Zach Brown
     [not found]               ` <20130925190620.GB30372-fypN+1c5dIyjpB87vu3CluTW4wlIGRCZ@public.gmane.org>
2013-09-25 19:55                 ` J. Bruce Fields
2013-09-25 21:07                   ` Zach Brown
2013-09-26  8:58                     ` Miklos Szeredi
2013-09-26 15:34                       ` J. Bruce Fields
     [not found]                         ` <20130926153359.GE704-uC3wQj2KruNg9hUCZPvPmw@public.gmane.org>
2013-09-26 16:46                           ` Ric Wheeler
2013-09-26 18:06                         ` Miklos Szeredi
2013-09-26 19:06                           ` Zach Brown
2013-09-26 19:53                             ` Miklos Szeredi
     [not found]                               ` <CAJfpegvvWhs+jv2J9kOQrB31PEO3kyn_sLm_e2w9YKp=y6EDhA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-09-26 21:23                                 ` Ric Wheeler
     [not found]                                   ` <5244A5E7.90808-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-09-27  4:47                                     ` Miklos Szeredi
2013-09-27 14:00                                       ` Ric Wheeler
2013-09-27 14:39                                         ` Miklos Szeredi
     [not found]                           ` <CAJfpegsUchb0eX+Hi3rN5Ypje3Y-dgo=pxgM1Y3BQbHVp=1hSw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-10-06  8:42                             ` Rob Landley
2013-09-26 18:55                       ` Zach Brown
     [not found]                         ` <20130926185508.GO30372-fypN+1c5dIyjpB87vu3CluTW4wlIGRCZ@public.gmane.org>
2013-09-26 21:26                           ` Ric Wheeler
     [not found]                             ` <5244A68F.906-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-09-27 20:05                               ` J. Bruce Fields
2013-09-27 20:50                                 ` Zach Brown
2013-09-28  5:49                                   ` Miklos Szeredi
2013-09-28 15:20                                     ` Myklebust, Trond
2013-09-28 21:20                                       ` Ric Wheeler [this message]
     [not found]                                         ` <52474839.2080201-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-09-30 12:20                                           ` Miklos Szeredi
2013-09-30 14:34                                             ` J. Bruce Fields
2013-09-30 14:48                                               ` Ric Wheeler
2013-09-30 14:51                                               ` Miklos Szeredi
     [not found]                                                 ` <CAJfpeguMCzv-UhrXrG7e9Q7F_0aEe3_ZMumFwLu3hxcewA_7gA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-09-30 14:52                                                   ` Ric Wheeler
2013-09-30 15:24                                                     ` Miklos Szeredi
2013-09-30 14:28                                                       ` Ric Wheeler
     [not found]                                                         ` <CAJfpegv_C6cLOuA-mNtgtf2QbmmmcHwjQVo8mA nhf_wbJ8iRhg@mail.gmail.com>
     [not found]                                                           ` <CAJfpegv_C6cLOuA-mNtgtf2QbmmmcHwjQVo8mAnhf_wbJ8iRhg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-09-30 14:41                                                             ` Ric Wheeler
     [not found]                                                               ` <52498DB6.7060901-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-09-30 15:46                                                                 ` Miklos Szeredi
2013-09-30 14:49                                                                   ` Ric Wheeler
     [not found]                                                                     ` <52498F68.8050200-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-09-30 15:57                                                                       ` Miklos Szeredi
     [not found]                                                                         ` <CAJfpegvvN_5c5oMv8UoODXQHc-DQnijhOtPDXmNamVpQLDoWMQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-09-30 16:31                                                                           ` Miklos Szeredi
2013-09-30 17:17                                                                             ` Bernd Schubert
     [not found]                                                                               ` <5249B21E.70603-mPn0NPGs4xGatNDF+KUbs4QuADTiUCJX@public.gmane.org>
2013-09-30 17:44                                                                                 ` Myklebust, Trond
     [not found]                                                                                   ` <1380563050.6501.15.camel-5lNtUQgoD8Pfa3cDbr2K10B+6BGkLq7r@public.gmane.org>
2013-09-30 17:48                                                                                     ` Bernd Schubert
     [not found]                                                                                       ` <5249B987.8020807-mPn0NPGs4xGatNDF+KUbs4QuADTiUCJX@public.gmane.org>
2013-09-30 18:02                                                                                         ` Myklebust, Trond
2013-09-30 18:49                                                                                           ` Bernd Schubert
2013-09-30 19:34                                                                                             ` Myklebust, Trond
2013-09-30 20:00                                                                                               ` Bernd Schubert
2013-09-30 20:08                                                                                                 ` Ric Wheeler
     [not found]                                                                                                   ` <5249DA50.5060105-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-09-30 20:27                                                                                                     ` Myklebust, Trond
     [not found]                                                                                                 ` <5249D86A.7080603-mPn0NPGs4xGatNDF+KUbs4QuADTiUCJX@public.gmane.org>
2013-09-30 20:10                                                                                                   ` Myklebust, Trond
     [not found]                                                                   ` <CAJfpegsvrr7x3MbdpvxUmzq0ZfDHfZkzAar6Od2G7wg8DgPLYQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-10-01 18:42                                                                     ` J. Bruce Fields
2013-09-30 15:33                                                         ` Myklebust, Trond
     [not found]                                                         ` <52498AA8.2090204-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-09-30 15:38                                                           ` Miklos Szeredi
     [not found]                                                       ` <CAJfpegtpXuh9070ALGy16Y8kdgioBqSf4JQqBBCF4FHvFJWAWQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-10-01 19:58                                                         ` Zach Brown
     [not found]                                                           ` <20131001195817.GE10831-fypN+1c5dIyjpB87vu3CluTW4wlIGRCZ@public.gmane.org>
2013-10-02 12:58                                                             ` Jan Kara
2013-10-02 13:31                                                               ` David Lang
2013-12-18 12:41   ` Christoph Hellwig
2013-12-18 17:10     ` Zach Brown
2013-12-18 17:26       ` Anna Schumaker
  -- strict thread matches above, loose matches on Subject: below --
2013-09-26 17:22 Steve French
     [not found] ` <CAH2r5muBuTK7ZZ+aKGC4q35gqaSWF4o07eoHypLKiNn5Y83RbQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-09-26 19:34   ` David Disseldorp
2013-10-10  2:18     ` Steve French
2013-10-01 21:05   ` J. Bruce Fields
     [not found]     ` <20131001210531.GA7093-uC3wQj2KruNg9hUCZPvPmw@public.gmane.org>
2013-10-02  1:19       ` Steve French

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=52474839.2080201@redhat.com \
    --to=rwheeler@redhat.com \
    --cc=Bryan.Schumaker@netapp.com \
    --cc=Trond.Myklebust@netapp.com \
    --cc=axboe@kernel.dk \
    --cc=bfields@fieldses.org \
    --cc=jlbec@evilplan.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-nfs@vger.kernel.org \
    --cc=mfasheh@suse.com \
    --cc=miklos@szeredi.hu \
    --cc=mkp@mkp.net \
    --cc=normalperson@yhbt.net \
    --cc=schumaker.anna@gmail.com \
    --cc=zab@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).