Received-SPF: softfail (google.com: domain of transitioning linux-nfs-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
From:   Chuck Lever III <chuck.lever@oracle.com>
To:     Trond Myklebust <trondmy@hammerspace.com>
CC:     Linux NFS Mailing List <linux-nfs@vger.kernel.org>,
        "bcodding@redhat.com" <bcodding@redhat.com>,
        Anna Schumaker <anna.schumaker@netapp.com>
Subject: Re: [PATCH] NFSv4: Tune the race to set and use the client id
 uniquifier
Thread-Topic: [PATCH] NFSv4: Tune the race to set and use the client id
 uniquifier
Thread-Index: AQHYI1yJP7y4xU9Y+kaLua2/7R9GAqyXsraAgBGBCwCAABW4AIAAG9GAgAARbYCAAAJVAA==
Date:   Mon, 28 Feb 2022 20:15:11 +0000
Message-ID: <5DB9D3E8-2687-4945-9542-C0ECC58C509A@oracle.com>
References: <61a5993a1f9bbed2ba1227bd3376e92232e0530a.1645033262.git.bcodding@redhat.com>
 <3EA14A5C-9FF9-4DDC-B530-768A86E2A4E8@redhat.com>
 <0F8CD466-6233-4386-94C5-FC8E5941F9D3@redhat.com>
 <73b61ba083df0a8954979fed11d6398d336ee1d1.camel@hammerspace.com>
 <F853D68C-C066-470E-BCFC-988C3D46ABA8@oracle.com>
 <01c6aeddf91d0e68d7c497456ea96f4f24145059.camel@hammerspace.com>
In-Reply-To: <01c6aeddf91d0e68d7c497456ea96f4f24145059.camel@hammerspace.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?hnmeYfVmyb1mDWPvisprf0QH4z/vku8eMCVNhCYsg4PGOqtVzGQk2Md97bcc?=
 =?us-ascii?Q?0JQIWF8hKseuAzY2NEwmWc3nzQ63a9NpNZGCyT1px9P9JNvcOVCrQpxfLbMM?=
 =?us-ascii?Q?RcOihf87qkXAuGy9NymK/BNvLlM5R1fTFVvTQ6zwTqsMGaV7lm9QIwkLaCg6?=
 =?us-ascii?Q?Ivv/dNmNJb2TTuqTa/7QCd55l5/++MP7Dy29HcnTK8491pFHYPj5z6PYZ7QQ?=
 =?us-ascii?Q?u5kHcyH6tAIGInfTHU+Uig0jjgQffy1fqxNFrBSiQCJJSfagQSh7m6oLFH3u?=
 =?us-ascii?Q?gI8NX2oHscLf3Etm9XvTD2AafA7yLaEQjyXyjoeSQeIN4tTR4B5d2H3Gptws?=
 =?us-ascii?Q?TenKsIbC+P0ZPp+R9W3b3BFdvU2vhwfQfFOUhUFGpfBonuV6lfiADVFBeT95?=
 =?us-ascii?Q?kWx5IFXwr51VLR1X8fDsIqLja8xwcr+U/evPmuno7fFwYtT2kLIlkMXBCgez?=
 =?us-ascii?Q?77tKSSKGYI8yyIwal8An9PLF8Sm1HnVpulU1AMjcAd+41vrcndu2MQyIMN32?=
 =?us-ascii?Q?WfxKXMGgV9AoGsGqs6Du5Jz3XdtCbsWeGHEI6lOw4ZKntU3stAifFKkGewT+?=
 =?us-ascii?Q?5ANtFWlho5qSAHA8VFezP9WYSSGLuWNL4WUUNMzeOgnavNFdQZ7iTwBCKlgS?=
 =?us-ascii?Q?BNACwoqQ7JwQyRanuK1E91hNR4f+3gsFKONvYHIPKcoZK9mmrq9rGgLtlJP7?=
 =?us-ascii?Q?H9ZwjLdqQDQ8uqLPsf6to0q1Mg+UmYEnEmvpwD5962mDYcL1CP/P3Cj0rWaw?=
 =?us-ascii?Q?XecQ++kR2mCzODuPb9bDq5wZkT9nazP1bWG6ZqW84ICzSUGtRvZPqqo+Ec4h?=
 =?us-ascii?Q?Gqh7sI+ZHrT+f8/eea4hm5Dl2UG2rS198jvyouzKhrFthEWzR+/AckJb+yXW?=
 =?us-ascii?Q?zLX12/d3TkoTcQtOR8WMOAVrzQY28JYwXCk8iLPZqXurQCLUsYzLVMyuOoA8?=
 =?us-ascii?Q?NdT1f52r/BKAC4xZBNDoAysQ2WEAPVd9S+VpLV3u5WCWeaxlCcIxtDUkVpVo?=
 =?us-ascii?Q?Hwy8viAs1qDBQvm0pMTRHQz+PveCaeP0ctaw4WHgymWDXnoxZZwGZOfSI1mt?=
 =?us-ascii?Q?v5wcAvb5SlUhIStYCpr7qEbtMBgmRub8XnswjuB+arlQUkKBfsd33EbAks1F?=
 =?us-ascii?Q?vK+ws8yoRJ+9s4v6YN8y4EYSFCGgm9bpCjl5pJZtqZ01MKw8nw/cMjxL7G3w?=
 =?us-ascii?Q?rqk2PfXQ+NsXuyS9/dueHq+Yy/wULG9whKmiju3LzVp2ySzHvaDGYJnnEC6G?=
 =?us-ascii?Q?Vfv2Ogsxltnmhj2hoElv7lVHj3bVAaWDPo2kjqNmn0/UPKBATRDGAybhv2hd?=
 =?us-ascii?Q?veB7O8Ky4aGXCgUWXXQ/qaEBEaZgpI+PkZM4l/HPtCWn+F8dLJOhjTrHq5WZ?=
 =?us-ascii?Q?uzsst5RQijFchzRgsX2Up0aBIc/phF8tmDWvIl3VfNU0jdUXs4D3aNs8p2/1?=
 =?us-ascii?Q?Qnfr/iZjlCD2RpEFx/68nnP45tSBkoazbVaHU5oUcdwHeQnIeHyWCyI0U5uy?=
 =?us-ascii?Q?1nS5KGcpdCWOOvbqHmL3TtBuudSb4cA7+PVnv/PTN5JGVPyizyGcPXthMDx9?=
 =?us-ascii?Q?wpMkQgY3yKWozEeurTBQG6povn+AeJov/vxXvL0zEzQBqZcjmz9ikiD3qYWE?=
 =?us-ascii?Q?eIK8pJkR1tOOmqCsgh84v+I=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <396D48C49A54DB41B4446F36B4510084@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: BN0PR10MB5128.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d52989bf-bc04-4f96-6494-08d9faf70626
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Feb 2022 20:15:11.6368
 (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: Tou7nbDlvKRpGrAauhqcOOX26xhhcQb4IfmcltC3KL10BrliNEkN5vC9lJThW50ttZtYeOjP2UP3iagOP3ANlA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR10MB3802
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10272 signatures=684655
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 mlxscore=0 phishscore=0
 bulkscore=0 adultscore=0 spamscore=0 suspectscore=0 mlxlogscore=999
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000
 definitions=main-2202280098
X-Proofpoint-GUID: IwT6DVzkXymdTc_zYNkUUpYGOB5valCC
X-Proofpoint-ORIG-GUID: IwT6DVzkXymdTc_zYNkUUpYGOB5valCC
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE
	autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
	lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-nfs.vger.kernel.org>
X-Mailing-List: linux-nfs@vger.kernel.org


> On Feb 28, 2022, at 3:06 PM, Trond Myklebust <trondmy@hammerspace.com> wr=
ote:
>=20
> On Mon, 2022-02-28 at 19:04 +0000, Chuck Lever III wrote:
>>=20
>>=20
>>> On Feb 28, 2022, at 12:24 PM, Trond Myklebust
>>> <trondmy@hammerspace.com> wrote:
>>>=20
>>>> Attempts to work toward solutions in this area seem to be
>>>> ignored,
>>>> the
>>>> questions still stand.  Until we can sort out and agree on a
>>>> direction,
>>>> self-NACK to this patch.
>>>=20
>>> A new mount option doesn't solve any problems that we can't solve
>>> with
>>> the existing framework.
>>=20
>> I don't think a mount option was proposed. Rather, the mechanics
>> of the udev rule would be done by mount.nfs without any changes
>> to the administrative interface.
>>=20
>=20
> That's not how I read this proposal:
>=20
>> Do you still want us to keep this approach, or will you accept a
> mount
>> option as:
>> - first mount in a namespace sets the identifier
>> - subsequent mounts with conflicting identifiers return an error
>=20
>=20
> Which is why I responded as I did.

Ah! Well, I read "mount option" as "mount alternative", I guess
I was filling in some context from previous dialog on the topic.

I agree with you that an actual mount option -- that is, a new
administrative interface -- is not desirable or necessary.

But I assert that having mount.nfs take care of initializing the
uniquifier for the net namespace is convenient, and can be done
automatically and reliably.

--
Chuck Lever