frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug #6583: Only open /proc/bus/pci/devices once. (Bill Nottingham)

This commit is contained in:
Daniel Stone 2006-06-01 22:30:52 +00:00
parent a9ed5a8790
commit 56f21bda1c
3 changed files with 166 additions and 183 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ChangeLog
View File

@ -94,6 +94,10 @@
* os/utils.c:
Remove LBX. Goodbye.
* hw/xfree86/os-support/bus/linuxPci.c:
* hw/xfree86/os-support/linux/lnx_pci.c:
Bug #6583: Only parse /proc/bus/pci/devices once. (Bill Nottingham)
2006-05-30 Matthieu Herrb <matthieu.herrb@laas.fr>
* hw/xfree86/os-support/bsd/Makefile.am:

15
hw/xfree86/os-support/bus/linuxPci.c
View File

@ -130,16 +130,28 @@ linuxPciOpenFile(PCITAG tag, Bool write)
{
static int lbus,ldev,lfunc,fd = -1,is_write = 0;
int bus, dev, func;
char file[32];
char file[64];
struct stat ignored;
static int is26 = -1;
bus = PCI_BUS_FROM_TAG(tag);
dev = PCI_DEV_FROM_TAG(tag);
func = PCI_FUNC_FROM_TAG(tag);
if (is26 == -1) {
if (stat("/sys/bus/pci",&ignored) < 0)
is26 = 0;
else
is26 = 1;
}
if (fd == -1 || (write && (!is_write))
|| bus != lbus || dev != ldev || func != lfunc) {
if (fd != -1)
close(fd);
if (is26)
sprintf(file,"/sys/bus/pci/devices/0000:%02x:%02x.%01x/config",
bus, dev, func);
else {
if (bus < 256) {
sprintf(file,"/proc/bus/pci/%02x",bus);
if (stat(file, &ignored) < 0)
@ -157,6 +169,7 @@ linuxPciOpenFile(PCITAG tag, Bool write)
sprintf(file, "/proc/bus/pci/%04x/%02x.%1x",
bus, dev, func);
}
}
if (write) {
fd = open(file,O_RDWR);
if (fd != -1) is_write = TRUE;

210
hw/xfree86/os-support/linux/lnx_pci.c
View File

@ -23,38 +23,41 @@
#define PCIADDR_FMT "%lx"
#endif
FILE *xf86OSLinuxPCIFile = NULL;
struct pci_dev {
unsigned int bus;
unsigned int devfn;
PCIADDR_TYPE offset[7];
PCIADDR_TYPE size[7];
struct pci_dev *next;
};
Bool
xf86GetPciSizeFromOS(PCITAG tag, int index, int* bits)
{
struct pci_dev *xf86OSLinuxPCIDevs = NULL;
static struct pci_dev *xf86OSLinuxGetPciDevs(void) {
char c[0x200];
char *res;
unsigned int bus, devfn, dev, fn;
unsigned PCIADDR_TYPE size[7];
FILE *file = NULL;
struct pci_dev *tmp, *ret = NULL;
unsigned int num;
signed PCIADDR_TYPE Size;
char *res;
if (index > 7)
return FALSE;
file = fopen("/proc/bus/pci/devices", "r");
if (!file) return NULL;
if (!xf86OSLinuxPCIFile && \
!(xf86OSLinuxPCIFile = fopen("/proc/bus/pci/devices","r")))
return FALSE;
do {
res = fgets(c,0x1ff,xf86OSLinuxPCIFile);
res = fgets(c, 0x1ff, file);
if (res) {
tmp = xcalloc(sizeof(struct pci_dev),1);
num = sscanf(res,
/*bus+dev vendorid deviceid irq */
"%02x%02x\t%*04x%*04x\t%*x"
/* 7 PCI resource base addresses */
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
/* 7 PCI resource sizes, and then optionally a driver name */
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
@ -63,30 +66,56 @@ xf86GetPciSizeFromOS(PCITAG tag, int index, int* bits)
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT,
&bus,&devfn,&size[0],&size[1],&size[2],&size[3],
&size[4],&size[5],&size[6]);
if (num != 9) { /* apparantly not 2.3 style */
fseek(xf86OSLinuxPCIFile, 0L, SEEK_SET);
return FALSE;
&tmp->bus,&tmp->devfn,&tmp->offset[0],&tmp->offset[1],&tmp->offset[2],&tmp->offset[3],
&tmp->offset[4],&tmp->offset[5],&tmp->offset[6], &tmp->size[0], &tmp->size[1], &tmp->size[2],
&tmp->size[3], &tmp->size[4], &tmp->size[5], &tmp->size[6]);
if (num != 16) { /* apparantly not 2.3 style */
xfree(tmp);
fclose(file);
return NULL;
}
dev = devfn >> 3;
fn = devfn & 0x7;
if (tag == pciTag(bus,dev,fn)) {
if (ret) {
tmp->next = ret;
}
ret = tmp;
}
} while (res);
fclose(file);
return ret;
}
Bool
xf86GetPciSizeFromOS(PCITAG tag, int index, int* bits)
{
unsigned int dev, fn;
signed PCIADDR_TYPE Size;
struct pci_dev *device;
if (index > 7)
return FALSE;
if (!xf86OSLinuxPCIDevs) {
xf86OSLinuxPCIDevs = xf86OSLinuxGetPciDevs();
}
if (!xf86OSLinuxPCIDevs)
return FALSE;
for (device = xf86OSLinuxPCIDevs; device; device = device->next) {
dev = device->devfn >> 3;
fn = device->devfn & 0x7;
if (tag == pciTag(device->bus,dev,fn)) {
*bits = 0;
if (size[index] != 0) {
Size = size[index] - ((PCIADDR_TYPE) 1);
if (device->size[index] != 0) {
Size = device->size[index] - ((PCIADDR_TYPE) 1);
while (Size & ((PCIADDR_TYPE) 0x01)) {
Size = Size >> ((PCIADDR_TYPE) 1);
(*bits)++;
}
}
fseek(xf86OSLinuxPCIFile, 0L, SEEK_SET);
return TRUE;
}
}
} while (res);
fseek(xf86OSLinuxPCIFile, 0L, SEEK_SET);
return FALSE;
}
@ -96,59 +125,28 @@ xf86GetPciSizeFromOS(PCITAG tag, int index, int* bits)
Bool
xf86GetPciOffsetFromOS(PCITAG tag, int index, unsigned long* bases)
{
FILE *file;
char c[0x200];
char *res;
unsigned int bus, devfn, dev, fn;
unsigned PCIADDR_TYPE offset[7];
unsigned int num;
unsigned int dev, fn;
struct pci_dev *device;
if (index > 7)
return FALSE;
if (!(file = fopen("/proc/bus/pci/devices","r")))
return FALSE;
do {
res = fgets(c,0x1ff,file);
if (res) {
num = sscanf(res,
/*bus+dev vendorid deviceid irq */
"%02x%02x\t%*04x%*04x\t%*x"
/* 7 PCI resource base addresses */
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
/* 7 PCI resource sizes, and then optionally a driver name */
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT
"\t" PCIADDR_IGNORE_FMT,
&bus,&devfn,&offset[0],&offset[1],&offset[2],&offset[3],
&offset[4],&offset[5],&offset[6]);
if (num != 9) { /* apparantly not 2.3 style */
fclose(file);
return FALSE;
if (!xf86OSLinuxPCIDevs) {
xf86OSLinuxPCIDevs = xf86OSLinuxGetPciDevs();
}
if (!xf86OSLinuxPCIDevs)
return FALSE;
dev = devfn >> 3;
fn = devfn & 0x7;
if (tag == pciTag(bus,dev,fn)) {
for (device = xf86OSLinuxPCIDevs; device; device = device->next) {
dev = device->devfn >> 3;
fn = device->devfn & 0x7;
if (tag == pciTag(device->bus,dev,fn)) {
/* return the offset for the index requested */
*bases = offset[index];
fclose(file);
*bases = device->offset[index];
return TRUE;
}
}
} while (res);
fclose(file);
return FALSE;
}
@ -156,50 +154,21 @@ xf86GetPciOffsetFromOS(PCITAG tag, int index, unsigned long* bases)
unsigned long
xf86GetOSOffsetFromPCI(PCITAG tag, int space, unsigned long base)
{
FILE *file;
char c[0x200];
char *res;
unsigned int bus, devfn, dev, fn;
unsigned PCIADDR_TYPE offset[7];
unsigned PCIADDR_TYPE size[7];
unsigned int num;
unsigned int dev, fn;
unsigned int ndx;
struct pci_dev *device;
if (!(file = fopen("/proc/bus/pci/devices","r")))
return 0;
do {
res = fgets(c,0x1ff,file);
if (res) {
num = sscanf(res,
/*bus+dev vendorid deviceid irq */
"%02x%02x\t%*04x%*04x\t%*x"
/* 7 PCI resource base addresses */
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
/* 7 PCI resource sizes, and then optionally a driver name */
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT
"\t" PCIADDR_FMT,
&bus,&devfn,&offset[0],&offset[1],&offset[2],&offset[3],
&offset[4],&offset[5],&offset[6], &size[0], &size[1], &size[2],
&size[3], &size[4], &size[5], &size[6]);
if (num != 16) { /* apparantly not 2.3 style */
fclose(file);
return 0;
if (!xf86OSLinuxPCIDevs) {
xf86OSLinuxPCIDevs = xf86OSLinuxGetPciDevs();
}
if (!xf86OSLinuxPCIDevs) {
return FALSE;
}
dev = devfn >> 3;
fn = devfn & 0x7;
if (tag == pciTag(bus,dev,fn)) {
for (device = xf86OSLinuxPCIDevs; device; device = device->next) {
dev = device->devfn >> 3;
fn = device->devfn & 0x7;
if (tag == pciTag(device->bus, dev, fn)) {
/* ok now look through all the BAR values of this device */
pciConfigPtr pDev = xf86GetPciConfigFromTag(tag);
@ -221,16 +190,13 @@ xf86GetOSOffsetFromPCI(PCITAG tag, int space, unsigned long base)
savePtr &= flagMask;
/* find the index of the incoming base */
if (base >= savePtr && base < (savePtr + size[ndx])) {
fclose(file);
return (offset[ndx] & flagMask) + (base - savePtr);
if (base >= savePtr && base < (savePtr + device->size[ndx])) {
return (device->offset[ndx] & flagMask) + (base - savePtr);
}
}
}
}
} while (res);
};
fclose(file);
return 0;
}