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;
Message-ID: <845b09da-6022-2fcd-7a5d-13e9307769b1@oracle.com>
Date:   Fri, 15 Apr 2022 08:36:45 -0700
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0)
 Gecko/20100101 Thunderbird/91.8.0
Subject: Re: [PATCH RFC v19 06/11] NFSD: Update find_clp_in_name_tree() to
 handle courtesy client
Content-Language: en-US
To:     Bruce Fields <bfields@fieldses.org>
Cc:     Chuck Lever III <chuck.lever@oracle.com>,
        Linux NFS Mailing List <linux-nfs@vger.kernel.org>
References: <1648742529-28551-1-git-send-email-dai.ngo@oracle.com>
 <1648742529-28551-7-git-send-email-dai.ngo@oracle.com>
 <20220401152109.GB18534@fieldses.org>
 <52CA1DBC-A0E2-4C1C-96DF-3E6114CDDFFD@oracle.com>
 <8dc762fc-dac8-b323-d0bc-4dbeada8c279@oracle.com>
 <20220413125550.GA29176@fieldses.org>
 <fae06919-13de-9ebb-8259-108f6e18c801@oracle.com>
 <eaf758e8-077d-f7fe-1e02-cfaa49830d97@oracle.com>
 <20220415151959.GA24358@fieldses.org>
From:   dai.ngo@oracle.com
In-Reply-To: <20220415151959.GA24358@fieldses.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TFB3MTVxQ29CWk1nbzNvZEFrTWZLODRINlJYQlllK0pPWUZhTVR2WW44UWZC?=
 =?utf-8?B?TDRvOXlRR3FSRmNSdHRNdmhlUnRiZTYydy8vcHBwLzJWNlVkYjQvWXNWaVRU?=
 =?utf-8?B?UnNUQXZxMmZTT3VBYkZhems4L0sxWGQrNDhqVTZ5N1d6UytsdGlrbnNaR0tX?=
 =?utf-8?B?TGIzNHE1RmZsMGgzeUswTWttZzdRUng1cnEyaUR6bWZKVk5IdEpJZWpOOUd0?=
 =?utf-8?B?YkZhSGZyWFlJZnR3bTRibWlFbkxYUDg3T3hiMUhKWll2aWRQcmY1S1V4dCtu?=
 =?utf-8?B?WWYzQlVHWEJrTUY5QmxPK0p2NDMrdjI1YncvMnZrTFJlcEJvNjVsdG1lSEhG?=
 =?utf-8?B?REVoTkJISDdBYy9MbUt5NUxKdCtqUEFXeUNyVjRvdGNEM3lXQWxZUXM2VVRv?=
 =?utf-8?B?c0Jpa1J1a2Y5WmRqcVJUWUF0MkFBVHlxQkRjWEd0R214MXV2T0pMTGV6Ly9K?=
 =?utf-8?B?Mm9JWmhySmYvRjBmOGI5Rm1GcE14OEl3MWpWTSs3TkNIazB0eW9XamVINHha?=
 =?utf-8?B?TDF4MzRraStudzJ5ZnltU1UwTTlLOThsSlZObEk2MDY1cFNIbGN5cGF3QnZx?=
 =?utf-8?B?ejVTRjJjOENVNFUzT2VDNEVMTUlXVG1oR290VDA1WmtCZ3pLd3hvL2ROYTc2?=
 =?utf-8?B?a1V2NEFrOGhjclBESDloTTVpT0VJdnUwVVl6MCtvMXNyemhSVmxSeFVmeVhy?=
 =?utf-8?B?Rld4QUN2dUlJZkx1ckpyUnNWdVhROTRGYnRGbTMwZ1pLazVCQnBPT3h3MTRQ?=
 =?utf-8?B?Q3R0SEpjSGIyc0JScEEyV1VOUFR4eHdwNkhFU2s3Q1QrRy8wWS9kRWJuRTRL?=
 =?utf-8?B?ZlZncmdMVk12K0dhT2oySy9VRnJDSlRra3RQcThpV2s3b295VHpCcVpwaWs2?=
 =?utf-8?B?WDJNVmpVazZKVkhmd3d6cnlycU5WREJGRFZJTHJhOFlma1pmV2dNSUlmcEtx?=
 =?utf-8?B?bW5rT1F6ekNtSjZMWFgvNVJ1WVQ3cm9EazlPVnMxNEd2eXcvTVhVejdwRGFK?=
 =?utf-8?B?VEl3OUdSTURIcitXT3hNTlNYMlYwMXpkdVdXUzZrT3VDajBBM3N1VytYSDU2?=
 =?utf-8?B?SU1oWHQzNXdZNkVMVGJxd3R0aW1BdXpIenFRTExxZ1E0Zk54alVOL2lIcjhS?=
 =?utf-8?B?b0U0MGRzZFFrTnROVWJ1TjVFTGx5eXJMNElpRERLNFNUS0xmWmtlSXJGYmxl?=
 =?utf-8?B?NldyaEMzNzExRnVBNk8rZCt2TTgxZ2hSNDR5V1kxR1FldnRjejdHK0pERzFR?=
 =?utf-8?B?aFNFUFoxYjNHaENVcVQxZmh3dGNmNWhOcVVTb2JGZmdmV3lqakF3dU00SkVE?=
 =?utf-8?B?eElMZC9uY2NFQWFOMkd4QmJJTjgxV0dYaldLbkZjK2h0WVphYmJqSm00R25z?=
 =?utf-8?B?TXlqcjEreGozTG5sZ0V6UmRISkdIbG4zV3ZLdjJXd3M5cDY4RWFCY01zZmNJ?=
 =?utf-8?B?MVpQN3Nyb2EzWkpEWkpab2ZXSEo3Y2R5eVMyNGt2aUlRZG5Rc21VZGQrUnVi?=
 =?utf-8?B?L2h1MC9qdXdyY2dVb0NoTFhTTkpyWkJiOHk2ZFQyMlJ1Ym9BNGIwRmduZytW?=
 =?utf-8?B?djZtNW5EalV2THJKSXArY2JOS29QemZGSkV5VEhGTVlLQzIycWZmSEpFTktP?=
 =?utf-8?B?dHZxdGVwZUFJUFZaRkdYVWVoYmxUMTg4NzE3NCtSMGFwWmY2WnI3dFh1UU9N?=
 =?utf-8?B?MkoyUlAxTnNVbmt3OW1INklrZDFXQlkrbkFJTHlDcjR1M2M0cEQ0UG5mNTd3?=
 =?utf-8?B?TFk4NHZDanhRSURMZWtEV0liLzJHMHJndTNjaXozUzhSWFhRWHZVWUR5M3dY?=
 =?utf-8?B?RzVBWDJVNXZLVjFhQ1Z0cWVPc0hkTjc4L2l6SGdBaUl2Qld5R1hBMGhGekMw?=
 =?utf-8?B?WkdmUDF2TmRCdFVtb1d6TTZSdnV2bVBjNmFoblJtd1RiYm9wMVdRNVo2c2Z3?=
 =?utf-8?B?bG00V2srdDRPVEJqRm9oaWsyWFgyZXRIeU5YMzI5UWtHTkM5SU5La1Q0N1Fv?=
 =?utf-8?B?bjhEeWYyVGUxbXlpZitQVEtJTkQvWlFGaDRuZXRYeEZ1dXEyNUlCVUtodTAy?=
 =?utf-8?B?Mi9Ec2hSazhBQTd2aVRaSTNmTktMaVE4c0NOWUo4NUdTMDZpSVExeGZaYXZM?=
 =?utf-8?B?cThianhNQWwwZGpxOVRRSGFMbTZRdVRpYUZ3ejZneWl2UWpibUY0OFVoRkFK?=
 =?utf-8?B?bmJid3RoSFhQYWFXckJaUmZ4Y1JYaHFDZDZLb3ZqRW43YTQwY0d3Mmp0cm1Q?=
 =?utf-8?B?dVRGdHJmNGlVVzJway9tTDY5N05yYWNzWXcwR202ck5qVERkdDBTbEFYUVpJ?=
 =?utf-8?B?c1ZDdCtkc2h3MUd6V3lBM0hXaWZ2cGNIdUNsa21EQXpIeXdOYzhqUT09?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 84aabd75-9c2e-4e37-7836-08da1ef5c15a
X-MS-Exchange-CrossTenant-AuthSource: BY5PR10MB4257.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Apr 2022 15:36:48.7505
 (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: XQJiwMkiJ9HPNWffCY8axk1xQyyLs2BX0I4Wa67x6OJvtzGniqtwRvIEd4IWaqyRrFt93Lak99vx+P82QV+fog==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN7PR10MB2754
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.486,18.0.858
 definitions=2022-04-15_01:2022-04-14,2022-04-15 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 adultscore=0 mlxscore=0
 bulkscore=0 phishscore=0 spamscore=0 mlxlogscore=982 suspectscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2202240000
 definitions=main-2204150091
X-Proofpoint-GUID: zKuc0DnDbIqic8ERGsH1dxdMzM5X5aJj
X-Proofpoint-ORIG-GUID: zKuc0DnDbIqic8ERGsH1dxdMzM5X5aJj
X-Spam-Status: No, score=-5.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,RDNS_NONE,SPF_HELO_NONE,
	T_SCC_BODY_TEXT_LINE autolearn=unavailable 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 4/15/22 8:19 AM, Bruce Fields wrote:
> On Fri, Apr 15, 2022 at 07:56:06AM -0700, dai.ngo@oracle.com wrote:
>> On 4/15/22 7:47 AM, dai.ngo@oracle.com wrote:
>> >From user space, how do we force the client to use the delegation
>>> state to read the file *after* doing a close(2)?
>> I guess we can write new pynfs test to do this, but I'd like to avoid
>> it if possible.
> Right, this would require pynfs tests.  If you don't think they'd be
> that bad.  But if you don't want to do tests for each step I think
> that's not the end of the world, up to you.

ok, thanks Bruce. I'll leave open state alone but don't do anything
with share reservation yet until patch 2.

-Dai