| From | Long Li <> | Subject | RE: [[PATCH v1] 00/37] Implement SMBD protocol: Series 1 | Date | Mon, 14 Aug 2017 17:04:00 +0000 |
| |
> -----Original Message----- > From: Christoph Hellwig [mailto:hch@infradead.org] > Sent: Sunday, August 13, 2017 3:28 AM > To: Long Li <longli@microsoft.com> > Cc: Steve French <sfrench@samba.org>; linux-cifs@vger.kernel.org; samba- > technical@lists.samba.org; linux-kernel@vger.kernel.org; Long Li > <longli@microsoft.com> > Subject: Re: [[PATCH v1] 00/37] Implement SMBD protocol: Series 1 > > Hi Long, > > a few meta-comments: > > first the split into lots of tiny lists makes the series really hard to read, I'd split > it into just a few: > > Patch 1: add the protocol constants > Patch 2-n: core cifs modifcation required to implement SMBD Patch n+1: > add the actual SMBD code Patch n+2..m: write up the core CIFS code to use > it. > > stcond a lot of the code doesn't seem to follow the usual Linux style. > It might be a good idea to run it through scripts/checkpach.pl and at least fix > the errors it generates. The warnings are much more opinioned so feel free > to ignore them if they sound odd for now.
Hi Christoph
Thank you for reviewing the patch set.
I'll address those comments and update the patches.
Long
|