From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kroah.com header.i=@kroah.com header.b="KrrT7nAN"; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="NISFLn7d" Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0D65DC2 for ; Mon, 11 Dec 2023 06:47:29 -0800 (PST) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id CF90C5C03B5; Mon, 11 Dec 2023 09:47:25 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute1.internal (MEProxy); Mon, 11 Dec 2023 09:47:25 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kroah.com; h=cc :cc:content-type: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=fm1; t=1702306045; x=1702392445; bh=aD bCmLvi9t4wqPy0P3gYOPOM7ZNVoaRUqU5RYRCbjgs=; b=KrrT7nANfapS8cJ673 poWaqPTcR54B66eOYZ9fsEYUXzlGGTTYlasYRYR+5c/7KFoh50XfbMiD9wpoNkU1 BeBazc/2ZK88faUbb7/CE36Pil3ZTlFajGYxPC0jBvBopacfERfnGwknbuJTLkcz CvJ6QfGD9a5HwNVKaEmlHGipFI0jL5p7I6JBtGLkYUJkPPQKf/gzlgqpTz2NvA87 q/zKzOfnH0gw2ANtxT27PIYbi5aPxlBt6VQa4cXE9xkURXvx5uyMitcrDwdyxAsn J6ticE2oaEnxrTe9qNBgWMvCio4Uvd4EcQMZPtVozUl/XpW3PSrMrFQSRlQmua2a KNwQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type: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=fm1; t=1702306045; x=1702392445; bh=aDbCmLvi9t4wq Py0P3gYOPOM7ZNVoaRUqU5RYRCbjgs=; b=NISFLn7dytRS20DU/CFP9hfKaTHO2 GYdfNSckznnhXRTVMSSpuhPUPayiNvphojYacrXdMvEA4Kb20VXIXEVZHOiojByi PeyN/s+tkD0hMUasLugM+BfBCT+IKgtQDqkV/K8n2H0sy0Tugsz4f6vTwfGaEBnx YsHWZr+ENc7+1yCMNWyTr3pXjWwmArnk72mqAw/NqT7LLWN56D+jdA2rn6WAf2UL jEL9hpnjz2oehW+5B7s3steTfqcaeaqiZbzCB7jHHLIC4N+15t9UFXCrzFrKMCHf xcxRTvXbmqHfDc2SLBk2eRhQZx0lt3B2/3JuIYYMrObHBlMG/CUIx/ubQ== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvkedrudelvddgjedtucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvfevuffkfhggtggujgesthdtredttddtvdenucfhrhhomhepifhrvghg ucfmjfcuoehgrhgvgheskhhrohgrhhdrtghomheqnecuggftrfgrthhtvghrnhepgeehue ehgfdtledutdelkeefgeejteegieekheefudeiffdvudeffeelvedttddvnecuffhomhgr ihhnpehkvghrnhgvlhdrohhrghenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmh epmhgrihhlfhhrohhmpehgrhgvgheskhhrohgrhhdrtghomh X-ME-Proxy: Feedback-ID: i787e41f1:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Mon, 11 Dec 2023 09:47:25 -0500 (EST) Date: Mon, 11 Dec 2023 15:47:23 +0100 From: Greg KH To: Adrian Hunter Cc: stable@vger.kernel.org Subject: Re: [PATCH 5.10] mmc: block: Be sure to wait while busy in CQE error recovery Message-ID: <2023121116-unvented-stipend-df75@gregkh> References: <2023120358-baking-anymore-b0c7@gregkh> <20231211144132.99257-1-adrian.hunter@intel.com> Precedence: bulk X-Mailing-List: stable@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: <20231211144132.99257-1-adrian.hunter@intel.com> On Mon, Dec 11, 2023 at 04:41:32PM +0200, Adrian Hunter wrote: > STOP command does not guarantee to wait while busy, but subsequent command > MMC_CMDQ_TASK_MGMT to discard the queue will fail if the card is busy, so > be sure to wait by employing mmc_poll_for_busy(). > > Backport to 5.10: Add mmc_busy_cmd MMC_BUSY_IO > > Fixes: 72a5af554df8 ("mmc: core: Add support for handling CQE requests") > Cc: stable@vger.kernel.org > Signed-off-by: Adrian Hunter > Reviewed-by: Avri Altman > Reviewed-by: Christian Loehle > Link: https://lore.kernel.org/r/20231103084720.6886-4-adrian.hunter@intel.com > Signed-off-by: Ulf Hansson > (cherry picked from commit c616696a902987352426fdaeec1b0b3240949e6b) > Tested-by: Adrian Hunter > Signed-off-by: Adrian Hunter > --- > drivers/mmc/core/core.c | 2 ++ > drivers/mmc/core/mmc_ops.c | 3 ++- > drivers/mmc/core/mmc_ops.h | 1 + > 3 files changed, 5 insertions(+), 1 deletion(-) Now queued up, thanks. greg k-h