diff options
author | bde <bde@FreeBSD.org> | 2000-04-30 10:01:56 +0000 |
---|---|---|
committer | bde <bde@FreeBSD.org> | 2000-04-30 10:01:56 +0000 |
commit | 2c5c997046c6e9a4b46da8c795733ef867dc7053 (patch) | |
tree | ae0bd2763d4f3ed2e02b85b96cbb170999920569 /sys/dev/pci | |
parent | 8b471affb9f3b118378c27a18ddbeff7fccf2186 (diff) | |
download | FreeBSD-src-2c5c997046c6e9a4b46da8c795733ef867dc7053.zip FreeBSD-src-2c5c997046c6e9a4b46da8c795733ef867dc7053.tar.gz |
Fixed the type of some ivar access functions. Ivars have type uintptr_t,
not u_long. On i386's with 64-bit longs, returning u_longs indirectly
in (more than) the space reserved for uintptr_t's tended to corrupt the
previous frame pointer in the stack frame, so it was not easy to debug.
The type mismatches are hidden by the bogus cast in DEVMETHOD().
Diffstat (limited to 'sys/dev/pci')
-rw-r--r-- | sys/dev/pci/pci.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/dev/pci/pci.c b/sys/dev/pci/pci.c index 23d9474..205fdfc 100644 --- a/sys/dev/pci/pci.c +++ b/sys/dev/pci/pci.c @@ -1267,7 +1267,7 @@ pci_probe_nomatch(device_t dev, device_t child) } static int -pci_read_ivar(device_t dev, device_t child, int which, u_long *result) +pci_read_ivar(device_t dev, device_t child, int which, uintptr_t *result) { struct pci_devinfo *dinfo; pcicfgregs *cfg; |