| To: | hadi@xxxxxxxxxx |
|---|---|
| Subject: | Re: [patch 4/10] s390: network driver. |
| From: | Tommy Christensen <tommy.christensen@xxxxxxxxx> |
| Date: | Thu, 06 Jan 2005 16:06:13 +0100 |
| Cc: | Jeff Garzik <jgarzik@xxxxxxxxx>, Thomas Spatzier <thomas.spatzier@xxxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, Hasso Tepper <hasso@xxxxxxxxx>, Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>, netdev@xxxxxxxxxxx, Paul Jakma <paul@xxxxxxxx> |
| In-reply-to: | <1105019912.2314.20.camel@jzny.localdomain> |
| References: | <OFB7F7E23F.EFB88418-ONC1256F7E.0031769E-C1256F7E.003270AD@de.ibm.com> <1104764710.1048.580.camel@jzny.localdomain> <41DB26A6.2070008@pobox.com> <1104895169.1117.63.camel@jzny.localdomain> <41DC0931.80603@tpack.net> <1105019912.2314.20.camel@jzny.localdomain> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Thu, 2005-01-06 at 14:58, jamal wrote: > On Wed, 2005-01-05 at 10:35, Tommy Christensen wrote: > > jamal wrote: > > > > > Except for the drivers that call netif_stop_queue() on link-down. These > > calls (and the corresponding netif_wake_queue) would have to be removed. > > If we assume all drivers do: > netif_stop then carrier_off then you dont need that extra check. > Thats the working assumption i had - maybe a comment is deserving or > we could say we dont think that all drivers are going to follow that > sequence. But qdisc_restart() isn't called any more after the queue is stopped. So how do we get to drain the packets? Another approach could be to reset the qdisc (kind of what dev_deactivate does) if the driver stays in queue_stopped and carrier_off for some period of time. -Tommy |
| Previous by Date: | Re: Network driver test suite?, David Hollis |
|---|---|
| Next by Date: | Re: [Prism54-devel] Re: [Prism54-users] Open hardware wireless cards, Lennart Sorensen |
| Previous by Thread: | Re: [patch 4/10] s390: network driver., jamal |
| Next by Thread: | Re: [patch 4/10] s390: network driver., jamal |
| Indexes: | [Date] [Thread] [Top] [All Lists] |