Page MenuHomeFreeBSD
Differential D47420

scmi: Make transports report the effective length of the received messages
AcceptedPublic
Actions

Authored by cristian.marussi_arm.com on Nov 4 2024, 8:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Feb 8, 8:23 AM
Unknown Object (File)
Sat, Feb 8, 8:21 AM
Unknown Object (File)
Tue, Feb 4, 7:09 PM
Unknown Object (File)
Sun, Feb 2, 4:18 PM
Unknown Object (File)
Jan 1 2025, 8:55 AM
Unknown Object (File)
Dec 25 2024, 7:12 AM
Unknown Object (File)
Dec 10 2024, 7:09 PM
Unknown Object (File)
Dec 8 2024, 11:50 PM
View All 26 Files
Subscribers
imp

Details

Reviewers
andrew
br
Group Reviewers
arm64
Summary

Once a message is received that fits into the RX buffer, update the
msg.rx_len field with the effective length of the message received.

Tested on: Arm Morello Board
Sponsored by: Arm Ltd
Signed-off-by: Cristian Marussi <cristian.marussi@arm.com>

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 60364
Build 57248: arc lint + arc unit

Event Timeline

cristian.marussi_arm.com created this revision.Nov 4 2024, 8:51 AM
Herald added a subscriber: imp. · View Herald TranscriptNov 4 2024, 8:51 AM
cristian.marussi_arm.com requested review of this revision.Nov 4 2024, 8:51 AM
cristian.marussi_arm.com added a parent revision: D47419: scmi: Fix race on timed out transmissions.
Harbormaster completed remote builds in B60364: Diff 145965.Nov 4 2024, 8:51 AM
cristian.marussi_arm.com added a child revision: D47421: scmi: Redefine max message payload size.Nov 4 2024, 8:51 AM
andrew added inline comments.Nov 4 2024, 3:46 PM
sys/dev/firmware/arm/scmi_shmem.c
307

This looks wrong, ret is a boolean type so shouldn't be compared with an integer.

cristian.marussi_arm.com added inline comments.Nov 4 2024, 4:51 PM
sys/dev/firmware/arm/scmi_shmem.c
307

Yes indeed, I will fix.

cristian.marussi_arm.com updated this revision to Diff 146812.Nov 21 2024, 8:54 PM

Fix mishandled retvalue on scmi_shmem_msg_poll and related funcs

Harbormaster completed remote builds in B60717: Diff 146812.Nov 21 2024, 8:54 PM
andrew accepted this revision.Nov 28 2024, 11:48 AM
This revision is now accepted and ready to land.Nov 28 2024, 11:48 AM
andrew mentioned this in rG9342829d578e: scmi: Make transports report the effective length of the received messages.Thu, Jan 23, 5:28 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
firmware/
arm/
2 lines
10 lines
10 lines
6 lines
22 lines
4 lines
5 lines

Diff 145965

View Options

sys/dev/firmware/arm/scmi.h

Loading...
View Options

sys/dev/firmware/arm/scmi.c

Loading...
View Options

sys/dev/firmware/arm/scmi_mailbox.c

Loading...
View Options

sys/dev/firmware/arm/scmi_shmem.h

Loading...
View Options

sys/dev/firmware/arm/scmi_shmem.c

Loading...
View Options

sys/dev/firmware/arm/scmi_smc.c

Loading...
View Options

sys/dev/firmware/arm/scmi_virtio.c

Loading...