Received: by 2002:a05:7412:3784:b0:e2:908c:2ebd with SMTP id jk4csp369309rdb; Sat, 30 Sep 2023 07:37:10 -0700 (PDT) X-Google-Smtp-Source: AGHT+IH2mT10tSbDQvq+KT5TgFAae2L6IlwWdxEdPuQOCGGUWTNaWBsHD8Y6HWxPxeNNgt0o9BVz X-Received: by 2002:a05:6a00:1c9b:b0:68f:cbd3:5b01 with SMTP id y27-20020a056a001c9b00b0068fcbd35b01mr10196024pfw.13.1696084630405; Sat, 30 Sep 2023 07:37:10 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1696084630; cv=none; d=google.com; s=arc-20160816; b=s2Ek4AgF3fDfsEjD8kzDutIe/CF94BGIn9fquuJwvzGcSjB6Y28caQks93IOxlslzV I94i9uMoJ3vgfOrjIZvZNcRpsM5g8xWsUB5faJKOIrfX4UzeVpgzEiuL4nLk4ii67zF6 iSvSn8dpUxVzWz8deZ4yi+nCxrjkM/TN8z6GFOhmEU4yJ/fWo8n4Qruz2SL11LpyARbC wiM/BujfgF0RtXNyA+p7sFvDu7qU67BjHXzHZuEJBrD8jaLsEzC/BXunHOzmzrSI/Bmk eYlmGZkulx+cuGRe2Hq+J9oR32vkHHejpXnewam81m0nZKjRqRY16XkPgPpgYlf1PA1e ME+g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=2HIIPtXVuUpNSoUCpPydGPPHcb8hSd3Ai4FPKT8qLco=; fh=snc54BWB4cPsLGTYt/0J4MaMXN3g69VEIQCkWkrlAU0=; b=UqVGM8Y0VpF1OM+QTeP+Fv2UuN2A4e6V1Z5C0G2bAKsddZBxs56MVhPAeaC102CGcA CwHfP7y6BS2NH3XGA1c1DEA6izerNQS8vDvEMNKviWutUSHQV+dS8OOFUl6m6BZPFN09 yKLfNE8XSZe9WPVERTDJryUQZG5bCjftBaxNGKnLBPx/iTg4SH8BzpkihChv2PHibeWC RypFxYvlArpLl666sU9MKeb2nDPbF7WMiyyzfaPY4Zhx674h24JBGyvWf0k5qIs/AgW+ AdeI+pzCqiTD+xd9/dYejH6UcbcRywJ0oZDhWlO50SQB39V6/dTPUwHG7V5QSTEpLF3U WD9Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=NnJO2sB5; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Return-Path: Received: from lipwig.vger.email (lipwig.vger.email. [23.128.96.33]) by mx.google.com with ESMTPS id b7-20020a056a000cc700b0068ff3a3c9d0si8818854pfv.91.2023.09.30.07.37.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 30 Sep 2023 07:37:10 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33; Authentication-Results: mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=NnJO2sB5; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id A0F0D808A427; Sat, 30 Sep 2023 06:36:06 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234194AbjI3NfE (ORCPT + 99 others); Sat, 30 Sep 2023 09:35:04 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43646 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234185AbjI3NfD (ORCPT ); Sat, 30 Sep 2023 09:35:03 -0400 Received: from madras.collabora.co.uk (madras.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e5ab]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D8DCDF9; Sat, 30 Sep 2023 06:35:01 -0700 (PDT) Received: from [192.168.2.47] (109-252-153-31.dynamic.spd-mgts.ru [109.252.153.31]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: dmitry.osipenko) by madras.collabora.co.uk (Postfix) with ESMTPSA id E6BE46607295; Sat, 30 Sep 2023 14:34:59 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1696080900; bh=0PxsPvfhCxmuM8OOhVgn6qAqAky1OTiS500Dk0oUnj8=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=NnJO2sB5C/86P2N1XLfIArFDbs/Wsi5YJZMvJG2bjYx+4nIsfc4XaeQvwjBZHcq2L 0IEskSh8jnMUO/bMBAbDeTJCwCBpedvgfrQyA+xM8Rlvl6/38dqCQp7qWKFU6iQVgZ JTJHcRqBQOZS/GSJL2N014ZqmwdX5VT6DR0vVufQC2YVH9jVIH7SfpiHGxqgrkEjMV yg472WlCqYmatismPMGmEHWDyrk4QcDOsc4HhY4FZTV8k69woEeBJqUeuottXbCX0S bBNSYm1iV1iyQ411HF/JXpDYfupBBfDBPQfgQgDjOQ40SpJldtTmo++l73VH7R6zPk pLjh+yGdNi6WQ== Message-ID: <9771886a-d60d-b273-9f0c-ba663acb1db8@collabora.com> Date: Sat, 30 Sep 2023 16:34:57 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 Subject: Re: [PATCH v9 6/6] KVM: x86/mmu: Handle non-refcounted pages Content-Language: en-US To: David Stevens Cc: Sean Christopherson , Yu Zhang , Isaku Yamahata , Zhi Wang , kvmarm@lists.linux.dev, linux-kernel@vger.kernel.org, kvm@vger.kernel.org References: <20230919022504.3153043-1-stevensd@chromium.org> From: Dmitry Osipenko In-Reply-To: <20230919022504.3153043-1-stevensd@chromium.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-3.4 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Sat, 30 Sep 2023 06:36:06 -0700 (PDT) On 9/19/23 05:25, David Stevens wrote: > On Mon, Sep 18, 2023 at 6:53 PM Dmitry Osipenko wrote: >> >> On 9/11/23 05:16, David Stevens wrote: >>> --- a/arch/x86/kvm/mmu/paging_tmpl.h >>> +++ b/arch/x86/kvm/mmu/paging_tmpl.h >>> @@ -848,7 +848,8 @@ static int FNAME(page_fault)(struct kvm_vcpu *vcpu, struct kvm_page_fault *fault >>> >>> out_unlock: >>> write_unlock(&vcpu->kvm->mmu_lock); >>> - kvm_release_pfn_clean(fault->pfn); >>> + if (fault->is_refcounted_page) >>> + kvm_set_page_accessed(pfn_to_page(fault->pfn)); >> >> The other similar occurrences in the code that replaced >> kvm_release_pfn_clean() with kvm_set_page_accessed() did it under the >> held mmu_lock. >> >> Does kvm_set_page_accessed() needs to be invoked under the lock? > > It looks like I made a mistake when folding the v8->v9 delta into the stack of > patches to get a clean v9 series. v8 of the series returned pfns without > reference counts from __kvm_follow_pfn, so the x86 MMU needed to mark the pages > as accessed under the lock. v9 instead returns pfns with a refcount (i.e. does > the same thing as __gfn_to_pfn_memslot), so the x86 MMU should instead call > kvm_release_page_clean outside of the lock. I've included the corrected version > of this patch in this email. [snip] I tested this series + the corrected version of the patch on Intel TGL using virgl/venus/virtio-intel on both qemu and crosvm on top of the recent linux-next. All is working good. Feel free to add my t-b to the v10: Tested-by: Dmitry Osipenko # virgl+venus+virtio-intel+i915 -- Best regards, Dmitry