sof icon indicating copy to clipboard operation
sof copied to clipboard

chain_dma: Fix chain DMA transfer

Open serhiy-katsyuba-intel opened this issue 1 year ago • 4 comments

Fixes chain DMA implementation: wrong calculation of bytes consumed from host and written into HDA link. The problem was reproduced on test with variable bitrate stream.

serhiy-katsyuba-intel avatar Sep 23 '24 13:09 serhiy-katsyuba-intel

codestyle / checkpatch failure is false positive:

WARNING: 'HDA' may be misspelled - perhaps 'HAD'?
#7: 
host and written into HDA link. The problem was reproduced on test with
                      ^^^

serhiy-katsyuba-intel avatar Sep 23 '24 13:09 serhiy-katsyuba-intel

Temporarily converted to Draft as test_06_05_pass_through_chain_hda_codec_loopback[dma_id=0-dma_id=0-sin_48khz_2ch_5s_mp3_48000Hz_16in16bit_2ch] started to randomly fail with this PR from time to time.

serhiy-katsyuba-intel avatar Sep 24 '24 11:09 serhiy-katsyuba-intel

@serhiy-katsyuba-intel any update ?

lgirdwood avatar Oct 21 '24 15:10 lgirdwood

@serhiy-katsyuba-intel any update ?

The fix triggers occasional chain DMA CI test failure. Most probably, the test would be fixed (or removed from CI). Once the test is fixed I'll un-draft the PR.

serhiy-katsyuba-intel avatar Oct 21 '24 16:10 serhiy-katsyuba-intel

@serhiy-katsyuba-intel still needed ?

lgirdwood avatar Feb 21 '25 12:02 lgirdwood