* [PATCH] kvm: remove invalid check for debugfs_create_dir() @ 2019-06-12 14:50 Greg Kroah-Hartman 2019-06-12 15:40 ` Sean Christopherson 0 siblings, 1 reply; 4+ messages in thread From: Greg Kroah-Hartman @ 2019-06-12 14:50 UTC (permalink / raw) To: Paolo Bonzini, Radim Krčmář; +Cc: kvm debugfs_create_dir() can never return NULL, so no need to check for an impossible thing. It's also not needed to ever check the return value of this function, so just remove the check entirely, and indent the previous line to a sane formatting :) Cc: Paolo Bonzini <pbonzini@redhat.com> Cc: "Radim Krčmář" <rkrcmar@redhat.com> Cc: kvm@vger.kernel.org Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> --- virt/kvm/kvm_main.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index ca54b09adf5b..4b4ef642d8fa 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm/kvm_main.c @@ -2605,9 +2605,7 @@ static int kvm_create_vcpu_debugfs(struct kvm_vcpu *vcpu) snprintf(dir_name, sizeof(dir_name), "vcpu%d", vcpu->vcpu_id); vcpu->debugfs_dentry = debugfs_create_dir(dir_name, - vcpu->kvm->debugfs_dentry); - if (!vcpu->debugfs_dentry) - return -ENOMEM; + vcpu->kvm->debugfs_dentry); ret = kvm_arch_create_vcpu_debugfs(vcpu); if (ret < 0) { -- 2.22.0 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] kvm: remove invalid check for debugfs_create_dir() 2019-06-12 14:50 [PATCH] kvm: remove invalid check for debugfs_create_dir() Greg Kroah-Hartman @ 2019-06-12 15:40 ` Sean Christopherson 2019-06-12 15:46 ` Greg Kroah-Hartman 0 siblings, 1 reply; 4+ messages in thread From: Sean Christopherson @ 2019-06-12 15:40 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: Paolo Bonzini, Radim Krčmář, kvm On Wed, Jun 12, 2019 at 04:50:33PM +0200, Greg Kroah-Hartman wrote: > debugfs_create_dir() can never return NULL, so no need to check for an > impossible thing. > > It's also not needed to ever check the return value of this function, so > just remove the check entirely, and indent the previous line to a sane > formatting :) > > Cc: Paolo Bonzini <pbonzini@redhat.com> > Cc: "Radim Krčmář" <rkrcmar@redhat.com> > Cc: kvm@vger.kernel.org > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > --- > virt/kvm/kvm_main.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > index ca54b09adf5b..4b4ef642d8fa 100644 > --- a/virt/kvm/kvm_main.c > +++ b/virt/kvm/kvm_main.c > @@ -2605,9 +2605,7 @@ static int kvm_create_vcpu_debugfs(struct kvm_vcpu *vcpu) > > snprintf(dir_name, sizeof(dir_name), "vcpu%d", vcpu->vcpu_id); > vcpu->debugfs_dentry = debugfs_create_dir(dir_name, > - vcpu->kvm->debugfs_dentry); > - if (!vcpu->debugfs_dentry) > - return -ENOMEM; > + vcpu->kvm->debugfs_dentry); > > ret = kvm_arch_create_vcpu_debugfs(vcpu); > if (ret < 0) { > -- > 2.22.0 Any objection to me pulling this into a series to clean up similar issues in arch/x86/kvm/debugfs.c -> kvm_arch_create_vcpu_debugfs(), and to change kvm_create_vcpu_debugfs() to not return success/failure? It'd be nice to fix everything in a single shot. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] kvm: remove invalid check for debugfs_create_dir() 2019-06-12 15:40 ` Sean Christopherson @ 2019-06-12 15:46 ` Greg Kroah-Hartman 2019-06-12 15:55 ` Sean Christopherson 0 siblings, 1 reply; 4+ messages in thread From: Greg Kroah-Hartman @ 2019-06-12 15:46 UTC (permalink / raw) To: Sean Christopherson; +Cc: Paolo Bonzini, Radim Krčmář, kvm On Wed, Jun 12, 2019 at 08:40:21AM -0700, Sean Christopherson wrote: > On Wed, Jun 12, 2019 at 04:50:33PM +0200, Greg Kroah-Hartman wrote: > > debugfs_create_dir() can never return NULL, so no need to check for an > > impossible thing. > > > > It's also not needed to ever check the return value of this function, so > > just remove the check entirely, and indent the previous line to a sane > > formatting :) > > > > Cc: Paolo Bonzini <pbonzini@redhat.com> > > Cc: "Radim Krčmář" <rkrcmar@redhat.com> > > Cc: kvm@vger.kernel.org > > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > --- > > virt/kvm/kvm_main.c | 4 +--- > > 1 file changed, 1 insertion(+), 3 deletions(-) > > > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > > index ca54b09adf5b..4b4ef642d8fa 100644 > > --- a/virt/kvm/kvm_main.c > > +++ b/virt/kvm/kvm_main.c > > @@ -2605,9 +2605,7 @@ static int kvm_create_vcpu_debugfs(struct kvm_vcpu *vcpu) > > > > snprintf(dir_name, sizeof(dir_name), "vcpu%d", vcpu->vcpu_id); > > vcpu->debugfs_dentry = debugfs_create_dir(dir_name, > > - vcpu->kvm->debugfs_dentry); > > - if (!vcpu->debugfs_dentry) > > - return -ENOMEM; > > + vcpu->kvm->debugfs_dentry); > > > > ret = kvm_arch_create_vcpu_debugfs(vcpu); > > if (ret < 0) { > > -- > > 2.22.0 > > Any objection to me pulling this into a series to clean up similar issues > in arch/x86/kvm/debugfs.c -> kvm_arch_create_vcpu_debugfs(), and to > change kvm_create_vcpu_debugfs() to not return success/failure? It'd be > nice to fix everything in a single shot. It was on my todo list to do the cleanup of the x86 kvm stuff. I figured this arch-neutral fix was independent, but if you want me to do it all at once, I'll be glad to do so later this week. thanks, greg k-h ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] kvm: remove invalid check for debugfs_create_dir() 2019-06-12 15:46 ` Greg Kroah-Hartman @ 2019-06-12 15:55 ` Sean Christopherson 0 siblings, 0 replies; 4+ messages in thread From: Sean Christopherson @ 2019-06-12 15:55 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: Paolo Bonzini, Radim Krčmář, kvm On Wed, Jun 12, 2019 at 05:46:22PM +0200, Greg Kroah-Hartman wrote: > On Wed, Jun 12, 2019 at 08:40:21AM -0700, Sean Christopherson wrote: > > On Wed, Jun 12, 2019 at 04:50:33PM +0200, Greg Kroah-Hartman wrote: > > > debugfs_create_dir() can never return NULL, so no need to check for an > > > impossible thing. > > > > > > It's also not needed to ever check the return value of this function, so > > > just remove the check entirely, and indent the previous line to a sane > > > formatting :) > > > > > > Cc: Paolo Bonzini <pbonzini@redhat.com> > > > Cc: "Radim Krčmář" <rkrcmar@redhat.com> > > > Cc: kvm@vger.kernel.org > > > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > > --- > > > virt/kvm/kvm_main.c | 4 +--- > > > 1 file changed, 1 insertion(+), 3 deletions(-) > > > > > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > > > index ca54b09adf5b..4b4ef642d8fa 100644 > > > --- a/virt/kvm/kvm_main.c > > > +++ b/virt/kvm/kvm_main.c > > > @@ -2605,9 +2605,7 @@ static int kvm_create_vcpu_debugfs(struct kvm_vcpu *vcpu) > > > > > > snprintf(dir_name, sizeof(dir_name), "vcpu%d", vcpu->vcpu_id); > > > vcpu->debugfs_dentry = debugfs_create_dir(dir_name, > > > - vcpu->kvm->debugfs_dentry); > > > - if (!vcpu->debugfs_dentry) > > > - return -ENOMEM; > > > + vcpu->kvm->debugfs_dentry); > > > > > > ret = kvm_arch_create_vcpu_debugfs(vcpu); > > > if (ret < 0) { > > > -- > > > 2.22.0 > > > > Any objection to me pulling this into a series to clean up similar issues > > in arch/x86/kvm/debugfs.c -> kvm_arch_create_vcpu_debugfs(), and to > > change kvm_create_vcpu_debugfs() to not return success/failure? It'd be > > nice to fix everything in a single shot. > > It was on my todo list to do the cleanup of the x86 kvm stuff. I > figured this arch-neutral fix was independent, but if you want me to do > it all at once, I'll be glad to do so later this week. My preference would be to get it done all at once, having a discrepancy between kvm_main and x86 leads to a bit of head scratching. The x86 change would be independent, the piece that's not is converting kvm_create_vcpu_debugfs() to handle failure internally and not return errors, which would depend on this patch's removal of "return -ENOMEM". At a minimum I think that change should be paired with this fix. ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-06-12 15:55 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-06-12 14:50 [PATCH] kvm: remove invalid check for debugfs_create_dir() Greg Kroah-Hartman 2019-06-12 15:40 ` Sean Christopherson 2019-06-12 15:46 ` Greg Kroah-Hartman 2019-06-12 15:55 ` Sean Christopherson
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.