linux.git/drivers/edac/debugfs.c, branch v6.12.80

ARM: 8892/1: EDAC: Add missing debugfs_create_x32 wrapper

2019-08-29T06:58:01+00:00

We already have wrappers for x8 and x16, so add the missing x32 one.

Signed-off-by: Jan Luebbe 
Reviewed-by: Borislav Petkov 
Signed-off-by: Chris Packham 
Signed-off-by: Russell King

EDAC: Make edac_debugfs_create_x*() return void

2019-06-20T18:44:36+00:00

The return values of edac_debugfs_create_x16() and
edac_debugfs_create_x8() are never checked (as they don't need to be),
so no need to have them return anything, just make the functions return
void instead.

This is done with the goal of being able to change the debugfs_create_x*
functions to also not return a value.

Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Borislav Petkov 
Cc: James Morse 
Cc: linux-edac 
Cc: Mauro Carvalho Chehab 
Link: https://lkml.kernel.org/r/20190611175433.GA5108@kroah.com

treewide: Add SPDX license identifier for missed files

2019-05-21T08:50:45+00:00

Add SPDX license identifiers to all files which:

 - Have no license information of any form

 - Have EXPORT_.*_SYMBOL_GPL inside which was used in the
   initial scan/conversion to ignore the file

These files fall under the project license, GPL v2 only. The resulting SPDX
license identifier is:

  GPL-2.0-only

Signed-off-by: Thomas Gleixner 
Signed-off-by: Greg Kroah-Hartman

EDAC: Do not check return value of debugfs_create() functions

2019-01-23T09:41:18+00:00

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.

 [ bp: Make edac_debugfs_init() return void too, while at it. ]

Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Borislav Petkov 
Cc: James Morse 
Cc: Mauro Carvalho Chehab 
Cc: linux-edac 
Link: https://lkml.kernel.org/r/20190122152151.16139-17-gregkh@linuxfoundation.org

EDAC: Use edac_debugfs_remove_recursive() in edac_debugfs_exit()

2016-02-10T09:37:46+00:00

debugfs_remove() is used to remove a file or a directory from the
debugfs filesystem on an EDAC device exit. However edac_debugfs might
not be empty. This is similar to

  30f84a891bf6 ("EDAC: Use edac_debugfs_remove_recursive()")

which changed the EDAC MCI code to use edac_debugfs_remove_recursive().

Suggested-by: Borislav Petkov 
Signed-off-by: Thor Thayer 
Cc: linux-edac 
Link: http://lkml.kernel.org/r/1455064165-3816-1-git-send-email-tthayer@opensource.altera.com
Signed-off-by: Borislav Petkov

EDAC: Use edac_debugfs_remove_recursive()

2015-10-14T16:50:32+00:00

debugfs_remove() is used to remove a file or a directory from the
debugfs filesystem, but mci->debugfs might not empty.

This can be triggered by the following sequence:

1) Enable CONFIG_EDAC_DEBUG
2) insmod an EDAC module (like i3000_edac or similar)
3) rmmod this module
4) we can see files remaining under /edac/ like
   "fake_inject", for example.

Removing edac_core then, causes a NULL pointer dereference.

Reported-by: Yun Wu (Abel) 
Signed-off-by: Tan Xiaojun 
Cc: Doug Thompson 
Cc: Mauro Carvalho Chehab 
Cc: linux-edac 
Link: http://lkml.kernel.org/r/1444787364-104353-1-git-send-email-tanxiaojun@huawei.com
Signed-off-by: Borislav Petkov

EDAC: Add debugfs wrappers

2015-09-22T16:10:22+00:00

Later patches will convert EDAC users to those.

Signed-off-by: Borislav Petkov

EDAC: Carve out debugfs functionality

2015-09-22T10:29:46+00:00

... into a separate compilation unit and drop a couple of
CONFIG_EDAC_DEBUG ifdefferies. Rename edac_create_debug_nodes() to
edac_create_debugfs_nodes(), while at it.

No functionality change.

Cc: 
Signed-off-by: Borislav Petkov