ACK/Cmnt: [SRU][Bionic][PATCH 1/1] nvme: Use admin command effects for admin commands

[Stefan Bader]

On 08.01.19 20:57, Joseph Salisbury wrote:
> From: Keith Busch <keith.busch at intel.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1797587
> 
> Signed-off-by: Keith Busch <keith.busch at intel.com>
> Signed-off-by: Jens Axboe <axboe at kernel.dk>
> (cherry picked from commit 62843c2e4226057c83f520c74fe9c81a1891c331)
> Signed-off-by: Joseph Salisbury <joseph.salisbury at canonical.com>
Acked-by: Stefan Bader <stefan.bader at canonical.com>
> ---

As Kleber already pouinted out there is nothing but the subject and s-o-b in the
original patch. Does not make it much  nicer to read but meh.

>  drivers/nvme/host/core.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c
> index c5dfca8..42f6202 100644
> --- a/drivers/nvme/host/core.c
> +++ b/drivers/nvme/host/core.c
> @@ -1054,7 +1054,7 @@ static u32 nvme_passthru_start(struct nvme_ctrl *ctrl, struct nvme_ns *ns,
>  	}
>  
>  	if (ctrl->effects)
> -		effects = le32_to_cpu(ctrl->effects->iocs[opcode]);
> +		effects = le32_to_cpu(ctrl->effects->acs[opcode]);
>  	else
>  		effects = nvme_known_admin_effects(opcode);
>  
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20190109/321e1e68/attachment.sig>