M-Labs
/
zynq-rs
11
0
Fork 15
Code Issues 6 Pull Requests 1 Releases Wiki Activity

synchronization primitives #57

Merged
sb10q merged 1 commits from pca006132/zynq-rs:sync into master 2020-08-04 13:57:58 +08:00
Conversation 3 Commits 1 Files Changed 7 +39 -34
pca006132 commented 2020-08-04 13:05:43 +08:00
Contributor
Copy Link

M-Labs/artiq-zynq#81
M-Labs/artiq-zynq#80

https://git.m-labs.hk/M-Labs/artiq-zynq/issues/81 https://git.m-labs.hk/M-Labs/artiq-zynq/issues/80
sb10q commented 2020-08-04 13:12:45 +08:00
Owner
Copy Link

We'll probably want to use asm::sev in artiq-zynq to trigger the ACP kernel initiator.
The function (and, for consistency, asm::wfe) should stay implemented even without the powersaving feature, it simply should not be used here.

We'll probably want to use ``asm::sev`` in ``artiq-zynq`` to trigger the ACP kernel initiator. The function (and, for consistency, ``asm::wfe``) should stay implemented even without the powersaving feature, it simply should not be used here.
pca006132 commented 2020-08-04 13:28:22 +08:00
Author
Contributor
Copy Link

We'll probably want to use asm::sev in artiq-zynq to trigger the ACP kernel initiator.
The function (and, for consistency, asm::wfe) should stay implemented even without the powersaving feature, it simply should not be used here.

What about adding spin_lock_yield and notify_spin_lock functions, which would be implemented as wfe and sev when power_saving feature is enabled, and nop when disabled?

> We'll probably want to use ``asm::sev`` in ``artiq-zynq`` to trigger the ACP kernel initiator. > The function (and, for consistency, ``asm::wfe``) should stay implemented even without the powersaving feature, it simply should not be used here. What about adding `spin_lock_yield` and `notify_spin_lock` functions, which would be implemented as `wfe` and `sev` when `power_saving` feature is enabled, and nop when disabled?
sb10q commented 2020-08-04 13:43:43 +08:00
Owner
Copy Link

Sounds good.

Sounds good.
sb10q closed this pull request 2020-08-04 13:57:58 +08:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/zynq-rs#57
No description provided.
Delete Branch "pca006132/zynq-rs:sync"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?