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 X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9A9C3C47089 for ; Thu, 27 May 2021 11:40:35 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 77A986113B for ; Thu, 27 May 2021 11:40:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233734AbhE0LmH (ORCPT ); Thu, 27 May 2021 07:42:07 -0400 Received: from verein.lst.de ([213.95.11.211]:38448 "EHLO verein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233044AbhE0LmF (ORCPT ); Thu, 27 May 2021 07:42:05 -0400 Received: by verein.lst.de (Postfix, from userid 2407) id D266468AFE; Thu, 27 May 2021 13:40:29 +0200 (CEST) Date: Thu, 27 May 2021 13:40:29 +0200 From: Christoph Hellwig To: Kai-Heng Feng Cc: Bjorn Helgaas , Christoph Hellwig , Keith Busch , Koba Ko , Jens Axboe , Sagi Grimberg , linux-nvme , Linux Kernel Mailing List , Henrik Juul Hansen , Bjorn Helgaas , Linux PCI Subject: Re: [PATCH] nvme-pci: Avoid to go into d3cold if device can't use npss. Message-ID: <20210527114029.GC17266@lst.de> References: <20210526150633.GA1291513@bjorn-Precision-5520> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, May 27, 2021 at 12:24:06AM +0800, Kai-Heng Feng wrote: > Yes, that's exactly what they said. Because Windows Modern Standby > always keep the NVMe at D0, so D3hot is untested by the vendors. So all the problems we've deal with latetly are that platforms cut the power off. So it is not kept always at D0 which would be the right thing, but moved into D3cold. Which makes this patch to disable D3cold rather counterintuitive.