From: Long Li <longli-0li6OtcxBFHby3iVrkZq2A@public.gmane.org> To: Pavel Shilovsky <piastryyy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Cc: Steve French <sfrench-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>, linux-cifs <linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, samba-technical <samba-technical-w/Ol4Ecudpl8XjKLYN78aQ@public.gmane.org>, Kernel Mailing List <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, Christoph Hellwig <hch-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>, Tom Talpey <ttalpey-0li6OtcxBFHby3iVrkZq2A@public.gmane.org>, Matthew Wilcox <mawilcox-0li6OtcxBFHby3iVrkZq2A@public.gmane.org> Subject: RE: [Patch v3 11/19] CIFS: SMBD: Define memory registration for I/O data Date: Wed, 30 Aug 2017 02:12:32 +0000 [thread overview] Message-ID: <MWHPR21MB0190E1D1698D183A6F0E8AA7CE9C0@MWHPR21MB0190.namprd21.prod.outlook.com> (raw) In-Reply-To: <CAKywueSMXWN1tsHbFDGK9d9STL1qcC9G_VMfV2hz9YZhxhhH+w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> > -----Original Message----- > From: Pavel Shilovsky [mailto:piastryyy@gmail.com] > Sent: Tuesday, August 29, 2017 5:49 PM > To: Long Li <longli@microsoft.com> > Cc: Steve French <sfrench@samba.org>; linux-cifs <linux- > cifs@vger.kernel.org>; samba-technical <samba-technical@lists.samba.org>; > Kernel Mailing List <linux-kernel@vger.kernel.org>; linux- > rdma@vger.kernel.org; Christoph Hellwig <hch@infradead.org>; Tom Talpey > <ttalpey@microsoft.com>; Matthew Wilcox <mawilcox@microsoft.com>; > Long Li <longli@microsoft.com> > Subject: Re: [Patch v3 11/19] CIFS: SMBD: Define memory registration for I/O > data > > 2017-08-29 12:29 GMT-07:00 Long Li <longli@exchange.microsoft.com>: > > From: Long Li <longli@microsoft.com> > > > > To prepare for RDMA read/write using memory registration, add memory > > registartion pointers to upper layer data I/O context. > > > > Signed-off-by: Long Li <longli@microsoft.com> > > --- > > fs/cifs/cifsglob.h | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/fs/cifs/cifsglob.h b/fs/cifs/cifsglob.h index > > dc5404d..dcd2b63 100644 > > --- a/fs/cifs/cifsglob.h > > +++ b/fs/cifs/cifsglob.h > > @@ -1166,6 +1166,7 @@ struct cifs_readdata { > > struct cifs_readdata *rdata, > > struct iov_iter *iter); > > struct kvec iov[2]; > > + struct smbd_mr *mr; > > unsigned int pagesz; > > unsigned int tailsz; > > unsigned int credits; > > @@ -1188,6 +1189,7 @@ struct cifs_writedata { > > pid_t pid; > > unsigned int bytes; > > int result; > > + struct smbd_mr *mr; > > unsigned int pagesz; > > unsigned int tailsz; > > unsigned int credits; > > -- > > 2.7.4 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-cifs" > > in the body of a message to majordomo@vger.kernel.org More > majordomo > > info at > > > https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fvger.k > > ernel.org%2Fmajordomo- > info.html&data=02%7C01%7Clongli%40microsoft.com% > > > 7C1a02c615906e4648835c08d4ef40e655%7C72f988bf86f141af91ab2d7cd011d > b47% > > > 7C1%7C0%7C636396509394018449&sdata=yjvt3xPYPMmBp8nRiAC%2FYUOOs > ganvPd%2 > > Ff6Y1uZtbqGU%3D&reserved=0 > > The same thing: let's not add structure fields that the code doesn't do > anything with. Okay. I will fold the patch into relevant patches. > > -- > Best regards, > Pavel Shilovsky
WARNING: multiple messages have this Message-ID (diff)
From: Long Li <longli@microsoft.com> To: Pavel Shilovsky <piastryyy@gmail.com> Cc: Steve French <sfrench@samba.org>, linux-cifs <linux-cifs@vger.kernel.org>, samba-technical <samba-technical@lists.samba.org>, Kernel Mailing List <linux-kernel@vger.kernel.org>, "linux-rdma@vger.kernel.org" <linux-rdma@vger.kernel.org>, Christoph Hellwig <hch@infradead.org>, "Tom Talpey" <ttalpey@microsoft.com>, Matthew Wilcox <mawilcox@microsoft.com> Subject: RE: [Patch v3 11/19] CIFS: SMBD: Define memory registration for I/O data Date: Wed, 30 Aug 2017 02:12:32 +0000 [thread overview] Message-ID: <MWHPR21MB0190E1D1698D183A6F0E8AA7CE9C0@MWHPR21MB0190.namprd21.prod.outlook.com> (raw) In-Reply-To: <CAKywueSMXWN1tsHbFDGK9d9STL1qcC9G_VMfV2hz9YZhxhhH+w@mail.gmail.com> > -----Original Message----- > From: Pavel Shilovsky [mailto:piastryyy@gmail.com] > Sent: Tuesday, August 29, 2017 5:49 PM > To: Long Li <longli@microsoft.com> > Cc: Steve French <sfrench@samba.org>; linux-cifs <linux- > cifs@vger.kernel.org>; samba-technical <samba-technical@lists.samba.org>; > Kernel Mailing List <linux-kernel@vger.kernel.org>; linux- > rdma@vger.kernel.org; Christoph Hellwig <hch@infradead.org>; Tom Talpey > <ttalpey@microsoft.com>; Matthew Wilcox <mawilcox@microsoft.com>; > Long Li <longli@microsoft.com> > Subject: Re: [Patch v3 11/19] CIFS: SMBD: Define memory registration for I/O > data > > 2017-08-29 12:29 GMT-07:00 Long Li <longli@exchange.microsoft.com>: > > From: Long Li <longli@microsoft.com> > > > > To prepare for RDMA read/write using memory registration, add memory > > registartion pointers to upper layer data I/O context. > > > > Signed-off-by: Long Li <longli@microsoft.com> > > --- > > fs/cifs/cifsglob.h | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/fs/cifs/cifsglob.h b/fs/cifs/cifsglob.h index > > dc5404d..dcd2b63 100644 > > --- a/fs/cifs/cifsglob.h > > +++ b/fs/cifs/cifsglob.h > > @@ -1166,6 +1166,7 @@ struct cifs_readdata { > > struct cifs_readdata *rdata, > > struct iov_iter *iter); > > struct kvec iov[2]; > > + struct smbd_mr *mr; > > unsigned int pagesz; > > unsigned int tailsz; > > unsigned int credits; > > @@ -1188,6 +1189,7 @@ struct cifs_writedata { > > pid_t pid; > > unsigned int bytes; > > int result; > > + struct smbd_mr *mr; > > unsigned int pagesz; > > unsigned int tailsz; > > unsigned int credits; > > -- > > 2.7.4 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-cifs" > > in the body of a message to majordomo@vger.kernel.org More > majordomo > > info at > > > https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fvger.k > > ernel.org%2Fmajordomo- > info.html&data=02%7C01%7Clongli%40microsoft.com% > > > 7C1a02c615906e4648835c08d4ef40e655%7C72f988bf86f141af91ab2d7cd011d > b47% > > > 7C1%7C0%7C636396509394018449&sdata=yjvt3xPYPMmBp8nRiAC%2FYUOOs > ganvPd%2 > > Ff6Y1uZtbqGU%3D&reserved=0 > > The same thing: let's not add structure fields that the code doesn't do > anything with. Okay. I will fold the patch into relevant patches. > > -- > Best regards, > Pavel Shilovsky
next prev parent reply other threads:[~2017-08-30 2:12 UTC|newest] Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-08-29 19:28 [Patch v3 00/19] CIFS: Implement SMB Direct Long Li 2017-08-29 19:28 ` [Patch v3 02/19] CIFS: SMBD: Add SMBDirect protocol and transport constants Long Li [not found] ` <20170829192915.26251-3-longli-Lp/cVzEoVyZiJJESP9tAQJZ3qXmFLfmx@public.gmane.org> 2017-08-29 22:44 ` Pavel Shilovsky 2017-08-29 22:44 ` Pavel Shilovsky [not found] ` <CAKywueQ3Vz9NUwfqgk36pnaXN=r3DDrrNAumMP418EkcXQPT9g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-08-29 23:00 ` Long Li 2017-08-29 23:00 ` Long Li [not found] ` <MWHPR21MB0190D3CB134749B54A0F709ECE9F0-saRRjQKJ25M/hL2NnenhuM1VXTxX1y3OvxpqHgZTriW3zl9H0oFU5g@public.gmane.org> 2017-08-30 18:19 ` Pavel Shilovsky 2017-08-30 18:19 ` Pavel Shilovsky [not found] ` <CAKywueTyhmP+p=bLCmvK8pzGNKRfeh1x_B9OGWKzt77eAz3Sdg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-08-30 18:24 ` Long Li 2017-08-30 18:24 ` Long Li [not found] ` <MWHPR21MB01907DFB1D1CBEDB17FD995ECE9C0-saRRjQKJ25M/hL2NnenhuM1VXTxX1y3OvxpqHgZTriW3zl9H0oFU5g@public.gmane.org> 2017-08-30 18:30 ` Pavel Shilovsky 2017-08-30 18:30 ` Pavel Shilovsky 2017-09-01 15:48 ` Steve French 2017-08-29 19:28 ` [Patch v3 03/19] CIFS: SMBD: Implement SMBDirect transport Long Li 2017-08-29 19:29 ` [Patch v3 04/19] CIFS: SMBD: Add SMBDirect transport to SMB connection and Makefile Long Li [not found] ` <20170829192915.26251-5-longli-Lp/cVzEoVyZiJJESP9tAQJZ3qXmFLfmx@public.gmane.org> 2017-08-29 23:11 ` Pavel Shilovsky 2017-08-29 23:11 ` Pavel Shilovsky [not found] ` <CAKywueRaTqR1yw_PbdcOYQTY9msweboej_6ZPcbsuyJfmiVgng-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-08-29 23:14 ` Long Li 2017-08-29 23:14 ` Long Li 2017-08-29 19:29 ` [Patch v3 05/19] CIFS: SMBD: Connect to SMBDirect session Long Li 2017-08-29 19:29 ` [Patch v3 06/19] CIFS: SMBD: Reconnect " Long Li [not found] ` <20170829192915.26251-7-longli-Lp/cVzEoVyZiJJESP9tAQJZ3qXmFLfmx@public.gmane.org> 2017-08-29 23:15 ` Pavel Shilovsky 2017-08-29 23:15 ` Pavel Shilovsky 2017-08-29 23:39 ` Long Li 2017-08-29 19:29 ` [Patch v3 08/19] CIFS: SMBD: Set SMBDirect maximum read or write size for I/O Long Li 2017-08-29 19:29 ` [Patch v3 10/19] CIFS: SMBD: Send data through SMBDirect Long Li 2017-08-29 19:29 ` [Patch v3 11/19] CIFS: SMBD: Define memory registration for I/O data Long Li [not found] ` <20170829192915.26251-12-longli-Lp/cVzEoVyZiJJESP9tAQJZ3qXmFLfmx@public.gmane.org> 2017-08-30 0:48 ` Pavel Shilovsky 2017-08-30 0:48 ` Pavel Shilovsky [not found] ` <CAKywueSMXWN1tsHbFDGK9d9STL1qcC9G_VMfV2hz9YZhxhhH+w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-08-30 2:12 ` Long Li [this message] 2017-08-30 2:12 ` Long Li 2017-08-29 19:29 ` [Patch v3 14/19] CIFS: SMBD: Deregister memory on finishing SMB write Long Li 2017-08-29 19:29 ` [Patch v3 15/19] CIFS: SMBD: Add parameter rdata to smb2_new_read_req Long Li [not found] ` <20170829192915.26251-1-longli-Lp/cVzEoVyZiJJESP9tAQJZ3qXmFLfmx@public.gmane.org> 2017-08-29 19:28 ` [Patch v3 01/19] CIFS: Add rdma mount option Long Li 2017-08-29 19:28 ` Long Li [not found] ` <20170829192915.26251-2-longli-Lp/cVzEoVyZiJJESP9tAQJZ3qXmFLfmx@public.gmane.org> 2017-08-29 22:37 ` Pavel Shilovsky 2017-08-29 22:37 ` Pavel Shilovsky [not found] ` <CAKywueRsjjG6BDbkZL_qeVn3V129N2OFwAaw60ZeZXmWbX-32A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-08-29 22:56 ` Long Li 2017-08-29 22:56 ` Long Li 2017-08-29 19:29 ` [Patch v3 07/19] CIFS: SMBD: Destroy SMBDirect session on shutdown or umount Long Li 2017-08-29 19:29 ` Long Li 2017-08-29 19:29 ` [Patch v3 09/19] CIFS: SMBD: Receive data from SMBDirect Long Li 2017-08-29 19:29 ` Long Li 2017-08-29 19:29 ` [Patch v3 12/19] CIFS: SMBD: Fix the definition for SMB2_CHANNEL_RDMA_V1_INVALIDATE Long Li 2017-08-29 19:29 ` Long Li [not found] ` <20170829192915.26251-13-longli-Lp/cVzEoVyZiJJESP9tAQJZ3qXmFLfmx@public.gmane.org> 2017-08-30 0:50 ` Pavel Shilovsky 2017-08-30 0:50 ` Pavel Shilovsky [not found] ` <CAKywueS4=RzCyodOYKDPEWvRrgpwsAX+s+v2KwfbrJWDTy10qg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-08-30 2:11 ` Long Li 2017-08-30 2:11 ` Long Li 2017-08-29 19:29 ` [Patch v3 13/19] CIFS: SMBD: Implement SMB write via RDMA read through memory registration Long Li 2017-08-29 19:29 ` Long Li 2017-08-29 19:29 ` [Patch v3 16/19] CIFS: SMBD: Read correct returned data length for RDMA write (SMB read) I/O Long Li 2017-08-29 19:29 ` Long Li 2017-08-29 19:29 ` [Patch v3 17/19] CIFS: SMBD: Implement SMB read via RDMA write through memory registration Long Li 2017-08-29 19:29 ` Long Li 2017-08-29 19:29 ` [Patch v3 18/19] CIFS: SMBD: Deregister memory when finishing SMB read Long Li 2017-08-29 19:29 ` Long Li 2017-08-30 18:40 ` [Patch v3 00/19] CIFS: Implement SMB Direct Long Li 2017-08-30 18:40 ` Long Li 2017-08-29 19:29 ` [Patch v3 19/19] CIFS: SMBD: Add SMBDirect debug counters Long Li
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=MWHPR21MB0190E1D1698D183A6F0E8AA7CE9C0@MWHPR21MB0190.namprd21.prod.outlook.com \ --to=longli-0li6otcxbfhby3ivrkzq2a@public.gmane.org \ --cc=hch-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \ --cc=linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=mawilcox-0li6OtcxBFHby3iVrkZq2A@public.gmane.org \ --cc=piastryyy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=samba-technical-w/Ol4Ecudpl8XjKLYN78aQ@public.gmane.org \ --cc=sfrench-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org \ --cc=ttalpey-0li6OtcxBFHby3iVrkZq2A@public.gmane.org \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.