[PATCH 0/2] Disable rdmavt and hfi1 drivers by default

[PATCH 0/2] Disable rdmavt and hfi1 drivers by default

[Bart Van Assche]

Hi Doug,

Since there is a strict policy in the Linux kernel that new drivers must 
be disabled by default, hence this series with two small patches that 
ensure that e.g. "make oldconfig" neither enables the rdmavt nor the 
hfi1 driver. The patches in this series are:

0001-IB-rdmavt-Disable-by-default.patch
0002-IB-hfi1-Disable-by-default.patch

Thanks,

Bart.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 1/2] IB/rdmavt: Disable by default

[Bart Van Assche]

There is a strict policy in the Linux kernel that new drivers must be
disabled by default. Hence leave out the "default m" line from Kconfig.

Fixes: 0194621b2253 ("IB/rdmavt: Create module framework and handle driver registration")
Signed-off-by: Bart Van Assche <bart.vanassche-XdAiOPVOjttBDgjK7y7TUQ@public.gmane.org>
Cc: Jubin John <jubin.john-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: Dennis Dalessandro <dennis.dalessandro-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: Ira Weiny <ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: Mike Marciniszyn <mike.marciniszyn-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: <stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> # v4.6+
---
 drivers/infiniband/sw/rdmavt/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/infiniband/sw/rdmavt/Kconfig b/drivers/infiniband/sw/rdmavt/Kconfig
index 11aa6a3..1da8d01 100644
--- a/drivers/infiniband/sw/rdmavt/Kconfig
+++ b/drivers/infiniband/sw/rdmavt/Kconfig
@@ -1,6 +1,5 @@
 config INFINIBAND_RDMAVT
 	tristate "RDMA verbs transport library"
 	depends on 64BIT
-	default m
 	---help---
 	This is a common software verbs provider for RDMA networks.
-- 
2.9.0

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] IB/rdmavt: Disable by default

[Dalessandro, Dennis]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1342 bytes --]

On Tue, 2016-07-19 at 10:03 -0700, Bart Van Assche wrote:
> There is a strict policy in the Linux kernel that new drivers must be
> disabled by default. Hence leave out the "default m" line from
> Kconfig.
> 
> Fixes: 0194621b2253 ("IB/rdmavt: Create module framework and handle
> driver registration")
> Signed-off-by: Bart Van Assche <bart.vanassche@sandisk.com>
> Cc: Jubin John <jubin.john@intel.com>
> Cc: Dennis Dalessandro <dennis.dalessandro@intel.com>
> Cc: Ira Weiny <ira.weiny@intel.com>
> Cc: Mike Marciniszyn <mike.marciniszyn@intel.com>
> Cc: <stable@vger.kernel.org> # v4.6+
> ---
>  drivers/infiniband/sw/rdmavt/Kconfig | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/infiniband/sw/rdmavt/Kconfig
> b/drivers/infiniband/sw/rdmavt/Kconfig
> index 11aa6a3..1da8d01 100644
> --- a/drivers/infiniband/sw/rdmavt/Kconfig
> +++ b/drivers/infiniband/sw/rdmavt/Kconfig
> @@ -1,6 +1,5 @@
>  config INFINIBAND_RDMAVT
>  	tristate "RDMA verbs transport library"
>  	depends on 64BIT
> -	default m
>  	---help---
>  	This is a common software verbs provider for RDMA networks.

Thanks.

Acked-by: Dennis Dalessandro <dennis.dalessandro@intel.com>

N‹§²æìr¸›yúèšØb²X¬¶Ç§vØ^–)Þº{.nÇ+‰·¥Š{±­ÙšŠ{ayº\x1dʇڙë,j\a­¢f£¢·hš‹»öì\x17/oSc¾™Ú³9˜uÀ¦æå‰È&jw¨®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿïêäz¹Þ–Šàþf£¢·hšˆ§~ˆmš

[PATCH 2/2] IB/hfi1: Disable by default

[Bart Van Assche]

There is a strict policy in the Linux kernel that new drivers must be
disabled by default. Hence leave out the "default m" line from Kconfig.

Fixes: f48ad614c100 ("IB/hfi1: Move driver out of staging")
Signed-off-by: Bart Van Assche <bart.vanassche-XdAiOPVOjttBDgjK7y7TUQ@public.gmane.org>
Cc: Jubin John <jubin.john-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: Dennis Dalessandro <dennis.dalessandro-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: Ira Weiny <ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: Mike Marciniszyn <mike.marciniszyn-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: <stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> # v4.7+
---
 drivers/infiniband/hw/hfi1/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/infiniband/hw/hfi1/Kconfig b/drivers/infiniband/hw/hfi1/Kconfig
index a925fb0..f846fd5 100644
--- a/drivers/infiniband/hw/hfi1/Kconfig
+++ b/drivers/infiniband/hw/hfi1/Kconfig
@@ -3,7 +3,6 @@ config INFINIBAND_HFI1
 	depends on X86_64 && INFINIBAND_RDMAVT
 	select MMU_NOTIFIER
 	select CRC32
-	default m
 	---help---
 	This is a low-level driver for Intel OPA Gen1 adapter.
 config HFI1_DEBUG_SDMA_ORDER
-- 
2.9.0

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] IB/hfi1: Disable by default

[Dalessandro, Dennis]

On Tue, 2016-07-19 at 10:03 -0700, Bart Van Assche wrote:
> There is a strict policy in the Linux kernel that new drivers must be
> disabled by default. Hence leave out the "default m" line from
> Kconfig.
> 
> Fixes: f48ad614c100 ("IB/hfi1: Move driver out of staging")
> Signed-off-by: Bart Van Assche <bart.vanassche@sandisk.com>
> Cc: Jubin John <jubin.john@intel.com>
> Cc: Dennis Dalessandro <dennis.dalessandro@intel.com>
> Cc: Ira Weiny <ira.weiny@intel.com>
> Cc: Mike Marciniszyn <mike.marciniszyn@intel.com>
> Cc: <stable@vger.kernel.org> # v4.7+
> ---
>  drivers/infiniband/hw/hfi1/Kconfig | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/infiniband/hw/hfi1/Kconfig
> b/drivers/infiniband/hw/hfi1/Kconfig
> index a925fb0..f846fd5 100644
> --- a/drivers/infiniband/hw/hfi1/Kconfig
> +++ b/drivers/infiniband/hw/hfi1/Kconfig
> @@ -3,7 +3,6 @@ config INFINIBAND_HFI1
>  	depends on X86_64 && INFINIBAND_RDMAVT
>  	select MMU_NOTIFIER
>  	select CRC32
> -	default m
>  	---help---
>  	This is a low-level driver for Intel OPA Gen1 adapter.
>  config HFI1_DEBUG_SDMA_ORDER

Thanks.

Acked-by: Dennis Dalessandro <dennis.dalessandro@intel.com>

Re: [PATCH 0/2] Disable rdmavt and hfi1 drivers by default

[Doug Ledford]

[-- Attachment #1: Type: text/plain, Size: 606 bytes --]

On Tue, 2016-07-19 at 10:02 -0700, Bart Van Assche wrote:
> Hi Doug,
> 
> Since there is a strict policy in the Linux kernel that new drivers
> must 
> be disabled by default, hence this series with two small patches
> that 
> ensure that e.g. "make oldconfig" neither enables the rdmavt nor the 
> hfi1 driver. The patches in this series are:
> 
> 0001-IB-rdmavt-Disable-by-default.patch
> 0002-IB-hfi1-Disable-by-default.patch
> 
> Thanks,
> 
> Bart.

Series applied, thanks.

-- 
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
              GPG KeyID: 0E572FDD

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]