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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8BDBBC001E0 for ; Mon, 31 Jul 2023 20:11:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230442AbjGaULN (ORCPT ); Mon, 31 Jul 2023 16:11:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45040 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229987AbjGaUK5 (ORCPT ); Mon, 31 Jul 2023 16:10:57 -0400 Received: from verein.lst.de (verein.lst.de [213.95.11.211]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 538D11BF9 for ; Mon, 31 Jul 2023 13:10:52 -0700 (PDT) Received: by verein.lst.de (Postfix, from userid 2407) id 9F65C68AA6; Mon, 31 Jul 2023 22:10:47 +0200 (CEST) Date: Mon, 31 Jul 2023 22:10:47 +0200 From: Christoph Hellwig To: "Limonciello, Mario" Cc: August Wikerfors , Keith Busch , axboe@fb.com, hch@lst.de, sagi@grimberg.me, linux-nvme@lists.infradead.org, linux-kernel@vger.kernel.org, nilskruse97@gmail.com, David.Chang@amd.com Subject: Re: [PATCH] nvme: Don't fail to resume if NSIDs change Message-ID: <20230731201047.GA14034@lst.de> References: <20230731185103.18436-1-mario.limonciello@amd.com> <040c5788-1a7b-26ea-23cc-ba239c76efa9@augustwikerfors.se> <39697f68-9dc8-7692-7210-b75cce32c6ce@amd.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <39697f68-9dc8-7692-7210-b75cce32c6ce@amd.com> User-Agent: Mutt/1.5.17 (2007-11-01) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jul 31, 2023 at 03:09:08PM -0500, Limonciello, Mario wrote: > So is it reasonable to just add a check for > > ctrl->oacs & NVME_CTRL_OACS_NS_MNGT_SUPP > > In the same error handling path as this patch? No. There are tons of NVMe devices that only support creating and deleting namespace out of band, especially in virtualized and cloud setups.