On Wed, Oct 30, 2024, Sean Christopherson wrote: > On Fri, Sep 06, 2024, Vipin Sharma wrote: > > + struct list_head tdp_mmu_possible_nx_huge_pages; > > + u64 tdp_mmu_nr_possible_nx_huge_pages; > > These obviously come in a pair, and must be passed around as such. To make the > relevant code easier on the eyes (long lines), and to avoid passing a mismatched > pair, add a parent structure. > > E.g. > > struct kvm_possible_nx_huge_pages { > struct list_head list; Actually, I vote for s/list/pages, as that makes the usage in code is more intuitive. > u64 nr_pages; > }