Guennadi Liakhovetski

Results 205 comments of Guennadi Liakhovetski

> Hmm, @lyakh can you check this https://sof-ci.01.org/sofpr/PR9116/build6415/devicetest/index.html?model=MTLP_RVP_HDA&testcase=multiple-pause-resume-50 > > ``` > [ 1032.371199] kernel: sof-audio-pci-intel-mtl 0000:00:1f.3: ipc timed out for 0x13010004|0x0 > [ 1032.371224] kernel: sof-audio-pci-intel-mtl 0000:00:1f.3: ------------[ IPC...

@marc-hb there are several reasons. One of them is https://github.com/thesofproject/sof/blob/8368a6c9f48eeecc8971d8fb422eb2545dcc4e02/src/platform/imx8/include/platform/lib/memory.h#L195-L198 Looking at that I'm actually wondering if our `__sparse_cache` annotations make sense for iMX8 and all other platforms with the...

> Maybe the macros should get annotations to avoid these issues without doing anything at runtime, just for the sake of it? I wouldn't have an issue with that. Just...

> > > Maybe the macros should get annotations to avoid these issues without doing anything at runtime, just for the sake of it? I wouldn't have an issue with...

> In that case I guess we won't need sparse coverage for IPC3 if/when it does not even build at all for neither TGL nor MTL? @marc-hb it looks like...

> @lyakh looks like build errors. Btw, on exception GDB stub should be in read only mode and use a polling based IPC. @lgirdwood why IPC? We don't use IPC...

> > > @lyakh looks like build errors. Btw, on exception GDB stub should be in read only mode and use a polling based IPC. > > > > >...

> RW Stub would be built in by developers for dev usage today, but RO crash stub would be in production build for end users to help report issues. @lgirdwood...

@softwarecki thanks for the review! Many useful comments, I'll respond to them when updating the series and will see which of them can be addressed