On Tue, May 18, 2021 at 06:16:25PM +0200, Greg Kroah-Hartman wrote: > There is no need to keep the dentry around for the debugfs trace files, > as we can just look it up when we want to remove it later on. Simplify > the structure by removing the dentries and relying on debugfs to find > the dentry to remove when we want to. > > By doing this change, we remove the last in-kernel user that was storing > the result of debugfs_create_bool(), so that api can be cleaned up. > > Cc: Karan Tilak Kumar <kartilak@xxxxxxxxx> > Cc: Sesidhar Baddela <sebaddel@xxxxxxxxx> > Cc: "James E.J. Bottomley" <jejb@xxxxxxxxxxxxx> > Cc: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx> > Cc: linux-scsi@xxxxxxxxxxxxxxx > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > --- > drivers/scsi/snic/snic_debugfs.c | 23 +++++++++-------------- > drivers/scsi/snic/snic_trc.h | 3 --- > 2 files changed, 9 insertions(+), 17 deletions(-) I can take this through my debugfs tree so I can clean up the debugfs_create_bool() api if no one objects. thanks, greg k-h
