On 10/05/2012 06:45 PM, Peter Maydell wrote: > On 4 October 2012 11:36, Avi Kivity <a...@redhat.com> wrote: >> diff --git a/targphys.h b/targphys.h >> index bd4938f..08cade9 100644 >> --- a/targphys.h >> +++ b/targphys.h >> @@ -3,25 +3,10 @@ >> #ifndef TARGPHYS_H >> #define TARGPHYS_H >> >> -#ifdef TARGET_PHYS_ADDR_BITS >> +#define TARGET_PHYS_ADDR_BITS 64 >> /* target_phys_addr_t is the type of a physical address (its size can >> be different from 'target_ulong'). */ > > I've just noticed that this change means that linux-user binaries > now get a definition of target_phys_addr_t (where previously they > did not get that type at all). Was this intentional,
No. > and does it make sense? Not much. Not very harmful either. If you want it removed, I can post a patch. -- error compiling committee.c: too many arguments to function