linux-kernel - RE: [Patch v3 00/16] CIFS: add support for direct I/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: <SN4PR2101MB07330E143E443576FBF08AC8CE180@SN4PR2101MB0733.namprd21.prod.outlook.com>
Date:   Sat, 15 Sep 2018 20:57:33 +0000
From:   Long Li <longli@...rosoft.com>
To:     Steve French <smfrench@...il.com>
CC:     Steve French <sfrench@...ba.org>,
        CIFS <linux-cifs@...r.kernel.org>,
        samba-technical <samba-technical@...ts.samba.org>,
        LKML <linux-kernel@...r.kernel.org>,
        "linux-rdma@...r.kernel.org" <linux-rdma@...r.kernel.org>
Subject: RE: [Patch v3 00/16] CIFS: add support for direct I/O

> From: Steve French <smfrench@...il.com>
> Sent: Saturday, September 15, 2018 2:28 AM
> To: Long Li <longli@...rosoft.com>
> Cc: Steve French <sfrench@...ba.org>; CIFS <linux-cifs@...r.kernel.org>;
> samba-technical <samba-technical@...ts.samba.org>; LKML <linux-
> kernel@...r.kernel.org>; linux-rdma@...r.kernel.org
> Subject: Re: [Patch v3 00/16] CIFS: add support for direct I/O
> 
> could you rebase these, patch 1 was merged quite a while ago, and patch 2
> etc. doesn't apply cleanly 

Sorry, I will rebase and resend.


On Fri, Sep 7, 2018 at 9:18 PM Long Li
> <longli@...uxonhyperv.com> wrote:
> >
> > From: Long Li <longli@...rosoft.com>
> >
> > This patch set implements direct I/O.
> >
> > In normal code path (even with cache=none), CIFS copies I/O data from
> > user-space to kernel-space for security reasons of possible protocol
> > required signing and encryption on user data.
> >
> > With this patch set, CIFS passes the I/O data directly from user-space
> > buffer to the transport layer, when file system is mounted with
> > "cache-none".
> >
> > Patch v2 addressed comments from Christoph Hellwig <hch@....de> and
> > Tom Talpey <ttalpey@...rosoft.com> to implement direct I/O for both
> > socket and RDMA.
> >
> > Patch v3 added support for kernel AIO.
> >
> >
> > Long Li (16):
> >   CIFS: Add support for direct pages in rdata
> >   CIFS: Use offset when reading pages
> >   CIFS: Add support for direct pages in wdata
> >   CIFS: pass page offset when issuing SMB write
> >   CIFS: Calculate the correct request length based on page offset and
> >     tail size
> >   CIFS: Introduce helper function to get page offset and length in
> >     smb_rqst
> >   CIFS: When sending data on socket, pass the correct page offset
> >   CIFS: SMBD: Support page offset in RDMA send
> >   CIFS: SMBD: Support page offset in RDMA recv
> >   CIFS: SMBD: Do not call ib_dereg_mr on invalidated memory registration
> >   CIFS: SMBD: Support page offset in memory registration
> >   CIFS: Pass page offset for calculating signature
> >   CIFS: Pass page offset for encrypting
> >   CIFS: Add support for direct I/O read
> >   CIFS: Add support for direct I/O write
> >   CIFS: Add direct I/O functions to file_operations
> >
> >  fs/cifs/cifsencrypt.c |   9 +-
> >  fs/cifs/cifsfs.c      |  10 +-
> >  fs/cifs/cifsfs.h      |   2 +
> >  fs/cifs/cifsglob.h    |  11 +-
> >  fs/cifs/cifsproto.h   |   9 +-
> >  fs/cifs/cifssmb.c     |  19 +-
> >  fs/cifs/connect.c     |   5 +-
> >  fs/cifs/file.c        | 477
> ++++++++++++++++++++++++++++++++++++++++++--------
> >  fs/cifs/misc.c        |  17 ++
> >  fs/cifs/smb2ops.c     |  22 ++-
> >  fs/cifs/smb2pdu.c     |  20 ++-
> >  fs/cifs/smbdirect.c   | 156 ++++++++++-------
> >  fs/cifs/smbdirect.h   |   2 +-
> >  fs/cifs/transport.c   |  34 ++--
> >  14 files changed, 606 insertions(+), 187 deletions(-)
> >
> > --
> > 2.7.4
> >
> 
> 
> --
> Thanks,
> 
> Steve