[lustre-devel] [PATCH 1/6] Autoconf option for rate-limiting Quality of Service (RLQOS)
Ben Evans
bevans at cray.com
Wed Mar 22 07:27:25 PDT 2017
I'd get rid of all the ENABLE_RLQOS blocks myself, but minimally the
lustre_idl.h ones.
-Ben Evans
On 3/22/17, 10:19 AM, "Yan Li" <yanli at ascar.io> wrote:
>
>On 03/21/2017 01:09 PM, Ben Evans wrote:
>> I would remove the #ifdef ENABLE_RLQOS blocks, especially in
>>lustre_idl.h
>> since you're proposing to add new fields and consume some of the padding
>> bits. It will cause a lot of headache for the next feature that comes
>> along and consumes some of those bits.
>
>Yeah, that's a good point. I'll remove it if all are ok with this.
>
>Yan
More information about the lustre-devel
mailing list