Re: [PATCH 2/2] ubifs: Allow O_DIRECT

From	Brian Norris <computersforpeace@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/2] ubifs: Allow O_DIRECT
Date	2015-08-20 23:00 +0200
Message-ID	<pZF6i-7Li-7@gated-at.bofh.it> (permalink)
References	<pZijn-8mM-15@gated-at.bofh.it> <pZijn-8mM-13@gated-at.bofh.it> <pZooN-tt-1@gated-at.bofh.it> <pZwmm-3AQ-27@gated-at.bofh.it> <pZww1-3MB-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Pardon the innocent bystander's comment, but:

On Thu, Aug 20, 2015 at 01:40:02PM +0200, Richard Weinberger wrote:
> Am 20.08.2015 um 13:31 schrieb Artem Bityutskiy:
> > On Thu, 2015-08-20 at 11:00 +0800, Dongsheng Yang wrote:
> >> On 08/20/2015 04:35 AM, Richard Weinberger wrote:
> >>> Currently UBIFS does not support direct IO, but some applications
> >>> blindly use the O_DIRECT flag.
> >>> Instead of failing upon open() we can do better and fall back
> >>> to buffered IO.
> >>
> >> Hmmmm, to be honest, I am not sure we have to do it as Dave
> >> suggested. I think that's just a work-around for current fstests.
> >>
> >> IMHO, perform a buffered IO when user request direct IO without
> >> any warning sounds not a good idea. Maybe adding a warning would
> >> make it better.
> >>
> >> I think we need more discussion about AIO&DIO in ubifs, and actually
> >> I have a plan for it. But I have not listed the all cons and pros of
> >> it so far.
> >>
> >> Artem, what's your opinion?
> > 
> > Yes, this is my worry too.
> > 
> > Basically, we need to see what is the "common practice" here, and
> > follow it. This requires a small research. What would be the most
> > popular Linux FS which does not support direct I/O? Can we check what
> > it does?
> 
> All popular filesystems seem to support direct IO.
> That's the problem, application do not expect O_DIRECT to fail.

Why can't we just suggest fixing the applications? The man pages for
O_DIRECT clearly document the EINVAL return code:

  EINVAL The filesystem does not support the O_DIRECT flag. See NOTES
  for more information.

and under NOTES:

  O_DIRECT  support  was added under Linux in kernel version 2.4.10.
  Older Linux kernels simply ignore this flag.  Some filesystems may not
  implement the flag and open() will fail with EINVAL if it is used.

Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH 2/2] ubifs: Allow O_DIRECT Richard Weinberger <richard@nod.at> - 2015-08-19 22:40 +0200
  Re: [PATCH 2/2] ubifs: Allow O_DIRECT Dongsheng Yang <yangds.fnst@cn.fujitsu.com> - 2015-08-20 05:10 +0200
    Re: [PATCH 2/2] ubifs: Allow O_DIRECT Richard Weinberger <richard@nod.at> - 2015-08-20 08:50 +0200
      Re: [PATCH 2/2] ubifs: Allow O_DIRECT Dongsheng Yang <yangds.fnst@cn.fujitsu.com> - 2015-08-20 09:30 +0200
    Re: [PATCH 2/2] ubifs: Allow O_DIRECT Artem Bityutskiy <dedekind1@gmail.com> - 2015-08-20 13:40 +0200
      Re: [PATCH 2/2] ubifs: Allow O_DIRECT Richard Weinberger <richard@nod.at> - 2015-08-20 13:50 +0200
        Re: [PATCH 2/2] ubifs: Allow O_DIRECT Artem Bityutskiy <dedekind1@gmail.com> - 2015-08-20 14:40 +0200
          Re: [PATCH 2/2] ubifs: Allow O_DIRECT Artem Bityutskiy <dedekind1@gmail.com> - 2015-08-24 09:20 +0200
            Re: [PATCH 2/2] ubifs: Allow O_DIRECT Dongsheng Yang <yangds.fnst@cn.fujitsu.com> - 2015-08-24 09:30 +0200
            Re: [PATCH 2/2] ubifs: Allow O_DIRECT Dongsheng Yang <yangds.fnst@cn.fujitsu.com> - 2015-08-24 09:30 +0200
              Re: [PATCH 2/2] ubifs: Allow O_DIRECT Christoph Hellwig <hch@infradead.org> - 2015-08-24 10:10 +0200
        Re: [PATCH 2/2] ubifs: Allow O_DIRECT Brian Norris <computersforpeace@gmail.com> - 2015-08-20 23:00 +0200
          Re: [PATCH 2/2] ubifs: Allow O_DIRECT Artem Bityutskiy <dedekind1@gmail.com> - 2015-08-24 09:20 +0200
            Re: [PATCH 2/2] ubifs: Allow O_DIRECT Christoph Hellwig <hch@infradead.org> - 2015-08-24 10:00 +0200
              Re: [PATCH 2/2] ubifs: Allow O_DIRECT Christoph Hellwig <hch@infradead.org> - 2015-08-24 10:10 +0200
                Re: [PATCH 2/2] ubifs: Allow O_DIRECT Dongsheng Yang <yangds.fnst@cn.fujitsu.com> - 2015-08-24 10:10 +0200
                Re: [PATCH 2/2] ubifs: Allow O_DIRECT Richard Weinberger <richard@nod.at> - 2015-08-24 11:40 +0200
                Re: [PATCH 2/2] ubifs: Allow O_DIRECT Artem Bityutskiy <dedekind1@gmail.com> - 2015-08-24 11:40 +0200
              Re: [PATCH 2/2] ubifs: Allow O_DIRECT Artem Bityutskiy <dedekind1@gmail.com> - 2015-08-24 10:10 +0200
            Re: [PATCH 2/2] ubifs: Allow O_DIRECT Brian Norris <computersforpeace@gmail.com> - 2015-08-24 18:20 +0200
              Re: [PATCH 2/2] ubifs: Allow O_DIRECT Jeff Moyer <jmoyer@redhat.com> - 2015-08-24 19:20 +0200
                Re: [PATCH 2/2] ubifs: Allow O_DIRECT Dave Chinner <david@fromorbit.com> - 2015-08-25 01:50 +0200
                Re: [PATCH 2/2] ubifs: Allow O_DIRECT Chris Mason <clm@fb.com> - 2015-08-25 03:30 +0200
                Re: [PATCH 2/2] ubifs: Allow O_DIRECT Artem Bityutskiy <dedekind1@gmail.com> - 2015-08-25 17:50 +0200
                Re: [PATCH 2/2] ubifs: Allow O_DIRECT Jeff Moyer <jmoyer@redhat.com> - 2015-08-25 16:10 +0200
                Re: [PATCH 2/2] ubifs: Allow O_DIRECT Jeff Moyer <jmoyer@redhat.com> - 2015-08-25 16:20 +0200
                Re: [PATCH 2/2] ubifs: Allow O_DIRECT Chris Mason <clm@fb.com> - 2015-08-25 16:20 +0200
  Re: [PATCH 2/2] ubifs: Allow O_DIRECT Artem Bityutskiy <dedekind1@gmail.com> - 2015-08-20 13:30 +0200

csiph-web