On Sun, 1 Nov 2020 03:37:28 +0200 Vladimir Oltean wrote: > On Sun, Nov 01, 2020 at 03:14:34AM +0200, Vladimir Oltean wrote: > > On Sat, Oct 31, 2020 at 06:00:43PM -0700, Jakub Kicinski wrote: > > > On Fri, 30 Oct 2020 03:48:59 +0200 Vladimir Oltean wrote: > > > > @@ -567,6 +591,17 @@ static netdev_tx_t dsa_slave_xmit(struct sk_buff > > > > *skb, struct net_device *dev) > > > > */ > > > > dsa_skb_tx_timestamp(p, skb); > > > > > > > > + if (dsa_realloc_skb(skb, dev)) { > > > > + kfree_skb(skb); > > > > > > dev_kfree_skb_any()? > > > > Just showing my ignorance, but where does the hardirq context come from? > > I mean I do see that netpoll_send_udp requires IRQs disabled, but is > that the only reason why all drivers need to assume hardirq context in > .xmit, or are there more?
netpoll is the only one that comes to my mind, maybe others know more..