[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 30 Apr 2008 15:51:39 +0200
From: "stephane eranian" <eranian@...glemail.com>
To: "Arjan van de Ven" <arjan@...radead.org>
Cc: linux-kernel@...r.kernel.org, ia64 <linux-ia64@...r.kernel.org>,
"Robert Richter" <robert.richter@....com>
Subject: Re: 2.6.25 new perfmon kernel patch + libpfm + pfmon available
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@...radead.org> wrote:
> On Wed, 30 Apr 2008 15:11:11 +0200
>
> "stephane eranian" <eranian@...glemail.com> 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-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists