Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1494724

Re: [Nbd] [PATCH][V3] nbd: add multi-connection support

Show all headers | View raw

On Mon, Oct 03, 2016 at 12:20:49AM -0700, Christoph Hellwig wrote:
> On Mon, Oct 03, 2016 at 01:47:06AM +0000, Josef Bacik wrote:
> > It's not "broken", it's working as designed, and any fs on top of this
> > patch will be perfectly safe because they all wait for their io to complete
> > before issuing the FLUSH.  If somebody wants to address the paranoid case
> > later then all the power to them, but this works for my use case and isn't
> > inherently broken.  If it doesn't work for yours then don't use the
> > feature, it's that simple.  Thanks,
> 
> Let's take one step back here.  I agree with Josef that sending
> one single flush is perfectly fine for all usual cases.  The issue
> that was brought up last time we had this discussion was that some
> (I think mostly theoretical) backends could not be coherent and
> this would be an issue.

Actually, I was pointing out the TCP head-of-line issue, where a delay
on the socket that contains the flush reply would result in the arrival
in the kernel block layer of a write reply before the said flush reply,
resulting in a write being considered part of the flush when in fact it
was not.

This is an edge case, and one highly unlikely to result in problems in
the common case (as per my other mail), but it is something to consider.

> So maybe the right way is to simply not support the current odd flush
> defintion in the kernel then and require a new properly defined flush
> version instead.

Can you clarify what you mean by that? Why is it an "odd flush
definition", and how would you "properly" define it?

Thanks,

-- 
< ron> I mean, the main *practical* problem with C++, is there's like a dozen
       people in the world who think they really understand all of its rules,
       and pretty much all of them are just lying to themselves too.
 -- #debian-devel, OFTC, 2016-02-12

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Wouter Verhelst <w@uter.be> - 2016-09-29 12:00 +0200
  Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Wouter Verhelst <w@uter.be> - 2016-09-29 18:50 +0200
    Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Alex Bligh <alex@alex.org.uk> - 2016-10-02 18:20 +0200
      Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Josef Bacik <jbacik@fb.com> - 2016-10-03 03:50 +0200
        Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Christoph Hellwig <hch@infradead.org> - 2016-10-03 09:30 +0200
          Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Wouter Verhelst <w@uter.be> - 2016-10-03 10:00 +0200
            Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Christoph Hellwig <hch@infradead.org> - 2016-10-03 10:00 +0200
              Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Alex Bligh <alex@alex.org.uk> - 2016-10-03 13:40 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Alex Bligh <alex@alex.org.uk> - 2016-10-03 16:50 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Wouter Verhelst <w@uter.be> - 2016-10-03 23:10 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Alex Bligh <alex@alex.org.uk> - 2016-10-04 11:40 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Wouter Verhelst <w@uter.be> - 2016-10-06 11:10 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Alex Bligh <alex@alex.org.uk> - 2016-10-06 11:50 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Wouter Verhelst <w@uter.be> - 2016-10-06 12:20 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Alex Bligh <alex@alex.org.uk> - 2016-10-06 13:10 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Christoph Hellwig <hch@infradead.org> - 2016-10-06 12:40 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Christoph Hellwig <hch@infradead.org> - 2016-10-06 15:20 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Wouter Verhelst <w@uter.be> - 2016-10-06 16:00 +0200
                Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Wouter Verhelst <w@uter.be> - 2016-10-06 15:20 +0200
      Re: [Nbd] [PATCH][V3] nbd: add multi-connection support Wouter Verhelst <w@uter.be> - 2016-10-03 09:50 +0200

csiph-web