[PATCH V2 1/5] nvme: add APIs for stopping/starting admin queue
Ming Lei
ming.lei at redhat.com
Mon Oct 4 19:23:14 PDT 2021
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 at redhat.com>
>
>
> 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.
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.
But if you really don't like this way, I am fine to merge the two since
merging is always easier than splitting, :-)
Thanks,
Ming
More information about the Linux-nvme
mailing list