d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed accesses to addresses between VM_MAXUSER_ADDRESS (normally

Browse Source 
0xefbfe000) and kernel_start (normally 0xf0100000).

Things are unnecessarily (?) difficult because procfs is used to
access user addresses in the live-kernel case although we must have
access to /dev/mem to work at all, and whatever works for the
dead-kernel case should work in all cases (modulo volatility of
live kernel variables).  We used the wrong range [0, kernel_start)
for user addresses.  Procfs should only work up to VM_MAXUSER_ADDRESS,
but it bogusly works for reads up to the address 2 pages higher
(the user area, including the kernel stack, is mapped to where the
user area used to be (WTUAUTB)).  Procfs can not work at all for
addresses between WTUAUTB and kernel_start.

Now we use procfs only to access addresses up to VM_MAXUSER_ADDRESS.
Higher addresses are translated normally using kvtophys(), so the
user ptd is used for addresses below the real kernel start (0xf0000000;
see INKERNEL()) and nothing is found WTUAUTB.

Strange accesses that cross the user-kernel boundary are now handled,
but such ranges are currently always errors because they necessarily
overlap the hole WTUAUTB.

Short reads are still not handled.
This commit is contained in:
Bruce Evans 1998-01-19 15:27:56 +00:00
parent 04822660a1
commit 9bb4a86cf0
3 changed files with 63 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
gnu/usr.bin/binutils/gdb/i386/kvm-fbsd.c
View File

@ -382,21 +382,30 @@ kcore_xfer_kmem (memaddr, myaddr, len, write, target)
int write;
struct target_ops *target;
{
int n;
int ns;
int nu;
if (!memaddr)
return (0);
if (memaddr >= (CORE_ADDR)VM_MAXUSER_ADDRESS)
nu = 0;
else
{
nu = xfer_umem (memaddr, myaddr, len, write);
if (nu <= 0)
return (0);
if (nu == len)
return (nu);
memaddr += nu;
if (memaddr != (CORE_ADDR)VM_MAXUSER_ADDRESS)
return (nu);
myaddr += nu;
len -= nu;
}
if (memaddr < kernel_start)
return xfer_umem (memaddr, myaddr, len, write);
ns = (write ? kvm_write : kvm_read) (core_kd, memaddr, myaddr, len);
if (ns < 0)
ns = 0;
n = write ?
kvm_write (core_kd, memaddr, myaddr, len) :
kvm_read (core_kd, memaddr, myaddr, len) ;
if (n < 0)
return 0;
return n;
return (nu + ns);
}
static int

33
gnu/usr.bin/binutils/gdb/kvm-fbsd.c
View File

@ -382,21 +382,30 @@ kcore_xfer_kmem (memaddr, myaddr, len, write, target)
int write;
struct target_ops *target;
{
int n;
int ns;
int nu;
if (!memaddr)
return (0);
if (memaddr >= (CORE_ADDR)VM_MAXUSER_ADDRESS)
nu = 0;
else
{
nu = xfer_umem (memaddr, myaddr, len, write);
if (nu <= 0)
return (0);
if (nu == len)
return (nu);
memaddr += nu;
if (memaddr != (CORE_ADDR)VM_MAXUSER_ADDRESS)
return (nu);
myaddr += nu;
len -= nu;
}
if (memaddr < kernel_start)
return xfer_umem (memaddr, myaddr, len, write);
ns = (write ? kvm_write : kvm_read) (core_kd, memaddr, myaddr, len);
if (ns < 0)
ns = 0;
n = write ?
kvm_write (core_kd, memaddr, myaddr, len) :
kvm_read (core_kd, memaddr, myaddr, len) ;
if (n < 0)
return 0;
return n;
return (nu + ns);
}
static int

33
gnu/usr.bin/gdb/gdb/kvm-fbsd.c
View File

@ -382,21 +382,30 @@ kcore_xfer_kmem (memaddr, myaddr, len, write, target)
int write;
struct target_ops *target;
{
int n;
int ns;
int nu;
if (!memaddr)
return (0);
if (memaddr >= (CORE_ADDR)VM_MAXUSER_ADDRESS)
nu = 0;
else
{
nu = xfer_umem (memaddr, myaddr, len, write);
if (nu <= 0)
return (0);
if (nu == len)
return (nu);
memaddr += nu;
if (memaddr != (CORE_ADDR)VM_MAXUSER_ADDRESS)
return (nu);
myaddr += nu;
len -= nu;
}
if (memaddr < kernel_start)
return xfer_umem (memaddr, myaddr, len, write);
ns = (write ? kvm_write : kvm_read) (core_kd, memaddr, myaddr, len);
if (ns < 0)
ns = 0;
n = write ?
kvm_write (core_kd, memaddr, myaddr, len) :
kvm_read (core_kd, memaddr, myaddr, len) ;
if (n < 0)
return 0;
return n;
return (nu + ns);
}
static int