netconsole: fix appending sysdata when sysdata_fields == SYSDATA_RELEASE
authorGustavo Luiz Duarte <gustavold@gmail.com>
Mon, 9 Jun 2025 18:24:20 +0000 (11:24 -0700)
committerJakub Kicinski <kuba@kernel.org>
Tue, 10 Jun 2025 22:52:08 +0000 (15:52 -0700)
commitc85bf1975108d2e2431c11d1cb7e95aca587dfbe
tree1d3924987cc100771d421c4041deebeb07b3dca3
parent2660a544fdc0940bba15f70508a46cf9a6491230
netconsole: fix appending sysdata when sysdata_fields == SYSDATA_RELEASE

Before appending sysdata, prepare_extradata() checks if any feature is
enabled in sysdata_fields (and exits early if none is enabled).

When SYSDATA_RELEASE was introduced, we missed adding it to the list of
features being checked against sysdata_fields in prepare_extradata().
The result was that, if only SYSDATA_RELEASE is enabled in
sysdata_fields, we incorreclty exit early and fail to append the
release.

Instead of checking specific bits in sysdata_fields, check if
sysdata_fields has ALL bit zeroed and exit early if true. This fixes
case when only SYSDATA_RELEASE enabled and makes the code more general /
less error prone in future feature implementation.

Signed-off-by: Gustavo Luiz Duarte <gustavold@gmail.com>
Reviewed-by: Breno Leitao <leitao@debian.org>
Fixes: cfcc9239e78a ("netconsole: append release to sysdata")
Link: https://patch.msgid.link/20250609-netconsole-fix-v1-1-17543611ae31@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/netconsole.c