On Mon, Jun 18, 2018 at 12:21:37PM +0200, Anders Selhammer wrote:
> Add of the first step of supporting SNMP communication to linuxptp.
> This patchset only adds the basics for SNMP. The support of published
> MIBs will be added later on.
> Support of cross compilation will also be included later on.

Finally getting around to this...
 
> Anders Selhammer (5):
>   pmc: Move new common funtionality into pmc_common
>   pdt: Added check if already defined

These first two patches are fine, and I'll merge them soon.

>   snmpd: Add snmp sub agent
>   snmpd: Added communication to ptp4l via the UDS port.
>   snmpd: Add snmpd_run_pmc for data collection from ptp4l program.

The code in snmpd.c looks fine to me.

The name 'snmpd' is a not a good choice.  The net-snmp project already
delivers a program of that name.  We are building an AgentX, are we
not?  So let's use a unique name that is also more descriptive.

How about 'ptpax' for PTP AgentX?

I've also looked at the build issues, and I have an idea how to
simplify the test for including snmp support and also support cross.
I'll post details as soon as we can agree on a name for the agentx
program.

Thanks,
Richard


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to