From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 97834C4321E for ; Tue, 29 Nov 2022 14:51:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235699AbiK2Ovn (ORCPT ); Tue, 29 Nov 2022 09:51:43 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54554 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234682AbiK2OvZ (ORCPT ); Tue, 29 Nov 2022 09:51:25 -0500 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F0992106 for ; Tue, 29 Nov 2022 06:50:01 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1669733401; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=rIj+Nbz6AgLqQCcGS/FsFSBmr6PKvS7lGQO6Xq5OhrQ=; b=L4Li9V3z1EgiFx7YIckYyacRBpgV6jkLsa7WyD6rL8c/MmY8onQYi4eo9BqfHacQ4rH2wQ dz7/oCPsef4Ls+eo6wuJdFyTspwqeDZvKWXoK7MS/1kpqD+pMnFdYqe9nnfS21FDtz7HIM ULOAGrwlf44lIn4RO1/CK7eC3/gEoEU= Received: from mail-pg1-f199.google.com (mail-pg1-f199.google.com [209.85.215.199]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-582-sfgV4i_BPJCT7e5xe4ViBg-1; Tue, 29 Nov 2022 09:49:59 -0500 X-MC-Unique: sfgV4i_BPJCT7e5xe4ViBg-1 Received: by mail-pg1-f199.google.com with SMTP id m9-20020a63ed49000000b00477f449365fso6638676pgk.15 for ; Tue, 29 Nov 2022 06:49:59 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-language:content-transfer-encoding:in-reply-to:mime-version :user-agent:date:message-id:from:references:cc:to:subject :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=rIj+Nbz6AgLqQCcGS/FsFSBmr6PKvS7lGQO6Xq5OhrQ=; b=A+aB2FhTtHi+AqeVBWJb8U6Ms83fQ9v/cIklsxReRVEzGIeTVIMVG2PhtVrppckgUr hbNZAp8ZW+RPBW8dz+YhX0hacE54pFvUGukOEDdgopTLRIMpcYkFZ8/VgmxhEixxtLOH ZzeoIXhiQXFVnFtAKBuxiwxtarz/7tas1302IEiqxZtpRPKoJ11OtkYg9QWtWy+Uq6Au sfYoVc8bBZnzRXQytkNhxjn+pECLoJvCUdaGFqd3QxQUZw4tPK1etuWx1l2ilZXZRMIq WfbAxb+BKtLs6GTZ9nv5gy0P1KAn1Dxsg6dPUsIVk6koyFDSEV3ESpSQ+2IUPlKd8Lqp 4ETA== X-Gm-Message-State: ANoB5pk6n2Iq2GcS11kW7/OAC940ARYpxHmMT0EITVf5UzTnXoLmqe+m lAJj6GIRTbywJGN0L+dSA50sN5DqVVFlyD4ddyfrw3oUbQJ/VJi7Xo6D3uTQ7IqRel1gLwsmDro ByfaIHHByFREnsPm0VGsgGcwe2BmiI859coCp8fOHJP3+IBHfuVzjvTsWTTLmDl6Q8YPdVLzJOA == X-Received: by 2002:a17:90a:77c9:b0:218:7bf1:386b with SMTP id e9-20020a17090a77c900b002187bf1386bmr57033022pjs.65.1669733398208; Tue, 29 Nov 2022 06:49:58 -0800 (PST) X-Google-Smtp-Source: AA0mqf5nZA8ECCkd9OhgwdfN5enInsLFG6AGdiZctNBhdjYv3mrM+xKnAW0AgSq0QNXPwjqRxxBzqw== X-Received: by 2002:a17:90a:77c9:b0:218:7bf1:386b with SMTP id e9-20020a17090a77c900b002187bf1386bmr57032990pjs.65.1669733397822; Tue, 29 Nov 2022 06:49:57 -0800 (PST) Received: from [10.72.12.126] ([209.132.188.80]) by smtp.gmail.com with ESMTPSA id nh10-20020a17090b364a00b002195819d541sm763567pjb.8.2022.11.29.06.49.54 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 29 Nov 2022 06:49:57 -0800 (PST) Subject: Re: [PATCH v4] ceph: mark directory as non-complete complete after loading key To: Ilya Dryomov , Venky Shankar , Gregory Farnum Cc: =?UTF-8?Q?Lu=c3=ads_Henriques?= , Jeff Layton , ceph-devel@vger.kernel.org, linux-kernel@vger.kernel.org References: <20221129103949.19737-1-lhenriques@suse.de> <4914a195-edc0-747b-6598-9ac9868593a1@redhat.com> From: Xiubo Li Message-ID: <20e0674a-4e51-a352-9ce2-d939cd4f3725@redhat.com> Date: Tue, 29 Nov 2022 22:49:49 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 29/11/2022 22:32, Ilya Dryomov wrote: > On Tue, Nov 29, 2022 at 3:15 PM Xiubo Li wrote: >> >> On 29/11/2022 18:39, Luís Henriques wrote: >>> When setting a directory's crypt context, ceph_dir_clear_complete() needs to >>> be called otherwise if it was complete before, any existing (old) dentry will >>> still be valid. >>> >>> This patch adds a wrapper around __fscrypt_prepare_readdir() which will >>> ensure a directory is marked as non-complete if key status changes. >>> >>> Signed-off-by: Luís Henriques >>> --- >>> Hi Xiubo, >>> >>> Here's a rebase of this patch. I did some testing but since this branch >>> doesn't really have full fscrypt support, I couldn't even reproduce the >>> bug. So, my testing was limited. >> I'm planing not to update the wip-fscrypt branch any more, except the IO >> path related fixes, which may introduce potential bugs each time as before. >> >> Since the qa tests PR has finished and the tests have passed, so we are >> planing to merge the first none IO part, around 27 patches. And then >> pull the reset patches from wip-fscrypt branch. > I'm not sure if merging metadata and I/O path patches separately > makes sense. What would a user do with just filename encryption? Hi Ilya, I think the IO ones should be followed soon. Currently the filename ones have been well testes. And the contents will be by passed for now. Since this is just for Dev Preview feature IMO it should be okay (?) > Once fscrypt test suite is merged, I think we should let it bake > for for a few weeks and then merge the whole thing together. Talked with Jeff before we both agreed to split the patches to make it to be easier to handle. But this also should be okay for me. Venky, Greg What do you think ? Thanks. - Xiubo > > Thanks, > > Ilya >