Messages in this thread | | | Subject | Re: [GIT PULL]: firmware patches for building firmware into kernel | From | David Dillow <> | Date | Fri, 08 Aug 2008 00:25:37 -0400 |
| |
On Fri, 2008-08-08 at 09:09 +0530, Jaswinder Singh wrote: > On Thu, 2008-08-07 at 22:59 -0400, David Dillow wrote: > > I don't like releasing the firmware before the pci_unregister_driver() > > call. I worry about ordering issues during cleanup, though I'll admit I > > have not yet researched if it will be a problem. In any event, if you're > > going to request it once per adapter in typhoon_init_one(), then it > > should be in the per-device struct, and released in > > typhoon_remove_one(). > > Here is updated patch :
You've fixed one part of the issues I described, but I think you've left open the possibility of dangling pointers when one card in a multi NIC system fails to come up. Again, I need to track down the code to see if I'm all wet or not.
> > Drop the typhoon patches, and once you fix the problems in the core, > > I'll respin the patch in a style I'm comfortable with. It will also need > > to be tested before it goes upstream. > > I can understand you are very worried about typhoon. > But this is only first version of patches. This will goto David > WoodHouse tree and he will again revise it. > > And driver is yours you can change it as per your comfort, No one can > stop you or typhoon ;)
I'd rather not have to undo the breakage when I fix it to my comfort -- best to get it right the first time if it is possible, and it is in this instance. I'm not just worried about typhoon, I'm worried about the core code and the other drivers you've converted. A brief skim over your tg3 and acenic patches suggests you took to heart the need to keep the firmware around while the device is up, but the basic locking issues in the core code suggest there are more lessons to be absorbed.
There's no way to get experience except by doing, and I applaud your efforts to learn the code, but please drop the typhoon part and concentrate on getting the core right. I've already committed to fixing typhoon once you do.
Dave
| |