Hi,
On Fri, Jan 29, 2021 at 2:54 AM Grygorii Strashko
wrote:
> I've being thinking - shouldn't be implemented as new api, like
> port_pr_xx(port, ...)?
> (similar to kernel dev_xx(dev)).
The problem is:
1) There are 8 basic log level macros, and then another 8 rate-limited
macros. If I
On 28/01/2021 10:30, Cliff Spradlin via Linuxptp-devel wrote:
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
Add a second UDS port to allow untrusted applications to monitor ptp4l.
On this "read-only" UDS port disable non-GET actions and forwarding.
The path can be configured with the uds_ro_address option (default is
/var/run/ptp4lro).
Forwarding is disabled to limit the access to the local ptp4l
This prevents conflicts on the new UDS-RO port.
Signed-off-by: Miroslav Lichvar
---
timemaster.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/timemaster.c b/timemaster.c
index fb27d72..616a99a 100644
--- a/timemaster.c
+++ b/timemaster.c
@@ -712,7 +712,7 @@
Drop non-management messages on the UDS port early in the processing to
prevent them from changing the port or clock state.
Signed-off-by: Miroslav Lichvar
---
port.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/port.c b/port.c
index 87f2c06..f491252 100644
--- a/port.c
+++ b/port.c
In port_open(), don't assume that UDS ports always have to have a zero
number. Check the transport directly to make the code cleaner. While at
it, switch all checks for the UDS in the port code to use a helper
function.
Signed-off-by: Miroslav Lichvar
---
port.c | 40
v2:
- renamed uds_address2 option to uds_ro_address
- added a helper function for UDS check in port.c
This patchset adds a new UDS port to be used by untrusted applications
for monitoring purposes.
The first four patches are cleanup and preparation.
The fifth patch is the main change. As this
In preparation for a new read-only UDS port, rename variables of the
current UDS port to make it clear it is read-write, as opposed to
read-only.
Signed-off-by: Miroslav Lichvar
---
clock.c | 49 +
1 file changed, 25 insertions(+), 24 deletions(-)
No COMMAND actions are currently supported, but check the port early in
clock_manage() before reaching port_manage().
Signed-off-by: Miroslav Lichvar
---
clock.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/clock.c b/clock.c
index 08c61eb..aff9589 100644
--- a/clock.c
+++ b/clock.c
Adds a string to the port struct that can easily identify a port.
This is intended to be used in log messages that refer to a port.
Signed-off-by: Cliff Spradlin
Change-Id: Ic45d61c05b0aa970c9caf9e17528693dc0090c31
---
port.c | 23 ++-
port.h | 7 +++
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
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
Signed-off-by: Cliff Spradlin
Change-Id:
12 matches
Mail list logo
