Re: [PATCH v4 2/6] KVM: x86: introduce memslot_set_lpage_disallowed
On 05/03/2013 10:10 AM, Takuya Yoshikawa wrote: > On Sat, 27 Apr 2013 11:13:18 +0800 > Xiao Guangrong wrote: > >> It is used to set disallowed large page on the specified level, can be >> used in later patch >> >> Signed-off-by: Xiao Guangrong >> --- >> arch/x86/kvm/x86.c | 53 >> ++- >> 1 files changed, 35 insertions(+), 18 deletions(-) >> >> diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c >> index 91dd9f4..52b4e97 100644 >> --- a/arch/x86/kvm/x86.c >> +++ b/arch/x86/kvm/x86.c >> @@ -6917,12 +6917,45 @@ void kvm_arch_free_memslot(struct kvm_memory_slot >> *free, >> } >> } >> >> +static void memslot_set_lpage_disallowed(struct kvm_memory_slot *slot, >> + unsigned long npages, >> + int lpage_size, int lpages) > > What this function does is to disable large page support for this slot > as can be seen in the comment below. > > Since setting lpage_info to something ("disallowed" ?) is an implementation > detail, we'd better hide such a thing from the function name. > > Taking into account that we have "kvm_largepages_enabled()", something like > disable_largepages_memslot() may be a candidate. > No. kvm_largepages_enabled effects on largepages_enabled, it is not related with this function. Actually, I really do not care the different between "disallowed" and "disabled". -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v4 2/6] KVM: x86: introduce memslot_set_lpage_disallowed
On Sat, 27 Apr 2013 11:13:18 +0800 Xiao Guangrong wrote: > It is used to set disallowed large page on the specified level, can be > used in later patch > > Signed-off-by: Xiao Guangrong > --- > arch/x86/kvm/x86.c | 53 ++- > 1 files changed, 35 insertions(+), 18 deletions(-) > > diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c > index 91dd9f4..52b4e97 100644 > --- a/arch/x86/kvm/x86.c > +++ b/arch/x86/kvm/x86.c > @@ -6917,12 +6917,45 @@ void kvm_arch_free_memslot(struct kvm_memory_slot > *free, > } > } > > +static void memslot_set_lpage_disallowed(struct kvm_memory_slot *slot, > + unsigned long npages, > + int lpage_size, int lpages) What this function does is to disable large page support for this slot as can be seen in the comment below. Since setting lpage_info to something ("disallowed" ?) is an implementation detail, we'd better hide such a thing from the function name. Taking into account that we have "kvm_largepages_enabled()", something like disable_largepages_memslot() may be a candidate. But I want to see suggestions from others as well. Takuya -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v4 2/6] KVM: x86: introduce memslot_set_lpage_disallowed
On Sat, 27 Apr 2013 11:13:18 +0800 Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com wrote: It is used to set disallowed large page on the specified level, can be used in later patch Signed-off-by: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com --- arch/x86/kvm/x86.c | 53 ++- 1 files changed, 35 insertions(+), 18 deletions(-) diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 91dd9f4..52b4e97 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c @@ -6917,12 +6917,45 @@ void kvm_arch_free_memslot(struct kvm_memory_slot *free, } } +static void memslot_set_lpage_disallowed(struct kvm_memory_slot *slot, + unsigned long npages, + int lpage_size, int lpages) What this function does is to disable large page support for this slot as can be seen in the comment below. Since setting lpage_info to something (disallowed ?) is an implementation detail, we'd better hide such a thing from the function name. Taking into account that we have kvm_largepages_enabled(), something like disable_largepages_memslot() may be a candidate. But I want to see suggestions from others as well. Takuya -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v4 2/6] KVM: x86: introduce memslot_set_lpage_disallowed
On 05/03/2013 10:10 AM, Takuya Yoshikawa wrote: On Sat, 27 Apr 2013 11:13:18 +0800 Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com wrote: It is used to set disallowed large page on the specified level, can be used in later patch Signed-off-by: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com --- arch/x86/kvm/x86.c | 53 ++- 1 files changed, 35 insertions(+), 18 deletions(-) diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 91dd9f4..52b4e97 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c @@ -6917,12 +6917,45 @@ void kvm_arch_free_memslot(struct kvm_memory_slot *free, } } +static void memslot_set_lpage_disallowed(struct kvm_memory_slot *slot, + unsigned long npages, + int lpage_size, int lpages) What this function does is to disable large page support for this slot as can be seen in the comment below. Since setting lpage_info to something (disallowed ?) is an implementation detail, we'd better hide such a thing from the function name. Taking into account that we have kvm_largepages_enabled(), something like disable_largepages_memslot() may be a candidate. No. kvm_largepages_enabled effects on largepages_enabled, it is not related with this function. Actually, I really do not care the different between disallowed and disabled. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/