If I recall, AMD said that this special initialization would eventually migrate into the core kernel. The pci_read() may be there because it was not exported by the kernel (this particular code lives in a kernel module). If you think there is now enough support in the core kernel to do this initialization then I'd be happy to remove this piece of code. On Tue, Apr 29, 2008 at 5:33 PM, Arjan van de Ven <arjan@xxxxxxxxxxxxx> wrote: > On Wed, 30 Apr 2008 15:11:11 +0200 > > "stephane eranian" <eranian@xxxxxxxxxxxxxx> wrote: > > > > Sorry, > > > > It's http://perfmon2.sf.net > > > > > > hummmmm, I wonder why perfmon wants to reimplement half of the PCI layer... > > +extern spinlock_t pci_config_lock; > + > +#define PCI_CONF1_ADDRESS(bus, devfn, reg) \ > + (0x80000000 | ((reg & 0xF00) << 16) | ((bus & 0xFF) << 16) \ > + | (devfn << 8) | (reg & 0xFC)) > + > +#define is_ibs_pmc(x) (x == 4 || x == 5) > + > +static int pci_read(unsigned int seg, unsigned int bus, > + unsigned int devfn, int reg, int len, u32 *value) > +{ > + unsigned long flags; > > especially since this doesn't seem to use the proper config space abstractions :( > -- To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |