Skip to content

Add changes for invokev2 #134

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: development
Choose a base branch
from

Conversation

quic-ekangupt
Copy link
Contributor

Invoke V2 is going to support enhanced invocations like CRC, poll mode, etc. Complete documentation of invoke_v2 is added under Docs/ as part of this change.

@quic-ekangupt quic-ekangupt force-pushed the invokev2 branch 2 times, most recently from 90efee2 to e04f215 Compare January 27, 2025 09:03
@lumag
Copy link
Contributor

lumag commented Apr 28, 2025

Please split the commit into logical chunks. It is impossible to review userspace changes if everything is thrown into a single bin.

Invoke V2 is going to support enhanced invocations like CRC,
poll mode, etc. This change add support for CRC mode using
invoke V2 request.

Signed-off-by: Ekansh Gupta <[email protected]>
For any remote call to DSP, after sending an invocation message,
fastRPC driver waits for glink response and during this time the
CPU can go into low power modes. With poll mode support fastrpc
driver will continously poll on memory after sending message to
remote subsystem which will eliminate CPU wakeup and schedule
latencies resulting in performance improvement. Invoke V2 is
used to support polling mode.

Signed-off-by: Ekansh Gupta <[email protected]>
Invoke V2 is adding few enhanced invocation requests. Add
documentation for CRC and polling mode explaining how the
feature works and how it can be enabled. Also add the performance
improvement data when using polling mode.

Signed-off-by: Ekansh Gupta <[email protected]>
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.

2 participants