Mikulas Patocka <mpato...@redhat.com> writes:
>       copy = min(bytes, iov->iov_len - skip);
>  
> +#ifdef CONFIG_HIGHMEM
>       if (!fault_in_pages_writeable(buf, copy)) {

If you use IS_ENABLED in the if here ...

>       kunmap(page);
> +
> +#ifdef CONFIG_HIGHMEM
>  done:
> +#endif

... you don't need this ifdef.

-Andi


-- 
a...@linux.intel.com -- Speaking for myself only

Reply via email to