RE: [PATCH] NAPI support for Sibyte MAC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello, 

> -----Original Message-----
> From: Sergei Shtylyov [mailto:sshtylyov@xxxxxxxxxxxxx] 
> Sent: Saturday, March 24, 2007 12:49 PM
> To: Mark E Mason
> Cc: linux-mips@xxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH] NAPI support for Sibyte MAC

[snip]
> > @@ -2075,12 +2143,52 @@
> >  		 */
> >  
> >  		if (isr & (M_MAC_INT_CHANNEL << S_MAC_TX_CH0)) {
> > -			sbdma_tx_process(sc,&(sc->sbm_txdma));
> > +			sbdma_tx_process(sc,&(sc->sbm_txdma), 0);
> > +#ifdef CONFIG_NETPOLL_TRAP
> > +		       if (netpoll_trap()) {
> > +			       if 
> (test_and_clear_bit(__LINK_STATE_XOFF, &dev->state)) 
> > +				       __netif_schedule(dev);
> > +		       }
> > +#endif
> >  		}
> 
>     This just doesn't make sense. That option is enabled to 
> *prevent* calls to 
> __netif_schedule() -- you can't override it that way. (Well, 
> how it works 
> currently, doesn't make much sense either since it totally 
> breaks the TX queue 
> control -- I was going to post a patch).

I'll admit - this was just copied over from the non-NAPI version we
started with ;-).

Are you saying that the #ifdef test should simply be an #ifndef here?

[I'm working on a revised patch based on feedback from Ralf and others
which removes the Kconfig option for NAPI - it'll just be on, all the
time.]

Thanks,
Mark



[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux