On 3/20/23 11:59, Jacob Pan wrote: > --- a/arch/x86/kernel/traps.c > +++ b/arch/x86/kernel/traps.c > @@ -40,7 +40,10 @@ > #include <linux/io.h> > #include <linux/hardirq.h> > #include <linux/atomic.h> > -#include <linux/ioasid.h> > + > +#ifdef CONFIG_IOMMU_SVA > +#include <linux/iommu.h> > +#endif What's the deal with the #ifdef'd #include? What's the harm of just making it unconditional?
