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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 24 Jul 2007 17:45:37 -0700 (PDT) From: David Miller <davem@...emloft.net> To: rusty@...tcorp.com.au Cc: netdev@...r.kernel.org, shemminger@...ux-foundation.org, jgarzik@...ox.com, hadi@...erus.ca Subject: Re: [PATCH RFX]: napi_struct V3 From: Rusty Russell <rusty@...tcorp.com.au> Date: Tue, 24 Jul 2007 16:21:43 +1000 > On Mon, 2007-07-23 at 22:47 -0700, David Miller wrote: > > Any objections? > > On the contrary, this looks good. It turns out the explicit restart logic isn't necessary. On the first driver I tried to "convert" this became apparent real fast. The key is what the ->poll() caller does if you don't complete the NAPI, from net_rx_action(): /* if napi_complete not called, reschedule */ if (test_bit(NAPI_STATE_SCHED, &n->state)) __napi_schedule(n); Let's look at ep93xx_poll() as it sits in my current tree, which used to use netif_rx_reschedule(): static int ep93xx_poll(struct napi_struct *napi, int budget) { struct ep93xx_priv *ep = container_of(napi, struct ep93xx_priv, napi); struct net_device *dev = ep->dev; int rx; /* * @@@ Have to stop polling if device is downed while we * are polling. */ rx = ep93xx_rx(dev, 0, budget); if (rx < budget) { spin_lock_irq(&ep->rx_lock); wrl(ep, REG_INTEN, REG_INTEN_TX | REG_INTEN_RX); if (ep93xx_have_more_rx(ep)) wrl(ep, REG_INTEN, REG_INTEN_TX); else netif_rx_complete(dev, napi); spin_unlock_irq(&ep->rx_lock); } return rx; } This driver handles TX in it's hardware interrupt handler, and RX via NAPI. So to NAPI poll it simply disables RX interrupts and schedules NAPI. if (status & REG_INTSTS_RX) { spin_lock(&ep->rx_lock); if (likely(__netif_rx_schedule_prep(dev, &ep->napi))) { wrl(ep, REG_INTEN, REG_INTEN_TX); __netif_rx_schedule(dev, &ep->napi); } spin_unlock(&ep->rx_lock); } anyways, if after re-enabling RX interrupts it sees some RX work, it can simply re-disable RX interrupts and leave the NAPI state alone. And that's how I've coded things above. The caller will requeue the NAPI instance onto the poll list, nothing more needs to be done to prevent event loss. I'm now going to go over the other resched cases and make sure things can be similarly handled in those drivers as well. To be honest I'm quite confident this will be the case. - 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
Powered by blists - more mailing lists