On Mon, Sep 29, 2014 at 12:09:06PM +0000, Gonglei (Arei) wrote: > > From: Cornelia Huck [mailto:cornelia.h...@de.ibm.com] > > Sent: Monday, September 29, 2014 6:53 PM > > Subject: Re: [PATCH RESEND 0/9] virtio: fix virtio child recount in > > transports > > > > On Sat, 27 Sep 2014 10:37:23 +0000 > > "Gonglei (Arei)" <arei.gong...@huawei.com> wrote: > > > > > > > One thing I noticed is that the various devices end up with similar > > > > > code in the end: > > > > > > > > > > object_initialize(&dev->vdev, sizeof(dev->vdev), TYPE_WHATEVER); > > > > > object_property_add_child(obj, "virtio-backend", > > > > OBJECT(&dev->vdev), > > > > > NULL); > > > > > object_unref(OBJECT(&dev->vdev)); > > > > > qdev_alias_all_properties(DEVICE(&dev->vdev), obj); > > > > > > > > > > Would it make sense to add a helper function for that? > > > > > > > > > > Sorry, I'm afraid this is not helpful. Because dev and dev->vdev is > > > different > > > for different virtio devices, like VirtIOBlkPCI(and its vdev is > > > VirtIOBlock), > > > VirtIONetPCI(and its vdev is VirtIONet). They have no the same parameters > > > for above code segment. :) > > > > Hm... > > > > void virtio_instance_init_common(Object *proxydev, > > DeviceState *vdev, > > size_t vdevsize, > > const char *vdevname) > > { > > object_initialize(vdev, vdevsize, vdevname); > > object_property_add_child(proxydev, "virtio-backend", OBJECT(vdev), > > NULL); > > object_unref(OBJECT(vdev)); > > qdev_alias_all_properties(vdev, proxydev); > > } > > > > and have the initializers call > > > > virtio_instance_init_common(obj, DEVICE(&dev->vdev), sizeof(dev->vdev), > > TYPE_WHATEVER); > > > > ? > > OK, it looks good that pass all parameters to one wrapper function. > Will do this in next version. > > Thanks, Cornelia. :) > > Best regards, > -Gonglei
I'm fine with doing the cleanup as a patch on top. Cornelia, fine with you?
