From: Antoine Tenart <aten...@redhat.com>

[redhat] Flip some diag modules from m to y

Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1948340
Upstream Status: RHEL only
Tested: Instructions in the first comment in bz1948340.

Diag modules are required when investigating sockets with the `ss` tool,
and are loaded on-demand. This is also what happens when using the
`sosreport` tool. Recent discussions pointed out it was not always
possible to load extra modules on a production system, for various
reasons, and the `sosreport` tool was updated to avoid modules to be
loaded by default. Links to the discussions are available in the bz.

The issue is with this policy the `ss` command is now skipped, except if
the administrator explicitly allows it to do so or if diag modules are
already loaded. This results in a loss of valuable information when
generating an `sosreport`, impacting the following investigation.

This patch enables a set of selected DIAG modules to be built-in, so
that `ss` and `sosreport` can be used without loading additional
modules (only diag modules not depending on another loadable module were
considered):

- CONFIG_INET_DIAG
- CONFIG_INET_RAW_DIAG
- CONFIG_INET_TCP_DIAG
- CONFIG_INET_UDP_DIAG
- CONFIG_INET_MPTCP_DIAG
- CONFIG_NETLINK_DIAG
- CONFIG_PACKET_DIAG
- CONFIG_UNIX_DIAG

The generated configuration do not enable any additional non-listed
options with this change. The kernel image increase in size is very
minimal.

Signed-off-by: Antoine Tenart <aten...@redhat.com>

diff a/redhat/configs/common/generic/CONFIG_INET_DIAG 
b/redhat/configs/common/generic/CONFIG_INET_DIAG
--- a/redhat/configs/common/generic/CONFIG_INET_DIAG
+++ b/redhat/configs/common/generic/CONFIG_INET_DIAG
@@ -1 +1 @@
-CONFIG_INET_DIAG=m
+CONFIG_INET_DIAG=y
diff a/redhat/configs/common/generic/CONFIG_INET_MPTCP_DIAG 
b/redhat/configs/common/generic/CONFIG_INET_MPTCP_DIAG
--- a/redhat/configs/common/generic/CONFIG_INET_MPTCP_DIAG
+++ b/redhat/configs/common/generic/CONFIG_INET_MPTCP_DIAG
@@ -1 +1 @@
-CONFIG_INET_MPTCP_DIAG=m
+CONFIG_INET_MPTCP_DIAG=y
diff a/redhat/configs/common/generic/CONFIG_INET_RAW_DIAG 
b/redhat/configs/common/generic/CONFIG_INET_RAW_DIAG
--- a/redhat/configs/common/generic/CONFIG_INET_RAW_DIAG
+++ b/redhat/configs/common/generic/CONFIG_INET_RAW_DIAG
@@ -1 +1 @@
-CONFIG_INET_RAW_DIAG=m
+CONFIG_INET_RAW_DIAG=y
diff a/redhat/configs/common/generic/CONFIG_INET_TCP_DIAG 
b/redhat/configs/common/generic/CONFIG_INET_TCP_DIAG
--- /dev/null
+++ b/redhat/configs/common/generic/CONFIG_INET_TCP_DIAG
@@ -0,0 +1 @@
+CONFIG_INET_TCP_DIAG=y
diff a/redhat/configs/common/generic/CONFIG_INET_UDP_DIAG 
b/redhat/configs/common/generic/CONFIG_INET_UDP_DIAG
--- a/redhat/configs/common/generic/CONFIG_INET_UDP_DIAG
+++ b/redhat/configs/common/generic/CONFIG_INET_UDP_DIAG
@@ -1 +1 @@
-CONFIG_INET_UDP_DIAG=m
+CONFIG_INET_UDP_DIAG=y
diff a/redhat/configs/common/generic/CONFIG_NETLINK_DIAG 
b/redhat/configs/common/generic/CONFIG_NETLINK_DIAG
--- a/redhat/configs/common/generic/CONFIG_NETLINK_DIAG
+++ b/redhat/configs/common/generic/CONFIG_NETLINK_DIAG
@@ -1 +1 @@
-CONFIG_NETLINK_DIAG=m
+CONFIG_NETLINK_DIAG=y
diff a/redhat/configs/common/generic/CONFIG_PACKET_DIAG 
b/redhat/configs/common/generic/CONFIG_PACKET_DIAG
--- a/redhat/configs/common/generic/CONFIG_PACKET_DIAG
+++ b/redhat/configs/common/generic/CONFIG_PACKET_DIAG
@@ -1 +1 @@
-CONFIG_PACKET_DIAG=m
+CONFIG_PACKET_DIAG=y
diff a/redhat/configs/common/generic/CONFIG_UNIX_DIAG 
b/redhat/configs/common/generic/CONFIG_UNIX_DIAG
--- a/redhat/configs/common/generic/CONFIG_UNIX_DIAG
+++ b/redhat/configs/common/generic/CONFIG_UNIX_DIAG
@@ -1 +1 @@
-CONFIG_UNIX_DIAG=m
+CONFIG_UNIX_DIAG=y

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1151
_______________________________________________
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to