diff options
Diffstat (limited to 'p12')
| -rw-r--r-- | p12 | 53 |
1 files changed, 53 insertions, 0 deletions
diff --git a/p12 b/p12 new file mode 100644 index 00000000000000..e23529bb75119c --- /dev/null +++ b/p12 @@ -0,0 +1,53 @@ +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: xen: 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: Boris Ostrovsky <boris.ostrovsky@oracle.com> +Cc: Juergen Gross <jgross@suse.com> +Cc: Stefano Stabellini <sstabellini@kernel.org> +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: <xen-devel@lists.xenproject.org> +Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> +--- + arch/x86/xen/debugfs.c | 7 +------ + arch/x86/xen/p2m.c | 3 --- + 2 files changed, 1 insertion(+), 9 deletions(-) + +--- a/arch/x86/xen/debugfs.c ++++ b/arch/x86/xen/debugfs.c +@@ -9,13 +9,8 @@ static struct dentry *d_xen_debug; + + struct dentry * __init xen_init_debugfs(void) + { +- if (!d_xen_debug) { ++ if (!d_xen_debug) + d_xen_debug = debugfs_create_dir("xen", NULL); +- +- if (!d_xen_debug) +- pr_warning("Could not create 'xen' debugfs directory\n"); +- } +- + return d_xen_debug; + } + +--- a/arch/x86/xen/p2m.c ++++ b/arch/x86/xen/p2m.c +@@ -810,9 +810,6 @@ static int __init xen_p2m_debugfs(void) + { + struct dentry *d_xen = xen_init_debugfs(); + +- if (d_xen == NULL) +- return -ENOMEM; +- + d_mmu_debug = debugfs_create_dir("mmu", d_xen); + + debugfs_create_file("p2m", 0600, d_mmu_debug, NULL, &p2m_dump_fops); |