Re: [PATCH 0/9] Assorted cleanups for staging/.../lustre/ldlm/ldlm_flock.c
From: NeilBrown
Date: Wed Oct 25 2017 - 17:55:21 EST
On Tue, Oct 24 2017, James Simmons wrote:
>> This file contains quite a bit of dead code and unused variables.
>> This patch series cleans it up in various ways.
>> It should change behaviour at all, just code
>> readability/maintainabilty.
>>
>> I sent the back in July but got not response, possibly because there
>> were included with other patches which caused a distraction.
>> So here they are by themselves.
>
> Thanks for separating them out. I will give them a spin.
>
Thanks. I have a growing series of cleanups (43 at present, including
these and the namei/dcache patches). Is posting them here the best
approach? I assume I should keep them in well-defined groups. Should I
wait for one set to be merged before sending more, or should I just send
them when I think they are ready and you (or others) will look at them
as time permits.
Thanks,
NeilBrown
Attachment:
signature.asc
Description: PGP signature