| Message ID | 39b1a3684880e1d85ef76e34403886e8f1d22508.1623149635.git.geert+renesas@glider.be |
|---|---|
| State | New |
| Headers | show |
| Series | nvme: NVME_TCP_OFFLOAD should not default to m | expand |
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 <geert+renesas@glider.be> Err, where did this appear from? This code has not been accepted into the NVMe tree and is indeed not acceptable in this form.
Hi Dave, please drop the nvme-offload code from net-next. Code for drivers/nvme/ needs ACKs from us nvme maintainers and for something this significant also needs to go through the NVMe tree. And this code is not ready yet.
diff --git a/drivers/nvme/host/Kconfig b/drivers/nvme/host/Kconfig index 9c6f4d776daf14cf..f76cc4690bfc37bc 100644 --- a/drivers/nvme/host/Kconfig +++ b/drivers/nvme/host/Kconfig @@ -88,7 +88,6 @@ config NVME_TCP config NVME_TCP_OFFLOAD tristate "NVM Express over Fabrics TCP offload common layer" - default m depends on BLOCK depends on INET select NVME_CORE
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 <geert+renesas@glider.be> --- drivers/nvme/host/Kconfig | 1 - 1 file changed, 1 deletion(-)