"Aneesh Kumar K.V" <aneesh.ku...@linux.vnet.ibm.com> writes:

> Also note that the `~` operation is wrong.
>
> Cc: Frederic Barrat <fbar...@linux.vnet.ibm.com>
> Cc: Andrew Donnellan <andrew.donnel...@au1.ibm.com>
> Acked-by: Ian Munsie <imun...@au1.ibm.com>
> Signed-off-by: Aneesh Kumar K.V <aneesh.ku...@linux.vnet.ibm.com>
> ---
>  drivers/misc/cxl/fault.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/misc/cxl/fault.c b/drivers/misc/cxl/fault.c
> index 9a8650bcb042..9a236543da23 100644
> --- a/drivers/misc/cxl/fault.c
> +++ b/drivers/misc/cxl/fault.c
> @@ -152,7 +152,7 @@ static void cxl_handle_page_fault(struct cxl_context *ctx,
>       access = _PAGE_PRESENT;
>       if (dsisr & CXL_PSL_DSISR_An_S)
>               access |= _PAGE_RW;
> -     if ((!ctx->kernel) || ~(dar & (1ULL << 63)))
> +     if ((!ctx->kernel) || (REGION_ID(dar) == USER_REGION_ID))
>               access |= _PAGE_USER;
>
>       if (dsisr & DSISR_NOHPTE)
> -- 
> 2.5.0

Posted an updated version of this patch alone with improved commit
message here

http://mid.gmane.org/1460482475-20782-1-git-send-email-aneesh.ku...@linux.vnet.ibm.com

-aneesh

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to