Skip to content

[nrf fromlist] modules: hal_nordic: nrfs: improve error handling#2449

Closed
lstnl wants to merge 1 commit intonrfconnect:mainfrom
lstnl:nrfs-improve-ipc-error-handling
Closed

[nrf fromlist] modules: hal_nordic: nrfs: improve error handling#2449
lstnl wants to merge 1 commit intonrfconnect:mainfrom
lstnl:nrfs-improve-ipc-error-handling

Conversation

@lstnl
Copy link
Contributor

@lstnl lstnl commented Jan 24, 2025

Added option to retry message send over ipc when for example there is -ENOMEM error (shared memory buffer full). Number of max retries and delay between them are Kconfigs. Fixed nrfs error reporting when sending directly over ipc service, status was not correct when negative value was returned from ipc service send function.

Upstream PR #: 84499

@lstnl lstnl requested review from Rafal-Nordic and nordic-krch and removed request for nordic-krch January 24, 2025 13:49
@lstnl lstnl force-pushed the nrfs-improve-ipc-error-handling branch from bc5b39e to 9a60593 Compare February 19, 2025 08:36
@lstnl lstnl force-pushed the nrfs-improve-ipc-error-handling branch from 9a60593 to 290ac00 Compare March 6, 2025 12:12
@lstnl lstnl marked this pull request as ready for review March 6, 2025 12:13
@lstnl lstnl requested a review from anangl March 6, 2025 12:13
@lstnl
Copy link
Contributor Author

lstnl commented Mar 6, 2025

It is approved in upstream but not merged because of zephyr release.

Added option to retry message send over ipc when for example
there is -ENOMEM error (shared memory buffer full).
Number of max retries and delay between them are Kconfigs.
Fixed nrfs error reporting when sending directly over
ipc service, status was not correct when negative value
was returned from ipc service send function.

Signed-off-by: Łukasz Stępnicki <lukasz.stepnicki@nordicsemi.no>
(cherry picked from commit 152207d37e21ce91b51a28831d1be5a449327011)
@lstnl lstnl force-pushed the nrfs-improve-ipc-error-handling branch from 290ac00 to 71cca3a Compare March 27, 2025 08:30
@lstnl lstnl closed this Mar 27, 2025
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments