Re: [PATCH 01/39] staging/lustre/hsm: remove hsm_nl proc file

From: Peng Tao
Date: Fri Nov 15 2013 - 05:32:58 EST


On Fri, Nov 15, 2013 at 12:26 PM, Greg Kroah-Hartman
<gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> On Fri, Nov 15, 2013 at 12:32:24AM +0800, Peng Tao wrote:
>> From: "John L. Hammond" <john.hammond@xxxxxxxxx>
>>
>> Remove the file /proc/fs/lustre/mdc/*/hsm_nl which was introduced "for
>> testing purposes."
>
> What does this mean? Why is this allowed? How do you know no one uses
> it? I need a much better changelog comment here, sorry.
>
Those proc entries were added for testing HSM
(http://en.wikipedia.org/wiki/Hierarchical_storage_management) during
the time of developing Lustre HSM feature. Now the HSM feature is
almost completed, those files no longer serve their purpose and can be
removed so that no one would develop dependency upon it after Lustre
HSM is announced released.

I'll add this to the commit message.

Thanks for reviewing,
Tao
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/