> -----Original Message-----
> From: Pankaj Gupta [mailto:pankaj.gupta.li...@gmail.com]
> Sent: Monday, November 30, 2020 6:19 PM
> To: ganqixin <ganqi...@huawei.com>
> Cc: Qemu Developers <qemu-devel@nongnu.org>; qemu-triv...@nongnu.org;
> Chenqun (kuhn) <kuhn.chen...@huawei.com>; th...@redhat.com;
> Zhanghailiang <zhang.zhanghaili...@huawei.com>; Michael S . Tsirkin
> <m...@redhat.com>
> Subject: Re: [PATCH v2 01/12] pc-dimm: put it into the 'storage' category
>
> > The category of the pc-dimm device is not set, put it into the 'storage'
> > category.
> >
> > Signed-off-by: Gan Qixin <ganqi...@huawei.com>
> > ---
> > Cc: Michael S. Tsirkin <m...@redhat.com>
> > ---
> > hw/mem/pc-dimm.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/hw/mem/pc-dimm.c b/hw/mem/pc-dimm.c index
> > 2ffc986734..017146e3d1 100644
> > --- a/hw/mem/pc-dimm.c
> > +++ b/hw/mem/pc-dimm.c
> > @@ -282,6 +282,7 @@ static void pc_dimm_class_init(ObjectClass *oc,
> void *data)
> > mdc->get_plugged_size = memory_device_get_region_size;
> > mdc->get_memory_region = pc_dimm_md_get_memory_region;
> > mdc->fill_device_info = pc_dimm_md_fill_device_info;
> > + set_bit(DEVICE_CATEGORY_STORAGE, dc->categories);
>
> Any reason why pc-dimm would be in the storage category?
Hi Pankaj,
Thanks for you reply. As far as I know, pc-dimm is a dimm device for memory
hotplug, described as a "DIMM memory module" in "-device help".
This device looks related to storage, so I put it into the "storage" category
to make it easy to find. I'm not sure if this is appropriate, do you have any
better ideas?
Gan Qixin
>
> > }
> >
> > static TypeInfo pc_dimm_info = {
> > --
> > 2.23.0
> >
> >
