Daniel Axtens <d...@axtens.net> writes:
>> +#ifdef CONFIG_PPC64
>> +static inline bool kfence_protect_page(unsigned long addr, bool protect)
>> +{
>> + struct page *page;
>> +
>> + page = virt_to_page(addr);
>> + if (protect)
>> + __kernel_map_pages(page, 1, 0);
>> + else
>> + __kernel_map_pages(page, 1, 1);
>
> I lose track of the type conversions and code conventions involved, but
> can we do something like __kernel_map_pages(page, 1, !!protect)?
Ah, I missed that the if changed the truth/falsity. !protect, not
!!protect :P
>
> Apart from that, this seems good.
>
> Kind regards,
> Daniel
