Re: [PATCH] Fix >4G i386 PAE grant table interface

[Steven Rostedt <srostedt@redhat.com>]

Jan Beulich wrote:
>>>> Steven Rostedt <srostedt@redhat.com> 02.11.06 17:53 >>>
>> It has been discovered that i386 boxes with more than 4G of RAM would 
>> randomly crash.  It was traced to the interface of blktap using 
>> gnttab_set_map_op.
>>
>> It would pass in the 64 bit pte entry, but the gnttab_set_map_op would 
>> only take a 32 bit (on i386) unsigned long as a parameter. So we lose 
>> the top 32bits.
> 
> Could you use maddr_t here rather than uint64_t? For non-PAE i386
> Linux, especially when using CONFIG_REGPARM, adding a useless
> argument slot seems wasteful...
> 

Actually, it makes no difference to me. In fact uint64_t was  my third 
incarnation, since I wasn't sure what the best would be. I started with 
unsigned long long, then switched to u64, and then noticed that since 
host_addr is uint64_t, that seemed the proper thing to use.

So a maddr_t would work too.

Do you want to do the patch, or would you like me to send another patch 
that would do this change?

-- Steve