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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 9DC9AC3DA6E for ; Sun, 17 Dec 2023 18:53:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Subject:Cc:To:From:Message-ID:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=y4UVyWg1BKOtMp5lfGWJjN/TkUw9zGlvLkPgEUE7QwQ=; b=mCOnYdcF4gAqXU i/AUDGOajOrSq2OwD461niuvdaQv75qmDNTUkPOxwsc4+uplt+Cxxu9RuvOUyrIpInqQHCpuvVpth xZmoJ318luNfGq6l5Cf1k6qU/hM36a/fbSJAPa3D/5zcWqiA2jAuiP9eWnYuOJ69oWkWE4Vs72iwr 9d6zY47qZABbbhZBTOLl9nQdo32y4xjAuRtroUuzJKJSL87S17sh+WtmrNcBm0HXEp4RDBHTVbHe6 1sg3ifBN/myz0fyd59IQdtjKxgAehoKKG8yIG7JVuHOezEuh3VchvD60gDkBRVI8ivAP4hoZ1rNmA uXI+zhe0jUu7trxidILg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rEwGA-008K25-2h; Sun, 17 Dec 2023 18:53:06 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rEwG7-008K1W-0q for linux-arm-kernel@lists.infradead.org; Sun, 17 Dec 2023 18:53:04 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by ams.source.kernel.org (Postfix) with ESMTP id B57EAB80AC5; Sun, 17 Dec 2023 18:53:00 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A5B7CC433C8; Sun, 17 Dec 2023 18:52:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1702839178; bh=h3U7iqYMSSXE3PLLsIKv7FMGZUFaSTPMFTW4G0Ip/kA=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=Zkwv67ry4jM9CrEDBCo8NgGmOjYoDUnAqgtuY0r9ceizWfkrO2uDiCA5ZoGQ2es2k JaZA5uIBcV3EI1gWgo/hI4iWAvVXO2ILW49yDGTmDjOLP6EG0e3G6gj6jSNMT0fHoS 1c0ZM6LQ5oFNpLeVzIOB+S/SeHeqUU7UntUOKl+/qBFq+85Nx07N0cH2ZsE5fMw4TT IZB1OPvjHXrHlDilVw/XOIdfmWEqlUZBwMrF+AZcBQADBhMfEroMybeZ0bW2senamY GMA8gq/WZusrHSgFECPH/nOqBqHV/90HIhvoasUg76Lxqqs+lSEngDNGZRfLt/o56e nSdgTHUz5GCFg== Received: from sofa.misterjones.org ([185.219.108.64] helo=wait-a-minute.misterjones.org) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1rEwFz-004rnx-TY; Sun, 17 Dec 2023 18:52:56 +0000 Date: Sun, 17 Dec 2023 18:52:55 +0000 Message-ID: <878r5s8xvc.wl-maz@kernel.org> From: Marc Zyngier To: Oliver Upton Cc: Kunkun Jiang , James Morse , Suzuki K Poulose , Zenghui Yu , Catalin Marinas , Will Deacon , Jean-Philippe Brucker , "moderated list:ARM SMMU DRIVERS" , kvmarm@lists.linux.dev, open list , "wanghaibin.wang@huawei.com" Subject: Re: [bug report] GICv4.1: vSGI remains pending across the guest reset In-Reply-To: References: <7e7f2c0c-448b-10a9-8929-4b8f4f6e2a32@huawei.com> <87a5q983zc.wl-maz@kernel.org> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/28.2 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: oliver.upton@linux.dev, jiangkunkun@huawei.com, james.morse@arm.com, suzuki.poulose@arm.com, yuzenghui@huawei.com, catalin.marinas@arm.com, will@kernel.org, jean-philippe@linaro.org, linux-arm-kernel@lists.infradead.org, kvmarm@lists.linux.dev, linux-kernel@vger.kernel.org, wanghaibin.wang@huawei.com X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231217_105303_596404_DEE45D14 X-CRM114-Status: GOOD ( 24.79 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sun, 17 Dec 2023 17:34:38 +0000, Oliver Upton wrote: > > On Sun, Dec 17, 2023 at 05:33:16PM +0000, Oliver Upton wrote: > > On Sun, Dec 17, 2023 at 11:26:15AM +0000, Marc Zyngier wrote: > > > > [...] > > > > > But this has *nothing* to do with the guest. This is the *host* > > > userspace performing a write to the redistributor view, which has > > > different semantics. Which is why your earlier description made no > > > sense to me. > > > > > > I think the problem is slightly larger than what you describe. A write > > > to ISPENDR0 should be propagated to the ITS for any values of the > > > latch, just like this happens on enabling HW-backed SGIs. > > > > > > Can you please give this a go? > > > > What do you think about using this as an opportunity for a bit of > > cleanup? It'd be nice unify the various MMIO and uaccess handlers for > > SPENDING + CPENDING while being careful about the arch_timer interrupt. What is special about the timer interrupt? > Cut myself off... Meant to say that user writes to SPENDING for GICv3 > can then be treated as: > > > clear = ~val; > > vgic_uaccess_write_spending(val); > > vgic_uaccess_write_cpending(clear); Could be. But I'd rather have separate fixes from more invasive reworks. Specially given that we have had multiple ugly bugs around this code in the past, which is why we ended up splitting userspace from guest accessors. M. -- Without deviation from the norm, progress is not possible. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel