Re: [xen 5/6] user_data: add register read function

On Tue, Aug 14, 2018 at 05:00:57PM +0200, Norbert Manthey wrote:
> Signed-off-by: Norbert Manthey <[email protected]>
> ---
>  check_user_data2.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/check_user_data2.c b/check_user_data2.c
> index 5d5606c..d95ba1d 100644
> --- a/check_user_data2.c
> +++ b/check_user_data2.c
> @@ -1238,6 +1238,9 @@ void check_user_data2(int id)
>  	for (i = 0; i < ARRAY_SIZE(xen_from_guest_funcs); i++)
>  		add_function_hook(xen_from_guest_funcs[i], &match_user_copy, INT_PTR(0));
>  
> +        // the Xen equivalent to kvm_register_read
> +	add_function_assign_hook("acpi_hw_register_read", &match_user_copy, NULL);
                                                                            ^^^^
This should be INT_PTR(1) because it's supposed to say which parameter
gets set.  It start counting parameters from zero.

regards,
dan carpenter