Merge pull request #34 from heitbaum/patch-1

change setcap to AmbientCapabilities in README.md

change setcap to AmbientCapabilities in README.md

Merge pull request #32 from qerub/patch-1

Add missing rm command to README.md

Merge pull request #31 from klemensn/openbsd

Support OpenBSD

Add missing rm command to README.md

Hoist control socket handling, restrict runtime on OpenBSD

Use pledge(2) to prevent fork/exec, filesystem access and other
unused subsets of system calls, effectively leaving only shared
memory and networking capabilities at runtime.

(Those might be further reduced, but that warrants further analysis
 and most likely more code shuffling.)

Hoist PTP socket handling, drop privileges on OpenBSD

bind(2)ing ports below 1024 is the only privileged operation NQPTP does.

Move its code up in main() before shared memory handling such that root
privileges can be dropped immediately after it;  no currently supported
system does that, thus this should be a NOOP.

Do so on OpenBSD where shm_open(3) does not allow access to shared memory
objects by multiple UIDs, i.e. to communicate, shairport-sync and NQPTP
must create them and run as the very same user.

OpenBSD's official audio/shairport-sync user provides an rc.d(8) daemon
script that runs as `_shairport` user.

Build on OpenBSD

Recognise the system to configure, build and start.
More work is required to actually work with shairport-sync.
Tested on OpenBSD/amd64 7.4-current.

Update RELEASE_NOTES.md

Change Linux installation to use AmbientCapabilities from systemd. Fix FreeBSD compilation errors.

reorder the include files for FreeBSD

Update RELEASE_NOTES.md

Update RELEASE_NOTES.md

Improve some of the error messages. Remove the setcap command from Makefile.am, since we are now using an AmbientCapabilities setting in the systemd service file.

Update README.md

Update README.md

Merge pull request #26 from Hs-Yeah/development

Added AmbientCapabilities to nqptp.service.in

Added AmbientCapabilities to nqptp.service.in

Added AmbientCapabilities=CAP_NET_BIND_SERVICE
so that the systemd service can be used without the  capability set on the built nqptp binary.

Update README.md

Updated release notes

Update release notes, readme and configuration version number.

Update README.md

Update RELEASE_NOTES.md

Update RELEASE_NOTES.md

Update RELEASE_NOTES.md

Update RELEASE_NOTES.md

Move to smi version 10, read-only interface, no mutex, write and read each record twice to ensure it is not inconsistent when read.

Update README.md

Update README.md

Update README.md

Update README.md

1.2.4d0

Fix a bug whereby a maliciously-crafted packet received on the control port could crash the program.

Fix a crashing bug with an empty command string and check for missing or bad parameters a bit more carefully

Update README.md

Remove references to the `development` branch!

1.2.2d0

bump version number

Include lightweight tags and note if broken.

merge updates from 1.2 main

actually make it 1.2

version 1.2rc0

update Release Notes

slight change in format of output of the smi in the -V command.

slight change in format of output of the smi in the -V command.

Merge branch 'main' into development
Typo

Merge pull request #21 from tsbernar/patch-1

Update README.md

Documentation and version string updates.

Fix a FreeBSD compilation error.

Update README.md

small typo

Update RELEASE_NOTES.md

Merge branch 'development' of github.com:mikebrady/nqptp into development

Online updates merged.

Change open_sockets_at_port to be satisfied with just one port on IPv4 or IPv6. Fix misleading message.

Update RELEASE_NOTES.md

Update RELEASE_NOTES.md

Merge branch 'development' of github.com:mikebrady/nqptp into development

Merge other update.

Tune the weights of additions and reductions.

Fix a bug -- use the previous offset if a negative jitter for the first period. Fix a misleading comment.

Update README.md

Update README.md

Update RELEASE_NOTES.md

Add modifications suggested by https://github.com/mikebrady/nqptp/issues/14 to restrict UDP write access to 9000 to the localhost.

Merge pull request #17 from Weidav/main

typo: remove duplicate you

Update RELEASE_NOTES.md

Enhance an error message.

Update README.md

Tidy up a few comments.

Bump the SHM Interface number to note new commands.

Add version string to initial debug message
Work out when the clock can be relied upon not to go to sleep, i.e. is "active". This is when SPS is playing (B) or paused (P) and for brief periods after the clock is selected (T) and after play ends (E).
SPS signals Play/Pause/End/Timer Select/Timer Release with B/P/E/T messages -- B for "Begin". "T <ip>" means select the clock at that address. "T" means release the clock.
Reset clock smoothing when the clock goes "active" but not at any other time.
Clamp reductions in the offset calculated as a uint64_t remote-time-now_ns - local-time-now_ns -- and which it is assumed are due to delays in the network when the clock is "active" -- to 2.5 mS.
Turn off clock updates when the clock is not "active" -- i.e. when SPS is not paying or paused.
Remove some redundant features and code.

Add ability to receive status information from SPS B/E/P for Begin/End/Pause play. Assume clocks don't sleep between Begin and End and are valid for short times after T and after E. More cleaning up to do.

Allow clock to resent if the grandmaster is the same as the master and it stops for two or more samples. Always send a wakeup to the master at the start.

Small documentation update

Update README.md

typo: remove duplicate you

Merge from main.

Small changes to debug messages.

Update RELEASE_NOTES.md

Only try to start a silent clock if no follow_ups have _ever_ been seen since it became master.

Update RELEASE_NOTES.md

Squashed commit of the following:

commit 1c610279b21ac778ab96fd2fb01fa0bbc8f363ce
Author: Mike Brady <4265913+mikebrady@users.noreply.github.com>
Date:   Mon Sep 12 15:07:15 2022 +0100

    * Bump NQPTP_SHM_STRUCTURES_VERSION to 8.
    * Remove the ability to handle multiple instances of AirPlay-2-capable Shairport Sync on the same system -- it seems clear that clients can not use this facility.
    * Greatly simplify NQPTP by only monitoring PTP information coming from the client, ignoring all other PTP information.
    * Improve the accuracy of the clock by taking correction field information into account.
    * In addition to trying to restart a clock that is silent, also send a restart to a clock if the clock's grandmaster appears to have stopped.

    Squashed commit of the following:
        Add some experimental exploratory code.
        Ignore the INSTALL document. Let INSTALL always be autogenerated.
        Enhanced and corrected ptp definitions.
        Fix shm name to work with FreeBSD. Update the SHM version.
        Change order of includes to avoid a compilation error in FreeBSD
        Fix potential bugs in finding a clock ID in a FreeBSD device.
        Only try to restart a stopped clock if it's the clock itself that has stopped and not a different grandmaster. Allow fast changes for up to one second after the start of mastership.
        Make the code to restart a clock a separate function and call it if a clock gets stuck. Include the correction field in the calculation of the precise origin timestamp. If a new clock is out by more than -4 seconds, treat it as a discontinuity rather than noise.
        Add a net-to-host converter for 64-bit arithmetic

* Bump NQPTP_SHM_STRUCTURES_VERSION to 8.
* Remove the ability to handle multiple instances of AirPlay-2-capable Shairport Sync on the same system -- it seems clear that clients can not use this facility.
* Greatly simplify NQPTP by only monitoring PTP information coming from the client, ignoring all other PTP information.
* Improve the accuracy of the clock by taking correction field information into account.
* In addition to trying to restart a clock that is silent, also send a restart to a clock if the clock's grandmaster appears to have stopped.

Squashed commit of the following:
    Add some experimental exploratory code.
    Ignore the INSTALL document. Let INSTALL always be autogenerated.
    Enhanced and corrected ptp defiinitions.
    Fix shm name to work with FreeBSD. Update the SHM version.
    Change order of includes to avoid a compilation error in FreeBSD
    Fix potential bugs in finding a clock ID in a FreeBSD device.
    Only try to restart a stopped clock if it's the clock itself that has stopped and not a different grandmaster. Allow fast changes for up to one second after the start of mastership.

Update RELEASE_NOTES.md

Ignore configure~

Update RELEASE_NOTES.md

Small change to wording.

Small change to wording.

General clang format. Quieten a few debug messages.

Add a handle_sync to see if a correctionfiled is ever used. Modify handle_follow_up to (mostly) only work on a master clock. Wait for a minute to restart a stopped clock. Always invalidate the clock when an empty peer list is sent.

Update handle_sync definition, remove handle_delay_resp definition -- not implemented.

Only try to waken a clock if it's a master. Shorten Announce "ping" reversion time to 150 ms.

Quieten a few debug messages.

Update README.md

Update README.md

Update RELEASE_NOTES.md

Small diagnostic message fix.

Make the code to restart a clock a separate function and call it if a clock gets stuck. Include the correction field in the calculation of the precise origin timestamp. If a new clock is out by more than -4 seconds, treat it as a discontinuity rather than noise.

Add a net-to-host converter for 64-bit arithmetic

Update RELEASE_NOTES.md

Update RELEASE_NOTES.md

If the previous offset's grandmaster is different from the current one, discard all the smoothing stuff, including previous offset data, and work as if it was a completely new clock.

Add is a field to store the grandmaster the previous offset refers to. The idea is that if the new grandmaster is different from the previous one, start over.