Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   <Wesley.Sheng@microchip.com>
To:     <logang@deltatee.com>, <kurt.schwemmer@microsemi.com>,
        <Kurt.Schwemmer@microchip.com>, <bhelgaas@google.com>
CC:     <linux-pci@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <Wesley.Sheng@microchip.com>, <Kelvin.Cao@microchip.com>
Subject: [PATCH 1/12 v2] switchtec: Remove immediate status check after submit
 a MRPC command
Thread-Topic: [PATCH 1/12 v2] switchtec: Remove immediate status check after
 submit a MRPC command
Thread-Index: AdRWJF1jbNxK+iTHQO26nowQGjGEMw==
Date:   Thu, 27 Sep 2018 05:57:19 +0000
Message-ID: <DM2PR11MB0105CD1B3A264290413ED1CA95140@DM2PR11MB0105.namprd11.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: microchip.com does not designate
 permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 079a3483-0c0c-4d44-3c14-08d6243e163e
X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Sep 2018 05:57:19.7534
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR11MB0057
X-OriginatorOrg: microchip.com
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

After submit a Firmware Download (Download sub-command) MRPC
command, Switchtec firmware refuses to response any management=20
EP's BAR access until current flash programming finished.
During this time, a READ TLP to the gas area in the BAR of the
management EP will complete with significant delay like more than=20
10ms.
It's a Switchtec firmware limitation that READ requests cannot get
prompt service during firmware download.
The delayed completion of READ TLP would be a problem on some system
 which is sensitive to READ timeout.
Current driver check status immediately after submit a MRPC command,
which triggers READ TLP to the gas area in the BAR of the management EP.
Also, other processes or functions, like NTB, would also trigger READ TLP b=
y
accessing the GAS.
To avoid this, the immediate check of status is removed in this patch, and
driver delays the status check to the occurrence of MSIx or MRPC timeout.
In the meantime, user must not initiate any gas access during a firmware
download.
Also, any process that issues MRPC command will be affected by the delay
in this patch.
However, this is only a software workaround to the READ issue in firmware
download. A complete fix of this should happen in firmware.

Note: For NTB function, the memory window access is handled by Switchtec
hardware. So it's not affected by this firmware limitation. But the other G=
AS
access which is handled by Switchtec firmware is are affected by this firmw=
are
limitation.

Signed-off-by: Kelvin Cao <Kelvin.Cao@microchip.com>
Signed-off-by: Wesley Sheng <Wesley.sheng@microchip.com>

---
 drivers/pci/switch/switchtec.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.=
c index 4591f15..b759228 100644
--- a/drivers/pci/switch/switchtec.c
+++ b/drivers/pci/switch/switchtec.c
@@ -142,10 +142,6 @@ static void mrpc_cmd_submit(struct switchtec_dev *stde=
v)
 		    stuser->data, stuser->data_len);
 	iowrite32(stuser->cmd, &stdev->mmio_mrpc->cmd);
=20
-	stuser->status =3D ioread32(&stdev->mmio_mrpc->status);
-	if (stuser->status !=3D SWITCHTEC_MRPC_STATUS_INPROGRESS)
-		mrpc_complete_cmd(stdev);
-
 	schedule_delayed_work(&stdev->mrpc_timeout,
 			      msecs_to_jiffies(500));
 }
--
2.7.4