Commit 79d89fab authored by Gal Pressman's avatar Gal Pressman Committed by Jakub Kicinski
Browse files

sfc: Remove newline at the end of a netlink error message 



Netlink error messages should not have a newline at the end of the
string.

Reviewed-by: default avatarTariq Toukan <tariqt@nvidia.com>
Signed-off-by: default avatarGal Pressman <gal@nvidia.com>
Reviewed-by: default avatarMichal Swiatkowski <michal.swiatkowski@linux.intel.com>
Link: https://patch.msgid.link/20250226093904.6632-4-gal@nvidia.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 3a2295ff

drivers/net/ethernet/sfc/mae.c

+1 −1
Original line number Diff line number Diff line
@@ -755,7 +755,7 @@ int efx_mae_match_check_caps_lhs(struct efx_nic *efx, 
	rc = efx_mae_match_check_cap_typ(supported_fields[MAE_FIELD_INGRESS_PORT],

					 ingress_port_mask_type);

	if (rc) {

		NL_SET_ERR_MSG_FMT_MOD(extack, "No support for %s mask in field %s\n",

		NL_SET_ERR_MSG_FMT_MOD(extack, "No support for %s mask in field %s",

				       mask_type_name(ingress_port_mask_type),

				       "ingress_port");

		return rc;

drivers/net/ethernet/sfc/tc.c

+3 −3
Original line number Diff line number Diff line
@@ -1043,7 +1043,7 @@ static int efx_tc_flower_handle_lhs_actions(struct efx_nic *efx, 
				return -EOPNOTSUPP;

			}

			if (fa->ct.action) {

				NL_SET_ERR_MSG_FMT_MOD(extack, "Unhandled ct.action %u for LHS rule\n",

				NL_SET_ERR_MSG_FMT_MOD(extack, "Unhandled ct.action %u for LHS rule",

						       fa->ct.action);

				return -EOPNOTSUPP;

			}
@@ -1056,7 +1056,7 @@ static int efx_tc_flower_handle_lhs_actions(struct efx_nic *efx, 
			act->zone = ct_zone;

			break;

		default:

			NL_SET_ERR_MSG_FMT_MOD(extack, "Unhandled action %u for LHS rule\n",

			NL_SET_ERR_MSG_FMT_MOD(extack, "Unhandled action %u for LHS rule",

					       fa->id);

			return -EOPNOTSUPP;

		}
@@ -1581,7 +1581,7 @@ static int efx_tc_flower_replace_foreign_lhs(struct efx_nic *efx, 


	type = efx_tc_indr_netdev_type(net_dev);

	if (type == EFX_ENCAP_TYPE_NONE) {

		NL_SET_ERR_MSG_MOD(extack, "Egress encap match on unsupported tunnel device\n");

		NL_SET_ERR_MSG_MOD(extack, "Egress encap match on unsupported tunnel device");

		return -EOPNOTSUPP;

	}