From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f48.google.com (mail-wm1-f48.google.com [209.85.128.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4DDC4207A2E for ; Fri, 21 Feb 2025 11:11:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740136291; cv=none; b=lFuh958Fa5FuSljRF1WK0PfH+eR3psdYCVS8ILA/lnthjzeEUgVy4rSm7Y/cqBSCbD9fdAQfLSBQiVbwYnnT+zmx854pOQrS58nHyfW8YgYxp+UfdDnNaFK6TL1+5LEEf7fcXpxS14WVuXD6ENqxNF9HTBC1RTFs+OfS51E1Ws4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740136291; c=relaxed/simple; bh=uFst0uZhDHecANQgSPcaTkQaZji1kbxiGdT8lgz3J6Y=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=F283dARJ4ridAdC93D5qRXTIBou1jiDGkJaYSdfDmnB6QFdHWgoWPhNRlyYGiXovFDBQ3m2nPIER3hlZ7QiKaMktjCtdu7dcSzxLWUkUS8Ol5pMMR7UFb9N0+GgCqJ1/thns4EJ4ufN5CQ7lRgfjeC6ia7eVtkSDkhovVsprQrc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ventanamicro.com; spf=pass smtp.mailfrom=ventanamicro.com; dkim=pass (2048-bit key) header.d=ventanamicro.com header.i=@ventanamicro.com header.b=KWpMP9QA; arc=none smtp.client-ip=209.85.128.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ventanamicro.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ventanamicro.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=ventanamicro.com header.i=@ventanamicro.com header.b="KWpMP9QA" Received: by mail-wm1-f48.google.com with SMTP id 5b1f17b1804b1-4399d14334aso17257175e9.0 for ; Fri, 21 Feb 2025 03:11:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ventanamicro.com; s=google; t=1740136287; x=1740741087; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=jSsWiZSoq00eYWyXg0OJO1m3DP0gspy/2ZFYEnCczbE=; b=KWpMP9QAKaSZKOF420rklwiTuC5+Y8wM1RIzB1olZYwtHg3vYK/zy5hXWL5cRXYgIE z9utogW2VMSxIGZnPCaIYTckrD7YEP53k5nMLrlI/Yh0OSq7FhvaOHkGq53SjipSC8aC 6dU6OI1g5x+Px8RjGm3DYgpIBQc6LRF0NXoGq/rmhYC73RYRj1Co2vL/ScVA2deX4Lou 38AmHoYTxNU9dIAshQcCsvrzcEp3cN4HZz7ZIepCmAu5eMEsPYKFE/u0uuD9CMhM3KNo btIPZlvvgofMVBJZF7UK67BensNJ3B1MnbmSSyMlcbfRoGjAxTw57lA0PYp2bV0iQLDV vxAA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740136287; x=1740741087; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=jSsWiZSoq00eYWyXg0OJO1m3DP0gspy/2ZFYEnCczbE=; b=FYj6ABbXOb7gKB7B1TPpJq3jd2hqjzee3evnS/WkKdAxVY9HDWbZwyF47u0SHumRAO RgOLTKaq2joOPxHj1fdble8sWk7lb5XUrwdcla9oWfu/NXgJZ56j9f0rhBPx7tWJpJ7r skhctZ1lkLIvHMg5IwKTOtFuI6VODNYWu0N43Fc6CcANXqwtDsEOtymkrBHXkiAZV3Vh UOtOOICAtNGm19+JeHQm1Qt52Gp1P5crNncpriwAjH0NfUVcFWfUUO67odHGe+Olcbmm ibHVGOmBuzV/c+8u2gi1blm+qAnQCPddVLMdmNjOOz59L16TnIeJFZV1HMXhPcm/Tx06 EvfA== X-Forwarded-Encrypted: i=1; AJvYcCW0P7dEmrgnbIuCams3BJlpjEZGVI5zC0y6Z8T6ye4wRUtNJpVF4dNatB8XWSX73tBt3px0tLQR2u9tZcQ=@vger.kernel.org X-Gm-Message-State: AOJu0YyPvsvn2l3zhqNZZcmfipW/aICeJDd8cah73K7fnLzu2qxqClDN A3I7OCGIn8Dyg9bEk9Ztxt4/LNNIirjj348N4hTFPqDMyjea0O0DCb6jsZGnUEU= X-Gm-Gg: ASbGncv5iIMVQFCtTnzNd+QHUtU9vjjLojSD1O7T+vwaZe8ukEMvxSuLtEbOldtbZl/ opW6Twl+l02xVQwhia345sJQKDvhWMI1q1eHdOhAnSQwuZBztZuJHEWHEMKQCZHcsTwRjR61FM6 2M3lqhLFXOy3p7EWeYwi1C6dak5B0RCIvSZ+t2dZaEgrxJ4o2YPDpeUiPTVKfm1M4QKatN6ZUwy Ws6lmzLUzCBLa04B/ABesNjzwEowd62+/RI+0MMAnRrs2AkHc320QnQm+egJB4scBTa04EpcV5D sBRlOsRSEwaUQA== X-Google-Smtp-Source: AGHT+IF0uAlwa6gBu1gPr2dqwSyuIZZV/tSdbeIren4YqG3M6YP4kMXrRqTqSrM5G5BkGMJt/vM5Cw== X-Received: by 2002:a05:600c:1396:b0:439:6d7c:48fd with SMTP id 5b1f17b1804b1-439ae1d877dmr24012595e9.4.1740136287415; Fri, 21 Feb 2025 03:11:27 -0800 (PST) Received: from localhost ([2a02:8308:a00c:e200::766e]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-38f258b439dsm22921657f8f.8.2025.02.21.03.11.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 21 Feb 2025 03:11:27 -0800 (PST) Date: Fri, 21 Feb 2025 12:11:26 +0100 From: Andrew Jones To: BillXiang Cc: anup@brainfault.org, kvm-riscv@lists.infradead.org, kvm@vger.kernel.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, atishp@atishpatra.org, paul.walmsley@sifive.com, palmer@dabbelt.com, aou@eecs.berkeley.edu, rkrcmar@ventanamicro.com Subject: Re: [PATCH v2] riscv: KVM: Remove unnecessary vcpu kick Message-ID: <20250221-11926c4f2ca3fab3b565bc92@orel> References: <20250221104538.2147-1-xiangwencheng@lanxincomputing.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250221104538.2147-1-xiangwencheng@lanxincomputing.com> On Fri, Feb 21, 2025 at 06:45:38PM +0800, BillXiang wrote: > Remove the unnecessary kick to the vCPU after writing to the vs_file > of IMSIC in kvm_riscv_vcpu_aia_imsic_inject. > > For vCPUs that are running, writing to the vs_file directly forwards > the interrupt as an MSI to them and does not need an extra kick. > > For vCPUs that are descheduled after emulating WFI, KVM will enable > the guest external interrupt for that vCPU in > kvm_riscv_aia_wakeon_hgei. This means that writing to the vs_file > will cause a guest external interrupt, which will cause KVM to wake > up the vCPU in hgei_interrupt to handle the interrupt properly. > > Signed-off-by: BillXiang > --- > v2: Revise the commit message to ensure it meets the required > standards for acceptance > > arch/riscv/kvm/aia_imsic.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/riscv/kvm/aia_imsic.c b/arch/riscv/kvm/aia_imsic.c > index a8085cd8215e..29ef9c2133a9 100644 > --- a/arch/riscv/kvm/aia_imsic.c > +++ b/arch/riscv/kvm/aia_imsic.c > @@ -974,7 +974,6 @@ int kvm_riscv_vcpu_aia_imsic_inject(struct kvm_vcpu *vcpu, > > if (imsic->vsfile_cpu >= 0) { > writel(iid, imsic->vsfile_va + IMSIC_MMIO_SETIPNUM_LE); > - kvm_vcpu_kick(vcpu); > } else { > eix = &imsic->swfile->eix[iid / BITS_PER_TYPE(u64)]; > set_bit(iid & (BITS_PER_TYPE(u64) - 1), eix->eip); > -- > 2.46.2 Reviewed-by: Andrew Jones