<div dir="auto">This is fairly minor but it reveals a few hidden warnings, could I get some feedback on it?<div dir="auto"><br></div><div dir="auto">Thanks</div><div dir="auto">--</div><div dir="auto">Guillermo</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 5 May 2017 10:53, "Tordek" <<a href="mailto:kedrot@gmail.com">kedrot@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This patch makes a lot of new warnings pop up, because the error is<br>
blocking the rest of the file from being processed.<br>
<br>
On 5 May 2017 at 10:51, Guillermo O. Freschi <<a href="mailto:kedrot@gmail.com">kedrot@gmail.com</a>> wrote:<br>
> Signed-off-by: Guillermo O. Freschi <<a href="mailto:kedrot@gmail.com">kedrot@gmail.com</a>><br>
> ---<br>
>  drivers/staging/lustre/lnet/<wbr>selftest/conrpc.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/drivers/staging/lustre/lnet/<wbr>selftest/conrpc.c b/drivers/staging/lustre/lnet/<wbr>selftest/conrpc.c<br>
> index da36c55b86d3..f4578200994f 100644<br>
> --- a/drivers/staging/lustre/lnet/<wbr>selftest/conrpc.c<br>
> +++ b/drivers/staging/lustre/lnet/<wbr>selftest/conrpc.c<br>
> @@ -487,7 +487,7 @@ lstcon_rpc_trans_interpreter(<wbr>struct lstcon_rpc_trans *trans,<br>
>                                    sizeof(struct list_head)))<br>
>                         return -EFAULT;<br>
><br>
> -               if (tmp.next == head_up)<br>
> +               if ((struct list_head __user *)tmp.next == head_up)<br>
>                         return 0;<br>
><br>
>                 next = tmp.next;<br>
> --<br>
> 2.11.0<br>
><br>
</blockquote></div></div>