From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E0376207E00 for ; Mon, 3 Feb 2025 12:48:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738586927; cv=none; b=XEiz7YKmupLx/+lfKMLCNBMrovugyGSkI608wzpy1KvkRyNQ+7K81BbxBJnuuf24gCbZbUnKz1ssvcuqPbH2SN22AAYgWaUX2p9H/AQOHLNYPc31J2ATD4DDRDNBX9Oflz7LDCxxEYAMl67O6ZvuHlA3lKLA7H9zI2/YUYpxM3s= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738586927; c=relaxed/simple; bh=dmp/OICvyzWbjsx8YA85YoXBDTLLpHmffMp0AD3Hxz8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=dw8iYaqhD3Q1T01U+TXHaYX7arZnd14wkWjufOLRgl0TYCCukn4jAkFFv+txiq1rKaCxxi27smd2Ohmx5fug24BofdoRmUIl0hNuY1Mgs6CARBQkoV4vTqLWrnpiy0KUCqKxA/6cU0LvEtj4IYju1UL0DlBy49iON2a32urM4G8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=COUy2V3z; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="COUy2V3z" Received: by smtp.kernel.org (Postfix) with ESMTPSA id E5BB5C4CEE4; Mon, 3 Feb 2025 12:48:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1738586926; bh=dmp/OICvyzWbjsx8YA85YoXBDTLLpHmffMp0AD3Hxz8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=COUy2V3z6dm4ScxiFFfn0MyhZ0oX/jMXPL7wyA090mHaSBa1DhIrAr6ixqLViOY+x iECsgjvHbEIVYDrWam1pvaEjwKp5diP+LBE5rO6eUHzz30c4rV1+FdQZ5EvbSRIYst cSSF9TZYoPxT4MphGVDPVnNlwgTtL4uP1edgI5hPvX2HRB/lA14A2yBljFrWj69TPG bH0hLufXgWThNLLRRJZg8nfQydSNKIe6eom3JDTkMI94GxyvZzzTRsOe7BMSoRQYbe rj0Sz9KpYNsDjWjMURJea2LGsvy52fR8In8WDwUWsmm8zMeEcOLEVyB3x2RIl+N1nl W0wNOGjeTAlug== From: Leon Romanovsky To: Jason Gunthorpe Cc: Maher Sanalla , Dennis Dalessandro , linux-rdma@vger.kernel.org Subject: [PATCH rdma-next v1 3/3] IB/hfi1: Remove state transition log message and opa_lstate_name() Date: Mon, 3 Feb 2025 14:48:06 +0200 Message-ID: <64e48bef00630e33f4b8c830cb7d9a69aedc34dc.1738586601.git.leon@kernel.org> X-Mailer: git-send-email 2.48.1 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-rdma@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Maher Sanalla Remove the state transition log message from the hfi1 driver, as the IB core now logs the same information when handling a cache update event. While at it, replace the hfi1-specific opa_lstate_name() function with the ib_verbs equivalent function, ib_port_state_to_str(), for converting IB port state to a string. Signed-off-by: Maher Sanalla Signed-off-by: Leon Romanovsky --- drivers/infiniband/hw/hfi1/chip.c | 18 ------------------ drivers/infiniband/hw/hfi1/chip.h | 1 - drivers/infiniband/hw/hfi1/driver.c | 2 +- drivers/infiniband/hw/hfi1/mad.c | 4 ++-- 4 files changed, 3 insertions(+), 22 deletions(-) diff --git a/drivers/infiniband/hw/hfi1/chip.c b/drivers/infiniband/hw/hfi1/chip.c index a442eca498b8..368b6be3226f 100644 --- a/drivers/infiniband/hw/hfi1/chip.c +++ b/drivers/infiniband/hw/hfi1/chip.c @@ -12882,22 +12882,6 @@ u32 chip_to_opa_pstate(struct hfi1_devdata *dd, u32 chip_pstate) } } -/* return the OPA port logical state name */ -const char *opa_lstate_name(u32 lstate) -{ - static const char * const port_logical_names[] = { - "PORT_NOP", - "PORT_DOWN", - "PORT_INIT", - "PORT_ARMED", - "PORT_ACTIVE", - "PORT_ACTIVE_DEFER", - }; - if (lstate < ARRAY_SIZE(port_logical_names)) - return port_logical_names[lstate]; - return "unknown"; -} - /* return the OPA port physical state name */ const char *opa_pstate_name(u32 pstate) { @@ -12956,8 +12940,6 @@ static void update_statusp(struct hfi1_pportdata *ppd, u32 state) break; } } - dd_dev_info(ppd->dd, "logical state changed to %s (0x%x)\n", - opa_lstate_name(state), state); } /** diff --git a/drivers/infiniband/hw/hfi1/chip.h b/drivers/infiniband/hw/hfi1/chip.h index 8841db16bde7..6992f6d40255 100644 --- a/drivers/infiniband/hw/hfi1/chip.h +++ b/drivers/infiniband/hw/hfi1/chip.h @@ -771,7 +771,6 @@ int is_bx(struct hfi1_devdata *dd); bool is_urg_masked(struct hfi1_ctxtdata *rcd); u32 read_physical_state(struct hfi1_devdata *dd); u32 chip_to_opa_pstate(struct hfi1_devdata *dd, u32 chip_pstate); -const char *opa_lstate_name(u32 lstate); const char *opa_pstate_name(u32 pstate); u32 driver_pstate(struct hfi1_pportdata *ppd); u32 driver_lstate(struct hfi1_pportdata *ppd); diff --git a/drivers/infiniband/hw/hfi1/driver.c b/drivers/infiniband/hw/hfi1/driver.c index 37a6794885d3..50826e7cdb7e 100644 --- a/drivers/infiniband/hw/hfi1/driver.c +++ b/drivers/infiniband/hw/hfi1/driver.c @@ -968,7 +968,7 @@ static bool __set_armed_to_active(struct hfi1_packet *packet) if (hwstate != IB_PORT_ACTIVE) { dd_dev_info(packet->rcd->dd, "Unexpected link state %s\n", - opa_lstate_name(hwstate)); + ib_port_state_to_str(hwstate)); return false; } diff --git a/drivers/infiniband/hw/hfi1/mad.c b/drivers/infiniband/hw/hfi1/mad.c index a9883295f4af..b39f63ce6dfc 100644 --- a/drivers/infiniband/hw/hfi1/mad.c +++ b/drivers/infiniband/hw/hfi1/mad.c @@ -1160,8 +1160,8 @@ static int port_states_transition_allowed(struct hfi1_pportdata *ppd, if (ret == HFI_TRANSITION_DISALLOWED || ret == HFI_TRANSITION_UNDEFINED) { pr_warn("invalid logical state transition %s -> %s\n", - opa_lstate_name(logical_old), - opa_lstate_name(logical_new)); + ib_port_state_to_str(logical_old), + ib_port_state_to_str(logical_new)); return ret; } -- 2.48.1