[bionic/azure][PATCH v2] UBUNTU: [Config] azure: Ensure CONFIG_MLX5_INFINIBAND=y

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[bionic/azure][PATCH v2] UBUNTU: [Config] azure: Ensure CONFIG_MLX5_INFINIBAND=y

Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1785822

Recent commit "infiniband: mlx5: fix build errors when
INFINIBAND_USER_ACCESS=m" changed the Kconfig dependencies for
CONFIG_MLX5_INFINIBAND, which caused it to revert to "m".

Force CONFIG_INFINIBAND_USER_ACCESS=y, to keep
CONFIG_MLX5_INFINIBAND=y, as requested by Bug #1759656.

Also enforce CONFIG_MLX4_INFINIBAND and CONFIG_MLX5_INFINIBAND to "y"
in the annotations file.

Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
---
 debian.azure/config/annotations          | 7 +++++--
 debian.azure/config/config.common.ubuntu | 2 +-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/debian.azure/config/annotations b/debian.azure/config/annotations
index 7e353e930dac..b206dc21ca74 100644
--- a/debian.azure/config/annotations
+++ b/debian.azure/config/annotations
@@ -3070,8 +3070,8 @@ CONFIG_INFINIBAND_CXGB3                         policy<{'amd64': 'm', 'arm64': '
 CONFIG_INFINIBAND_CXGB3_DEBUG                   policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
 CONFIG_INFINIBAND_CXGB4                         policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
 CONFIG_INFINIBAND_I40IW                         policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
-CONFIG_MLX4_INFINIBAND                          policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'm'}>
-CONFIG_MLX5_INFINIBAND                          policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'm'}>
+CONFIG_MLX4_INFINIBAND                          policy<{'amd64': 'y'}>
+CONFIG_MLX5_INFINIBAND                          policy<{'amd64': 'y'}>
 CONFIG_INFINIBAND_NES                           policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'n'}>
 CONFIG_INFINIBAND_NES_DEBUG                     policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
 CONFIG_INFINIBAND_OCRDMA                        policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'n'}>
@@ -3096,6 +3096,9 @@ CONFIG_SDMA_VERBOSITY                           policy<{'amd64': 'n'}>
 CONFIG_INFINIBAND_QEDR                          policy<{'amd64': 'm', 'arm64': 'm', 'ppc64el': 'm'}>
 CONFIG_INFINIBAND_BNXT_RE                       policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'n'}>
 
+CONFIG_MLX4_INFINIBAND                          mark<ENFORCED> note<LP:1759656>
+CONFIG_MLX5_INFINIBAND                          mark<ENFORCED> note<LP:1759656>
+
 # Menu: Device Drivers >> Input device support
 
 # Menu: Device Drivers >> Input device support >> Generic input layer (needed for keyboard, mouse, ...)
diff --git a/debian.azure/config/config.common.ubuntu b/debian.azure/config/config.common.ubuntu
index 8ea51473f3dd..a5230477f7a5 100644
--- a/debian.azure/config/config.common.ubuntu
+++ b/debian.azure/config/config.common.ubuntu
@@ -1971,7 +1971,7 @@ CONFIG_INFINIBAND_OPA_VNIC=m
 # CONFIG_INFINIBAND_RDMAVT is not set
 CONFIG_INFINIBAND_SRP=m
 # CONFIG_INFINIBAND_SRPT is not set
-CONFIG_INFINIBAND_USER_ACCESS=m
+CONFIG_INFINIBAND_USER_ACCESS=y
 # CONFIG_INFINIBAND_USER_MAD is not set
 CONFIG_INFINIBAND_USER_MEM=y
 CONFIG_INFINIBAND_USNIC=m
--
2.17.1


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK: [bionic/azure][PATCH v2] UBUNTU: [Config] azure: Ensure CONFIG_MLX5_INFINIBAND=y

Stefan Bader-2
On 07.08.2018 16:12, Marcelo Henrique Cerri wrote:

> BugLink: http://bugs.launchpad.net/bugs/1785822
>
> Recent commit "infiniband: mlx5: fix build errors when
> INFINIBAND_USER_ACCESS=m" changed the Kconfig dependencies for
> CONFIG_MLX5_INFINIBAND, which caused it to revert to "m".
>
> Force CONFIG_INFINIBAND_USER_ACCESS=y, to keep
> CONFIG_MLX5_INFINIBAND=y, as requested by Bug #1759656.
>
> Also enforce CONFIG_MLX4_INFINIBAND and CONFIG_MLX5_INFINIBAND to "y"
> in the annotations file.
>
> Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
Acked-by: Stefan Bader <[hidden email]>

> ---
>  debian.azure/config/annotations          | 7 +++++--
>  debian.azure/config/config.common.ubuntu | 2 +-
>  2 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/debian.azure/config/annotations b/debian.azure/config/annotations
> index 7e353e930dac..b206dc21ca74 100644
> --- a/debian.azure/config/annotations
> +++ b/debian.azure/config/annotations
> @@ -3070,8 +3070,8 @@ CONFIG_INFINIBAND_CXGB3                         policy<{'amd64': 'm', 'arm64': '
>  CONFIG_INFINIBAND_CXGB3_DEBUG                   policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
>  CONFIG_INFINIBAND_CXGB4                         policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
>  CONFIG_INFINIBAND_I40IW                         policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
> -CONFIG_MLX4_INFINIBAND                          policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'm'}>
> -CONFIG_MLX5_INFINIBAND                          policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'm'}>
> +CONFIG_MLX4_INFINIBAND                          policy<{'amd64': 'y'}>
> +CONFIG_MLX5_INFINIBAND                          policy<{'amd64': 'y'}>
>  CONFIG_INFINIBAND_NES                           policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'n'}>
>  CONFIG_INFINIBAND_NES_DEBUG                     policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
>  CONFIG_INFINIBAND_OCRDMA                        policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'n'}>
> @@ -3096,6 +3096,9 @@ CONFIG_SDMA_VERBOSITY                           policy<{'amd64': 'n'}>
>  CONFIG_INFINIBAND_QEDR                          policy<{'amd64': 'm', 'arm64': 'm', 'ppc64el': 'm'}>
>  CONFIG_INFINIBAND_BNXT_RE                       policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'n'}>
>  
> +CONFIG_MLX4_INFINIBAND                          mark<ENFORCED> note<LP:1759656>
> +CONFIG_MLX5_INFINIBAND                          mark<ENFORCED> note<LP:1759656>
> +
>  # Menu: Device Drivers >> Input device support
>  
>  # Menu: Device Drivers >> Input device support >> Generic input layer (needed for keyboard, mouse, ...)
> diff --git a/debian.azure/config/config.common.ubuntu b/debian.azure/config/config.common.ubuntu
> index 8ea51473f3dd..a5230477f7a5 100644
> --- a/debian.azure/config/config.common.ubuntu
> +++ b/debian.azure/config/config.common.ubuntu
> @@ -1971,7 +1971,7 @@ CONFIG_INFINIBAND_OPA_VNIC=m
>  # CONFIG_INFINIBAND_RDMAVT is not set
>  CONFIG_INFINIBAND_SRP=m
>  # CONFIG_INFINIBAND_SRPT is not set
> -CONFIG_INFINIBAND_USER_ACCESS=m
> +CONFIG_INFINIBAND_USER_ACCESS=y
>  # CONFIG_INFINIBAND_USER_MAD is not set
>  CONFIG_INFINIBAND_USER_MEM=y
>  CONFIG_INFINIBAND_USNIC=m
>


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

ACK: [bionic/azure][PATCH v2] UBUNTU: [Config] azure: Ensure CONFIG_MLX5_INFINIBAND=y

Thadeu Lima de Souza Cascardo-3
In reply to this post by Marcelo Henrique Cerri
Acked-by: Thadeu Lima de Souza Cascardo <[hidden email]>

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[Acked/cmt] [bionic/azure][PATCH v2] UBUNTU: [Config] azure: Ensure CONFIG_MLX5_INFINIBAND=y

Andy Whitcroft-3
In reply to this post by Marcelo Henrique Cerri
On Tue, Aug 07, 2018 at 11:12:28AM -0300, Marcelo Henrique Cerri wrote:

> BugLink: http://bugs.launchpad.net/bugs/1785822
>
> Recent commit "infiniband: mlx5: fix build errors when
> INFINIBAND_USER_ACCESS=m" changed the Kconfig dependencies for
> CONFIG_MLX5_INFINIBAND, which caused it to revert to "m".
>
> Force CONFIG_INFINIBAND_USER_ACCESS=y, to keep
> CONFIG_MLX5_INFINIBAND=y, as requested by Bug #1759656.
>
> Also enforce CONFIG_MLX4_INFINIBAND and CONFIG_MLX5_INFINIBAND to "y"
> in the annotations file.
>
> Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
> ---
>  debian.azure/config/annotations          | 7 +++++--
>  debian.azure/config/config.common.ubuntu | 2 +-
>  2 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/debian.azure/config/annotations b/debian.azure/config/annotations
> index 7e353e930dac..b206dc21ca74 100644
> --- a/debian.azure/config/annotations
> +++ b/debian.azure/config/annotations
> @@ -3070,8 +3070,8 @@ CONFIG_INFINIBAND_CXGB3                         policy<{'amd64': 'm', 'arm64': '
>  CONFIG_INFINIBAND_CXGB3_DEBUG                   policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
>  CONFIG_INFINIBAND_CXGB4                         policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
>  CONFIG_INFINIBAND_I40IW                         policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
> -CONFIG_MLX4_INFINIBAND                          policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'm'}>
> -CONFIG_MLX5_INFINIBAND                          policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'm'}>
> +CONFIG_MLX4_INFINIBAND                          policy<{'amd64': 'y'}>
> +CONFIG_MLX5_INFINIBAND                          policy<{'amd64': 'y'}>
>  CONFIG_INFINIBAND_NES                           policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'n'}>
>  CONFIG_INFINIBAND_NES_DEBUG                     policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
>  CONFIG_INFINIBAND_OCRDMA                        policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'n'}>
> @@ -3096,6 +3096,9 @@ CONFIG_SDMA_VERBOSITY                           policy<{'amd64': 'n'}>
>  CONFIG_INFINIBAND_QEDR                          policy<{'amd64': 'm', 'arm64': 'm', 'ppc64el': 'm'}>
>  CONFIG_INFINIBAND_BNXT_RE                       policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm', 's390x': 'n'}>
>  

IIRC this blank line should actually be a comment line with no contents.
The sorting by menu update expects this form though it has not effect on
its effectivness.

> +CONFIG_MLX4_INFINIBAND                          mark<ENFORCED> note<LP:1759656>
> +CONFIG_MLX5_INFINIBAND                          mark<ENFORCED> note<LP:1759656>
> +
>  # Menu: Device Drivers >> Input device support
>  
>  # Menu: Device Drivers >> Input device support >> Generic input layer (needed for keyboard, mouse, ...)
> diff --git a/debian.azure/config/config.common.ubuntu b/debian.azure/config/config.common.ubuntu
> index 8ea51473f3dd..a5230477f7a5 100644
> --- a/debian.azure/config/config.common.ubuntu
> +++ b/debian.azure/config/config.common.ubuntu
> @@ -1971,7 +1971,7 @@ CONFIG_INFINIBAND_OPA_VNIC=m
>  # CONFIG_INFINIBAND_RDMAVT is not set
>  CONFIG_INFINIBAND_SRP=m
>  # CONFIG_INFINIBAND_SRPT is not set
> -CONFIG_INFINIBAND_USER_ACCESS=m
> +CONFIG_INFINIBAND_USER_ACCESS=y
>  # CONFIG_INFINIBAND_USER_MAD is not set
>  CONFIG_INFINIBAND_USER_MEM=y
>  CONFIG_INFINIBAND_USNIC=m

Acked-by: Andy Whitcroft <[hidden email]>

-apw

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

APPLIED: [bionic/azure][PATCH v2] UBUNTU: [Config] azure: Ensure CONFIG_MLX5_INFINIBAND=y

Marcelo Henrique Cerri
In reply to this post by Marcelo Henrique Cerri
--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (499 bytes) Download Attachment