[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 1/3] x86/mem_access: Make the mem_access ops generic



>>> On 11.04.14 at 11:35, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 11/04/14 04:49, Aravindh Puthiyaparambil wrote:
>> --- a/xen/arch/x86/mm/mem_access.c
>> +++ b/xen/arch/x86/mm/mem_access.c
>> @@ -21,31 +21,98 @@
>>   */
>>  
>>  
>> +#include <xen/sched.h>
>> +#include <xen/guest_access.h>
>> +#include <xen/hypercall.h>
>>  #include <asm/p2m.h>
>>  #include <asm/mem_event.h>
>> +#include <xsm/xsm.h>
>>  
>>  
>> -int mem_access_memop(struct domain *d, xen_mem_event_op_t *meo)
>> +#define ACCESS_op_mask  0xff
>> +
>> +int mem_access_memop(XEN_GUEST_HANDLE_PARAM(void) arg, unsigned long 
> start_iter)
>>  {
>> -    int rc;
>> +    long rc;
> 
> int function and long rc? The function wants promoting to long.

No, what makes it out of the function fits in an "int", but the result
from p2m_set_mem_access() needs the variable to be "long".

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.