linux-kernel - Re: [PATCH] losetup: add option for O

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <3bc8237c0808130708w389123fet5f8d0bb6ff0429bb@mail.gmail.com>
Date:	Wed, 13 Aug 2008 09:08:25 -0500
From:	"Archie Cobbs" <archie@...lroad.org>
To:	"Karel Zak" <kzak@...hat.com>
Cc:	linux-kernel@...r.kernel.org, util-linux-ng@...r.kernel.org
Subject: Re: [PATCH] losetup: add option for O_DIRECT

On Wed, Aug 13, 2008 at 3:15 AM, Karel Zak <kzak@...hat.com> wrote:
> On Tue, Aug 12, 2008 at 04:26:10PM -0500, Archie Cobbs wrote:
> > On Thu, Aug 7, 2008 at 1:26 PM, Archie Cobbs <archie@...lroad.org> wrote:
> > > I have an application involving the loopback device where it would be
> > > valuable for the underlying file to be opened with the O_DIRECT flag.
> > >
> > > Any thoughts one way or the other about adding support to losetup(8)
> > > and mount(8) for a 'direct' option that would enable this?
> > >
> > > I will propose a patch if people think this would be a reasonable thing to do.
> >
> > Add support for opening loopback files with O_DIRECT.
> [...]
>
> >         mode = (*options & SETLOOP_RDONLY) ? O_RDONLY : O_RDWR;
> > +       if (*options & SETLOOP_DIRECT)
> > +               mode |= O_DIRECT;
> >         if ((ffd = open(file, mode)) < 0) {
>
>
>  Do you remember  http://lkml.org/lkml/2007/1/10/233  ?

Yes... I understand there is a valid debate on the merits of O_DIRECT.
It is an interesting question and I don't claim to know the "right" answer.

For example, another way to achieve this same effect, though it would
require kernel changes, is to have some additional way(s) to configure
the loopback device using new LOOP_* ioctls.

In any case, that debate does not seem to be over yet. So my opinion:
if and when O_DIRECT is removed from Linux then we can revert this
patch too :-)

>  Sorry, but I have to see ACK/NACK from relevant kernel person.
>  Redirecting to lkml.

No problem. Please CC me on lkml responses if possible.

Thanks,
-Archie

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