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..