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 EAC5FC54EBD for ; Fri, 13 Jan 2023 12:38:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Awx3Ut4SGrw2bnO5rBD9JVRTiduH9lOsI4n2DNZJjiY=; b=erOJSBpJaXLneN4VENtfX4u9VR 9OrVIUo5QqXpcSM7NqAcTxbkuM+4bEt5JPZh0O3z9tLR3j0azxtQ8tAjxUkxMPrvDaLpJa/U2oQ4s IUC9tMfUwLiNPiFs/PpHwU1lzP2ehwCLroZvDF3ul7mYhATOAZHzS3CDs9A7y9It03xZqcLk6JYv/ BJMUmucBNT+D2DnmaQCj/Otu49xc/NZNPfKN/Gvhsi3HFgVjnG1THgTr+OIyCg5gfDGgAy2sFOoy0 xy8FG+tj2WAEdCXmQgrfDJaczhMpPcVkOe/QNoCcDkIcLvE3VYQN89H041Mi3YrDqucUPV8Noi0O5 hhYolndQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pGJJu-002a3W-Bc; Fri, 13 Jan 2023 12:38:06 +0000 Received: from out4-smtp.messagingengine.com ([66.111.4.28]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pGJJq-002a11-Kp for linux-nvme@lists.infradead.org; Fri, 13 Jan 2023 12:38:04 +0000 Received: from compute2.internal (compute2.nyi.internal [10.202.2.46]) by mailout.nyi.internal (Postfix) with ESMTP id CA0905C0051; Fri, 13 Jan 2023 07:37:56 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute2.internal (MEProxy); Fri, 13 Jan 2023 07:37:56 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=irrelevant.dk; h=cc:cc:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm2; t=1673613476; x=1673699876; bh=Aw x3Ut4SGrw2bnO5rBD9JVRTiduH9lOsI4n2DNZJjiY=; b=pzf6qZ5WJy6E/jZdLV hW89Etprgi/KATrVuP9CX9/0v1eAqhh6sjxH7EYrLv22gqWrff0aU3XhyA128FD+ yH5DKjMTaoCZq0icpfLrdnYZ2SfU38X3KOSeUdpHROPaARt+R+FZIz19tk+kHdgb sEV0qTcNlHOH7Z1xWbz42xGZ/QVMkDVRqmOGHSFgTXdQkdyzxCJmXslSkfdl0950 Hy9fEYfzo8nxABEG2zdWx41FKXLasMU2wr6soGt/2q4+f1v1dS82mfeBsgFm1r4s LrglR6N8iKz8KSn22tadES7jVVaLKtXpE8vd9YJUr6cXzw28GP+G8cEeCI6/BrNU E3/Q== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:date:date:feedback-id :feedback-id:from:from:in-reply-to:in-reply-to:message-id :mime-version:references:reply-to:sender:subject:subject:to:to :x-me-proxy:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s= fm3; t=1673613476; x=1673699876; bh=Awx3Ut4SGrw2bnO5rBD9JVRTiduH 9lOsI4n2DNZJjiY=; b=SqyNWkn9Hz2NFKfM4TOi9SHK7qlvTVQePBqztIvYJrQP 9lr+VcXBkbI1VQcHkViRCOkbOpzRd9sC0XM7k9FqLMf42Dl89+yijg2goDDCsTnx Cpy+NX0iXIsRVAcncrw9qi6PxXpJKgbQPq28QjtGhQbuX/e/wkMZpYazFEN6Jt22 unuwlWNWq+Uu8JxKZ3tABaM9cBfecn8umIaGXc34pRdGO7kVoFRfjeAOp9wWrO8l JJNnrlMeN4c0ev66/5kXDwngnIBHj/rTMrlT7teFqV+eeexEZo8b23A6avulNYff ZUHSgCQzUmckV6czHZIG2MiP0uCOg0SXx8PCajDQ6w== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrleekgdefkecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpeffhffvvefukfhfgggtuggjsehgtderredttdejnecuhfhrohhmpefmlhgruhhs ucflvghnshgvnhcuoehithhssehirhhrvghlvghvrghnthdrughkqeenucggtffrrghtth gvrhhnpeejgfejfeffvdeuhfeifefhgffgueelhedukeevjeevtdduudegieegteffffej veenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpehith hssehirhhrvghlvghvrghnthdrughk X-ME-Proxy: Feedback-ID: idc91472f:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Fri, 13 Jan 2023 07:37:53 -0500 (EST) Date: Fri, 13 Jan 2023 13:37:50 +0100 From: Klaus Jensen To: Peter Maydell Cc: Keith Busch , Jens Axboe , Christoph Hellwig , Sagi Grimberg , linux-nvme@lists.infradead.org, qemu-block@nongnu.org, qemu-devel@nongnu.org, Guenter Roeck , "Michael S. Tsirkin" , Marcel Apfelbaum Subject: Re: completion timeouts with pin-based interrupts in QEMU hw/nvme Message-ID: References: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="GDbzrrQ6Xxiht77F" Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230113_043803_540893_6A38A105 X-CRM114-Status: GOOD ( 17.67 ) X-BeenThere: linux-nvme@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-nvme" Errors-To: linux-nvme-bounces+linux-nvme=archiver.kernel.org@lists.infradead.org --GDbzrrQ6Xxiht77F Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Jan 13 12:32, Peter Maydell wrote: > On Fri, 13 Jan 2023 at 08:55, Klaus Jensen wrote: > > > > +CC qemu pci maintainers > > > > Michael, Marcel, > > > > Do you have any comments on this thread? As you can see one solution is > > to simply deassert prior to asserting, the other is to reintroduce a > > pci_irq_pulse(). Both seem to solve the issue. >=20 > Both seem to be missing any analysis of "this is what is > happening, this is where we differ from hardware, this > is why this is the correct fix". We shouldn't put in > random "this seems to happen to cause the guest to boot" > fixes, please. >=20 No, I'd like to get to the bottom of this, which is why I'm reaching out to the pci maintainers to get an idea about if this is a general/known issue with pin based interrupts on pci. There are a fair amount of uses of pci_irq_pulse() still left in the tree. --GDbzrrQ6Xxiht77F Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAEBCgAdFiEEUigzqnXi3OaiR2bATeGvMW1PDekFAmPBUJ4ACgkQTeGvMW1P Dek3xwf/arpcGt6Lgj2IiknKb/+M7NYjx9H90bEk28jB0sZ4MCcsOt6Ltl3CxmIf kImorYKhR3/Pr+jjNlMrI7b92uA1UXxSQVIMFuXioj79dS/Vk/NSnytb4Z7rGnOB Kxk/ePDY+bVXxS65wRay05g+f6dbxYkxYies4QZqqvQ6GCKzJAI7NLGgK46koFSz nwj4phUhunGlyTiOEWd4KyJQMtiaMvf8rj01VVQ0/a8o9yxaQN/PUHm/dO/R+Cpy PsFsmnChtMh5UfByRnm4BF0B7dk7Bjmr7mBNP2/3PBFQgPl9hBO09hsw2QFUy+yp vy3YwNuajIPJT8a/qorlYehyJXXV+A== =WpH1 -----END PGP SIGNATURE----- --GDbzrrQ6Xxiht77F--