Repository navigation
schedule: ll: refinement low latency queue - #1149
Conversation
df25e11 to
4730c6e
Compare
4730c6e to
f2cd433
Compare
11c066f to
94f2eec
Compare
There was a problem hiding this comment.
Can we not use the existing list_item_prepend() and list_item_append() here ?
There was a problem hiding this comment.
list_item_prepend() adds to the beginning of the list and list_item_append() at the end. These new functions allow to insert into specific location in the middle of the list.
There was a problem hiding this comment.
yes, but by changing the list head passed to list_item_prepend() should do the same.
There was a problem hiding this comment.
@lgirdwood
Yes, you are right. If we replace list head, list_item_prepend() do the same as list_item_add_after() and list_item_append() do the same list_item_add_before(). I can use "old" ones, but it can enter some misunderstanding by function names.
There was a problem hiding this comment.
@lgirdwood
I've removed list_item_add_after() and list_item_add_before() definitions. I'm using list_item_append() instead of list_item_add_before() now.
94f2eec to
ad63982
Compare
|
We are seeing regression with DMIC tests on CI, @bkokoszx please check it and confirm. |
|
@zrombel @lgirdwood |
I've removed queue per priority in low latency scheduler. Now there is only one queue for all priorities. Scheduler adds tasks to queue in descending order based on ther priority. Signed-off-by: Bartosz Kokoszko <bartoszx.kokoszko@linux.intel.com>
ad63982 to
a32be94
Compare
|
I have modified |
|
With those changes I don't see any regression in python tests. Looks ready for merging. |
I've removed queue per priority in low latency scheduler. Now there is only one queue for all priorities. Scheduler adds tasks to queue in descending order based on their priority.