Re: [RESEND PATCH v11 0/3] AX88796C SPI Ethernet Adapter

From: Jakub Kicinski
Date: Thu May 06 2021 - 20:02:04 EST


On Wed, 05 May 2021 19:11:39 +0200 Łukasz Stelmach wrote:
> It was <2021-03-02 wto 16:22>, when Łukasz Stelmach wrote:
> > This is a driver for AX88796C Ethernet Adapter connected in SPI mode as
> > found on ARTIK5 evaluation board. The driver has been ported from a
> > v3.10.9 vendor kernel for ARTIK5 board.
> >
> > Changes in v11:
> > - changed stat counters to 64-bit
> > - replaced WARN_ON(!mutex_is_locked()) with lockdep_assert_held()
> > - replaced ax88796c_free_skb_queue() with __skb_queue_purge()
> > - added cancel_work_sync() for ax_work
> > - removed unused fields of struct skb_data
> > - replaced MAX() with max() from minmax.h
> > - rebased to net-next (resend)
>
> Hi,
>
> What is current status? Should I rebase once more?

Unfortunately it seems so :( The patches got marked as Not Applicable
in patchwork. And now we are once again in the merge window period.
Please repost late next week, and make sure to prefix the subjects with
[PATCH net-next v12], this will hopefully prevent mis-categorization.

Looking at the code again is the use of netif_stop_queue(ndev) in
ax88796c_close() not a little racy? The work may be running in parallel
and immediately re-enable the queue.