Re: [PATCH] btrfs: remove pointless debugfs interface

2016-09-01 Thread David Sterba
On Wed, Aug 31, 2016 at 06:36:34PM -0400, Jeff Mahoney wrote: > On 8/31/16 3:08 PM, David Sterba wrote: > > On Wed, Aug 31, 2016 at 10:13:49AM -0500, Eric Sandeen wrote: > >> A /sys/kernel/debug/btrfs/test file was added nearly > >> two and a half years ago, but it serves no purpose; > > > > It

Re: [PATCH] btrfs: remove pointless debugfs interface

2016-09-01 Thread David Sterba
On Wed, Aug 31, 2016 at 04:38:16PM -0500, Eric Sandeen wrote: > On 8/31/16 2:08 PM, David Sterba wrote: > > On Wed, Aug 31, 2016 at 10:13:49AM -0500, Eric Sandeen wrote: > >> A /sys/kernel/debug/btrfs/test file was added nearly > >> two and a half years ago, but it serves no purpose; > > > > It

Re: [PATCH] btrfs: remove pointless debugfs interface

2016-08-31 Thread Jeff Mahoney
On 8/31/16 3:08 PM, David Sterba wrote: > On Wed, Aug 31, 2016 at 10:13:49AM -0500, Eric Sandeen wrote: >> A /sys/kernel/debug/btrfs/test file was added nearly >> two and a half years ago, but it serves no purpose; > > It does. Introduced in 1bae30982bc86ab66d61ccb6e22792593b45d44d says >

Re: [PATCH] btrfs: remove pointless debugfs interface

2016-08-31 Thread Eric Sandeen
On 8/31/16 2:08 PM, David Sterba wrote: > On Wed, Aug 31, 2016 at 10:13:49AM -0500, Eric Sandeen wrote: >> A /sys/kernel/debug/btrfs/test file was added nearly >> two and a half years ago, but it serves no purpose; > > It does. Introduced in 1bae30982bc86ab66d61ccb6e22792593b45d44d says >

Re: [PATCH] btrfs: remove pointless debugfs interface

2016-08-31 Thread David Sterba
On Wed, Aug 31, 2016 at 10:13:49AM -0500, Eric Sandeen wrote: > A /sys/kernel/debug/btrfs/test file was added nearly > two and a half years ago, but it serves no purpose; It does. Introduced in 1bae30982bc86ab66d61ccb6e22792593b45d44d says something about helping developers to easily export

[PATCH] btrfs: remove pointless debugfs interface

2016-08-31 Thread Eric Sandeen
A /sys/kernel/debug/btrfs/test file was added nearly two and a half years ago, but it serves no purpose; it stores and returns a value, but nothing in the btrfs code uses this value in any way. There are no other btrfs files in this debugfs dir. This was brought to my attention because it is