| To: | Jeff Garzik <jgarzik@xxxxxxxxx> |
|---|---|
| Subject: | Re: 4/x: [NETDRV] Merge register_netdev calls |
| From: | Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> |
| Date: | Fri, 11 Jun 2004 12:08:55 +1000 |
| Cc: | netdev@xxxxxxxxxxx |
| In-reply-to: | <40C8F824.50904@pobox.com> |
| References: | <20040313025859.GA8186@gondor.apana.org.au> <405C294D.5040508@pobox.com> <20040520111937.GA21804@gondor.apana.org.au> <20040522074435.GA9628@gondor.apana.org.au> <20040529084109.GA13032@gondor.apana.org.au> <40BE3778.1020404@pobox.com> <20040605052737.GA27406@gondor.apana.org.au> <40C8F824.50904@pobox.com> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.5.1+cvs20040105i |
On Thu, Jun 10, 2004 at 08:09:08PM -0400, Jeff Garzik wrote: > > Unfortunately this and patch #5 are moving in the opposition direction > from what we want to be doing, I think. > > AFAICS these patches move register_netdev() to points in each driver > earlier in the probe phase, when the driver is not fully set up and > ready to receive packets. > > Am I missing something? The only change made by these two patches is moving the call to register_netdev() from the outer probe() function to the inner probe() function where the the printk's are. There is absolutely no code in between the two locations. That is, the driver would've immediately called register_netdev after the inner probe() has succeeded. In fact it's really making these ISA/MCA probe() functions more like the ones we have for PCI. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: Allow IP header alignment to be overriden, Anton Blanchard |
|---|---|
| Next by Date: | Re: Allow IP header alignment to be overriden, Andi Kleen |
| Previous by Thread: | 5/x: [NETDRV] Move register_netdev into probe function, Herbert Xu |
| Next by Thread: | Re: 4/x: [NETDRV] Merge register_netdev calls, Herbert Xu |
| Indexes: | [Date] [Thread] [Top] [All Lists] |