mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-21 08:19:28 +00:00
[PATCH] i386/x86-64 Fall back to type 1 access when no entry found
When there is no entry for a bus in MCFG fall back to type1. This is especially important on K8 systems where always some devices can't be accessed using mmconfig (in particular the builtin northbridge doesn't support it for its own devices) Cc: <gregkh@suse.de> Cc: <jgarzik@pobox.com> Signed-off-by: Andi Kleen <ak@suse.de> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
parent
bf5421c309
commit
928cf8c627
4 changed files with 45 additions and 19 deletions
|
@ -13,7 +13,7 @@
|
||||||
#define PCI_CONF1_ADDRESS(bus, devfn, reg) \
|
#define PCI_CONF1_ADDRESS(bus, devfn, reg) \
|
||||||
(0x80000000 | (bus << 16) | (devfn << 8) | (reg & ~3))
|
(0x80000000 | (bus << 16) | (devfn << 8) | (reg & ~3))
|
||||||
|
|
||||||
static int pci_conf1_read(unsigned int seg, unsigned int bus,
|
int pci_conf1_read(unsigned int seg, unsigned int bus,
|
||||||
unsigned int devfn, int reg, int len, u32 *value)
|
unsigned int devfn, int reg, int len, u32 *value)
|
||||||
{
|
{
|
||||||
unsigned long flags;
|
unsigned long flags;
|
||||||
|
@ -42,7 +42,7 @@ static int pci_conf1_read(unsigned int seg, unsigned int bus,
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int pci_conf1_write(unsigned int seg, unsigned int bus,
|
int pci_conf1_write(unsigned int seg, unsigned int bus,
|
||||||
unsigned int devfn, int reg, int len, u32 value)
|
unsigned int devfn, int reg, int len, u32 value)
|
||||||
{
|
{
|
||||||
unsigned long flags;
|
unsigned long flags;
|
||||||
|
|
|
@ -30,10 +30,8 @@ static u32 get_base_addr(unsigned int seg, int bus)
|
||||||
while (1) {
|
while (1) {
|
||||||
++cfg_num;
|
++cfg_num;
|
||||||
if (cfg_num >= pci_mmcfg_config_num) {
|
if (cfg_num >= pci_mmcfg_config_num) {
|
||||||
/* something bad is going on, no cfg table is found. */
|
/* Not found - fallback to type 1 */
|
||||||
/* so we fall back to the old way we used to do this */
|
return 0;
|
||||||
/* and just rely on the first entry to be correct. */
|
|
||||||
return pci_mmcfg_config[0].base_address;
|
|
||||||
}
|
}
|
||||||
cfg = &pci_mmcfg_config[cfg_num];
|
cfg = &pci_mmcfg_config[cfg_num];
|
||||||
if (cfg->pci_segment_group_number != seg)
|
if (cfg->pci_segment_group_number != seg)
|
||||||
|
@ -44,9 +42,9 @@ static u32 get_base_addr(unsigned int seg, int bus)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void pci_exp_set_dev_base(unsigned int seg, int bus, int devfn)
|
static inline void pci_exp_set_dev_base(unsigned int base, int bus, int devfn)
|
||||||
{
|
{
|
||||||
u32 dev_base = get_base_addr(seg, bus) | (bus << 20) | (devfn << 12);
|
u32 dev_base = base | (bus << 20) | (devfn << 12);
|
||||||
if (dev_base != mmcfg_last_accessed_device) {
|
if (dev_base != mmcfg_last_accessed_device) {
|
||||||
mmcfg_last_accessed_device = dev_base;
|
mmcfg_last_accessed_device = dev_base;
|
||||||
set_fixmap_nocache(FIX_PCIE_MCFG, dev_base);
|
set_fixmap_nocache(FIX_PCIE_MCFG, dev_base);
|
||||||
|
@ -57,13 +55,18 @@ static int pci_mmcfg_read(unsigned int seg, unsigned int bus,
|
||||||
unsigned int devfn, int reg, int len, u32 *value)
|
unsigned int devfn, int reg, int len, u32 *value)
|
||||||
{
|
{
|
||||||
unsigned long flags;
|
unsigned long flags;
|
||||||
|
u32 base;
|
||||||
|
|
||||||
if (!value || (bus > 255) || (devfn > 255) || (reg > 4095))
|
if (!value || (bus > 255) || (devfn > 255) || (reg > 4095))
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
|
base = get_base_addr(seg, bus);
|
||||||
|
if (!base)
|
||||||
|
return pci_conf1_read(seg,bus,devfn,reg,len,value);
|
||||||
|
|
||||||
spin_lock_irqsave(&pci_config_lock, flags);
|
spin_lock_irqsave(&pci_config_lock, flags);
|
||||||
|
|
||||||
pci_exp_set_dev_base(seg, bus, devfn);
|
pci_exp_set_dev_base(base, bus, devfn);
|
||||||
|
|
||||||
switch (len) {
|
switch (len) {
|
||||||
case 1:
|
case 1:
|
||||||
|
@ -86,13 +89,18 @@ static int pci_mmcfg_write(unsigned int seg, unsigned int bus,
|
||||||
unsigned int devfn, int reg, int len, u32 value)
|
unsigned int devfn, int reg, int len, u32 value)
|
||||||
{
|
{
|
||||||
unsigned long flags;
|
unsigned long flags;
|
||||||
|
u32 base;
|
||||||
|
|
||||||
if ((bus > 255) || (devfn > 255) || (reg > 4095))
|
if ((bus > 255) || (devfn > 255) || (reg > 4095))
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
|
base = get_base_addr(seg, bus);
|
||||||
|
if (!base)
|
||||||
|
return pci_conf1_write(seg,bus,devfn,reg,len,value);
|
||||||
|
|
||||||
spin_lock_irqsave(&pci_config_lock, flags);
|
spin_lock_irqsave(&pci_config_lock, flags);
|
||||||
|
|
||||||
pci_exp_set_dev_base(seg, bus, devfn);
|
pci_exp_set_dev_base(base, bus, devfn);
|
||||||
|
|
||||||
switch (len) {
|
switch (len) {
|
||||||
case 1:
|
case 1:
|
||||||
|
|
|
@ -74,3 +74,10 @@ extern spinlock_t pci_config_lock;
|
||||||
|
|
||||||
extern int (*pcibios_enable_irq)(struct pci_dev *dev);
|
extern int (*pcibios_enable_irq)(struct pci_dev *dev);
|
||||||
extern void (*pcibios_disable_irq)(struct pci_dev *dev);
|
extern void (*pcibios_disable_irq)(struct pci_dev *dev);
|
||||||
|
|
||||||
|
extern int pci_conf1_write(unsigned int seg, unsigned int bus,
|
||||||
|
unsigned int devfn, int reg, int len, u32 value);
|
||||||
|
extern int pci_conf1_read(unsigned int seg, unsigned int bus,
|
||||||
|
unsigned int devfn, int reg, int len, u32 *value);
|
||||||
|
|
||||||
|
|
||||||
|
|
|
@ -19,7 +19,7 @@ struct mmcfg_virt {
|
||||||
};
|
};
|
||||||
static struct mmcfg_virt *pci_mmcfg_virt;
|
static struct mmcfg_virt *pci_mmcfg_virt;
|
||||||
|
|
||||||
static char *get_virt(unsigned int seg, int bus)
|
static char *get_virt(unsigned int seg, unsigned bus)
|
||||||
{
|
{
|
||||||
int cfg_num = -1;
|
int cfg_num = -1;
|
||||||
struct acpi_table_mcfg_config *cfg;
|
struct acpi_table_mcfg_config *cfg;
|
||||||
|
@ -27,10 +27,9 @@ static char *get_virt(unsigned int seg, int bus)
|
||||||
while (1) {
|
while (1) {
|
||||||
++cfg_num;
|
++cfg_num;
|
||||||
if (cfg_num >= pci_mmcfg_config_num) {
|
if (cfg_num >= pci_mmcfg_config_num) {
|
||||||
/* something bad is going on, no cfg table is found. */
|
/* Not found - fall back to type 1. This happens
|
||||||
/* so we fall back to the old way we used to do this */
|
e.g. on the internal devices of a K8 northbridge. */
|
||||||
/* and just rely on the first entry to be correct. */
|
return NULL;
|
||||||
return pci_mmcfg_virt[0].virt;
|
|
||||||
}
|
}
|
||||||
cfg = pci_mmcfg_virt[cfg_num].cfg;
|
cfg = pci_mmcfg_virt[cfg_num].cfg;
|
||||||
if (cfg->pci_segment_group_number != seg)
|
if (cfg->pci_segment_group_number != seg)
|
||||||
|
@ -43,18 +42,25 @@ static char *get_virt(unsigned int seg, int bus)
|
||||||
|
|
||||||
static inline char *pci_dev_base(unsigned int seg, unsigned int bus, unsigned int devfn)
|
static inline char *pci_dev_base(unsigned int seg, unsigned int bus, unsigned int devfn)
|
||||||
{
|
{
|
||||||
|
char *addr = get_virt(seg, bus);
|
||||||
return get_virt(seg, bus) + ((bus << 20) | (devfn << 12));
|
if (!addr)
|
||||||
|
return NULL;
|
||||||
|
return addr + ((bus << 20) | (devfn << 12));
|
||||||
}
|
}
|
||||||
|
|
||||||
static int pci_mmcfg_read(unsigned int seg, unsigned int bus,
|
static int pci_mmcfg_read(unsigned int seg, unsigned int bus,
|
||||||
unsigned int devfn, int reg, int len, u32 *value)
|
unsigned int devfn, int reg, int len, u32 *value)
|
||||||
{
|
{
|
||||||
char *addr = pci_dev_base(seg, bus, devfn);
|
char *addr;
|
||||||
|
|
||||||
|
/* Why do we have this when nobody checks it. How about a BUG()!? -AK */
|
||||||
if (unlikely(!value || (bus > 255) || (devfn > 255) || (reg > 4095)))
|
if (unlikely(!value || (bus > 255) || (devfn > 255) || (reg > 4095)))
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
|
addr = pci_dev_base(seg, bus, devfn);
|
||||||
|
if (!addr)
|
||||||
|
return pci_conf1_read(seg,bus,devfn,reg,len,value);
|
||||||
|
|
||||||
switch (len) {
|
switch (len) {
|
||||||
case 1:
|
case 1:
|
||||||
*value = readb(addr + reg);
|
*value = readb(addr + reg);
|
||||||
|
@ -73,11 +79,16 @@ static int pci_mmcfg_read(unsigned int seg, unsigned int bus,
|
||||||
static int pci_mmcfg_write(unsigned int seg, unsigned int bus,
|
static int pci_mmcfg_write(unsigned int seg, unsigned int bus,
|
||||||
unsigned int devfn, int reg, int len, u32 value)
|
unsigned int devfn, int reg, int len, u32 value)
|
||||||
{
|
{
|
||||||
char *addr = pci_dev_base(seg, bus, devfn);
|
char *addr;
|
||||||
|
|
||||||
|
/* Why do we have this when nobody checks it. How about a BUG()!? -AK */
|
||||||
if (unlikely((bus > 255) || (devfn > 255) || (reg > 4095)))
|
if (unlikely((bus > 255) || (devfn > 255) || (reg > 4095)))
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
|
addr = pci_dev_base(seg, bus, devfn);
|
||||||
|
if (!addr)
|
||||||
|
return pci_conf1_write(seg,bus,devfn,reg,len,value);
|
||||||
|
|
||||||
switch (len) {
|
switch (len) {
|
||||||
case 1:
|
case 1:
|
||||||
writeb(value, addr + reg);
|
writeb(value, addr + reg);
|
||||||
|
|
Loading…
Reference in a new issue