Re: [PATCH V2 1/5] nvme: add APIs for stopping/starting admin queue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/4/21 19:23, Ming Lei wrote:
> External email: Use caution opening links or attachments
> 
> 
> Hello Chaitanya,
> 
> On Fri, Oct 01, 2021 at 05:56:04AM +0000, Chaitanya Kulkarni wrote:
>> On 9/30/2021 5:56 AM, Ming Lei wrote:
>>> External email: Use caution opening links or attachments
>>>
>>>
>>> Add two APIs for stopping and starting admin queue.
>>>
>>> Signed-off-by: Ming Lei <ming.lei@xxxxxxxxxx>
>>
>>
>> this patch looks good to me, but from the feedback I've received in past
>> we need to add the new functions in the patch where they are actually
>> used than adding it in a separate patch.
> 
> The added two APIs are exported via EXPORT_SYMBOL_GPL(), so it won't
> cause any build warning. I see lots of such practise too.
> 

the comment was not related to any build or warning.

> It is easier for reviewing in this way since the 1st patch focuses on
> API implementation, and the 2nd patch focuses on using the API,
> especially there are lots of users in patch 2.
> 

I am aware of that, just sharing what I got the comments in the past.

> But if you really don't like this way, I am fine to merge the two since
> merging is always easier than splitting, :-)
> 

it will be good if we can keep the consistency ... nothing else ..

> 
> Thanks,
> Ming
> 




[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux