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:     Steve Dickson <steved@redhat.com>,
        Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH 0/1] Enable inter server to server copies on a export
Thread-Topic: [PATCH 0/1] Enable inter server to server copies on a export
Thread-Index: AQHXzAr3yIMqaUdei0uEQ5nE8lcyw6vp/mgAgAATygCAAB03AIAADd4AgAAdDoCABHh4AIAAAskAgAAU8gCAABztAIAAAWYA
Date:   Mon, 1 Nov 2021 18:44:17 +0000
Message-ID: <F1C2437C-ADE2-4CF3-87B8-8FF818B35146@oracle.com>
References: <20211028144851.644018-1-steved@redhat.com>
 <20211029134534.GA19967@fieldses.org>
 <3e928624-6a7a-8583-7ea4-4eef9c22488e@redhat.com>
 <20211029164058.GE19967@fieldses.org>
 <65b31c94-54aa-5035-546c-75eb0048ba96@redhat.com>
 <20211029191435.GI19967@fieldses.org>
 <ce34c1f2-a0ad-fcb0-99fa-a1ccea8abfd7@redhat.com>
 <20211101154046.GA12965@fieldses.org>
 <93DAB7C7-D0BB-48BA-9BFF-2821D88582EA@oracle.com>
 <20211101183916.GA14427@fieldses.org>
In-Reply-To: <20211101183916.GA14427@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?Xh0zGmvj3jO7O54CYniEPML4ndpNAc9qh7Q/lT9LM8ooxtInJPXcUfjQC4Zx?=
 =?us-ascii?Q?aBGJLW0C52hEXG+oihT5W4qLDcZ31ZDDVieH5zgDbg2dBKWyBnhwO4mwy+ei?=
 =?us-ascii?Q?DrqRyXuaKACDRDbNgLToa01Ew9/W/WTaC1Q+XpO5wtKZfvjTjiBFBVMRN+lZ?=
 =?us-ascii?Q?ry/Mbf1EMX4AsOVjNGoBAlrhDSK3mW3xgwqqTRxYJjWPn5dxMuQZOlFODRVo?=
 =?us-ascii?Q?1Xpf0Cr51uhu8GuzEe8su+tuBa/tQXHJF0nu1tWzWgzbitmWGAzHdKZq1uo2?=
 =?us-ascii?Q?+USRRUimg63LqOhmcYxMPMOvnx9Z+LfmKb5Rq1l/90dI6uKzwVKZhjOMLTGZ?=
 =?us-ascii?Q?SBxWx1lM+17SE0+G5PbD5ERQThIvlysyVKO3Dh6nGhp3u90UQJpsSwuYerCK?=
 =?us-ascii?Q?pvmu3VcL2uDo+INheEow6iKokbbieWwAJ/hKnRUi0dom52W1zs6DmhhkrxpB?=
 =?us-ascii?Q?0AltG/8ifvUzFshuYArNkeCAV4bFxuWuGz+SEIAqsgahp/G+K3Ot9ZQC3Ddn?=
 =?us-ascii?Q?vqPZTgIvjrobL7VNia4fzVmx9K3PCs0Opvs/GnC+r7xadf72Tm6h/vQrpHyW?=
 =?us-ascii?Q?nTK0L8KEJPISf6RwQhrIg3qH5Y0LMcyBcoEt6ZhnxNxc8aG6fXFqo0u959Jh?=
 =?us-ascii?Q?9O6cJvk8hgSVccJIaxs3+131I6e5DSWsCpTvOve58yNpO0Zg5nRkmgLwn3ZZ?=
 =?us-ascii?Q?e1i8ka9wsmcwX7Rvrd81Fyh0vCfAVf8+FM2mHaPycAu9/PIVwa0r/46GzyCS?=
 =?us-ascii?Q?kQeXDXWQ10YNvuRi2waucS3UddTqHlm1W2+qQnvBlLnfxszweD0j9ruJV6SD?=
 =?us-ascii?Q?gkWUu2pSARLie5Evdc5WitXm6JK7z7pBdJ9feBm4rDAestA7uieL65bAXqa9?=
 =?us-ascii?Q?tVdLNbSRdpbL0dJfMH7YCvHJfpe6EyKYZdIFo+U2RnoKnLcKypq8copJtWLf?=
 =?us-ascii?Q?TSGirDImL6hydLiD+oKksY1wasXbcjrT+a4AkzK9+rehQJJMRMpXgx/M0C6k?=
 =?us-ascii?Q?FOQS3tt52gL5OgBXJ758VW9fZ1TkAvdZrE3ZnWlk0qf0gndRtvzYKyRgerBb?=
 =?us-ascii?Q?ksiLGUecxx9i6PhyKqYwirjPnQbk33i3ViqxGnfSFMX4OAlISqfQCpULn6A5?=
 =?us-ascii?Q?Iuh3ldg/1N7lchiq40JA0b1oRZuimdyTPunDwmS3NpK9Dws5thBBloIRNY48?=
 =?us-ascii?Q?WZdGFZeuoLDIpRrr+ahftIak68f7MSicmFYgYS1cZOV+6HcKNSLEX2CHyE2F?=
 =?us-ascii?Q?EFmUHmqWzjNPahLT63l94J54NMNM5u1hkoLRs/QAaxfqn6/dXj4iTBdNv9Vd?=
 =?us-ascii?Q?F2qJTmDv3PRqVSCAk/mogcjDr2T4QuDp/LfzMgWXaDo+Cv3b8zkpEnLJS3Ix?=
 =?us-ascii?Q?lSYfM9aZnOGBayM2eMmGos08zuXxjSuWDHLtlQ+b71FprZUgh9igtZWtZpGg?=
 =?us-ascii?Q?MOpJB79hZYuU8OR+yhqgOcMead65uBsFSj1H3rP5NdKOgy3DK3lbiv6NRbk9?=
 =?us-ascii?Q?vM+NAtyNLmyHnWG3Z0iZJ6f5YN+XKn5/qChXZn2MUQ8iN3+UZWPj52/UIxyO?=
 =?us-ascii?Q?VAPFta1DGbX1lQhYnZhGFjI+ZLswzdq9zpZ4aTQPggQzmkNCISQRfNTQMYfB?=
 =?us-ascii?Q?qhN/yNOKJ+GqWw3Hn2aUOxA=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <EC2DF0BF2E4AE4498A0410B6885292B3@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: 3c28f93a-ffc8-4b28-0c43-08d99d679bfe
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Nov 2021 18:44:17.2820
 (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: VYzZgf+S99OvwsEknF1AGw+np6tnsAl5k33EdP8JPE6+SnigqeCoVFsueePfLGz6uYY6Z42SO/J3/bc5QzVX5Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR10MB4624
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10155 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 phishscore=0 malwarescore=0
 mlxscore=0 suspectscore=0 mlxlogscore=999 adultscore=0 spamscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2110150000
 definitions=main-2111010100
X-Proofpoint-GUID: hfx3MQcVvxO8IXq4gaQAFZXxjoFFDQic
X-Proofpoint-ORIG-GUID: hfx3MQcVvxO8IXq4gaQAFZXxjoFFDQic
Precedence: bulk
List-ID: <linux-nfs.vger.kernel.org>
X-Mailing-List: linux-nfs@vger.kernel.org


> On Nov 1, 2021, at 2:39 PM, Bruce Fields <bfields@fieldses.org> wrote:
>=20
> On Mon, Nov 01, 2021 at 04:55:45PM +0000, Chuck Lever III wrote:
>>=20
>>> On Nov 1, 2021, at 11:40 AM, J. Bruce Fields <bfields@fieldses.org> wro=
te:
>>>=20
>>> On Mon, Nov 01, 2021 at 11:30:48AM -0400, Steve Dickson wrote:
>>>> Hey!
>>>>=20
>>>> On 10/29/21 15:14, J. Bruce Fields wrote:
>>>>> On Fri, Oct 29, 2021 at 01:30:36PM -0400, Steve Dickson wrote:
>>>>>> On 10/29/21 12:40, J. Bruce Fields wrote:
>>>>>>> Let's just stick with that for now, and leave it off by default unt=
il
>>>>>>> we're sure it's mature enough.  Let's not introduce new configurati=
on to
>>>>>>> work around problems that we haven't really analyzed yet.
>>>>>> How is this going to find problems? At least with the export option
>>>>>> it is documented
>>>>>=20
>>>>> That sounds fixable.  We need documentation of module parameters anyw=
ay.
>>>> Yeah I just took I don't see any documentation of module
>>>> parameters anywhere for any of the modules. But by documentation
>>>> I meant having the feature in the exports(5) manpage.
>>>=20
>>> I think I'd probably create a new page for sysctls (this isn't the only
>>> one needing documentation), and make sure it's listed in the "SEE ALSO"
>>> section of the other man pages.
>>=20
>> Aren't sysctls documented under Documentation/ ?
>=20
> Sorry, I meant "module parameter".  Anyway, yes, looks like both are
> listed in Documentation/admin-guide/kernel-parameters.txt.
>=20
> Might be nice if these were in a man page too somewhere, but maybe
> that's not the most important thing these days.
>=20
> --b.

This looks like a step forward to me.


> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentat=
ion/admin-guide/kernel-parameters.txt
> index 91ba391f9b32..14bc3f0b0149 100644
> --- a/Documentation/admin-guide/kernel-parameters.txt
> +++ b/Documentation/admin-guide/kernel-parameters.txt
> @@ -3243,6 +3243,19 @@
> 			driver. A non-zero value sets the minimum interval
> 			in seconds between layoutstats transmissions.
>=20
> +	nfsd.inter_copy_offload_enable =3D
> +			[NFSv4.2] When set to 1, the server will support
> +			server-to-server copies for which this server is
> +			the destination of the copy.
> +
> +	nfsd.nfsd4_ssc_umount_timeout =3D
> +			[NFSv4.2] When used as the destination of a
> +			server-to-server copy, knfsd temporarily mounts
> +			the source server.  It caches the mount in case
> +			it will be needed again, and discards it if not
> +			used for the number of milliseconds specified by
> +			this parameter.
> +
> 	nfsd.nfs4_disable_idmapping=3D
> 			[NFSv4] When set to the default of '1', the NFSv4
> 			server will return only numeric uids and gids to
> @@ -3250,6 +3263,7 @@
> 			and gids from such clients.  This is intended to ease
> 			migration from NFSv2/v3.
>=20
> +
> 	nmi_backtrace.backtrace_idle [KNL]
> 			Dump stacks even of idle CPUs in response to an
> 			NMI stack-backtrace request.

--
Chuck Lever