Re: [PATCH v4 13/14] hw/mem/system-memory: add a memory sysbus device

[Damien Hedde]

On 2/24/22 10:55, Igor Mammedov wrote:
> On Wed, 23 Feb 2022 11:19:49 +0100
> Damien Hedde <damien.hedde@greensocs.com> wrote:
>
> > On 2/23/22 10:44, Igor Mammedov wrote:
> > > On Wed, 23 Feb 2022 10:07:05 +0100
> > > Damien Hedde <damien.hedde@greensocs.com> wrote:
> > >    
> > > > This device can be used to create a memory wrapped into a
> > > > sysbus device.
> > > > This device has one property 'readonly' which allows
> > > > to choose between a ram or a rom.
> > > >   
> > >    
> > > > The purpose for this device is to be used with qapi command
> > > > device_add.
> > > that's the way to add a device to QEMU but a don't actual
> > > purpose described here, i.e. how board will use this
> > > device/actual usecase and how it will be wired to board
> > > and why it does have to be a sysbus device.
> > >    
> > Sorry, this was unclear.
> >
> > It is a sysbus device in order to use it like any other sysbus device.
> > The memory region it contains is exposed as a sysbus mmio.
>
> aside of that sysbus is legacy fictional bus (albeit widely used),
> it doesn't scale to non sysbus devices (me thinking about buss-less
> pc-dimm & co) since eventually we would like to create mainstream
> machine types via QMP as well.
>
> > I can replace the commit message by the following paragraph:
> >
> > Boards instantiate memories by creating memory region objects which is
> > not possible using QAPI commands.
>
> That's not entirely true, you can use object-add with hostmem backends
> which do provide a means to allocate memory_region.
> (there is no rom hostmem backend probably (i.e. one that return rom memory 
> region)
> but that could be added).
> Another benefit of approach is that one can replace backing
> memory any other backend (file/memfd/pmem...) without affecting
> device model.

I'm not familiar with memory backends. I need to take a look at them.
> > To create a memory, the user can instantiate and map this device by
> > issuing the following commands:
> > device_add driver=sysbus-memory size=0x1000 id=someram
> > sysbus-mmio-map device=someram addr=0
>
> I'd imagine more generic approach would look like:
>
> object-add memory-backend-ram,id=mem1,size=0x1000,other_backend_twiks
> device_add memdevice_frontend,memdev=mem1,addr=0x0
>
> where [pre]plug hooks in machine can map device to
> an appropriate address space/place at device realize time.
> (see memory_device_[pre_]plug() for starters).

We cannot rely on hooks the machine would define, because we start
from an empty machine. So anything must come from qapi and we would
need to do something like that I suppose:
object-add memory-backend-ram,id=mem1,size=0x1000,other_backend_twiks
device_add sysbus-memory-frontend,memdev=mem1,id=memdev_fe
sysbus-mmio-map device=memdev_fe addr=0

Thanks,
--
Damien