netdev - Re: [PATCH net] drivers : stmmac : Fixes for IEEE 1588 Timestamping

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <52B2DD9A.50003@st.com>
Date:	Thu, 19 Dec 2013 12:50:50 +0100
From:	Giuseppe CAVALLARO <peppe.cavallaro@...com>
To:	Vince Bridgers <vbridgers2013@...il.com>,
	Rayagond K <rayagond@...avyalabs.com>
Cc:	<netdev@...r.kernel.org>
Subject: Re: [PATCH net] drivers : stmmac : Fixes for IEEE 1588 Timestamping

On 12/19/2013 12:35 AM, Vince Bridgers wrote:
> >From e9278f13a922433adbf3a81ef9d7823a99bd8d88 Mon Sep 17 00:00:00 2001
> From: Vince Bridgers <vbridger2013@...il.com>
> Date: Wed, 18 Dec 2013 14:29:26 -0600
> Subject: [PATCH 1/1] Fix release of incorrect spinlock in 
> stmmac_ptp.c, and
>   masked configuration of control variable adv_ts
>
> Two issues addressed with this patch:
> 1) The functions stmmac_adjust_freq and stmmac_adjust_time were
> releasing the incorrect spinlock
>     at the end of the function - priv->lock instead of priv->ptp_lock.
> 2) Setting the control data variable priv->adv_ts according to
> hardware capabilities was masked
>     by message level settings, so it was not being set correctly in the
> case that the Synopsys
>     MAC only supported advanced timestamping.
>
> Signed-off-by: Vince Bridgers <vbridgers2013@...il.com>

Hello Vince

thanks for the patch that provides useful fixes. Pls can you check if it 
applies
fine on net.git master branch? It failed on my side.


peppe

> ---
>   drivers/net/ethernet/stmicro/stmmac/stmmac_main.c |   20 
> +++++++++-----------
>   drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c  |    4 ++--
>   2 files changed, 11 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
> b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
> index 8a7a23a..797b56a 100644
> --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
> +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
> @@ -622,17 +622,15 @@ static int stmmac_init_ptp(struct stmmac_priv *priv)
>       if (!(priv->dma_cap.time_stamp || priv->dma_cap.atime_stamp))
>           return -EOPNOTSUPP;
>
> -    if (netif_msg_hw(priv)) {
> -        if (priv->dma_cap.time_stamp) {
> -            pr_debug("IEEE 1588-2002 Time Stamp supported\n");
> -            priv->adv_ts = 0;
> -        }
> -        if (priv->dma_cap.atime_stamp && priv->extend_desc) {
> -            pr_debug
> -                ("IEEE 1588-2008 Advanced Time Stamp supported\n");
> -            priv->adv_ts = 1;
> -        }
> -    }
> +    priv->adv_ts = 0;
> +    if (priv->dma_cap.atime_stamp && priv->extend_desc)
> +        priv->adv_ts = 1;
> +
> +    if (netif_msg_hw(priv) && priv->dma_cap.time_stamp)
> +        pr_debug("IEEE 1588-2002 Time Stamp supported\n");
> +
> +    if (netif_msg_hw(priv) && priv->adv_ts)
> +        pr_debug("IEEE 1588-2008 Advanced Time Stamp supported\n");
>
>       priv->hw->ptp = &stmmac_ptp;
>       priv->hwts_tx_en = 0;
> diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c
> b/drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c
> index b8b0eee..7680581 100644
> --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c
> +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c
> @@ -56,7 +56,7 @@ static int stmmac_adjust_freq(struct ptp_clock_info
> *ptp, s32 ppb)
>
>       priv->hw->ptp->config_addend(priv->ioaddr, addend);
>
> -    spin_unlock_irqrestore(&priv->lock, flags);
> +    spin_unlock_irqrestore(&priv->ptp_lock, flags);
>
>       return 0;
>   }
> @@ -91,7 +91,7 @@ static int stmmac_adjust_time(struct ptp_clock_info
> *ptp, s64 delta)
>
>       priv->hw->ptp->adjust_systime(priv->ioaddr, sec, nsec, neg_adj);
>
> -    spin_unlock_irqrestore(&priv->lock, flags);
> +    spin_unlock_irqrestore(&priv->ptp_lock, flags);
>
>       return 0;
>   }
>

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html