On Fri, Mar 26, 2021 at 01:22:24PM -0500, Brijesh Singh wrote: > Should I do the same for the sev-es.c ? Currently, I am keeping all the > SEV-SNP specific changes in sev-snp.{c,h}. After a rename of > sev-es.{c,h} from both the arch/x86/kernel and arch-x86/boot/compressed > I can add the SNP specific stuff to it. > > Thoughts ? SNP depends on the whole functionality in SEV-ES, right? Which means, SNP will need all the functionality of sev-es.c. But sev-es.c is a lot more code than the header and snp is arch/x86/kernel/sev-snp.c | 269 ++++++++++++++++++++++++ oh well, not so much. I guess a single arch/x86/kernel/sev.c is probably ok. We can always do arch/x86/kernel/sev/ later and split stuff then when it starts getting real fat and impacts complication times. Btw, there's also arch/x86/kernel/sev-es-shared.c and that can be arch/x86/kernel/sev-shared.c then. Thx. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette