Received-SPF: pass (google.com: domain of linux-kernel+bounces-192931-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
From: admiyo@os.amperecomputing.com
To: Sudeep Holla <sudeep.holla@arm.com>,
	Jassi Brar <jassisinghbrar@gmail.com>,
	Robert Moore <robert.moore@intel.com>,
	"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
	Len Brown <lenb@kernel.org>
Cc: netdev@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH v2 1/3] mctp pcc: Check before sending MCTP PCC response ACK
Date: Tue, 28 May 2024 15:18:21 -0400
Message-Id: <20240528191823.17775-2-admiyo@os.amperecomputing.com>
In-Reply-To: <20240528191823.17775-1-admiyo@os.amperecomputing.com>
References: <20240513173546.679061-1-admiyo@os.amperecomputing.com>
 <20240528191823.17775-1-admiyo@os.amperecomputing.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?5kyw9Dn9L26l4i92b623uWdeTLOYnvYB0E87g1hS3QHGYJV8lBlAfTrQSHto?=
 =?us-ascii?Q?NmK+E/1EEyfI/aq1HmCD5uDxToEZf86MhPnjHx95vM0k3ipuf90Ez/v+r0yS?=
 =?us-ascii?Q?nOM+9fJShCcIKowqrmqR7QaB0ZV9gNPIMbSbS5sLWynNX9ZPO2FCVeXfwHYQ?=
 =?us-ascii?Q?xpeHFv9lRAMXiP3F7/Ux6dLTh6Ws3LapTF5pqV1nByBQUSzZq3NOhzfjdbxL?=
 =?us-ascii?Q?yTC1WrfH7VVqElq+9P/JG+zKZ6YsmyxlLbe1ANsHGh/66rl3EltdkGnFS33h?=
 =?us-ascii?Q?29qZpx946fyDqGNTN5t6HdaY12fbfIbBquPI0TMviwJrOHlkHbNzuu6NX7S4?=
 =?us-ascii?Q?r2NXMfz1GSEYsteQKe9UmOY/lOODGLBFTMjB/byCeIiGELIoHwUCwEoO8A9W?=
 =?us-ascii?Q?XWxf2WlZ/njRNdyu6o+Ke6zDWwyrqjeRzfoGDl4V4p+Zd16AX6DQf78lG2TB?=
 =?us-ascii?Q?eu/t4RF+xc2lsj9ogdGWLl3q3y/zzzqmBZOYZEOWgyOoN1tlcJv+S2Fb9kXJ?=
 =?us-ascii?Q?L4BwfzieXcmySXzT02Rh2nwPfkujqoRr2PcruFNLkHjHXDJzzkk4KfDVajpx?=
 =?us-ascii?Q?Sk5dHipzd7qewcroTspWQ2WJ/Ud7tk+6F9qd+Vhzb3LgbeatZvXPLuapi3WL?=
 =?us-ascii?Q?FXBbLoOFbneII+b9j1aSTcwSb7Wz6GEvmNbGiZAxhBKBQd5LjhD4cBWi0B4i?=
 =?us-ascii?Q?cUcpKuuBsPZgzDSG+kUjfo1sH2wWjR7zlH5Fs2lPJ+QVNniCBChmDCQVUS3b?=
 =?us-ascii?Q?rDGHtpPbqEPh2C2WGozGghIra3DpGCfUyugqPPMnhn9E2SoKVKKi/Tp3v70/?=
 =?us-ascii?Q?ki5/mPamUggoBv6lJr38fwIYFTkbtgMtKpedecg9TFoff0M83kd5UgpMgMpb?=
 =?us-ascii?Q?wxFcNJhRqbW8dyCGLkkT+JDhC5ChwU9scO7KUoOH9N01R3RJA4g05nfL91K7?=
 =?us-ascii?Q?NxaFjSVasxyyr5RXAxhY3sbaxiYoIYj23Y6R4Lrl/EHIXaiISIr2nOPRS2vu?=
 =?us-ascii?Q?XBzRFG/WRT3j/09un5+HOTEuBF+NznUxG3sONFVqMkj7lhdiGhNLxo7SzeYS?=
 =?us-ascii?Q?uMK09JMSOvmzWLnRVA9uwEEfWzVK0uHf0fZ2NKXzvZRkbQhVt3USGZC5mWBg?=
 =?us-ascii?Q?22aWZrLZLfDXMQ/ZyKoU+Wp06H4MVTipQz48u7HzJWUgdoUjuzrcPCsEJ8bO?=
 =?us-ascii?Q?yBNIfwK5UGpb/2pMDU96fEpnXXxcZk5715eAZ88dX0lRvbsrfo80W8dyog9g?=
 =?us-ascii?Q?bHGrhbnGztF9QO/vwaclWpOfsBYHT0gIxmo3Hp0/NPUYi9XOpJ3JQkvh1NL6?=
 =?us-ascii?Q?i33AjcpN2fnniWScs7oxZvfrfB20B6G/+p14liUEr6nuGOpsMvHC4C8KlO0H?=
 =?us-ascii?Q?LubceofmJoSHA8lpoYvMzGCxLVdVrk1kIw8QF/JW5aApzevrDgqvVrtjgmBs?=
 =?us-ascii?Q?CtVX2pRj65qZFs8ByQj/aVV5rou87jP/Pd0YiIcQCg7cuTj0/VCGUi0mMe60?=
 =?us-ascii?Q?mL/lzj7IOleEzCngeAu6kQSgKynGEfrW9p0H+MlekcTgcShJfZfhqhQb71sG?=
 =?us-ascii?Q?KESr2tHQy+ybLL63IH7H94bRZL7reKypfLwNS7b4ZN8jKqfzQk+NdeTrEVeB?=
 =?us-ascii?Q?YXW3+Fbm3l4pSRuMVcec31qAsVXRdwxE7pfd2/nXpIaoMKvRZqGPLY9u7etU?=
 =?us-ascii?Q?lP6ucOmBBZ1nnILPbqq5Cnj2z7c=3D?=
X-OriginatorOrg: os.amperecomputing.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0acc3724-55ca-4925-dc85-08dc7f4af6f5
X-MS-Exchange-CrossTenant-AuthSource: SA0PR01MB6171.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 May 2024 19:18:32.8016
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: zj1at55oUSdLohhICX3mGrI/++RoqZE9T/XKhUeHIlRGx4aM6b555Vz744ZM9lUxAa7tcdmosopKGRLrqV2XaWmiQtyqClOkG2doMu0nqaXkZhpLuStDWpaJ1iBtLiMR
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ2PR01MB8403

From: Adam Young <admiyo@amperecomputing.com>

Type 4 PCC channels have an option to send back a response
to the platform when they are done processing the request.
The flag to indicate whether or not to respond is inside
the message body, and thus is not available to the pcc
mailbox.  Since only one message can be processed at once per
channel, the value of this flag is checked during message processing
and passed back via the channels global structure.

Ideally, the mailbox callback function would return a value
indicating whether the message requires an ACK, but that
would be a change to the mailbox API.  That would involve
some change to all (about 12) of the mailbox based drivers,
and the majority of them would not need to know about the
ACK call.

Signed-off-by: Adam Young <admiyo@os.amperecomputing.com>
---
 drivers/mailbox/pcc.c | 5 ++++-
 include/acpi/pcc.h    | 1 +
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/mailbox/pcc.c b/drivers/mailbox/pcc.c
index 94885e411085..774727b89693 100644
--- a/drivers/mailbox/pcc.c
+++ b/drivers/mailbox/pcc.c
@@ -280,6 +280,7 @@ static irqreturn_t pcc_mbox_irq(int irq, void *p)
 {
 	struct pcc_chan_info *pchan;
 	struct mbox_chan *chan = p;
+	struct pcc_mbox_chan *pmchan;
 	u64 val;
 	int ret;
 
@@ -304,6 +305,8 @@ static irqreturn_t pcc_mbox_irq(int irq, void *p)
 	if (pcc_chan_reg_read_modify_write(&pchan->plat_irq_ack))
 		return IRQ_NONE;
 
+	pmchan = &pchan->chan;
+	pmchan->ack_rx = true;  //TODO default to False
 	mbox_chan_received_data(chan, NULL);
 
 	/*
@@ -312,7 +315,7 @@ static irqreturn_t pcc_mbox_irq(int irq, void *p)
 	 *
 	 * The PCC master subspace channel clears chan_in_use to free channel.
 	 */
-	if (pchan->type == ACPI_PCCT_TYPE_EXT_PCC_SLAVE_SUBSPACE)
+	if ((pchan->type == ACPI_PCCT_TYPE_EXT_PCC_SLAVE_SUBSPACE) && pmchan->ack_rx)
 		pcc_send_data(chan, NULL);
 	pchan->chan_in_use = false;
 
diff --git a/include/acpi/pcc.h b/include/acpi/pcc.h
index 9b373d172a77..297913378c2b 100644
--- a/include/acpi/pcc.h
+++ b/include/acpi/pcc.h
@@ -16,6 +16,7 @@ struct pcc_mbox_chan {
 	u32 latency;
 	u32 max_access_rate;
 	u16 min_turnaround_time;
+	bool ack_rx;
 };
 
 /* Generic Communications Channel Shared Memory Region */
-- 
2.34.1