Re: [PATCH] add PCI ROMs to sysfs

From: Greg KH (greg_at_kroah.com)
Date: 07/30/04

  • Next message: Jesse Barnes: "Re: [PATCH] add PCI ROMs to sysfs"
    Date:	Fri, 30 Jul 2004 14:29:30 -0700
    To: Jesse Barnes <jbarnes@engr.sgi.com>
    
    

    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...

    > +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.

    thanks,

    greg k-h
    -
    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/


  • Next message: Jesse Barnes: "Re: [PATCH] add PCI ROMs to sysfs"

    Relevant Pages

    • Re: -mm -> 2.6.13 merge status
      ... fails totally in a single routine. ... then putting all of the cleanup code in a ... single function and using the enums eliminates duplicate code and ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: question on memory map cleanup stuff
      ... > I've got a small feature that maps a page of kernel memory to userspace via ... > a syscall, then uses that page for various things. ... > It appears that I should be able to put my cleanup code in exit_mmap. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: inter-module-* removal.. small next step
      ... > Michael's patches put it back in. ... What's the fix? ... basically replaces the cleanup code, and thus puts it back in. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [PATCH] add PCI ROMs to sysfs
      ... On Friday, July 30, 2004 2:29 pm, Greg KH wrote: ... I've tried to add cleanup code, ... I'll add code to kill them at any rate. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: 2.6.10: "[permanent]" modules?
      ... On Iau, 2005-01-06 at 13:58, Bartlomiej Zolnierkiewicz wrote: ... > It can be unloaded given that the needed locking ... > and cleanup code are in place... ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)