On Sun, 5 Sept 2021 at 18:07, Bin Meng <bmeng...@gmail.com> wrote: > > On Mon, Sep 6, 2021 at 12:54 AM Peter Maydell <peter.mayd...@linaro.org> > wrote: > > I mean that before commit 62a0db942dec leaving the pointers all > > NULL was not allowed, and after that commit leaving the pointers all > > NULL was still not allowed. It's been a requirement that you > > provide at least one function pointer for read and one for > > write ever since the MemoryRegion APIs were introduced in 2012. > > You're proposing an API change; it might be a good one, but it > > isn't a 'Fixes' to anything. > > Where is the requirement documented? I don't see anything mentioned in > docs/devel/memory.rst
It's not documented, but lots of fiddly details of QEMU functions aren't documented... > If it's a requirement since 2012, then I agree "Fixes" can be dropped. > But a doc fix should be made to document the "requirement". Agreed. -- PMM
