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().

1 files changed, 1 insertions, 1 deletions

diff --git a/sys/isa/fd.c b/sys/isa/fd.c
index c1b02b1..cd4df13 100644
--- a/sys/isa/fd.c
+++ b/sys/isa/fd.c
@@ -660,7 +660,7 @@ static struct isa_pnp_id fdc_ids[] = {
 };
 
 static int
-fdc_read_ivar(device_t dev, device_t child, int which, u_long *result)
+fdc_read_ivar(device_t dev, device_t child, int which, uintptr_t *result)
 {
 	struct fdc_ivars *ivars = device_get_ivars(child);