On Thu, Jun 24, 2021 at 11:47:31AM +0100, Robin Murphy wrote:

Hmm, I think the Kconfig reshuffle has actually left a slight wrinkle here.
For DMA_DIRECT_REMAP=y we can assume an atomic pool is always needed, since
that was the original behaviour anyway. However the implications of
AMD_MEM_ENCRYPT=y are different - even if support is enabled, it still
should only be relevant if mem_encrypt_active(), so it probably does make
sense to have an additional runtime gate on that.

Yeah, a check for that would probably be useful.