kernel/linux.git/drivers/staging/lustre, branch master

staging: lustre: delete the filesystem from the tree.

2018-06-05T17:22:35+00:00

The Lustre filesystem has been in the kernel tree for over 5 years now. While it has been an endless source of enjoyment for new kernel developers learning how to do basic codingstyle cleanups, as well as an semi-entertaining source of bewilderment from the vfs developers any time they have looked into the codebase to try to figure out how to port their latest api changes to this filesystem, it has not really moved forward into the "this is in shape to get out of staging" despite many half-completed attempts. And getting code out of staging is the main goal of that portion of the kernel tree. Code should not stagnate and it feels like having this code in staging is only causing the development cycle of the filesystem to take longer than it should. There is a whole separate out-of-tree copy of this codebase where the developers work on it, and then random changes are thrown over the wall at staging at some later point in time. This dual-tree development model has never worked, and the state of this codebase is proof of that. So, let's just delete the whole mess. Now the lustre developers can go off and work in their out-of-tree codebase and not have to worry about providing valid changelog entries and breaking their patches up into logical pieces. They can take the time they have spend doing those types of housekeeping chores and get the codebase into a much better shape, and it can be submitted for inclusion into the real part of the kernel tree when ready. Cc: Oleg Drokin Cc: Andreas Dilger Cc: James Simmons Signed-off-by: Greg Kroah-Hartman

staging: lustre: get rid of ldebugfs_remove()

2018-06-01T08:47:44+00:00

It was just a dumb wrapper around debugfs_remove_recursive() so just call the function properly. Also, there is no need to set the dentry to NULL, it's gone, who cares about it anymore... Cc: Oleg Drokin Cc: Andreas Dilger Cc: James Simmons Cc: NeilBrown Cc: Ingo Molnar Cc: Roman Storozhenko Cc: Aastha Gupta Cc: Ben Evans Cc: Quentin Bouget Cc: Arvind Yadav Cc: Arushi Singhal Cc: Patrick Farrell Cc: Aliaksei Karaliou Cc: Mathias Rav Cc: Andriy Skulysh Cc: Dafna Hirschfeld Cc: "Gustavo A. R. Silva" Cc: Bob Glosman Cc: lustre-devel@lists.lustre.org Signed-off-by: Greg Kroah-Hartman

staging: lustre: make ldebugfs_add_vars a void function

2018-06-01T08:47:44+00:00

The call to ldebugfs_add_vars() can not really fail, so have it just return nothing, which allows us to clean up a lot of unused error handling code. Cc: Oleg Drokin Cc: Andreas Dilger Cc: James Simmons Cc: NeilBrown Cc: Roman Storozhenko Cc: Arnd Bergmann Cc: Quentin Bouget Cc: Aastha Gupta Cc: Ben Evans Cc: Arvind Yadav Cc: Arushi Singhal Cc: Frank Zago Cc: Patrick Farrell Cc: Simo Koskinen Cc: Andriy Skulysh Cc: "John L. Hammond" Cc: Mathias Rav Cc: Dafna Hirschfeld Cc: "Gustavo A. R. Silva" Cc: lustre-devel@lists.lustre.org Signed-off-by: Greg Kroah-Hartman

staging: lustre: remove last two users of ldebugfs_register()

2018-06-01T08:47:44+00:00

ldebugfs_register() is just a call to debugfs_create_dir() and ldebugfs_add_vars() if the list option is set. Fix up the last two users of this function to just call these two functions instead, and delete the now unused ldebugfs_register() call. This ends up cleaning up more code and making things smaller, always a good thing. Cc: Oleg Drokin Cc: Andreas Dilger Cc: James Simmons Cc: NeilBrown Cc: Ben Evans Cc: Quentin Bouget Cc: Arvind Yadav Cc: Arushi Singhal Cc: Julia Lawall Cc: Mathias Rav Cc: Dafna Hirschfeld Cc: "Gustavo A. R. Silva" Cc: Patrick Farrell Cc: Nadav Amit Cc: lustre-devel@lists.lustre.org Signed-off-by: Greg Kroah-Hartman

staging: lustre: unwrap some ldebugfs_register() calls

2018-06-01T08:47:44+00:00

When the third option (list) to ldebugfs_register() is NULL, it's the same as just calling debugfs_create_dir(). So unwind this and call debugfs_create_dir() directly. This ends up saving lots of code as we do not need to do any error checking of the return value (because it does not matter). The ldebugfs_register() call will be removed in a later patch when it is fully removed, right now there are 2 outstanding users of it in the tree. Cc: Oleg Drokin Cc: Andreas Dilger Cc: James Simmons Cc: NeilBrown Cc: Aastha Gupta Cc: Arnd Bergmann Cc: Ingo Molnar Cc: Quentin Bouget Cc: Patrick Farrell Cc: Aliaksei Karaliou Cc: "John L. Hammond" Cc: Mathias Rav Cc: Andriy Skulysh Cc: Ben Evans Cc: Bob Glosman Cc: lustre-devel@lists.lustre.org Signed-off-by: Greg Kroah-Hartman

staging: lustre: remove ldebugfs_obd_seq_create() wrapper function

2018-06-01T08:47:44+00:00

It was just calling debugfs_create_file() so unwind things and just call the real function instead. This ends up saving a number of lines as there was never any error handling happening anyway, so that all can be removed as well. Cc: Oleg Drokin Cc: Andreas Dilger Cc: James Simmons Cc: Quentin Bouget Cc: Ben Evans Cc: NeilBrown Cc: Arushi Singhal Cc: Arvind Yadav Cc: Julia Lawall Cc: Dafna Hirschfeld Cc: Mathias Rav Cc: "Gustavo A. R. Silva" Cc: Roman Storozhenko Cc: lustre-devel@lists.lustre.org Signed-off-by: Greg Kroah-Hartman

staging: lustre: remove ldebugfs_seq_create() wrapper function

2018-06-01T08:47:44+00:00

It was just calling debugfs_create_file() so unwind things and just call the real function instead. This ends up saving a number of lines as there was never any error handling happening anyway, so that all can be removed as well. Cc: Oleg Drokin Cc: Andreas Dilger Cc: James Simmons Cc: Quentin Bouget Cc: NeilBrown Cc: Ben Evans Cc: Arushi Singhal Cc: Arvind Yadav Cc: "John L. Hammond" Cc: Vitaly Fertman Cc: Julia Lawall Cc: Dafna Hirschfeld Cc: Mathias Rav Cc: "Gustavo A. R. Silva" Cc: Bob Glosman Cc: lustre-devel@lists.lustre.org Signed-off-by: Greg Kroah-Hartman

staging: lustre: remove ldebugfs_register_stats() wrapper function

2018-06-01T08:47:44+00:00

It was just calling debugfs_create_file() so unwind things and just call the real function instead. This ends up saving a number of lines as there was never any error handling happening anyway, so that all can be removed as well. Cc: Oleg Drokin Cc: Andreas Dilger Cc: James Simmons Cc: Ben Evans Cc: Quentin Bouget Cc: NeilBrown Cc: Arushi Singhal Cc: Arvind Yadav Cc: Patrick Farrell Cc: Aliaksei Karaliou Cc: Aastha Gupta Cc: Dafna Hirschfeld Cc: Mathias Rav Cc: "Gustavo A. R. Silva" Cc: Bob Glosman Cc: lustre-devel@lists.lustre.org Signed-off-by: Greg Kroah-Hartman

staging: lustre: remove ldebugfs_add_simple() wrapper

2018-06-01T08:47:43+00:00

It was only being called in one place, and is an unneeded wrapper function around debugfs_create_file() so just call the real debugfs function instead. This ends up cleaning up some unneeded error handling logic that was never needed as well. Cc: Oleg Drokin Cc: Andreas Dilger Cc: James Simmons Cc: Quentin Bouget Cc: Ben Evans Cc: NeilBrown Cc: Arvind Yadav Cc: Arushi Singhal Cc: Dafna Hirschfeld Cc: Mathias Rav Cc: "Gustavo A. R. Silva" Cc: lustre-devel@lists.lustre.org Signed-off-by: Greg Kroah-Hartman

staging: lustre: no need to check debugfs return values

2018-06-01T08:47:43+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. Clean up the lustre core code by not caring about the value of debugfs calls. This ends up removing a number of lines of code that are not needed. Note, more work is needed to remove the unneeded debugfs wrapper functions in the future. Cc: Oleg Drokin Cc: Andreas Dilger Cc: James Simmons Cc: NeilBrown Cc: Ben Evans Cc: Arvind Yadav Cc: "John L. Hammond" Cc: Dafna Hirschfeld Cc: Mathias Rav Cc: "Gustavo A. R. Silva" Cc: lustre-devel@lists.lustre.org Signed-off-by: Greg Kroah-Hartman