Re: [Linuxptp-devel] [PATCH 3/4] Add string and pointer array utility functions.

2014-10-03 Thread Richard Cochran
On Thu, Oct 02, 2014 at 10:38:33AM +0200, Miroslav Lichvar wrote: Add some functions to work with strings and arrays of pointers that will be useful later. Signed-off-by: Miroslav Lichvar mlich...@redhat.com --- incdefs.sh | 3 ++ util.c | 95

[Linuxptp-devel] [PATCHv3 1/4] Don't print messages in signal handler.

2014-10-03 Thread Miroslav Lichvar
Only reentrant functions should be called here. Signed-off-by: Miroslav Lichvar mlich...@redhat.com --- util.c | 1 - 1 file changed, 1 deletion(-) diff --git a/util.c b/util.c index ae66bb1..cb428b1 100644 --- a/util.c +++ b/util.c @@ -318,7 +318,6 @@ int get_arg_val_d(int op, const char

[Linuxptp-devel] [PATCHv3 4/4] Add timemaster.

2014-10-03 Thread Miroslav Lichvar
timemaster is a program that uses ptp4l and phc2sys in combination with chronyd or ntpd to synchronize the system clock to NTP and PTP time sources. The PTP time is provided by phc2sys and ptp4l via SHM reference clocks to chronyd/ntpd, which can compare all time sources and use the best sources

[Linuxptp-devel] [PATCHv3 0/4] timemaster

2014-10-03 Thread Miroslav Lichvar
Changes since v2: - fix warnings reported by Clang static analyzer - don't define _GNU_SOURCE twice Changes since v1: - add support for posix_spawn() to compile on systems missing fork() - define _GNU_SOURCE for vasprintf() - resolve conflict with config.h - minor updates in man page timemaster

[Linuxptp-devel] [PATCHv3 2/4] Don't include config.h in util.h

2014-10-03 Thread Miroslav Lichvar
The config module is used by ptp4l only, but util is shared with other programs. Signed-off-by: Miroslav Lichvar mlich...@redhat.com --- config.h | 8 phc_ctl.c | 1 + util.h| 12 +++- 3 files changed, 12 insertions(+), 9 deletions(-) diff --git a/config.h b/config.h

Re: [Linuxptp-devel] [PATCHv3 0/4] timemaster

2014-10-03 Thread Richard Cochran
On Fri, Oct 03, 2014 at 02:13:48PM +0200, Miroslav Lichvar wrote: Changes since v2: - fix warnings reported by Clang static analyzer - don't define _GNU_SOURCE twice Changes since v1: - add support for posix_spawn() to compile on systems missing fork() - define _GNU_SOURCE for vasprintf()