Re: [PATCH RFC net-next 3/5] net: phy: add Marvell PHY PTP support

[Kory Maincent <kory.maincent@bootlin.com>]

On Mon, 14 Apr 2025 17:21:37 +0200
Kory Maincent <kory.maincent@bootlin.com> wrote:

> On Mon, 14 Apr 2025 15:55:09 +0100
> "Russell King (Oracle)" <linux@armlinux.org.uk> wrote:
> 
> > On Mon, Apr 14, 2025 at 04:43:14PM +0200, Kory Maincent wrote:  
> > > On Fri, 11 Apr 2025 22:26:42 +0100
> > > Russell King <rmk+kernel@armlinux.org.uk> wrote:
> > >     
>  [...]  
> > > 
> > > It seems a lot less stable than the first version of the driver.

FYI there is also an invalid wait context error:
[   65.041285] =============================
[   65.045286] [ BUG: Invalid wait context ]
[   65.049289] 6.14.0-13314-g04846c13cbec-dirty #91 Not tainted
[   65.054938] -----------------------------
[   65.058939] swapper/0/0 is trying to lock:
[   65.063028] ffff000805b0c110 (&rxq->rx_mutex){....}-{4:4}, at:
marvell_ptp_rxtstamp+0xf0/0x228 
[   65.071665] other info that might help us debug this: 
[   65.076707] context-{3:3}
[   65.079320] no locks held by swapper/0/0.
[   65.083321] stack backtrace:
[   65.086198] CPU: 0 UID: 0 PID: 0 Comm: swapper/0 Not tainted
6.14.0-13314-g04846c13cbec-dirty #91 PREEMPT(voluntary)
[   65.086207] Hardware name: Fluyt Prototype 1 Carrier Board ZU9 mpsoc  (DT)
[   65.086211] Call trace: 
[   65.086214]  show_stack+0x18/0x24 (C)
[   65.086224]  dump_stack_lvl+0xa4/0xf4
[   65.086233]  dump_stack+0x18/0x24
[   65.086240]  __lock_acquire+0xa14/0x2000
[   65.086250]  lock_acquire+0x1c4/0x360
[   65.086259]  __mutex_lock+0xa0/0x52c
[   65.086266]  mutex_lock_nested+0x24/0x30
[   65.086273]  marvell_ptp_rxtstamp+0xf0/0x228
[   65.086282]  marvell_phy_ptp_rxtstamp+0x18/0x24
[   65.086290]  skb_defer_rx_timestamp+0x104/0x31c

Regards,
-- 
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com