On Wed, Apr 23, 2025 at 01:22:00PM -0700, Ackerley Tng wrote:
> Yan Zhao <yan.y.zhao@xxxxxxxxx> writes:
>
> > Hi Ackerley,
> >
> > Not sure if below nits have been resolved in your latest code.
> > I came across them and felt it's better to report them anyway.
> >
> > Apologies for any redundancy if you've already addressed them.
>
> No worries, thank you so much for your reviews!
>
> >
> > On Tue, Sep 10, 2024 at 11:43:44PM +0000, Ackerley Tng wrote:
> >> +static void kvm_gmem_init_mount(void)
> >> +{
> >> + kvm_gmem_mnt = kern_mount(&kvm_gmem_fs);
> >> + BUG_ON(IS_ERR(kvm_gmem_mnt));
> >> +
> >> + /* For giggles. Userspace can never map this anyways. */
> >> + kvm_gmem_mnt->mnt_flags |= MNT_NOEXEC;
> >> +}
> >> +
> >> static struct file_operations kvm_gmem_fops = {
> >> .open = generic_file_open,
> >> .release = kvm_gmem_release,
> >> @@ -311,6 +348,8 @@ static struct file_operations kvm_gmem_fops = {
> >> void kvm_gmem_init(struct module *module)
> >> {
> >> kvm_gmem_fops.owner = module;
> >> +
> >> + kvm_gmem_init_mount();
> >> }
> > When KVM is compiled as a module, looks "kern_unmount(kvm_gmem_mnt)" is
> > missing in the kvm_exit() path.
> >
> > This may lead to kernel oops when executing "sync" after KVM is unloaded or
> > reloaded.
> >
>
> Thanks, Fuad will be addressing this in a revision of [1].
>
> > BTW, there're lots of symbols not exported under mm.
> >
>
> Thanks again, is there a good way to do a build test for symbols not
> being exported? What CONFIG flags do you use?
I compiled kvm.ko and kvm-intel.ko as modules.
CONFIG_KVM=m
CONFIG_KVM_INTEL=m
CONFIG_KVM_INTEL_TDX=y
