| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <aHgMdcOAWujOPOUC@lizhi-Precision-Tower-5810>
Date: Wed, 16 Jul 2025 16:32:53 -0400
From: Frank Li <Frank.li@....com>
To: Wei Fang <wei.fang@....com>
Cc: robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
richardcochran@...il.com, claudiu.manoil@....com,
vladimir.oltean@....com, xiaoning.wang@....com,
andrew+netdev@...n.ch, davem@...emloft.net, edumazet@...gle.com,
kuba@...nel.org, pabeni@...hat.com, vadim.fedorenko@...ux.dev,
shawnguo@...nel.org, s.hauer@...gutronix.de, festevam@...il.com,
fushi.peng@....com, devicetree@...r.kernel.org,
netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
imx@...ts.linux.dev, kernel@...gutronix.de
Subject: Re: [PATCH v2 net-next 07/14] ptp: netc: add debugfs support to loop
back pulse signal
On Wed, Jul 16, 2025 at 03:31:04PM +0800, Wei Fang wrote:
> The NETC Timer supports to loop back the output pulse signal of Fiper-n
> into Trigger-n input, so that we can leverage this feature to validate
> some other features without external hardware support. For example, we
> can use it to test external trigger stamp (EXTTS). And we can combine
> EXTTS with loopback mode to check whether the generation time of PPS is
> aligned with an integral second of PHC, or the periodic output signal
> (PTP_CLK_REQ_PEROUT) whether is generated at the specified time. So add
> the debugfs interfaces to enable the loopback mode of Fiper1 and Fiper2.
>
> An example to test the generation time of PPS event.
>
> $ echo 1 > /sys/kernel/debug/netc_timer0/fiper1-loopback
> $ echo 1 > /sys/class/ptp/ptp0/pps_enable
> $ testptp -d /dev/ptp0 -e 3
> external time stamp request okay
> event index 0 at 108.000000018
> event index 0 at 109.000000018
> event index 0 at 110.000000018
>
> An example to test the generation time of the periodic output signal.
>
> $ echo 1 > /sys/kernel/debug/netc_timer0/fiper1-loopback
> $ echo 0 260 0 1 500000000 > /sys/class/ptp/ptp0/period
> $ testptp -d /dev/ptp0 -e 3
> external time stamp request okay
> event index 0 at 260.000000016
> event index 0 at 261.500000015
> event index 0 at 263.000000016
>
> Signed-off-by: Wei Fang <wei.fang@....com>
Reviewed-by: Frank Li <Frank.Li@....com>
>
> ---
> v2 changes:
> 1. Remove the check of the return value of debugfs_create_dir()
> ---
> drivers/ptp/ptp_netc.c | 114 +++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 114 insertions(+)
>
> diff --git a/drivers/ptp/ptp_netc.c b/drivers/ptp/ptp_netc.c
> index c2fc6351db5b..2a077eb2f0eb 100644
> --- a/drivers/ptp/ptp_netc.c
> +++ b/drivers/ptp/ptp_netc.c
> @@ -6,6 +6,7 @@
>
> #include <linux/bitfield.h>
> #include <linux/clk.h>
> +#include <linux/debugfs.h>
> #include <linux/fsl/netc_global.h>
> #include <linux/module.h>
> #include <linux/of.h>
> @@ -22,6 +23,8 @@
> #define TMR_ETEP2 BIT(9)
> #define TMR_COMP_MODE BIT(15)
> #define TMR_CTRL_TCLK_PERIOD GENMASK(25, 16)
> +#define TMR_CTRL_PP2L BIT(26)
> +#define TMR_CTRL_PP1L BIT(27)
> #define TMR_CTRL_FS BIT(28)
> #define TMR_ALARM1P BIT(31)
>
> @@ -129,6 +132,7 @@ struct netc_timer {
> u8 fs_alarm_num;
> u8 fs_alarm_bitmap;
> struct netc_pp pp[NETC_TMR_FIPER_NUM]; /* periodic pulse */
> + struct dentry *debugfs_root;
> };
>
> #define netc_timer_rd(p, o) netc_read((p)->base + (o))
> @@ -991,6 +995,114 @@ static int netc_timer_get_global_ip_rev(struct netc_timer *priv)
> return val & IPBRR0_IP_REV;
> }
>
> +static int netc_timer_get_fiper_loopback(struct netc_timer *priv,
> + int fiper, u64 *val)
> +{
> + unsigned long flags;
> + u32 tmr_ctrl;
> +
> + spin_lock_irqsave(&priv->lock, flags);
> + tmr_ctrl = netc_timer_rd(priv, NETC_TMR_CTRL);
> + spin_unlock_irqrestore(&priv->lock, flags);
> +
> + switch (fiper) {
> + case 0:
> + *val = tmr_ctrl & TMR_CTRL_PP1L ? 1 : 0;
> + break;
> + case 1:
> + *val = tmr_ctrl & TMR_CTRL_PP2L ? 1 : 0;
> + break;
> + default:
> + return -EINVAL;
> + }
> +
> + return 0;
> +}
> +
> +static int netc_timer_set_fiper_loopback(struct netc_timer *priv,
> + int fiper, u64 val)
> +{
> + unsigned long flags;
> + u32 tmr_ctrl;
> + int err = 0;
> +
> + spin_lock_irqsave(&priv->lock, flags);
> +
> + tmr_ctrl = netc_timer_rd(priv, NETC_TMR_CTRL);
> + switch (fiper) {
> + case 0:
> + tmr_ctrl = u32_replace_bits(tmr_ctrl, val ? 1 : 0,
> + TMR_CTRL_PP1L);
> + break;
> + case 1:
> + tmr_ctrl = u32_replace_bits(tmr_ctrl, val ? 1 : 0,
> + TMR_CTRL_PP2L);
> + break;
> + default:
> + err = -EINVAL;
> + }
> +
> + if (!err)
> + netc_timer_wr(priv, NETC_TMR_CTRL, tmr_ctrl);
> +
> + spin_unlock_irqrestore(&priv->lock, flags);
> +
> + return err;
> +}
> +
> +static int netc_timer_get_fiper1_loopback(void *data, u64 *val)
> +{
> + struct netc_timer *priv = data;
> +
> + return netc_timer_get_fiper_loopback(priv, 0, val);
> +}
> +
> +static int netc_timer_set_fiper1_loopback(void *data, u64 val)
> +{
> + struct netc_timer *priv = data;
> +
> + return netc_timer_set_fiper_loopback(priv, 0, val);
> +}
> +
> +DEFINE_DEBUGFS_ATTRIBUTE(netc_timer_fiper1_fops, netc_timer_get_fiper1_loopback,
> + netc_timer_set_fiper1_loopback, "%llu\n");
> +
> +static int netc_timer_get_fiper2_loopback(void *data, u64 *val)
> +{
> + struct netc_timer *priv = data;
> +
> + return netc_timer_get_fiper_loopback(priv, 1, val);
> +}
> +
> +static int netc_timer_set_fiper2_loopback(void *data, u64 val)
> +{
> + struct netc_timer *priv = data;
> +
> + return netc_timer_set_fiper_loopback(priv, 1, val);
> +}
> +
> +DEFINE_DEBUGFS_ATTRIBUTE(netc_timer_fiper2_fops, netc_timer_get_fiper2_loopback,
> + netc_timer_set_fiper2_loopback, "%llu\n");
> +
> +static void netc_timer_create_debugfs(struct netc_timer *priv)
> +{
> + char debugfs_name[24];
> +
> + snprintf(debugfs_name, sizeof(debugfs_name), "netc_timer%d",
> + priv->phc_index);
> + priv->debugfs_root = debugfs_create_dir(debugfs_name, NULL);
> + debugfs_create_file("fiper1-loopback", 0600, priv->debugfs_root,
> + priv, &netc_timer_fiper1_fops);
> + debugfs_create_file("fiper2-loopback", 0600, priv->debugfs_root,
> + priv, &netc_timer_fiper2_fops);
> +}
> +
> +static void netc_timer_remove_debugfs(struct netc_timer *priv)
> +{
> + debugfs_remove(priv->debugfs_root);
> + priv->debugfs_root = NULL;
> +}
> +
> static int netc_timer_probe(struct pci_dev *pdev,
> const struct pci_device_id *id)
> {
> @@ -1038,6 +1150,7 @@ static int netc_timer_probe(struct pci_dev *pdev,
> }
>
> priv->phc_index = ptp_clock_index(priv->clock);
> + netc_timer_create_debugfs(priv);
>
> return 0;
>
> @@ -1055,6 +1168,7 @@ static void netc_timer_remove(struct pci_dev *pdev)
> {
> struct netc_timer *priv = pci_get_drvdata(pdev);
>
> + netc_timer_remove_debugfs(priv);
> ptp_clock_unregister(priv->clock);
> netc_timer_free_msix_irq(priv);
> clk_disable_unprepare(priv->src_clk);
> --
> 2.34.1
>
Powered by blists - more mailing lists