diff options
| author | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-01-13 11:28:06 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-01-13 11:28:06 +0100 |
| commit | b74a449f1a312d7d30c4738ead06c5751e5160f4 (patch) | |
| tree | c364049eca1c6690bfac8197fe0f4c326a241ff0 /p09 | |
| parent | e89b7388846e2b8aa288676365671f7ab9b7cee7 (diff) | |
| download | patches-b74a449f1a312d7d30c4738ead06c5751e5160f4.tar.gz | |
updated patches
Diffstat (limited to 'p09')
| -rw-r--r-- | p09 | 59 |
1 files changed, 59 insertions, 0 deletions
diff --git a/p09 b/p09 new file mode 100644 index 00000000000000..69ca9d73f7e270 --- /dev/null +++ b/p09 @@ -0,0 +1,59 @@ +From e82f2d8eab210dc7eb0dba2bd8e3c64ad485442b Mon Sep 17 00:00:00 2001 +From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> +Date: Wed, 30 May 2018 16:15:35 +0200 +Subject: [PATCH] x86: kvm: no need to check return value of debugfs_create functions + +When calling debugfs functions, there is no need to ever check the +return value. The function can work or not, but the code logic should +never do something different based on this. + +Cc: Paolo Bonzini <pbonzini@redhat.com> +Cc: "Radim Krčmář" <rkrcmar@redhat.com> +Cc: Thomas Gleixner <tglx@linutronix.de> +Cc: Ingo Molnar <mingo@redhat.com> +Cc: Borislav Petkov <bp@alien8.de> +Cc: "H. Peter Anvin" <hpa@zytor.com> +Cc: <x86@kernel.org> +Cc: <kvm@vger.kernel.org> +Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> +--- + arch/x86/kvm/debugfs.c | 26 ++++++++------------------ + 1 file changed, 8 insertions(+), 18 deletions(-) + +--- a/arch/x86/kvm/debugfs.c ++++ b/arch/x86/kvm/debugfs.c +@@ -43,26 +43,16 @@ DEFINE_SIMPLE_ATTRIBUTE(vcpu_tsc_scaling + + int kvm_arch_create_vcpu_debugfs(struct kvm_vcpu *vcpu) + { +- struct dentry *ret; +- +- ret = debugfs_create_file("tsc-offset", 0444, +- vcpu->debugfs_dentry, +- vcpu, &vcpu_tsc_offset_fops); +- if (!ret) +- return -ENOMEM; ++ debugfs_create_file("tsc-offset", 0444, vcpu->debugfs_dentry, vcpu, ++ &vcpu_tsc_offset_fops); + + if (kvm_has_tsc_control) { +- ret = debugfs_create_file("tsc-scaling-ratio", 0444, +- vcpu->debugfs_dentry, +- vcpu, &vcpu_tsc_scaling_fops); +- if (!ret) +- return -ENOMEM; +- ret = debugfs_create_file("tsc-scaling-ratio-frac-bits", 0444, +- vcpu->debugfs_dentry, +- vcpu, &vcpu_tsc_scaling_frac_fops); +- if (!ret) +- return -ENOMEM; +- ++ debugfs_create_file("tsc-scaling-ratio", 0444, ++ vcpu->debugfs_dentry, vcpu, ++ &vcpu_tsc_scaling_fops); ++ debugfs_create_file("tsc-scaling-ratio-frac-bits", 0444, ++ vcpu->debugfs_dentry, vcpu, ++ &vcpu_tsc_scaling_frac_fops); + } + + return 0; |