You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Regression: sof-logger doesn't work with -t parameter #551
@akloniex@lrgirdwo May we request any PR/commit with ABI change add keyword "ABI" in the subject to highlight the risk? And this PR should mention the soft PR in the body. Ideally, the SOFT update should be backward compatible and be merged at first before the SOF PR.
@bkokoszx
Confirmed that this issue is fixed on UP^2.
For Yorp, there's one blocker issue with top sof-master :thesofproject/linux#277
so there's no trace log available, but the "Error: Invalid filename length." is gone. @markyang
please help to confirm this issue on other platform. Thanks~
Activity
[-]sof-logger doesn[/-][+]sof-logger doesn't work with -t parameter[/+][-]sof-logger doesn't work with -t parameter[/-][+]Regression: sof-logger doesn't work with -t parameter[/+]akloniex commentedon Nov 8, 2018
@lgirdwood @bkokoszx @keqiaozhang
Issue is caused by misalignment between FW and sof-logger.
Expected to be solved as soon as thesofproject/soft#118 is merged and sof-logger rebuilt.
mengdonglin commentedon Nov 8, 2018
@akloniex @lrgirdwo May we request any PR/commit with ABI change add keyword "ABI" in the subject to highlight the risk? And this PR should mention the soft PR in the body. Ideally, the SOFT update should be backward compatible and be merged at first before the SOF PR.
ZhendanYang commentedon Nov 9, 2018
sof-logger doesn't work even without -t on CNL:
mengdonglin commentedon Nov 9, 2018
@bkokoszx Are all the PR low necessary to fix this issue? Anything missing?
SOFT: thesofproject/soft#118
FW part: #530
Linux part: thesofproject/linux#222
bkokoszx commentedon Nov 9, 2018
@mengdonglin
Also FW PR I've made yesterday: #548
bkokoszx commentedon Nov 9, 2018
@keqiaozhang
Could you verify if issue still reproduces?
keqiaozhang commentedon Nov 13, 2018
@bkokoszx
Confirmed that this issue is fixed on UP^2.
For Yorp, there's one blocker issue with top sof-master :thesofproject/linux#277
so there's no trace log available, but the "Error: Invalid filename length." is gone.
@markyang
please help to confirm this issue on other platform. Thanks~
markyang commentedon Nov 13, 2018
@bkokoszx @keqiaozhang
This issue has been fixed on BYT, APL and CNL.
markyang commentedon Nov 13, 2018
There's no trace log available on BYT because of a blocker issue thesofproject/linux#233
keqiaozhang commentedon Nov 13, 2018
Thanks @markyang for your confirmation, this issue can be closed now.