Re: [PATCH] pci_proc_domain
On Mon, Jan 17, 2005 at 07:23:35PM +, Matthew Wilcox wrote: > > There's no need for the architectures to know how to name busses, > so replace pci_name_bus with pci_proc_domain -- a predicate to allow > architectures to choose whether domains are included in /proc/bus/pci > or not. I've converted all architectures but only tested ia64 and a > CONFIG_PCI_DOMAINS=n build. Applied, thanks. greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] pci_proc_domain
On Mon, Jan 17, 2005 at 07:23:35PM +, Matthew Wilcox wrote: There's no need for the architectures to know how to name busses, so replace pci_name_bus with pci_proc_domain -- a predicate to allow architectures to choose whether domains are included in /proc/bus/pci or not. I've converted all architectures but only tested ia64 and a CONFIG_PCI_DOMAINS=n build. Applied, thanks. greg k-h - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] pci_proc_domain
There's no need for the architectures to know how to name busses, so replace pci_name_bus with pci_proc_domain -- a predicate to allow architectures to choose whether domains are included in /proc/bus/pci or not. I've converted all architectures but only tested ia64 and a CONFIG_PCI_DOMAINS=n build. Index: linux-2.6/drivers/pci/proc.c === RCS file: /var/cvs/linux-2.6/drivers/pci/proc.c,v retrieving revision 1.6 diff -u -p -r1.6 proc.c --- linux-2.6/drivers/pci/proc.c21 Oct 2004 18:59:15 - 1.6 +++ linux-2.6/drivers/pci/proc.c17 Jan 2005 19:03:01 - @@ -384,26 +384,32 @@ static struct proc_dir_entry *proc_bus_p int pci_proc_attach_device(struct pci_dev *dev) { struct pci_bus *bus = dev->bus; - struct proc_dir_entry *de, *e; + struct proc_dir_entry *e; char name[16]; if (!proc_initialized) return -EACCES; - if (!(de = bus->procdir)) { - if (pci_name_bus(name, bus)) - return -EEXIST; - de = bus->procdir = proc_mkdir(name, proc_bus_pci_dir); - if (!de) + if (!bus->procdir) { + if (pci_proc_domain(bus)) { + sprintf(name, "%04x:%02x", pci_domain_nr(bus), + bus->number); + } else { + sprintf(name, "%02x", bus->number); + } + bus->procdir = proc_mkdir(name, proc_bus_pci_dir); + if (!bus->procdir) return -ENOMEM; } + sprintf(name, "%02x.%x", PCI_SLOT(dev->devfn), PCI_FUNC(dev->devfn)); - e = dev->procent = create_proc_entry(name, S_IFREG | S_IRUGO | S_IWUSR, de); + e = create_proc_entry(name, S_IFREG | S_IRUGO | S_IWUSR, bus->procdir); if (!e) return -ENOMEM; e->proc_fops = _bus_pci_operations; e->data = dev; e->size = dev->cfg_size; + dev->procent = e; return 0; } Index: linux-2.6/arch/ppc64/kernel/pci.c === RCS file: /var/cvs/linux-2.6/arch/ppc64/kernel/pci.c,v retrieving revision 1.12 diff -u -p -r1.12 pci.c --- linux-2.6/arch/ppc64/kernel/pci.c 12 Jan 2005 20:15:46 - 1.12 +++ linux-2.6/arch/ppc64/kernel/pci.c 17 Jan 2005 19:03:02 - @@ -300,19 +300,15 @@ int pci_domain_nr(struct pci_bus *bus) EXPORT_SYMBOL(pci_domain_nr); -/* Set the name of the bus as it appears in /proc/bus/pci */ -int pci_name_bus(char *name, struct pci_bus *bus) +/* Decide whether to display the domain number in /proc */ +int pci_proc_domain(struct pci_bus *bus) { -#ifndef CONFIG_PPC_ISERIES +#ifdef CONFIG_PPC_ISERIES + return 0; +#else struct pci_controller *hose = pci_bus_to_host(bus); - - if (hose->buid) - sprintf(name, "%04x:%02x", pci_domain_nr(bus), bus->number); - else + return hose->buid; #endif - sprintf(name, "%02x", bus->number); - - return 0; } /* Index: linux-2.6/arch/sparc64/kernel/pci.c === RCS file: /var/cvs/linux-2.6/arch/sparc64/kernel/pci.c,v retrieving revision 1.7 diff -u -p -r1.7 pci.c --- linux-2.6/arch/sparc64/kernel/pci.c 4 Dec 2004 07:02:25 - 1.7 +++ linux-2.6/arch/sparc64/kernel/pci.c 17 Jan 2005 19:03:02 - @@ -794,12 +794,6 @@ int pci_domain_nr(struct pci_bus *pbus) } EXPORT_SYMBOL(pci_domain_nr); -int pci_name_bus(char *name, struct pci_bus *bus) -{ - sprintf(name, "%04x:%02x", pci_domain_nr(bus), bus->number); - return 0; -} - int pcibios_prep_mwi(struct pci_dev *dev) { /* We set correct PCI_CACHE_LINE_SIZE register values for every Index: linux-2.6/include/asm-alpha/pci.h === RCS file: /var/cvs/linux-2.6/include/asm-alpha/pci.h,v retrieving revision 1.7 diff -u -p -r1.7 pci.h --- linux-2.6/include/asm-alpha/pci.h 13 Sep 2004 15:23:56 - 1.7 +++ linux-2.6/include/asm-alpha/pci.h 17 Jan 2005 19:03:02 - @@ -228,17 +228,10 @@ extern void pcibios_resource_to_bus(stru #define pci_domain_nr(bus) ((struct pci_controller *)(bus)->sysdata)->index -static inline int -pci_name_bus(char *name, struct pci_bus *bus) +static inline int pci_proc_domain(struct pci_bus *bus) { struct pci_controller *hose = bus->sysdata; - - if (likely(hose->need_domain_info == 0)) { - sprintf(name, "%02x", bus->number); - } else { - sprintf(name, "%04x:%02x", hose->index, bus->number); - } - return 0; + return hose->need_domain_info; } static inline void Index: linux-2.6/include/asm-ia64/pci.h === RCS file: /var/cvs/linux-2.6/include/asm-ia64/pci.h,v retrieving revision 1.8 diff -u -p
[PATCH] pci_proc_domain
There's no need for the architectures to know how to name busses, so replace pci_name_bus with pci_proc_domain -- a predicate to allow architectures to choose whether domains are included in /proc/bus/pci or not. I've converted all architectures but only tested ia64 and a CONFIG_PCI_DOMAINS=n build. Index: linux-2.6/drivers/pci/proc.c === RCS file: /var/cvs/linux-2.6/drivers/pci/proc.c,v retrieving revision 1.6 diff -u -p -r1.6 proc.c --- linux-2.6/drivers/pci/proc.c21 Oct 2004 18:59:15 - 1.6 +++ linux-2.6/drivers/pci/proc.c17 Jan 2005 19:03:01 - @@ -384,26 +384,32 @@ static struct proc_dir_entry *proc_bus_p int pci_proc_attach_device(struct pci_dev *dev) { struct pci_bus *bus = dev-bus; - struct proc_dir_entry *de, *e; + struct proc_dir_entry *e; char name[16]; if (!proc_initialized) return -EACCES; - if (!(de = bus-procdir)) { - if (pci_name_bus(name, bus)) - return -EEXIST; - de = bus-procdir = proc_mkdir(name, proc_bus_pci_dir); - if (!de) + if (!bus-procdir) { + if (pci_proc_domain(bus)) { + sprintf(name, %04x:%02x, pci_domain_nr(bus), + bus-number); + } else { + sprintf(name, %02x, bus-number); + } + bus-procdir = proc_mkdir(name, proc_bus_pci_dir); + if (!bus-procdir) return -ENOMEM; } + sprintf(name, %02x.%x, PCI_SLOT(dev-devfn), PCI_FUNC(dev-devfn)); - e = dev-procent = create_proc_entry(name, S_IFREG | S_IRUGO | S_IWUSR, de); + e = create_proc_entry(name, S_IFREG | S_IRUGO | S_IWUSR, bus-procdir); if (!e) return -ENOMEM; e-proc_fops = proc_bus_pci_operations; e-data = dev; e-size = dev-cfg_size; + dev-procent = e; return 0; } Index: linux-2.6/arch/ppc64/kernel/pci.c === RCS file: /var/cvs/linux-2.6/arch/ppc64/kernel/pci.c,v retrieving revision 1.12 diff -u -p -r1.12 pci.c --- linux-2.6/arch/ppc64/kernel/pci.c 12 Jan 2005 20:15:46 - 1.12 +++ linux-2.6/arch/ppc64/kernel/pci.c 17 Jan 2005 19:03:02 - @@ -300,19 +300,15 @@ int pci_domain_nr(struct pci_bus *bus) EXPORT_SYMBOL(pci_domain_nr); -/* Set the name of the bus as it appears in /proc/bus/pci */ -int pci_name_bus(char *name, struct pci_bus *bus) +/* Decide whether to display the domain number in /proc */ +int pci_proc_domain(struct pci_bus *bus) { -#ifndef CONFIG_PPC_ISERIES +#ifdef CONFIG_PPC_ISERIES + return 0; +#else struct pci_controller *hose = pci_bus_to_host(bus); - - if (hose-buid) - sprintf(name, %04x:%02x, pci_domain_nr(bus), bus-number); - else + return hose-buid; #endif - sprintf(name, %02x, bus-number); - - return 0; } /* Index: linux-2.6/arch/sparc64/kernel/pci.c === RCS file: /var/cvs/linux-2.6/arch/sparc64/kernel/pci.c,v retrieving revision 1.7 diff -u -p -r1.7 pci.c --- linux-2.6/arch/sparc64/kernel/pci.c 4 Dec 2004 07:02:25 - 1.7 +++ linux-2.6/arch/sparc64/kernel/pci.c 17 Jan 2005 19:03:02 - @@ -794,12 +794,6 @@ int pci_domain_nr(struct pci_bus *pbus) } EXPORT_SYMBOL(pci_domain_nr); -int pci_name_bus(char *name, struct pci_bus *bus) -{ - sprintf(name, %04x:%02x, pci_domain_nr(bus), bus-number); - return 0; -} - int pcibios_prep_mwi(struct pci_dev *dev) { /* We set correct PCI_CACHE_LINE_SIZE register values for every Index: linux-2.6/include/asm-alpha/pci.h === RCS file: /var/cvs/linux-2.6/include/asm-alpha/pci.h,v retrieving revision 1.7 diff -u -p -r1.7 pci.h --- linux-2.6/include/asm-alpha/pci.h 13 Sep 2004 15:23:56 - 1.7 +++ linux-2.6/include/asm-alpha/pci.h 17 Jan 2005 19:03:02 - @@ -228,17 +228,10 @@ extern void pcibios_resource_to_bus(stru #define pci_domain_nr(bus) ((struct pci_controller *)(bus)-sysdata)-index -static inline int -pci_name_bus(char *name, struct pci_bus *bus) +static inline int pci_proc_domain(struct pci_bus *bus) { struct pci_controller *hose = bus-sysdata; - - if (likely(hose-need_domain_info == 0)) { - sprintf(name, %02x, bus-number); - } else { - sprintf(name, %04x:%02x, hose-index, bus-number); - } - return 0; + return hose-need_domain_info; } static inline void Index: linux-2.6/include/asm-ia64/pci.h === RCS file: /var/cvs/linux-2.6/include/asm-ia64/pci.h,v retrieving revision 1.8 diff -u -p -r1.8 pci.h ---