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=-10.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham 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 89ADBC4743E for ; Tue, 8 Jun 2021 12:19:31 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 69A5861182 for ; Tue, 8 Jun 2021 12:19:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232518AbhFHMVX (ORCPT ); Tue, 8 Jun 2021 08:21:23 -0400 Received: from verein.lst.de ([213.95.11.211]:50559 "EHLO verein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232504AbhFHMVW (ORCPT ); Tue, 8 Jun 2021 08:21:22 -0400 Received: by verein.lst.de (Postfix, from userid 2407) id E93F468AFE; Tue, 8 Jun 2021 14:19:25 +0200 (CEST) Date: Tue, 8 Jun 2021 14:19:25 +0200 From: Christoph Hellwig To: Geert Uytterhoeven Cc: "David S . Miller" , Keith Busch , Jens Axboe , Christoph Hellwig , Sagi Grimberg , Omkar Kulkarni , Hannes Reinecke , Dean Balandin , Himanshu Madhani , Shai Malin , Petr Mladek , linux-nvme@lists.infradead.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] nvme: NVME_TCP_OFFLOAD should not default to m Message-ID: <20210608121925.GA24201@lst.de> References: <39b1a3684880e1d85ef76e34403886e8f1d22508.1623149635.git.geert+renesas@glider.be> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <39b1a3684880e1d85ef76e34403886e8f1d22508.1623149635.git.geert+renesas@glider.be> User-Agent: Mutt/1.5.17 (2007-11-01) Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Tue, Jun 08, 2021 at 12:56:09PM +0200, Geert Uytterhoeven wrote: > The help text for the symbol controlling support for the NVM Express > over Fabrics TCP offload common layer suggests to not enable this > support when unsure. > > Hence drop the "default m", which actually means "default y" if > CONFIG_MODULES is not enabled. > > Fixes: f0e8cb6106da2703 ("nvme-tcp-offload: Add nvme-tcp-offload - NVMeTCP HW offload ULP") > Signed-off-by: Geert Uytterhoeven Err, where did this appear from? This code has not been accepted into the NVMe tree and is indeed not acceptable in this form.