| Message ID | 1430811548-34140-1-git-send-email-heiko.carstens@de.ibm.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Am 05.05.2015 um 09:39 schrieb Heiko Carstens: > On cpu hotplug only KVM emits an unconditional message that its notifier > has been called. It certainly can be assumed that calling cpu hotplug > notifiers work, therefore there is no added value if KVM prints a message. > > If an error happens on cpu online KVM will still emit a warning. > > So let's remove this superfluous message. > > Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com> Its really the only code that I know of that is chatty on cpu hotplug. Acked-by: Christian Borntraeger <borntraeger@de.ibm.com> > --- > virt/kvm/kvm_main.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > index 90977418aeb6..5fb52af5b975 100644 > --- a/virt/kvm/kvm_main.c > +++ b/virt/kvm/kvm_main.c > @@ -2882,18 +2882,12 @@ static int hardware_enable_all(void) > static int kvm_cpu_hotplug(struct notifier_block *notifier, unsigned long val, > void *v) > { > - int cpu = (long)v; > - > val &= ~CPU_TASKS_FROZEN; > switch (val) { > case CPU_DYING: > - pr_info("kvm: disabling virtualization on CPU%d\n", > - cpu); > hardware_disable(); > break; > case CPU_STARTING: > - pr_info("kvm: enabling virtualization on CPU%d\n", > - cpu); > hardware_enable(); > break; > } > -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 05/05/2015 09:39, Heiko Carstens wrote: > On cpu hotplug only KVM emits an unconditional message that its notifier > has been called. It certainly can be assumed that calling cpu hotplug > notifiers work, therefore there is no added value if KVM prints a message. > > If an error happens on cpu online KVM will still emit a warning. > > So let's remove this superfluous message. > > Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com> > --- > virt/kvm/kvm_main.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > index 90977418aeb6..5fb52af5b975 100644 > --- a/virt/kvm/kvm_main.c > +++ b/virt/kvm/kvm_main.c > @@ -2882,18 +2882,12 @@ static int hardware_enable_all(void) > static int kvm_cpu_hotplug(struct notifier_block *notifier, unsigned long val, > void *v) > { > - int cpu = (long)v; > - > val &= ~CPU_TASKS_FROZEN; > switch (val) { > case CPU_DYING: > - pr_info("kvm: disabling virtualization on CPU%d\n", > - cpu); > hardware_disable(); > break; > case CPU_STARTING: > - pr_info("kvm: enabling virtualization on CPU%d\n", > - cpu); > hardware_enable(); > break; > } > Applied, thanks. Paolo -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index 90977418aeb6..5fb52af5b975 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm/kvm_main.c @@ -2882,18 +2882,12 @@ static int hardware_enable_all(void) static int kvm_cpu_hotplug(struct notifier_block *notifier, unsigned long val, void *v) { - int cpu = (long)v; - val &= ~CPU_TASKS_FROZEN; switch (val) { case CPU_DYING: - pr_info("kvm: disabling virtualization on CPU%d\n", - cpu); hardware_disable(); break; case CPU_STARTING: - pr_info("kvm: enabling virtualization on CPU%d\n", - cpu); hardware_enable(); break; }
On cpu hotplug only KVM emits an unconditional message that its notifier has been called. It certainly can be assumed that calling cpu hotplug notifiers work, therefore there is no added value if KVM prints a message. If an error happens on cpu online KVM will still emit a warning. So let's remove this superfluous message. Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com> --- virt/kvm/kvm_main.c | 6 ------ 1 file changed, 6 deletions(-)