Re: [PATCH] nullb: make config_item_type const

From: Bhumika Goyal
Date: Thu Oct 12 2017 - 10:51:02 EST


On Thu, Oct 12, 2017 at 4:47 PM, Jens Axboe <axboe@xxxxxxxxx> wrote:
> On 10/12/2017 08:42 AM, Bhumika Goyal wrote:
>> On Thu, Oct 12, 2017 at 4:27 PM, Jens Axboe <axboe@xxxxxxxxx> wrote:
>>> On 10/12/2017 04:19 AM, Bhumika Goyal wrote:
>>>> This is a followup patch for: https://lkml.org/lkml/2017/10/11/375 and
>>>> https://patchwork.kernel.org/patch/9999649/
>>>>
>>>> Make these structures const as they are either passed to the functions
>>>> having the argument as const or stored as a reference in the "ci_type"
>>>> const field of a config_item structure.
>>>
>>> Thanks, applied.
>>>
>>
>> Hello,
>>
>> Actually, this patch is dependent on the patches in the links
>> https://lkml.org/lkml/2017/10/11/375 and
>> https://patchwork.kernel.org/patch/9999649/. Therefore, this patch
>> won't be correct unless the patches in these links gets applied.
>
> Gah, I do see that now. That should have been made much clearer in
> the patch description!
>
> When you send out patches with hard dependencies like that, either
> wait until the dependency has been solved in mainline, or make it
> explicitly clear in the patch submission.
>

Sorry, for not mentioning the dependencies clearly. I will keep the
pointer in mind.

Thanks,
Bhumika

> --
> Jens Axboe
>