From nobody Thu Feb 12 19:05:12 2026 Received: from mail-yw1-f201.google.com (mail-yw1-f201.google.com [209.85.128.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4178D19B3FB for ; Fri, 7 Jun 2024 17:57:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1717783071; cv=none; b=hAVuLv/mr2deLOLqzJOeaNp/01/nrLPxPc3IVmc5IuhvylPkAlxJxKyLwTbCZkKRtAC4ekDBHYzi/Xkxslrj/R06jIkfEPBWKn0AAVnSVspp2Mp9rwbqXBayObNXxSdE2Bn6fd7rX5vOVD5I6oywH5LHsFu+jXX0kQj+vuKPLZg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1717783071; c=relaxed/simple; bh=kceaFwwjaSlBmkhXtusA3USI+23jbBoLODkMZKgX7RE=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=dsJP2jaFvS0e2fy5OMSVeS5ZC46d3jtInT/aPUJOPInzCRpGcpKZCyn4VrpBz4kuKWOUyJm3UyN7Z9NaP2dEu9ovsUzOfQLtAuFLtTNGZnH/AmjaOVjS/Pph82q9pbeebOirrUjj8qfyNbzVVAA8JujvXX75EPYMixWcaBVBTSY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--tadamsjr.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=PX6kx7Pi; arc=none smtp.client-ip=209.85.128.201 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--tadamsjr.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="PX6kx7Pi" Received: by mail-yw1-f201.google.com with SMTP id 00721157ae682-629638f1cb0so36730547b3.3 for ; Fri, 07 Jun 2024 10:57:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1717783069; x=1718387869; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=68M9FutlL8cAjPUlj6bX6OztbAKg98sW3h68noYav98=; b=PX6kx7Pi7Pi5pBl4e6DZGHy18AgFLMWz4iuwOpWW3nxYSqlabtLJCQeUoCv01bEybO SN8FN8z4/b29sxLG60rOdmweMU3ixP+Y8/NHFwXirPyZfdt4AHGHMZ1ll+X1O6K4LKmt QlMh2DVFJd87n+hiEtcUFR8RbGha79j8ouNBUUGr1ajHWkbC+umY7qj4Yh9ZLvx31sFQ Hp5GTgGkZAO60xigUFeThLcahCeJZsXjQjFrtlbozJSyPnMHiUKWNBuvPS2xnX8YIRcO cl0Tf9YDGhslrApG5Omn53rGlYB/+piL302rNxeBhMrSCRU3mn4J79ys3+5HhrCTjaDr Wtug== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1717783069; x=1718387869; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=68M9FutlL8cAjPUlj6bX6OztbAKg98sW3h68noYav98=; b=r24NKLj2Or+3LLscPDKEMKIM9F3fuXJ22hQE78Ajo+tdprGzZem2HkO1y0XhnrctRc Rxvmq5waJNwZu6RIVTObwErE3aGTDg+9Elr9k6EE3OUI7qMPk0NvZ1jl3SqHRGtWjSji MIHaT5kubq55sLY9HnPzMY4OAxw/SOMSAVvbEkyYGablCIWAA6HORp3UYR8T9zQ57nlt fyimmF6axa1rVatnXLOSJk0AQmK2BQY9yUDqcOqrzEq6y0SnYsoP8USSIDRp/7mWIxDf 9YoUZrPgmdnf56+Wa590nmK+YogqhU8MlK2YPIznx0Zz8Ngeej3qyqwFbkW4Cskhweqh yEJg== X-Forwarded-Encrypted: i=1; AJvYcCXUq9WNRpxgQB9bYzZY808OEUqIvXMm8gysbgQw8LZokQo8iMcHXhPQCZUgTXj0j14tGfB86U0kR6drHCCPSOCPeX4x/BhbcKwN5+Qg X-Gm-Message-State: AOJu0YwsbNliaw4p3MP1QB1zd6MApT+jBMYAaNLrOxp6h8YGwIg6COIO JTVs2BaJ9bgvn6e4R2rYZqnBQmleDToUhsljLhoSlY/7OTfsdg27Kc/Rrk9acIDN7I0oiuW8Jqf RA7ZZsDRhmg== X-Google-Smtp-Source: AGHT+IFSVowxqqPvr0+4CvkqgeikyT0vf4vjJ2LiCufzLnHvtpyVCokVVjJIiQzdT4oPCJcaV2sRwUt/x9KYIw== X-Received: from tadamsjr.c.googlers.com ([fda3:e722:ac3:cc00:20:ed76:c0a8:177c]) (user=tadamsjr job=sendgmr) by 2002:a05:690c:4:b0:62a:4932:68de with SMTP id 00721157ae682-62cd566adc9mr9535477b3.8.1717783069258; Fri, 07 Jun 2024 10:57:49 -0700 (PDT) Date: Fri, 7 Jun 2024 17:57:41 +0000 In-Reply-To: <20240607175743.3986625-1-tadamsjr@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20240607175743.3986625-1-tadamsjr@google.com> X-Mailer: git-send-email 2.45.2.505.gda0bf45e8d-goog Message-ID: <20240607175743.3986625-2-tadamsjr@google.com> Subject: [PATCH 1/3] scsi: pm80xx: Set phy->enable_completion only when we wait for it From: TJ Adams To: Jack Wang , "James E . J . Bottomley" , "Martin K . Petersen" Cc: linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Igor Pylypiv , Terrence Adams Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Igor Pylypiv pm8001_phy_control() populates the enable_completion pointer with a stack address, sends a PHY_LINK_RESET / PHY_HARD_RESET, waits 300 ms, and returns. The problem arises when a phy control response comes late. After 300 ms the pm8001_phy_control() function returns and the passed enable_completion stack address is no longer valid. Late phy control response invokes complete() on a dangling enable_completion pointer which leads to a kernel crash. Signed-off-by: Igor Pylypiv Signed-off-by: Terrence Adams Acked-by: Jack Wang --- drivers/scsi/pm8001/pm8001_sas.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/scsi/pm8001/pm8001_sas.c b/drivers/scsi/pm8001/pm8001_= sas.c index a5a31dfa4512..ee2da8e49d4c 100644 --- a/drivers/scsi/pm8001/pm8001_sas.c +++ b/drivers/scsi/pm8001/pm8001_sas.c @@ -166,7 +166,6 @@ int pm8001_phy_control(struct asd_sas_phy *sas_phy, enu= m phy_func func, unsigned long flags; pm8001_ha =3D sas_phy->ha->lldd_ha; phy =3D &pm8001_ha->phy[phy_id]; - pm8001_ha->phy[phy_id].enable_completion =3D &completion; =20 if (PM8001_CHIP_DISP->fatal_errors(pm8001_ha)) { /* @@ -190,6 +189,7 @@ int pm8001_phy_control(struct asd_sas_phy *sas_phy, enu= m phy_func func, rates->maximum_linkrate; } if (pm8001_ha->phy[phy_id].phy_state =3D=3D PHY_LINK_DISABLE) { + pm8001_ha->phy[phy_id].enable_completion =3D &completion; PM8001_CHIP_DISP->phy_start_req(pm8001_ha, phy_id); wait_for_completion(&completion); } @@ -198,6 +198,7 @@ int pm8001_phy_control(struct asd_sas_phy *sas_phy, enu= m phy_func func, break; case PHY_FUNC_HARD_RESET: if (pm8001_ha->phy[phy_id].phy_state =3D=3D PHY_LINK_DISABLE) { + pm8001_ha->phy[phy_id].enable_completion =3D &completion; PM8001_CHIP_DISP->phy_start_req(pm8001_ha, phy_id); wait_for_completion(&completion); } @@ -206,6 +207,7 @@ int pm8001_phy_control(struct asd_sas_phy *sas_phy, enu= m phy_func func, break; case PHY_FUNC_LINK_RESET: if (pm8001_ha->phy[phy_id].phy_state =3D=3D PHY_LINK_DISABLE) { + pm8001_ha->phy[phy_id].enable_completion =3D &completion; PM8001_CHIP_DISP->phy_start_req(pm8001_ha, phy_id); wait_for_completion(&completion); } --=20 2.45.2.505.gda0bf45e8d-goog From nobody Thu Feb 12 19:05:12 2026 Received: from mail-yb1-f201.google.com (mail-yb1-f201.google.com [209.85.219.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1D2E219B5BB for ; Fri, 7 Jun 2024 17:57:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1717783073; cv=none; b=Y/J0keGVgLWt3C5y+HsncTCmfmY29UQp4VaTafAy8FiZv0TET0Lphk1jKZBAmvhdZ6WaeJC+n4OH45wQhAwEi6Nl6yhFByZwUky99enp9grMkzveNMgoFrFAIKeM8pMcybK+rs4HWJrsC69DDZfNgUawS1kmSEaGOnAwzLOrwsw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1717783073; c=relaxed/simple; bh=77RomTbxoT5IvK9cD9wwGOCTWkN2mvquDo1FazG6ADk=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=j262SjYTdlTpkxkkUQIrFRsZsxV9TF5ijYM5X+ZC8LmxwnUkNHYSyooCT/jTGYs9Gd8pHbjSgd3YvhoV2NqTlXFbAq+Q8ITjaN05f1Cs2fSEOxfbR6YFG04O4O3iZm6lmS724Ead2DHo9Gr9L84EiUyKmbFG5k2kta4pMCSOAGg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--tadamsjr.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=z5+BBEM3; arc=none smtp.client-ip=209.85.219.201 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--tadamsjr.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="z5+BBEM3" Received: by mail-yb1-f201.google.com with SMTP id 3f1490d57ef6-df4da3446beso3976203276.1 for ; Fri, 07 Jun 2024 10:57:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1717783071; x=1718387871; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=Gu46/3S2zZA6k5Oex8T+jABLwZXyDcysB54lkLA/uRc=; b=z5+BBEM35CAisqPQ8iOIkk3zM+3f4OaBQWVsgWIXteJ9sqIeOOryNZzKAIS5UngGnh QAYDdC+8f/+mbuRiEbR9x6A+OJhwToQyjvgrjM3g/f+3Czhx2lTw+INa2aM89BDdK62y ADyoazOy502P5IIivIBaD77Ojh2UWvIovoCxR7OvuYj1r/GEetMpyYBu9V3PGpFBs25O +skTXMyD1WiXms61Zs0eI1p8XIo4BIENA5E7vr8+cZVD9rU3Rq/05FqWig0jHcloFniT fq5AXfCHXQ3W+mupsXsfzFUTCvFwA/11FOCXGOS4Gdulbbpp1Jd+2oEKrzZlDGW4Euq9 BT3w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1717783071; x=1718387871; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Gu46/3S2zZA6k5Oex8T+jABLwZXyDcysB54lkLA/uRc=; b=e4OoDWQCHekEMnWfmj5RG/orlSdrasLgqzcNTRrtthq8V2q+uHFIKkWrbEPCTQ/in4 3nOExOobU9RF+9TnfOsSyDxGTr1hTuAq3YYIEk9iBCNFMDX+NuUrZp3ItgB2r9KNgGyO 9km59CkBxb/UhPhXtluLI30Az8w3XSmzPFumWXmiQgazs52ZqlVb8jLC1zPbZj/tqU5I pI7c6MpD/UxUAhKYaIYAumyqQ0POwjdKRvNK/QdEfvSTlz0KP70WpDU6MERtbz/JxX7w i4GWEBA32cFW9ibhMQTK5pau7fXjU/t3fLIndXqXLIOqmidHa328bC2IgxtN1f8U6iNI cLWQ== X-Forwarded-Encrypted: i=1; AJvYcCVKQm/0vHky1KCDaXXqbR7xn8y/t2b3Vi19kjSXnB+6bc9M37FgxJSreUj1cmZ0nQ1om1p9rxApdyFvOvrHtmUTe9JBKViF5DghGk1K X-Gm-Message-State: AOJu0Yzs0+bi0QvQ2adzO/h2qVfJI8tm60GjxkUS9zNTjlbhpGfyv25C 1CSvr/PO92UI5DiOZ+4kx0mAJ8G4HpS0RywyhS5JKhDHkoO9C/NMaBn/YGiC1XY857QC6Q/wtC5 yaFOc0BIytw== X-Google-Smtp-Source: AGHT+IERE/EYXtc8MUrFi8tAzHgVN6/66kMWeUDhMDvzhrkKK4ZBgSlyAwdYRCe+7vczJj8PruT3vRNLZRsyaA== X-Received: from tadamsjr.c.googlers.com ([fda3:e722:ac3:cc00:20:ed76:c0a8:177c]) (user=tadamsjr job=sendgmr) by 2002:a05:6902:c04:b0:dfa:59bc:8857 with SMTP id 3f1490d57ef6-dfaf66268a4mr939243276.9.1717783071064; Fri, 07 Jun 2024 10:57:51 -0700 (PDT) Date: Fri, 7 Jun 2024 17:57:42 +0000 In-Reply-To: <20240607175743.3986625-1-tadamsjr@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20240607175743.3986625-1-tadamsjr@google.com> X-Mailer: git-send-email 2.45.2.505.gda0bf45e8d-goog Message-ID: <20240607175743.3986625-3-tadamsjr@google.com> Subject: [PATCH 2/3] scsi: pm80xx: Do not issue hard reset before NCQ EH From: TJ Adams To: Jack Wang , "James E . J . Bottomley" , "Martin K . Petersen" Cc: linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Igor Pylypiv , Terrence Adams Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Igor Pylypiv v6.2 commit 811be570a9a8 ("scsi: pm8001: Use sas_ata_device_link_abort() to handle NCQ errors") removed duplicate NCQ EH from the pm80xx driver and started relying on libata to handle the NCQ errors. The PM8006 controller has a special EH sequence that was added in v4.15 commit 869ddbdcae3b ("scsi: pm80xx: corrected SATA abort handling sequence."). The special EH sequence issues a hard reset to a drive before libata EH has a chance to read the NCQ log page. Libata EH gets confused by empty NCQ log page which results in HSM violation. The failed command gets retried a few times and each time fails with the same HSM violation. Finally, libata decides to disable NCQ due to subsequent HSM vioaltions. To avoid unwanted hard resets we can initiate abort all from the driver to prevent libsas EH from calling lldd_abort_task()/pm8001_abort_task(). Signed-off-by: Igor Pylypiv Signed-off-by: Terrence Adams --- drivers/scsi/pm8001/pm8001_hwi.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/drivers/scsi/pm8001/pm8001_hwi.c b/drivers/scsi/pm8001/pm8001_= hwi.c index dec1e2d380f1..f19f76dc6e1c 100644 --- a/drivers/scsi/pm8001/pm8001_hwi.c +++ b/drivers/scsi/pm8001/pm8001_hwi.c @@ -1672,7 +1672,18 @@ void pm8001_work_fn(struct work_struct *work) break; case IO_XFER_ERROR_ABORTED_NCQ_MODE: { + struct pm8001_hba_info *pm8001_ha =3D pw->pm8001_ha; dev =3D pm8001_dev->sas_device; + /* + * pm8001_abort_task() issues a hard reset to a drive + * before libata EH has a chance to read the NCQ log page. + * + * Initiate abort all from the driver to prevent libsas EH + * from calling lldd_abort_task() / pm8001_abort_task(). + */ + if (pm8001_ha->chip_id =3D=3D chip_8006) + sas_execute_internal_abort_dev(dev, 0, NULL); + sas_ata_device_link_abort(dev, false); } break; --=20 2.45.2.505.gda0bf45e8d-goog From nobody Thu Feb 12 19:05:12 2026 Received: from mail-yw1-f202.google.com (mail-yw1-f202.google.com [209.85.128.202]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B67C019CD13 for ; Fri, 7 Jun 2024 17:57:53 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.202 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1717783075; cv=none; b=bPNFWfIiijzCtRa8DKa0uxYgstpdn6v/bIX2/gcj6rzojRmla9ivX+nHCQsKp9c4HZGNCouow3uVslAs1VD7SW7jVtZq5ZylXeTJm4aAfrhT9ZUO3uY3IEzHVGBAVLtDdrZ+GKlggOs/QfnWLPw2TCZLUx7nkfE9SZFozVdmxHo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1717783075; c=relaxed/simple; bh=iAbPGXH/r5gL1hZgWkLgKCD1/WIGTyanOjQWhd5s0L4=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=TfE5M3MfGkzNAsvYJPSDbOzZSfnJ8b4X2boQRnUYwf1qxIgKVlxBcQXQRIwdFwpyJNu22FO0pHumPxQ129EKZHDm41UGx6adkm9KUUXEY/eqOojAmPjNIleY0sumqUVI/vw+SzcVGoEE4jONybFlZrj2M9cgp8LNv6VtK1Dolvc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--tadamsjr.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=zaZOeBjV; arc=none smtp.client-ip=209.85.128.202 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--tadamsjr.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="zaZOeBjV" Received: by mail-yw1-f202.google.com with SMTP id 00721157ae682-62a083e617aso41160507b3.2 for ; Fri, 07 Jun 2024 10:57:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1717783073; x=1718387873; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=+1aQwu1iGrHhm3SengJCu2MFtSP+MYd6cSL40GeM53Y=; b=zaZOeBjV3FRTaOZGcExh1nORxNM9GASo6bSRwgEKeHlidel0IQxAfdhQmWJdsgFYce Q9kAPcM39ZhUhkan92MMVGqpuyBKIVlxKwFgOEHi2N37YBzmIMYUawdlbvI+zRFrSQ5W QBSraZN1ybITmMYV5QpiizboXcgpCbbYqNS2L1gUDBANQDzhmzoeTwj/4vIOce4llrUP w9wbwnc0GXULBzA1dFEtqaOwbOR7RkiKvreAcFnP+pofQN7Z+bxEYAaivuh3R0XLCvho fpKKbe4y6Xk8PeDk8+CzwEyeMR9XNOrlfyvaS3Hzg/O7zZxvDzbSx9KyiLLnjTlXJ+wH wlyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1717783073; x=1718387873; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=+1aQwu1iGrHhm3SengJCu2MFtSP+MYd6cSL40GeM53Y=; b=Jomuuz0LucQ/VyE6ntYDT0rEjVpdA3ziCPbVCgckntm56v01HcXDzkxYI92iWaEKVL 97evNDRL5PvLllGrZC6kWY10+FNVe3Pcv33Oz65maJf5c+NOVOYDIb9yJY02yBArjpMA OiXQlc10bjcHN/DxAgDxBRnvomgqUczkP7fyUL9Qp/Yz+gbx5dLqUgnMQDUDJiBChKI3 xaLuGw8XyAlQM4fMHECV5NsyeAe3vFqNCCEDfYGPzKgIvlMsqH7TW3gdnlcgWQkTSx76 mPMIbUJibqKlGI125HLqfxrVSgYadjX1DH1ifEsh6Tus8ykGDtJ4EVofriGAIrOg3hBy H9fw== X-Forwarded-Encrypted: i=1; AJvYcCV97uBco4tWRZUCH26rdEbYwft9Aix1d0lOfWIDWO9XmM3ZPX8L7k0B7Pp9+tD+CHX5X8wEm+5+8emjBwTdUszj3+vZkmo9WMSxyo11 X-Gm-Message-State: AOJu0YxS+OwguQQMet8JYjY45Nng2/hA17BMdiKFaJGVKy3vaYRcuL/1 /aUEbpkuAfURHg/Q2WIJ1FNvf2pEOu90DGJ8joLzu6LAU9RNB6rO75E2neknznnMYoULVyUfV58 t9DkQ6T5ADw== X-Google-Smtp-Source: AGHT+IHp7YoVWlbDnzAYutrdvwxNbknx4DiQAHnpB/HfNYrXL6Srw00AbWWvHVrb7rZt95ERJDHNm49UM8p4ZQ== X-Received: from tadamsjr.c.googlers.com ([fda3:e722:ac3:cc00:20:ed76:c0a8:177c]) (user=tadamsjr job=sendgmr) by 2002:a05:6902:1245:b0:dcb:e982:4e40 with SMTP id 3f1490d57ef6-dfaf66c5d0fmr801100276.12.1717783072744; Fri, 07 Jun 2024 10:57:52 -0700 (PDT) Date: Fri, 7 Jun 2024 17:57:43 +0000 In-Reply-To: <20240607175743.3986625-1-tadamsjr@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20240607175743.3986625-1-tadamsjr@google.com> X-Mailer: git-send-email 2.45.2.505.gda0bf45e8d-goog Message-ID: <20240607175743.3986625-4-tadamsjr@google.com> Subject: [PATCH 3/3] scsi: pm8001: Update log level when reading config table From: TJ Adams To: Jack Wang , "James E . J . Bottomley" , "Martin K . Petersen" Cc: linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Igor Pylypiv , Terrence Adams Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Terrence Adams Reading the main config table occurs as a part of initialization in pm80xx_chip_init(). Because of this it makes more sense to have it be a part of the INIT logging. Signed-off-by: Terrence Adams Acked-by: Jack Wang --- drivers/scsi/pm8001/pm80xx_hwi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/pm8001/pm80xx_hwi.c b/drivers/scsi/pm8001/pm80xx_= hwi.c index a52ae6841939..8fe886dc5e47 100644 --- a/drivers/scsi/pm8001/pm80xx_hwi.c +++ b/drivers/scsi/pm8001/pm80xx_hwi.c @@ -568,13 +568,13 @@ static void read_main_config_table(struct pm8001_hba_= info *pm8001_ha) pm8001_ha->main_cfg_tbl.pm80xx_tbl.inc_fw_version =3D pm8001_mr32(address, MAIN_MPI_INACTIVE_FW_VERSION); =20 - pm8001_dbg(pm8001_ha, DEV, + pm8001_dbg(pm8001_ha, INIT, "Main cfg table: sign:%x interface rev:%x fw_rev:%x\n", pm8001_ha->main_cfg_tbl.pm80xx_tbl.signature, pm8001_ha->main_cfg_tbl.pm80xx_tbl.interface_rev, pm8001_ha->main_cfg_tbl.pm80xx_tbl.firmware_rev); =20 - pm8001_dbg(pm8001_ha, DEV, + pm8001_dbg(pm8001_ha, INIT, "table offset: gst:%x iq:%x oq:%x int vec:%x phy attr:%x\n", pm8001_ha->main_cfg_tbl.pm80xx_tbl.gst_offset, pm8001_ha->main_cfg_tbl.pm80xx_tbl.inbound_queue_offset, @@ -582,7 +582,7 @@ static void read_main_config_table(struct pm8001_hba_in= fo *pm8001_ha) pm8001_ha->main_cfg_tbl.pm80xx_tbl.int_vec_table_offset, pm8001_ha->main_cfg_tbl.pm80xx_tbl.phy_attr_table_offset); =20 - pm8001_dbg(pm8001_ha, DEV, + pm8001_dbg(pm8001_ha, INIT, "Main cfg table; ila rev:%x Inactive fw rev:%x\n", pm8001_ha->main_cfg_tbl.pm80xx_tbl.ila_version, pm8001_ha->main_cfg_tbl.pm80xx_tbl.inc_fw_version); --=20 2.45.2.505.gda0bf45e8d-goog