]> git.dujemihanovic.xyz Git - linux.git/commit
net: bridge: switchdev: Improve error message for port_obj_add/del functions
authorOleksij Rempel <o.rempel@pengutronix.de>
Mon, 6 May 2024 10:32:05 +0000 (12:32 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 8 May 2024 11:19:12 +0000 (12:19 +0100)
commitb7ffab29a8e4705e7b9f05293669dc8a30272bc6
treea5a599d9441d7246b64d74b3c0f72cce15194c72
parentdb3efdcf70c752e8a8deb16071d8e693c3ef8746
net: bridge: switchdev: Improve error message for port_obj_add/del functions

Enhance the error reporting mechanism in the switchdev framework to
provide more informative and user-friendly error messages.

Following feedback from users struggling to understand the implications
of error messages like "failed (err=-28) to add object (id=2)", this
update aims to clarify what operation failed and how this might impact
the system or network.

With this change, error messages now include a description of the failed
operation, the specific object involved, and a brief explanation of the
potential impact on the system. This approach helps administrators and
developers better understand the context and severity of errors,
facilitating quicker and more effective troubleshooting.

Example of the improved logging:

[   70.516446] ksz-switch spi0.0 uplink: Failed to add Port Multicast
               Database entry (object id=2) with error: -ENOSPC (-28).
[   70.516446] Failure in updating the port's Multicast Database could
               lead to multicast forwarding issues.
[   70.516446] Current HW/SW setup lacks sufficient resources.

This comprehensive update includes handling for a range of switchdev
object IDs, ensuring that most operations within the switchdev framework
benefit from clearer error reporting.

Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/switchdev/switchdev.c