Messages in this thread |  | | | From | Jesse Barnes <> | | Subject | Re: [PATCH] add PCI ROMs to sysfs | | Date | Fri, 30 Jul 2004 14:34:50 -0700 |
| |
On Friday, July 30, 2004 2:29 pm, Greg KH wrote: > On Fri, Jul 30, 2004 at 02:09:05PM -0700, Jesse Barnes wrote: > > Thoughts? I've tried to add cleanup code, but I'm not sure how > > acceptable it is and I don't have any way of testing it. > > You don't have access to a cardbus machine? Or how about using the > fakephp driver to "remove" pci devices? You should be able to test this > code path with either one of those methods...
I've never tried fakephp, I'll look into it. And I do have a laptop at home that I could mess around on too, though I don't think I have any devices with ROMs...
> > +void pci_remove_sysfs_dev_files(struct pci_dev *pdev) > > +{ > > + /* Don't need to free config entries since they're static & global */ > > What do you mean by this? You should still remove all files we added in > the pci_create_sysfs_dev_files() function here, not just the rom file.
What will happen if we don't? They didn't used to be removed, and there's only one global copy afaict. I'll add code to kill them at any rate.
Thanks, Jesse - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |