On 5/18/20 11:46 AM, Peter Maydell wrote:
On Sun, 17 May 2020 at 17:48, Philippe Mathieu-Daudé <f4...@amsat.org> wrote:
Various places ignore the MemTxResult indicator of
transaction failed. Some cases might be justified
(DMA?) while other are probably bugs. To avoid
ignoring transaction errors, suggestion is to mark
functions returning MemTxResult with
warn_unused_result attribute.
Not necessarily a bad idea, but don't we have an awful
lot of places that ignore the result that we'd need
to fix first?
Yes, there are various places to fix. I wanted to have a preview first,
and not start working on this if it is later rejected. Most of the cases
are hardware specific and require studying each hardware behavior.
