On Wed, May 18, 2022, Oliver Upton wrote: > Initialize the field alongside the other struct kvm fields. No > functional change intended. > > Signed-off-by: Oliver Upton <oupton@xxxxxxxxxx> > --- > virt/kvm/kvm_main.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > index 6d971fb1b08d..36dc9271d039 100644 > --- a/virt/kvm/kvm_main.c > +++ b/virt/kvm/kvm_main.c > @@ -1101,6 +1101,9 @@ static struct kvm *kvm_create_vm(unsigned long type) > */ > kvm->debugfs_dentry = ERR_PTR(-ENOENT); > > + snprintf(kvm->stats_id, sizeof(kvm->stats_id), > + "kvm-%d", task_pid_nr(current)); After looking at the next patch, can you opportunistically tweak this to (a) have the string on the first line, and (b) align indentation? I.e. snprintf(kvm->stats_id, sizeof(kvm->stats_id), "kvm-%d", task_pid_nr(current)); That makes it a lot easier to see what the string will look like.
