Re: [PATCH v1 08/12] tmem: Handle 'struct tmem_info' as a seperate field in the

[Konrad Rzeszutek Wilk <konrad@kernel.org>]

On Fri, Sep 30, 2016 at 08:56:03AM -0600, Jan Beulich wrote:
> >>> On 30.09.16 at 16:36, <konrad@kernel.org> wrote:
> > On Wed, Sep 28, 2016 at 06:56:40AM -0600, Jan Beulich wrote:
> >> >>> On 28.09.16 at 11:42, <konrad.wilk@oracle.com> wrote:
> >> > Note: We still have to do this awkward 'guest_handle_cast'
> >> > otherwise it will not compile on ARM - which defines _two_
> >> > of these macros (__guest_handle_64_xen_sysctl_tmem_client_t
> >> > and __guest_handle_xen_sysctl_tmem_client_t). And if cast is
> >> > not used then a compile error comes up as we use the wrong one.
> >> 
> >> This seems suspicious, but it's hard to judge without knowing what
> >> exactly the errors were.
> > 
> > tmem_control.c: In function ‘tmem_control’:
> > tmem_control.c:426:9: error: incompatible type for argument 2 of
> > ‘tmemc_set_client_info’
> >          ret = tmemc_set_client_info(op->cli_id, op->u.client);
> >          ^
> > tmem_control.c:302:12: note: expected
> > ‘__guest_handle_xen_sysctl_tmem_client_t’ but argument is of type
> > ‘__guest_handle_64_xen_sysctl_tmem_client_t’
> >  static int tmemc_set_client_info(domid_t cli_id,
> >             ^
> 
> Looks like you want to pass around a 64-bit handle then?


<nods> Which I found is XEN_GUEST_HANDLE, not XEN_GUEST_HANDLE_PARAM.

.. snip..
> >> >      if ( copy_from_guest(&info, buf, 1) )
> >> >          return -EFAULT;
> >> 
> >> The adjustments above look pretty unrelated to the purpose of the
> >> patch, but well - you're the maintainer of this code.
> > 
> > <smacks himself in the head>
> 
> Don't be that harsh to yourself.

:-)

> 
> Jan

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