M-Labs
/
zynq-rs
10
0
Fork 12
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
pca006132 commented 2020-08-04 13:05:43 +08:00

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

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
Poster

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

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
There is no content yet.
Delete Branch "pca006132/zynq-rs:sync"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?