On 16/04/2025 17:40, Dikshita Agarwal wrote:
On 4/16/2025 5:40 PM, Bryan O'Donoghue wrote:
On 15/04/2025 05:58, Dikshita Agarwal wrote:
Although firmware makes sure that during session close, all buffers are returned to driver and driver will release them but still we shouldn't rely for this on firmware and should handle in driver. Will fix this in next patch set.
Shouldn't we reset iris in this case ?
Not required.
OK sure.
Could you at least add an error message on close() if any buffer is not released ?
That way we can "trust but verify". What makes me suspicious is that we have one instance where a buffer hasn't been released which we expected to have been released - that may be reasons for that which we can't interrogate from APSS - fine but, then how can we be sure the software contract on close() is respected ?
So yes, I accept what you say that its not required but for peace of mind we should at the very least be noisy on close() about unreleased buffers and if we start to see kernel logs about unreleased bufs we should revisit resetting firmware.
--- bod