Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Chuck Lever III <chuck.lever@oracle.com>
To:     Bruce Fields <bfields@fieldses.org>
CC:     Linux NFS Mailing List <linux-nfs@vger.kernel.org>,
        Dai Ngo <dai.ngo@oracle.com>,
        Olga Kornievskaia <olga.kornievskaia@gmail.com>,
        Steve Dickson <steved@redhat.com>
Subject: Re: server-to-server copy by default
Thread-Topic: server-to-server copy by default
Thread-Index: AQHXxcrDI0R5Oq8J10KODCEElXm6xqvcC5sA
Date:   Wed, 20 Oct 2021 16:00:28 +0000
Message-ID: <18E32DF5-3F1D-4C23-8C2F-A7963103CF8C@oracle.com>
References: <20211020155421.GC597@fieldses.org>
In-Reply-To: <20211020155421.GC597@fieldses.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?Kjuz80g66Sa3C4wdbLxR+yBbyZazb9Sk4VJs4L3WRwcge+ZhUUJW/xYpKyKr?=
 =?us-ascii?Q?S2WJNAbg/alMTn9ClanGyY9mj1gVfhCEZ0UrlBSRdfGbxLcUqg6xrsDVePPV?=
 =?us-ascii?Q?a5onkc3Zpc9coy4RWaU0qIF5S6jLykJ+f0EjDsPNVUfLZ+5KokK2c87j4nps?=
 =?us-ascii?Q?vOOGhGxwyuQ6WU7StD48dmLomXB445BIH3yZTvvb2hQLrBLoUh7PmYaksCJP?=
 =?us-ascii?Q?NdcW1stvDTIHjqrITxa1sjNZDLZZtoj9LzKqZ7I2taM4m85NocrIgY6Amy1f?=
 =?us-ascii?Q?axftumrD2QyfE04iz3tAJ6MYH5ytTnFBVAqT1L5nE8G2hocdOuuMCvtBsMyp?=
 =?us-ascii?Q?boiwQ4iGp+HdeeTVNhJR+6uf9LJAbfm1Ur6x13+Kx9WfVw3JWLsK1UoWfyYg?=
 =?us-ascii?Q?LLIJ36BE4UH83YS0ZSR5PWaHMNuJKp/jQxF4FCXmNElSQoQ4G0Wt3PfUYKz5?=
 =?us-ascii?Q?iS/f8FDg1gkrBLSKPkXWNMXq8k8MnFiQag3JhgT3YYhGXCDSoM84oyemWgtm?=
 =?us-ascii?Q?5muJmh76LIKHKJBDZR4njW2im1J23sWIpjIFHI4wKbTURKRu0J17fk2APJpv?=
 =?us-ascii?Q?M/82czfxMZm/x9b/GgCmREOrOSHsWWiR2E17ftqDnl4ZMWSe7Q4t+1cWDVVn?=
 =?us-ascii?Q?n9UV04aaOZqtgL0B67Vgsq0Wlp6OPFNNK53G4ZQMuWUvOl9vPk8IiGiGN3+r?=
 =?us-ascii?Q?9fgUtEzO0VPvqW/zdONhpiNjR40jindfn+YKYq/KQ4nrvc21VZ7OPDyTlk8d?=
 =?us-ascii?Q?N+mYh+vOY1yPEAKXDvmJvC9akI51ImjOTI+jkeCWLE5Qx4dF10qb4mD114rJ?=
 =?us-ascii?Q?rIKJpd6HszblivXvkwshR1cRE8eLqWKxqZJeLHB3OcrwIa4jR8aN5NU75tXc?=
 =?us-ascii?Q?jBQnC/t6w2fxQ+i+N+dwLT4MzIy0ad0GR4X8USc7zYYITJWR4NpI9Koi1yDN?=
 =?us-ascii?Q?UVO6pbmfYAfQYsOgwDn6xN+p73PqfJ7WsYqOmIoJDzQvm+l1anH7kax4MhRs?=
 =?us-ascii?Q?qpTCsVrXDuu7mvJ16U9PTp40kRcbvqkqd3KPxUY7ugqFwy8XDpICYdbkLRGz?=
 =?us-ascii?Q?+p4uUM0cqIsnPNJFyjRJ8l2QdBF8gVEG1O+NutltK9CxbD+4ZNz+6QI5nrwk?=
 =?us-ascii?Q?tWC6u+95QFYCb/+F0ZJHtYd8qAzj9dYdYTVUd8TCIhH/MPlK/6qPRcCtSkNd?=
 =?us-ascii?Q?PU9d9eUN7JHbJHMadAk9dpwbHC3zk3v4h3VyoYdb9/nibyCvl/StnYJeTwhy?=
 =?us-ascii?Q?UILee51rUgRNe+PNXdG5hDZFq1Iu+0C5oDBSeZrtSXr1t7JzpcYb31YxIs3v?=
 =?us-ascii?Q?6oTMNtabTYlSoQJugcgjDfW8?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <FD34C1DB79519945B761EEBECF189AA3@namprd10.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR10MB4688.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ca62c0c8-5ac0-4c35-af93-08d993e2bc4d
X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Oct 2021 16:00:28.0068
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: G3mo6dzZdM3RoGVmjO6LI9F3tAjPS8w3X6YI7hMAhExTlnMZFgNxjHbm64sa/DExN3Jwd4UBuIHlCbmZ5gYjFQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR10MB4687
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10143 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxlogscore=999 mlxscore=0 adultscore=0
 spamscore=0 phishscore=0 bulkscore=0 suspectscore=0 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2109230001
 definitions=main-2110200090
X-Proofpoint-GUID: BjeIp8eArfwE-rptW67myr2T_6_6Ptn0
X-Proofpoint-ORIG-GUID: BjeIp8eArfwE-rptW67myr2T_6_6Ptn0
Precedence: bulk
List-ID: <linux-nfs.vger.kernel.org>
X-Mailing-List: linux-nfs@vger.kernel.org


> On Oct 20, 2021, at 11:54 AM, J. Bruce Fields <bfields@fieldses.org> wrot=
e:
>=20
> knfsd has supported server-to-server copy for a couple years (since
> 5.5).  You have set a module parameter to enable it.  I'm getting asked
> when we could turn that parameter on by default.
>=20
> I've got a couple vague criteria: one just general maturity, the other a
> security question:
>=20
> 1. General maturity: the only reports I recall seeing are from testers.
> Is anyone using this?  Does it work for them?  Do they find a benefit?
> Maybe we could turn it on by default in one distro (Fedora?) and promote
> it a little and see what that turns up?

I like the idea of enabling it in one of the technology
preview distributions.

But wrt the maturity question, is the work finished? Or,
perhaps a better question is do we have a minimum viable
product here that can be enabled, or is more work needed
to meet even that bar?

One thing that I recall is missing is support for Kerberos
in the server-to-server copy operation. Is that in plan,
or deemed unimportant?


> 2. Security question: with server-to-server copy enabled, you can send
> the server a COPY call with any random address, and the server will
> mount that address, open a file, and read from it.  Is that safe?
>=20
> Normally we only mount servers that were chosen by root.  Here we'll
> mount any random server that some client told us to.  What's the worst
> that random server can do?  Do we trust our xdr decoding?  Can it DOS us
> by throwing the client's state recovery code into some loop with weird
> error returns?  Etc.

A basic question is what is in distribution QE test suites
that could exercise this feature? Should upstream be tasked
with providing any missing pieces (as part of, say, pynfs,
or nfstests)?


> Maybe it's fine.  I'm OK with some level of risk.  I just want to make
> sure somebody's thought this through.
>=20
> There's also interest in allowing unprivileged NFS mounts, but I don't
> think we've turned that on yet, partly for similar reasons.  This is a
> subset of that problem.
>=20
> --b.

--
Chuck Lever