Re: [PATCH v15 2/9] nvme: Create helper function to obtain command effects

From: Christoph Hellwig
Date: Mon Jul 20 2020 - 10:04:15 EST


On Thu, Jul 16, 2020 at 02:33:12PM -0600, Logan Gunthorpe wrote:
> Separate the code to obtain command effects from the code
> to start a passthru request and open code nvme_known_admin_effects()
> in the new helper.
>
> The new helper function will be necessary for nvmet passthru
> code to determine if we need to change out of interrupt context
> to handle the effects.

I actually found the nvme_known_admin_effects pretty nice, as the
function name documents why it is there. The rest looks fine to me.