thesofproject
modules: Apparently asking 0 alignment is dangerous, use 4 as default
The current rballoc_align() implementation has this line: assert(IS_ALIGNED(mem, align));
Which will cause division by zero due to IS_ALIGNED() Zephyr implementation: define IS_ALIGNED(ptr, align) (((uintptr_t)(ptr)) % (align) == 0)
So better use 4 as the default value.
@lrudyX can you check quickbuild? Both me and Jyri checked the logs and doesn't seem related to this PR. There are no failures in FW log, but TestBasicHdaLinkRandomDma test is reported as timed out. The IPC log doesn't show any IPC timing out, so not sure what goes wrong. FYI @jsarha
@lrudyX can you check quickbuild? Both me and Jyri checked the logs and doesn't seem related to this PR. There are no failures in FW log, but TestBasicHdaLinkRandomDma test is reported as timed out. The IPC log doesn't show any IPC timing out, so not sure what goes wrong. FYI @jsarha
No, this is sporadic problem seen in many others PR
@lrudyX can you check quickbuild? Both me and Jyri checked the logs and doesn't seem related to this PR. There are no failures in FW log, but TestBasicHdaLinkRandomDma test is reported as timed out. The IPC log doesn't show any IPC timing out, so not sure what goes wrong. FYI @jsarha
No, this is sporadic problem seen in many others PR
Can we rerun @lrudyX ? thanks.
@lrudyX can you check quickbuild? Both me and Jyri checked the logs and doesn't seem related to this PR. There are no failures in FW log, but TestBasicHdaLinkRandomDma test is reported as timed out. The IPC log doesn't show any IPC timing out, so not sure what goes wrong. FYI @jsarha
No, this is sporadic problem seen in many others PR
Can we rerun @lrudyX ? thanks. We have DUT problem at the moment, I will rerun as soon as possible.
Ok, here is an alternative solution. We need one or the other, preferrably soon. So please agree on either one (I would still vote for both). https://github.com/thesofproject/sof/pull/10345