Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Makefile: Changes to make jclklib available from libptpmgmt Makefile #67

Open
wants to merge 93 commits into
base: main
Choose a base branch
from

Conversation

AzuraTarmiziIntel
Copy link
Contributor

No description provided.

Noor Azura Ahmad Tarmizi and others added 30 commits May 16, 2024 11:51
Add upper and lower limits for threshold

Fix name (SPC->'.') of client side message queue and add unlinking in finalize to cleanup queue
This document will provide basic architecture and simplified code flow
for connect/subscribe/notification messages.

This is a live document.

Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com>
Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com>
Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com>
Add workflow to build the code using coverity tool and upload the result to
https://scan.coverity.com/builds?project=libptpmgmt_iaclocklib.

The workflow will run automatically when new changes are pushed to main
branch.

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
Add workflow for security vulnerabilities scan and upload the result to
https://securityscorecards.dev/viewer/?uri=github.com/intel-staging/libptpmgmt_iaclocklib

The workflow will run automatically when new changes are pushed to main
branch.

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
1. In GitHub action:
    - set permissions as read
    - add Harden Runner
    - use SHA commit id instead of version
2. Add codeql scanning

Signed-off-by: StepSecurity Bot <bot@stepsecurity.io>
Replace autobuild command with proper build steps

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
Try to point to erezgeva container as we dint plan to
create our own image.

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
The connect message now will return the client session ID to the client.
It will also have the timeout of 5 sec by default to receive the proxy
reply.

This could be change in the future.

Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com>
The virus scan will be run for every PR agaist master branch
and whenever master branch got changes.

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
Add two new subscribe events:
  1. NOTIFY_PORT_STATE_NP
  2. NOTIFY_ALL

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
Signed-off-by: Lai Peter Jun Ann <peter.jun.ann.lai@intel.com>
Signed-off-by: Lai Peter Jun Ann <peter.jun.ann.lai@intel.com>
This document is created for clock manager installation and test guidance reference.
Can be used by user and validation alike.

This is a work in progress.
Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com>
Execute the script with "sh ./run_ptp4l.sh" and start ptp4l program.

Signed-off-by: Lai Peter Jun Ann <peter.jun.ann.lai@intel.com>
Added subscription message tx and rx pathway between the proxy and client.

Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com>
Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com>
When Proxy receive subscription message, send
the initial state of PTP back to Client.
Client will parse the data and print it out.

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
Signed-off-by: Lai Peter Jun Ann <peter.jun.ann.lai@intel.com>
…proxy sub msg

This is to ensure the whole message with the data :

Msgid, msgack, jcl_subscription, ptp_event struct are sent by the proxy and
received correctly by the Client.

Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com>
Signed-off-by: Lai Peter Jun Ann <peter.jun.ann.lai@intel.com>
Proxy will send notification message to client to report
the latest data it received from ptp4l.

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
This commit removes the less -R command from the default target in the
Makefile of client, common and proxy.

The less -R command was originally used to paginate the output of the make
command. However, this caused an issue where the build process would pause
until the user manually quit less by pressing "q".

This is not a typical or recommended use of less, especially in a Makefile.
less is designed for interactive use in a shell to view the contents of a
file or the output of a command page by page. In a Makefile, it can cause
problems if the Makefile is run in a non-interactive environment where
there's no user to scroll through the pages.

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
Will add back when we figure out why we need it.

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
Signed-off-by: Lai Peter Jun Ann <peter.jun.ann.lai@intel.com>
Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com>
yoongsiang2 and others added 2 commits May 21, 2024 13:20
If the value of ascapable is same as previous, end the event handler.
This is to avoid unneccesary messages in message queue.

Signed-off-by: Song Yoong Siang <yoong.siang.song@intel.com>
Signed-off-by: Noor Azura Ahmad Tarmizi <noor.azura.ahmad.tarmizi@intel.com>
@yoongsiang2 yoongsiang2 force-pushed the dev/noorazur/makefile_change_v1 branch from 4db0888 to 342010a Compare May 22, 2024 06:56
@yoongsiang2 yoongsiang2 force-pushed the main branch 3 times, most recently from 0d46f24 to c5388be Compare August 13, 2024 02:21
@yoongsiang2 yoongsiang2 force-pushed the main branch 3 times, most recently from 8f74ba2 to efef9e2 Compare August 26, 2024 06:48
@yoongsiang2 yoongsiang2 force-pushed the main branch 2 times, most recently from c55a424 to e71ee68 Compare August 30, 2024 07:09
@yoongsiang2 yoongsiang2 force-pushed the main branch 4 times, most recently from d99fc61 to 3f5915c Compare September 11, 2024 07:42
@yoongsiang2 yoongsiang2 force-pushed the main branch 4 times, most recently from 3727930 to c27072b Compare September 19, 2024 02:18
@yoongsiang2 yoongsiang2 force-pushed the main branch 2 times, most recently from ac9137d to f97db78 Compare September 30, 2024 05:53
@yoongsiang2 yoongsiang2 force-pushed the main branch 2 times, most recently from 5b23e36 to d5e6aaa Compare October 7, 2024 07:57
@yoongsiang2 yoongsiang2 force-pushed the main branch 5 times, most recently from c39e500 to 59c7506 Compare October 22, 2024 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants