Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   "Namjae Jeon" <namjae.jeon@samsung.com>
To:     "'J. Bruce Fields'" <bfields@fieldses.org>,
        =?UTF-8?Q?'Aur=C3=A9lien_Aptel'?= <aaptel@suse.com>
Cc:     <linux-fsdevel@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <smfrench@gmail.com>, <senozhatsky@chromium.org>,
        <hyc.lee@gmail.com>, <viro@zeniv.linux.org.uk>, <hch@lst.de>,
        <hch@infradead.org>, <ronniesahlberg@gmail.com>,
        <aurelien.aptel@gmail.com>, <sandeen@sandeen.net>,
        <dan.carpenter@oracle.com>, <colin.king@canonical.com>,
        <rdunlap@infradead.org>, <willy@infradead.org>
In-Reply-To: <20210428235702.GE7400@fieldses.org>
Subject: RE: [PATCH v2 00/10] cifsd: introduce new SMB3 kernel server
Date:   Thu, 29 Apr 2021 09:18:28 +0900
Message-ID: <005d01d73c8d$2dd5ca80$89815f80$@samsung.com>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Thread-Index: AQEyw8UcQFiOkygfL6nbawEwj6LAkgNL6ZwbARcXS5ACyQCCswIY1UerAQPpMTkCx5elNKurDbhA
Content-Language: ko
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 101P
DLP-Filter: Pass
References: <CGME20210422003835epcas1p246c40c6a6bbc0e9f5d4ccf9b69bef0d7@epcas1p2.samsung.com>
        <20210422002824.12677-1-namjae.jeon@samsung.com>
        <20210428191829.GB7400@fieldses.org> <878s52w49d.fsf@suse.com>
        <20210428204035.GD7400@fieldses.org> <875z06vyi6.fsf@suse.com>
        <20210428235702.GE7400@fieldses.org>
Precedence: bulk

> On Thu, Apr 29, 2021 at 12:24:17AM +0200, Aur=C3=A9lien=20Aptel=20wrote:=
=0D=0A>=20>=20=22J.=20Bruce=20Fields=22=20<bfields=40fieldses.org>=20writes=
:=0D=0A>=20>=20>=20I'd=20rather=20see=20multiple=20patches=20that=20were=20=
actually=20functional=20at=0D=0A>=20>=20>=20each=0D=0A>=20>=20>=20stage:=20=
e.g.,=20start=20with=20a=20server=20that=20responds=20to=20some=20sort=20of=
=0D=0A>=20>=20>=20rpc-level=20ping=20but=20does=20nothing=20else,=20then=20=
add=20basic=20file=20IO,=20etc.=0D=0A>=20>=20>=0D=0A>=20>=20>=20I=20don't=
=20know=20if=20that's=20practical.=0D=0A>=20>=0D=0A>=20>=20Although=20it=20=
would=20certainly=20be=20nice=20I=20don't=20think=20it's=20realistic=20to=
=0D=0A>=20>=20expect=20this=20kind=20of=20retro-logical-rewriting.=20AFAIK=
=20the=20other=20new=0D=0A>=20>=20fs-related=20addition=20(ntfs=20patchset)=
=20is=20using=20the=20same=20trick=20of=20adding=0D=0A>=20>=20the=20Makefil=
e=20at=20the=20end=20after=20it=20was=20suggested=20on=20the=20mailing=20li=
st.=20So=0D=0A>=20>=20there's=20a=20precedent.=0D=0A>=20=0D=0A>=20OK,=20I=
=20wondered=20if=20that=20might=20be=20the=20case.=0D=0A>=20=0D=0A>=20I=20d=
on't=20love=20it,=20but,=20fair=20enough,=20maybe=20that's=20the=20best=20c=
ompromise.=0D=0AThank=20you=20for=20your=20understanding.=20One=20big=20pat=
ch=20seems=20to=20be=20unable=20to=20review=0D=0Abecause=20the=20code=20see=
m=20to=20be=20cut=20off=20in=20the=20mail.=20So=20it=20does=20not=20exceed=
=20300KB=0D=0Aper=20patch.=20So=20we=20split=20patches=20out=20by=20layer=
=20and=20added=20Makefile,=20Kconfig=0D=0Aat=20the=20end.=0D=0A=0D=0AThanks=
=21=0D=0A=0D=0A>=20=0D=0A>=20--b.=0D=0A=0D=0A