Mikulas Patocka <mpatocka@xxxxxxxxxx> 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 -- ak@xxxxxxxxxxxxxxx -- Speaking for myself only -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>