On Mon, 2024-02-26 at 00:25 -0800, isaku.yamahata@xxxxxxxxx wrote: > +#include <linux/compiler.h> > + > +#include <asm/cacheflush.h> > +#include <asm/asm.h> > +#include <asm/kvm_host.h> None of above are needed to build this file, except the <asm/cacheflush.h>. I am removing them. I am also adding <asm/tdx.h> because this file uses 'struct tdx_module_args'. And <asm/page.h> for __va(). > + > +#include "tdx_errno.h" > +#include "tdx_arch.h" I am moving them to "tdx.h", and make "tdx.h" to include this "tdx_ops.h" as well, and declare the C file should never include "tdx_ops.h" directly: https://lore.kernel.org/lkml/72a718244456db291fc3bb243fcbafcb03f854e7.camel@xxxxxxxxx/