From nobody Mon Feb 9 05:20:55 2026 Received: from out-170.mta1.migadu.com (out-170.mta1.migadu.com [95.215.58.170]) (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 B7628261564 for ; Tue, 11 Feb 2025 17:59:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=95.215.58.170 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739296799; cv=none; b=c+pHcUpeDlibeuFhBT/9yAyMVAYWRz6be1ipBTEVJ7KmWj/0mYOmJDpbEy3POGoBzK18G/Pp2lAvHo32UeGV2a29gTYHNzDTTqRGdoB6zSy0Fzx5rfqC0+awGFhNjP8G4y8FGgnKy3S3UJ32DOw1cK4HNmn5yJ8Vmw80HmCumrw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739296799; c=relaxed/simple; bh=R/HfJb0sLCsimdhY0pWkZLGmdv8yhi0WfSk5mHIqSuA=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=s5GzDWClJDpzRduk+eVWv2w6pdMt4Uu54xDcQ2nLfgrYkm3+FbULdo+F5rOAzZf07I1AeCS32TKLf8g2eYZUXCCXquUGSKSlSKWNpAIbUSO3NzYMdnXO/QAatd/KvrpcjvsF6+4hGJnrnU6rd4zqnNYMJEGiBquWHEN2cJCqff8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=rosenzweig.io; spf=pass smtp.mailfrom=rosenzweig.io; dkim=pass (2048-bit key) header.d=rosenzweig.io header.i=@rosenzweig.io header.b=PzSnSzCa; arc=none smtp.client-ip=95.215.58.170 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=rosenzweig.io Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=rosenzweig.io Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=rosenzweig.io header.i=@rosenzweig.io header.b="PzSnSzCa" X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=rosenzweig.io; s=key1; t=1739296792; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=K2PhGXiwtStqYCjcm40EkOiEETfXthr8C7sGbtb4GB8=; b=PzSnSzCaVuNGetsX0o2fDzCsGE8F7yqo+iKPBxpMqC7xhY1NPm5PtsvA+kqvdgPTgXD3fD RAkIZWft6B4osZpHIs/BLhJWJy53WYtrNeOiCMcV4fuG43w853CgM3DcE3m20dIt9bNegI ScrNTl5dDR22PVne47royC6W3KywD6hfqzO44RPSS7A9a1Rl8aHR6QOoFYMwdhpf99QbzY LyTtiuCibP8A2Os4BpEl5Y/GTqetXSvZtFJfT5kQkMV+cpyzCoLBhpgX5bC+lFJSs/VkDT z6K/CUv8Q/g20Ti/25CrhKwmx8/CdikwHEql1fBbDYUOZfvOpP4lUMaOP3i4NA== From: Alyssa Rosenzweig Date: Tue, 11 Feb 2025 12:59:44 -0500 Subject: [PATCH] soc: apple: rtkit: Check & log more failures Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250211-rtkit-more-logging-v1-1-93334e9c1c77@rosenzweig.io> X-B4-Tracking: v=1; b=H4sIAA+Qq2cC/6tWKk4tykwtVrJSqFYqSi3LLM7MzwNyDHUUlJIzE vPSU3UzU4B8JSMDI1MDI0ND3aKS7MwS3dz8olTdnPz09My8dF1zy+SkpKRUI8PU1FQloMaCotS 0zAqwodGxtbUAZVmi8GQAAAA= X-Change-ID: 20250211-rtkit-more-logging-79cbbbe21eee To: Sven Peter Cc: asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Asahi Lina , Alyssa Rosenzweig X-Developer-Signature: v=1; a=openpgp-sha256; l=4147; i=alyssa@rosenzweig.io; h=from:subject:message-id; bh=a9qAM//3ptsLIAwPktL4UIoeDeVyl/yRaYAY2sOTCIM=; b=owEBbQKS/ZANAwAIAf7+UFoK9VgNAcsmYgBnq5AWtR1CyqDvp60CNnJA/EMbjYudSzf3Yk8k6 ECS5lrq/eqJAjMEAAEIAB0WIQRDXuCbsK8A0B2q9jj+/lBaCvVYDQUCZ6uQFgAKCRD+/lBaCvVY DcXIEACN9+AIOYd+j81n1VusjwjWoLMSLUWBpu5TcZPhuXDy3wbTYfPrFVsD6CGvOaF/LSZdLex J04pELymqBm4d12uP1wMCKq+6UAerkJV0rwDmA/wCOSsWgaVbVl6G8le8y9X0ZeBI1VuYszNbKM zaGk6gLz4APfsZWYoH5l6JaOnjyosUFPjJxBJN3NCZgAEVu5oXBeI/pqyKp5BHYwcZEjj90Ofr9 ICe35C9b6czn0JOiv+Lr/p85RjajSUmt77LDrSrKS6Xg4NnxlFa5sPvZDYsKx3aekhxzAcgIOQ9 ZB2X/ozY1Ia8QipJqUwbJY8/9Oy054ld+ZJJBEE+xf3kEHZtgbDZDFyZRsNeAFEZyzLPTpm9rzx u1h8SdhF8CqeQZe+Hm9+44beleFqDISaNz9R7a6JMGQD9J8P0g5DDBZHwBxwbOrPJsnu79caj+8 /5XU8hbTlT9csHNHGIOmjQn6S+rLM1cccZoxy9zqtAT6d0h5qlq67WeIjf/+9UhgUXpq6HWIWyU zGDQAUpaXvERwFN9DWCpjd+iOPaC9oQyweUzs2NGbd7QDofCvipIzSUggRipVUSGptFAYogFDvH OnZKO1bTy4QNNtOuJtk2A6N/nlhjOLhBBdKut8F0rwfaZ/jkq13cveD9bUF9zSwfho3SLNWvWux 3A1fxjFe2SP47Vw== X-Developer-Key: i=alyssa@rosenzweig.io; a=openpgp; fpr=435EE09BB0AF00D01DAAF638FEFE505A0AF5580D X-Migadu-Flow: FLOW_OUT From: Asahi Lina Check and log the following failures: * regular messages * management messages * failed buffer requests This helps debugging. Signed-off-by: Asahi Lina Signed-off-by: Alyssa Rosenzweig Reviewed-by: Neal Gompa --- Originally multiple commits by Asahi Lina, squashed here and checkpatch.pl warn fixed. --- drivers/soc/apple/rtkit.c | 44 ++++++++++++++++++++++++++++++++++---------- 1 file changed, 34 insertions(+), 10 deletions(-) diff --git a/drivers/soc/apple/rtkit.c b/drivers/soc/apple/rtkit.c index e6d940292c9fbdfc4cd42020e89aca2662c5cdce..f8077a1ec3a42265eb9565a0ea1= ca6a4cf7e79dc 100644 --- a/drivers/soc/apple/rtkit.c +++ b/drivers/soc/apple/rtkit.c @@ -97,12 +97,19 @@ bool apple_rtkit_is_crashed(struct apple_rtkit *rtk) } EXPORT_SYMBOL_GPL(apple_rtkit_is_crashed); =20 -static void apple_rtkit_management_send(struct apple_rtkit *rtk, u8 type, +static int apple_rtkit_management_send(struct apple_rtkit *rtk, u8 type, u64 msg) { + int ret; + msg &=3D ~APPLE_RTKIT_MGMT_TYPE; msg |=3D FIELD_PREP(APPLE_RTKIT_MGMT_TYPE, type); - apple_rtkit_send_message(rtk, APPLE_RTKIT_EP_MGMT, msg, NULL, false); + ret =3D apple_rtkit_send_message(rtk, APPLE_RTKIT_EP_MGMT, msg, NULL, fal= se); + + if (ret) + dev_err(rtk->dev, "RTKit: Failed to send management message: %d\n", ret); + + return ret; } =20 static void apple_rtkit_management_rx_hello(struct apple_rtkit *rtk, u64 m= sg) @@ -295,6 +302,9 @@ static int apple_rtkit_common_rx_get_buffer(struct appl= e_rtkit *rtk, return 0; =20 error: + dev_err(rtk->dev, "RTKit: failed buffer request for 0x%zx bytes (%d)\n", + buffer->size, err); + buffer->buffer =3D NULL; buffer->iomem =3D NULL; buffer->iova =3D 0; @@ -588,11 +598,18 @@ int apple_rtkit_send_message(struct apple_rtkit *rtk,= u8 ep, u64 message, .msg1 =3D ep, }; =20 - if (rtk->crashed) + if (rtk->crashed) { + dev_warn(rtk->dev, + "RTKit: Device is crashed, cannot send message\n"); return -EINVAL; + } + if (ep >=3D APPLE_RTKIT_APP_ENDPOINT_START && - !apple_rtkit_is_running(rtk)) + !apple_rtkit_is_running(rtk)) { + dev_warn(rtk->dev, + "RTKit: Endpoint 0x%02x is not running, cannot send message\n", ep); return -EINVAL; + } =20 /* * The message will be sent with a MMIO write. We need the barrier @@ -742,8 +759,10 @@ static int apple_rtkit_set_ap_power_state(struct apple= _rtkit *rtk, reinit_completion(&rtk->ap_pwr_ack_completion); =20 msg =3D FIELD_PREP(APPLE_RTKIT_MGMT_PWR_STATE, state); - apple_rtkit_management_send(rtk, APPLE_RTKIT_MGMT_SET_AP_PWR_STATE, - msg); + ret =3D apple_rtkit_management_send(rtk, APPLE_RTKIT_MGMT_SET_AP_PWR_STAT= E, + msg); + if (ret) + return ret; =20 ret =3D apple_rtkit_wait_for_completion(&rtk->ap_pwr_ack_completion); if (ret) @@ -763,8 +782,10 @@ static int apple_rtkit_set_iop_power_state(struct appl= e_rtkit *rtk, reinit_completion(&rtk->iop_pwr_ack_completion); =20 msg =3D FIELD_PREP(APPLE_RTKIT_MGMT_PWR_STATE, state); - apple_rtkit_management_send(rtk, APPLE_RTKIT_MGMT_SET_IOP_PWR_STATE, - msg); + ret =3D apple_rtkit_management_send(rtk, APPLE_RTKIT_MGMT_SET_IOP_PWR_STA= TE, + msg); + if (ret) + return ret; =20 ret =3D apple_rtkit_wait_for_completion(&rtk->iop_pwr_ack_completion); if (ret) @@ -865,6 +886,7 @@ EXPORT_SYMBOL_GPL(apple_rtkit_quiesce); int apple_rtkit_wake(struct apple_rtkit *rtk) { u64 msg; + int ret; =20 if (apple_rtkit_is_running(rtk)) return -EINVAL; @@ -876,8 +898,10 @@ int apple_rtkit_wake(struct apple_rtkit *rtk) * will wait for the completion anyway. */ msg =3D FIELD_PREP(APPLE_RTKIT_MGMT_PWR_STATE, APPLE_RTKIT_PWR_STATE_ON); - apple_rtkit_management_send(rtk, APPLE_RTKIT_MGMT_SET_IOP_PWR_STATE, - msg); + ret =3D apple_rtkit_management_send(rtk, APPLE_RTKIT_MGMT_SET_IOP_PWR_STA= TE, + msg); + if (ret) + return ret; =20 return apple_rtkit_boot(rtk); } --- base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b change-id: 20250211-rtkit-more-logging-79cbbbe21eee Best regards, --=20 Alyssa Rosenzweig