This patch improves the readability of many log messages.

Previously, only the logical port number was emitted for most
port-related log messages. Now, the interface name is included.

old: port 12: assuming the grand master role
new: port 12 (eth8): assuming the grand master role

This changes differs from the first version by introducing a cached string
port identifier prepended to all port-related log messages. Other options were
considered (see the review thread for the v1 version), but this one seemed the
most flexible and fitting.

I believe this should also cover the bonded interface failover scenario that
Richard had mentioned, because the interface name for a port never changes.

Cliff Spradlin (2):
  port: Cache display name for logs.
  Improve port-related log messages.

 clock.c           |  22 +++---
 e2e_tc.c          |  10 +--
 p2p_tc.c          |  10 +--
 port.c            | 181 +++++++++++++++++++++++++---------------------
 port.h            |   7 ++
 port_private.h    |   1 +
 port_signaling.c  |   2 +-
 tc.c              |  24 +++---
 unicast_client.c  |  42 +++++------
 unicast_service.c |   2 +-
 10 files changed, 161 insertions(+), 140 deletions(-)

-- 
2.30.0.280.ga3ce27912f-goog



_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to