[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH] efault - add data type to put_user()/get_us
From: |
Thiemo Seufer |
Subject: |
Re: [Qemu-devel] Re: [PATCH] efault - add data type to put_user()/get_user() |
Date: |
Sat, 3 Nov 2007 15:56:00 +0000 |
User-agent: |
Mutt/1.5.16 (2007-06-11) |
Thayne Harbaugh wrote:
>
> On Wed, 2007-10-31 at 16:44 -0600, Thayne Harbaugh wrote:
> > This patch updates get_user() and put_user() to take a third argument of
> > data type. get_user() and put_user() use target address which are
> > target_ulong and don't reflect the data type pointed to in target
> > memory.
> >
> > Simply casting the target_ulong to a type before passing to
> > get/put_user() is poor because target_ulong isn't always a simple cast
> > to a host type (consider 32 bit on 64 bit where address are either
> > extended or truncate). Also, simple casting of the argument to
> > get/put_user() results in several warnings when target and long pointer
> > sizes don't match.
> >
> > This patch has additional updates to fix places where get/put_user() are
> > already used.
>
> This is an updated patch that doesn't conflict with the
> abi_long/abi_ulong changes from a couple weeks ago.
Still misses the sparc64 bits in linux-user/signal.c.
Thiemo