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 lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 17291CD8C92 for ; Mon, 8 Jun 2026 14:53:52 +0000 (UTC) Received: from boromir.ozlabs.org (localhost [127.0.0.1]) by lists.ozlabs.org (Postfix) with ESMTP id 4gYw4W2X2nz3cCt; Tue, 09 Jun 2026 00:52:27 +1000 (AEST) Authentication-Results: lists.ozlabs.org; arc=none smtp.remote-ip=85.214.250.239 ARC-Seal: i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1780930345; cv=none; b=GRb+x/+beRhQar+2DbbgZoIzJnKtWRBDxxc7BIrRHZiParSoNcrsMtv2ByPcuYhT7QYjSWIGE2rV17KrH1NUIz/I4FFEi6nvfj9SokP4DRtsJBy2jAclcTLaX4xco9Kdm70WiqwvQPqQjit1Wq1NK0DRq/XVOyY37QvIEq3CX6dLQu1MiRBhuBvvN5vSnUbmDGtE3BW111KMdxqnyPIY/4a/h+vp8A6rd2AWnMYo3/SzkgPeugmR8QgV0g2l3NAZmk+lVhKMTHnYusTQZL7G93MObcEO4XwRa7nvcZIvIRLiGiar1wPilCUhS+3hWER5q1aezgz4zVm9vJLZz2qPPg== ARC-Message-Signature: i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1780930345; c=relaxed/relaxed; bh=mbEehg9LdTGdSOiAAIn3KdGBhg9eDEx+AekEW+rMGwM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=DzPqgvQmu9Wi62HPO6sL7Td1bMgd1bV7BY2kn20DMBXJq+5DPzlwNFajcJ7VqW5KoE3/sXd14KajP8VCln0P0vqdU17zu+0c9ypKhpiDKCWS15+9QPQOwKY8aAhSKIsJkoXdN2Y0MnvdQYPgtbmHdZyBDKwP8/tuRX1/12NLn9GqBkAcW3IujXGG0JXLFZ1ismAlqDdnyHUYlQqY2ddRWoSzgmIib5LtfJuf9dcl/3o7DYaUEMC7HgzpLGaYkGdPOnWxpxqzYDYkz6vf3vq9WjDVCqV2+EsP2I1HV6+Ap6P1hUA0h08rcX2SMJheKYmgA62JJgk05TM0roQ2sUHX7A== ARC-Authentication-Results: i=1; lists.ozlabs.org; dmarc=none (p=none dis=none) header.from=8bytes.org; spf=pass (client-ip=85.214.250.239; helo=mail.8bytes.org; envelope-from=joro@8bytes.org; receiver=lists.ozlabs.org) smtp.mailfrom=8bytes.org Authentication-Results: lists.ozlabs.org; dmarc=none (p=none dis=none) header.from=8bytes.org Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=8bytes.org (client-ip=85.214.250.239; helo=mail.8bytes.org; envelope-from=joro@8bytes.org; receiver=lists.ozlabs.org) Received: from mail.8bytes.org (mail.8bytes.org [85.214.250.239]) by lists.ozlabs.org (Postfix) with ESMTP id 4gYw4L19j1z3bwp for ; Tue, 09 Jun 2026 00:52:17 +1000 (AEST) Received: from io.home.8bytes.org (p4ffe1d30.dip0.t-ipconnect.de [79.254.29.48]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mail.8bytes.org (Postfix) with ESMTPSA id 67A2E2028BC; Mon, 8 Jun 2026 16:43:10 +0200 (CEST) From: =?UTF-8?q?J=C3=B6rg=20R=C3=B6del?= To: Paolo Bonzini , Sean Christopherson Cc: Tom Lendacky , ashish.kalra@amd.com, michael.roth@amd.com, nsaenz@amazon.com, anelkz@amazon.de, James.Bottomley@HansenPartnership.com, Melody Wang , kvm@vger.kernel.org, linux-kernel@vger.kernel.org, kvmarm@lists.linux.dev, loongarch@lists.linux.dev, linux-mips@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, kvm-riscv@lists.infradead.org, x86@kernel.org, coconut-svsm@lists.linux.dev, joerg.roedel@amd.com Subject: [PATCH 25/60] kvm: Move kvm_vcpu->dirty_ring to struct kvm_vcpu_common Date: Mon, 8 Jun 2026 16:42:17 +0200 Message-ID: <20260608144252.351443-26-joro@8bytes.org> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260608144252.351443-1-joro@8bytes.org> References: <20260608144252.351443-1-joro@8bytes.org> X-Mailing-List: linuxppc-dev@lists.ozlabs.org List-Id: List-Help: List-Owner: List-Post: List-Archive: , List-Subscribe: , , List-Unsubscribe: Precedence: list MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Joerg Roedel The dirty tracking should happen across all planes of a given VCPU, so move the dirty_ring to struct kvm_vcpu_common. Signed-off-by: Joerg Roedel --- include/linux/kvm_host.h | 3 ++- virt/kvm/dirty_ring.c | 4 ++-- virt/kvm/kvm_main.c | 22 ++++++++++------------ 3 files changed, 14 insertions(+), 15 deletions(-) diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h index f6e8a0b653b3..7d06459a06f3 100644 --- a/include/linux/kvm_host.h +++ b/include/linux/kvm_host.h @@ -373,6 +373,8 @@ struct kvm_vcpu_common { bool preempted; bool ready; bool scheduled_out; + + struct kvm_dirty_ring dirty_ring; }; struct kvm_vcpu { @@ -413,7 +415,6 @@ struct kvm_vcpu { struct kvm_vcpu_arch arch; struct kvm_vcpu_stat stat; char stats_id[KVM_STATS_NAME_SIZE]; - struct kvm_dirty_ring dirty_ring; /* * The most recently used memslot by this vCPU and the slots generation diff --git a/virt/kvm/dirty_ring.c b/virt/kvm/dirty_ring.c index 572b854edf74..c6f46b93bddb 100644 --- a/virt/kvm/dirty_ring.c +++ b/virt/kvm/dirty_ring.c @@ -218,7 +218,7 @@ int kvm_dirty_ring_reset(struct kvm *kvm, struct kvm_dirty_ring *ring, void kvm_dirty_ring_push(struct kvm_vcpu *vcpu, u32 slot, u64 offset) { - struct kvm_dirty_ring *ring = &vcpu->dirty_ring; + struct kvm_dirty_ring *ring = &vcpu->common->dirty_ring; struct kvm_dirty_gfn *entry; /* It should never get full */ @@ -250,7 +250,7 @@ bool kvm_dirty_ring_check_request(struct kvm_vcpu *vcpu) * the dirty ring is reset by userspace. */ if (kvm_check_request(KVM_REQ_DIRTY_RING_SOFT_FULL, vcpu) && - kvm_dirty_ring_soft_full(&vcpu->dirty_ring)) { + kvm_dirty_ring_soft_full(&vcpu->common->dirty_ring)) { kvm_make_request(KVM_REQ_DIRTY_RING_SOFT_FULL, vcpu); vcpu->run->exit_reason = KVM_EXIT_DIRTY_RING_FULL; trace_kvm_dirty_ring_exit(vcpu); diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index 24ff8748a317..f85ddb0fc781 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm/kvm_main.c @@ -483,6 +483,13 @@ static int kvm_vcpu_init_common(struct kvm_vcpu *vcpu, struct kvm *kvm, unsigned common->ready = false; preempt_notifier_init(&common->preempt_notifier, &kvm_preempt_ops); + if (kvm->dirty_ring_size) { + r = kvm_dirty_ring_alloc(kvm, &common->dirty_ring, + id, kvm->dirty_ring_size); + if (r) + goto out_drop_counter; + } + vcpu->common = no_free_ptr(common); kvm_vcpu_set_in_spin_loop(vcpu, false); @@ -547,6 +554,7 @@ static void kvm_vcpu_common_destroy(struct kvm_vcpu *vcpu) * are already gone. */ put_pid(common->pid); + kvm_dirty_ring_free(&common->dirty_ring); kfree(common); } @@ -555,7 +563,6 @@ static void kvm_vcpu_destroy(struct kvm_vcpu *vcpu) kvm_arch_vcpu_destroy(vcpu); kvm_vcpu_common_destroy(vcpu); - kvm_dirty_ring_free(&vcpu->dirty_ring); free_page((unsigned long)vcpu->run); kmem_cache_free(kvm_vcpu_cache, vcpu); @@ -4209,7 +4216,7 @@ static vm_fault_t kvm_vcpu_fault(struct vm_fault *vmf) #endif else if (kvm_page_in_dirty_ring(vcpu->kvm, vmf->pgoff)) page = kvm_dirty_ring_get_page( - &vcpu->dirty_ring, + &vcpu->common->dirty_ring, vmf->pgoff - KVM_DIRTY_LOG_PAGE_OFFSET); else return kvm_arch_vcpu_fault(vcpu, vmf); @@ -4338,13 +4345,6 @@ static int kvm_vm_ioctl_create_vcpu(struct kvm *kvm, unsigned long id) if (r) goto vcpu_free_run_page; - if (kvm->dirty_ring_size) { - r = kvm_dirty_ring_alloc(kvm, &vcpu->dirty_ring, - id, kvm->dirty_ring_size); - if (r) - goto arch_vcpu_destroy; - } - mutex_lock(&kvm->lock); if (kvm_get_vcpu_by_id(kvm, id)) { @@ -4385,8 +4385,6 @@ static int kvm_vm_ioctl_create_vcpu(struct kvm *kvm, unsigned long id) xa_erase(&kvm->planes[0]->vcpu_array, vcpu->vcpu_idx); unlock_vcpu_destroy: mutex_unlock(&kvm->lock); - kvm_dirty_ring_free(&vcpu->dirty_ring); -arch_vcpu_destroy: kvm_arch_vcpu_destroy(vcpu); vcpu_free_run_page: free_page((unsigned long)vcpu->run); @@ -5120,7 +5118,7 @@ static int kvm_vm_ioctl_reset_dirty_pages(struct kvm *kvm) mutex_lock(&kvm->slots_lock); kvm_for_each_vcpu(i, vcpu, kvm) { - r = kvm_dirty_ring_reset(vcpu->kvm, &vcpu->dirty_ring, &cleared); + r = kvm_dirty_ring_reset(vcpu->kvm, &vcpu->common->dirty_ring, &cleared); if (r) break; } -- 2.53.0