Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 28 Oct 2020 14:55:14 +0100
From:   Pali =?utf-8?B?Um9ow6Fy?= <pali@kernel.org>
To:     Sascha Hauer <s.hauer@pengutronix.de>, Jens Axboe <axboe@kernel.dk>
Cc:     linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org,
        kernel@pengutronix.de
Subject: Re: [PATCH v2] libata: Fix retrieving of active qcs
Message-ID: <20201028135514.r2m3kxchbg6nljht@pali>
References: <20191213080408.27032-1-s.hauer@pengutronix.de>
 <20191225181840.ooo6mw5rffghbmu2@pali>
 <20200106081605.ffjz7xy6e24rfcgx@pengutronix.de>
 <20200127111630.bqqzhj57tzt7geds@pali>
 <20200127112428.sdfxvlqdox5efzcb@pengutronix.de>
 <20200503214627.gerb3ipcwek2h3h7@pali>
 <20200508054644.GJ5877@pengutronix.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20200508054644.GJ5877@pengutronix.de>
User-Agent: NeoMutt/20180716
Precedence: bulk

On Friday 08 May 2020 07:46:44 Sascha Hauer wrote:
> From fcdcfa9e7a4ee4faf411de1df4f3c4e12c78545c Mon Sep 17 00:00:00 2001
> From: Sascha Hauer <s.hauer@pengutronix.de>
> Date: Fri, 8 May 2020 07:28:19 +0200
> Subject: [PATCH] ata: sata_nv: Fix retrieving of active qcs
> 
> ata_qc_complete_multiple() has to be called with the tags physically
> active, that is the hw tag is at bit 0. ap->qc_active has the same tag
> at bit ATA_TAG_INTERNAL instead, so call ata_qc_get_active() to fix that
> up. This is done in the vein of 8385d756e114 ("libata: Fix retrieving of
> active qcs").
> 
> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

I tested this second change on nforce4 box with sata_nv controllers:

  00:07.0 IDE interface [0101]: NVIDIA Corporation CK804 Serial ATA Controller [10de:0054] (rev f3)
  00:08.0 IDE interface [0101]: NVIDIA Corporation CK804 Serial ATA Controller [10de:0055] (rev f3)

Both disks are working fine, I do not see any regression or change, so
you can add my:

Tested-by: Pali Rohár <pali@kernel.org>

Ideally add also Fixes line:

Fixes: 28361c403683 ("libata: add extra internal command")

Jens, do you need something more from me? Some special tests, etc?

> ---
>  drivers/ata/sata_nv.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/ata/sata_nv.c b/drivers/ata/sata_nv.c
> index eb9dc14e5147..20190f66ced9 100644
> --- a/drivers/ata/sata_nv.c
> +++ b/drivers/ata/sata_nv.c
> @@ -2100,7 +2100,7 @@ static int nv_swncq_sdbfis(struct ata_port *ap)
>  	pp->dhfis_bits &= ~done_mask;
>  	pp->dmafis_bits &= ~done_mask;
>  	pp->sdbfis_bits |= done_mask;
> -	ata_qc_complete_multiple(ap, ap->qc_active ^ done_mask);
> +	ata_qc_complete_multiple(ap, ata_qc_get_active(ap) ^ done_mask);
>  
>  	if (!ap->qc_active) {
>  		DPRINTK("over\n");
> -- 
> 2.26.2