On 2026-10-20 10:47, Popov, Pavel E wrote:
> If preferred, we could use for cpu_supports_p2pdma() the pattern Logan
> suggested for hypervisor_supports_p2pdma() in v1: the whole function
> under #ifdef CONFIG_X86 with a stub returning false for !CONFIG_X86.
> The Intel CPU list then moves into the same #ifdef block, which saves
> one #ifdef, at the cost of also touching the existing AMD part.

I personally would prefer cleaning up the #ifdef style but that's an
issue that's already in the code. I don't feel the need to make it
conditional on my Reviewed-by. If you are able to clean that up, I would
appreciate it.

Thanks,

Logan

Reply via email to