On Thu, Dec 07, 2017 at 04:50:52PM +0100, David Hildenbrand wrote: > On 07.12.2017 15:18, Roman Kagan wrote: > > Move kvm.arch.hyperv initialization and cleanup to separate functions. > > > > For now only a mutex is inited in the former, and the latter is empty; > > more stuff will go in there in a followup patch. > > > > Signed-off-by: Roman Kagan <rkagan@xxxxxxxxxxxxx> > > --- > > v1 -> v2: > > - fix subject prefix > > should be "kvm: x86: " ;) > > > @@ -8158,6 +8157,8 @@ int kvm_arch_init_vm(struct kvm *kvm, unsigned long type) > > INIT_DELAYED_WORK(&kvm->arch.kvmclock_update_work, kvmclock_update_fn); > > INIT_DELAYED_WORK(&kvm->arch.kvmclock_sync_work, kvmclock_sync_fn); > > > > + kvm_hv_init_vm(kvm); > > + > > wonder if that empty line is really needed. Compensating for the missing space in the subject prefix ;) I'll kill it when respinning (to take into account your comments to patch 2/2). Thanks, Roman.
